drm/amdkfd: Fix a memory limit issue
authorEric Huang <jinhuieric.huang@amd.com>
Mon, 14 Nov 2022 16:43:48 +0000 (11:43 -0500)
committerAlex Deucher <alexander.deucher@amd.com>
Tue, 15 Nov 2022 18:24:32 +0000 (13:24 -0500)
It is to resolve a regression, which fails to allocate
VRAM due to no free memory in application, the reason
is we add check of vram_pin_size for memory limit, and
application is pinning the memory for Peerdirect, KFD
should not count it in memory limit. So removing
vram_pin_size will resolve it.

Signed-off-by: Eric Huang <jinhuieric.huang@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_amdkfd_gpuvm.c

index 84f44f7..6d1ff7b 100644 (file)
@@ -171,9 +171,7 @@ int amdgpu_amdkfd_reserve_mem_limit(struct amdgpu_device *adev,
            (kfd_mem_limit.ttm_mem_used + ttm_mem_needed >
             kfd_mem_limit.max_ttm_mem_limit) ||
            (adev && adev->kfd.vram_used + vram_needed >
-            adev->gmc.real_vram_size -
-            atomic64_read(&adev->vram_pin_size) -
-            reserved_for_pt)) {
+            adev->gmc.real_vram_size - reserved_for_pt)) {
                ret = -ENOMEM;
                goto release;
        }