drm/amdgpu: stop adding dummy entry in amdgpu_ttm_placement_init
authorChristian König <christian.koenig@amd.com>
Mon, 15 Aug 2016 12:08:54 +0000 (14:08 +0200)
committerAlex Deucher <alexander.deucher@amd.com>
Mon, 22 Aug 2016 17:47:24 +0000 (13:47 -0400)
AMDGPU_GEM_CREATE_NO_CPU_ACCESS and AMDGPU_GEM_CREATE_CPU_ACCESS_REQUIRED are
obviously mutual exclusive. So stop adding a dummy entry without effect when
both are specified.

Signed-off-by: Christian König <christian.koenig@amd.com>
Reviewed-by: Alex Deucher <alexander.deucher@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/amdgpu/amdgpu_object.c

index 04c376f27611ef052f1f5a1bc412578cff8d43ed..b17734e0ecc83727799f66249ca9d0389fa837f6 100644 (file)
@@ -119,18 +119,16 @@ static void amdgpu_ttm_placement_init(struct amdgpu_device *adev,
                                      struct ttm_place *places,
                                      u32 domain, u64 flags)
 {
-       u32 c = 0, i;
+       u32 c = 0;
 
        if (domain & AMDGPU_GEM_DOMAIN_VRAM) {
                unsigned visible_pfn = adev->mc.visible_vram_size >> PAGE_SHIFT;
 
                if (flags & AMDGPU_GEM_CREATE_NO_CPU_ACCESS &&
+                   !(flags & AMDGPU_GEM_CREATE_CPU_ACCESS_REQUIRED) &&
                    adev->mc.visible_vram_size < adev->mc.real_vram_size) {
                        places[c].fpfn = visible_pfn;
-                       if (flags & AMDGPU_GEM_CREATE_CPU_ACCESS_REQUIRED)
-                               places[c].lpfn = visible_pfn;
-                       else
-                               places[c].lpfn = 0;
+                       places[c].lpfn = 0;
                        places[c].flags = TTM_PL_FLAG_WC |
                                TTM_PL_FLAG_UNCACHED | TTM_PL_FLAG_VRAM |
                                TTM_PL_FLAG_TOPDOWN;