drm/amdgpu: Increase hmm range get pages timeout
authorPhilip Yang <Philip.Yang@amd.com>
Tue, 6 Jun 2023 19:46:33 +0000 (15:46 -0400)
committerAlex Deucher <alexander.deucher@amd.com>
Thu, 15 Jun 2023 21:55:41 +0000 (17:55 -0400)
If hmm_range_fault returns -EBUSY, we should call hmm_range_fault again
to validate the remaining pages. On one system with NUMA auto balancing
enabled, hmm_range_fault takes 6 seconds for 1GB range because CPU
migrate the range one page at a time. To be safe, increase timeout value
to 1 second for 128MB range.

Signed-off-by: Philip Yang <Philip.Yang@amd.com>
Reviewed-by: Felix Kuehling <Felix.Kuehling@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/amdgpu/amdgpu_hmm.c

index 2dadcfe..0812671 100644 (file)
@@ -190,8 +190,8 @@ int amdgpu_hmm_range_get_pages(struct mmu_interval_notifier *notifier,
                pr_debug("hmm range: start = 0x%lx, end = 0x%lx",
                        hmm_range->start, hmm_range->end);
 
-               /* Assuming 512MB takes maxmium 1 second to fault page address */
-               timeout = max((hmm_range->end - hmm_range->start) >> 29, 1UL);
+               /* Assuming 128MB takes maximum 1 second to fault page address */
+               timeout = max((hmm_range->end - hmm_range->start) >> 27, 1UL);
                timeout *= HMM_RANGE_DEFAULT_TIMEOUT;
                timeout = jiffies + msecs_to_jiffies(timeout);