drm/radeon: Clean up assignment of TTM placement lpfn member for pinning
authorMichel Dänzer <michel.daenzer@amd.com>
Tue, 9 Sep 2014 01:09:23 +0000 (10:09 +0900)
committerAlex Deucher <alexander.deucher@amd.com>
Wed, 10 Sep 2014 15:29:46 +0000 (11:29 -0400)
This sets the lpfn member to 0 instead of the full domain size. TTM uses
the full domain size when lpfn is 0.

Signed-off-by: Michel Dänzer <michel.daenzer@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/radeon/radeon_object.c

index eef60aaf4e64f81ed01244e7a319f4b8d9a511a8..3dbbd65336d50ea723c7aa4579e25bde375875f6 100644 (file)
@@ -311,18 +311,14 @@ int radeon_bo_pin_restricted(struct radeon_bo *bo, u32 domain, u64 max_offset,
        }
        radeon_ttm_placement_from_domain(bo, domain);
        for (i = 0; i < bo->placement.num_placement; i++) {
-               unsigned lpfn = 0;
-
                /* force to pin into visible video ram */
-               if (bo->placements[i].flags & TTM_PL_FLAG_VRAM)
-                       lpfn = bo->rdev->mc.visible_vram_size >> PAGE_SHIFT;
+               if ((bo->placements[i].flags & TTM_PL_FLAG_VRAM) &&
+                   (!max_offset || max_offset > bo->rdev->mc.visible_vram_size))
+                       bo->placements[i].lpfn =
+                               bo->rdev->mc.visible_vram_size >> PAGE_SHIFT;
                else
-                       lpfn = bo->rdev->mc.gtt_size >> PAGE_SHIFT; /* ??? */
-
-               if (max_offset)
-                       lpfn = min (lpfn, (unsigned)(max_offset >> PAGE_SHIFT));
+                       bo->placements[i].lpfn = max_offset >> PAGE_SHIFT;
 
-               bo->placements[i].lpfn = lpfn;
                bo->placements[i].flags |= TTM_PL_FLAG_NO_EVICT;
        }