Revert "drm/amdgpu: remove TOPDOWN flags when allocating VRAM in large bar system"
authorArunpravin Paneer Selvam <Arunpravin.PaneerSelvam@amd.com>
Fri, 9 Jun 2023 13:18:41 +0000 (06:18 -0700)
committerAlex Deucher <alexander.deucher@amd.com>
Thu, 15 Jun 2023 14:43:36 +0000 (10:43 -0400)
This reverts commit c105518679b6e87232874ffc989ec403bee59664.

This patch disables the TOPDOWN flag for APU and few dGPU cards
which has the VRAM size equal to the BAR size.

When we enable the TOPDOWN flag, we get the free blocks at
the highest available memory region and we don't split the
lower order blocks. This change is required to keep off
the fragmentation related issues particularly in ASIC
which has VRAM space <= 500MiB

Hence, we are reverting this patch.

Link: https://gitlab.freedesktop.org/drm/amd/-/issues/2270
Signed-off-by: Arunpravin Paneer Selvam <Arunpravin.PaneerSelvam@amd.com>
Reviewed-by: Christian König <christian.koenig@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/amdgpu/amdgpu_object.c

index f76649e523a0de11acb5e2b8e15a2a013b623f39..54cd21c06a44fb39757e3f39c9b78f25ddfd3b0c 100644 (file)
@@ -150,7 +150,7 @@ void amdgpu_bo_placement_from_domain(struct amdgpu_bo *abo, u32 domain)
 
                if (flags & AMDGPU_GEM_CREATE_CPU_ACCESS_REQUIRED)
                        places[c].lpfn = min_not_zero(places[c].lpfn, visible_pfn);
-               else if (adev->gmc.real_vram_size != adev->gmc.visible_vram_size)
+               else
                        places[c].flags |= TTM_PL_FLAG_TOPDOWN;
 
                if (flags & AMDGPU_GEM_CREATE_VRAM_CONTIGUOUS)