summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorChris Wilson <chris@chris-wilson.co.uk>2016-12-22 11:36:32 +0300
committerDaniel Vetter <daniel.vetter@ffwll.ch>2016-12-28 13:50:42 +0300
commit268c6498fba2f6555d215408ae4de3ca1a08fb77 (patch)
treed2e3ed7fef057aa6fc28b7f4a20cc03f4a9a29ca
parent2c4b389518fbe552188928aadcd3815d5116a05c (diff)
downloadlinux-268c6498fba2f6555d215408ae4de3ca1a08fb77.tar.xz
drm: Fix application of color vs range restriction when scanning drm_mm
The range restriction should be applied after the color adjustment, or else we may inadvertently apply the color adjustment to the restricted hole (and not against its neighbours). Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> Reviewed-by: Joonas Lahtinen <joonas.lahtinen@linux.intel.com> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch> Link: http://patchwork.freedesktop.org/patch/msgid/20161222083641.2691-30-chris@chris-wilson.co.uk
-rw-r--r--drivers/gpu/drm/drm_mm.c15
1 files changed, 9 insertions, 6 deletions
diff --git a/drivers/gpu/drm/drm_mm.c b/drivers/gpu/drm/drm_mm.c
index c68f79149b9a..1b5613bcb35e 100644
--- a/drivers/gpu/drm/drm_mm.c
+++ b/drivers/gpu/drm/drm_mm.c
@@ -772,6 +772,7 @@ bool drm_mm_scan_add_block(struct drm_mm_scan *scan,
struct drm_mm *mm = scan->mm;
struct drm_mm_node *hole;
u64 hole_start, hole_end;
+ u64 col_start, col_end;
u64 adj_start, adj_end;
DRM_MM_BUG_ON(node->mm != mm);
@@ -789,14 +790,16 @@ bool drm_mm_scan_add_block(struct drm_mm_scan *scan,
node->node_list.next = &scan->prev_scanned_node->node_list;
scan->prev_scanned_node = node;
- hole_start = drm_mm_hole_node_start(hole);
- hole_end = drm_mm_hole_node_end(hole);
-
- adj_start = max(hole_start, scan->range_start);
- adj_end = min(hole_end, scan->range_end);
+ hole_start = __drm_mm_hole_node_start(hole);
+ hole_end = __drm_mm_hole_node_end(hole);
+ col_start = hole_start;
+ col_end = hole_end;
if (mm->color_adjust)
- mm->color_adjust(hole, scan->color, &adj_start, &adj_end);
+ mm->color_adjust(hole, scan->color, &col_start, &col_end);
+
+ adj_start = max(col_start, scan->range_start);
+ adj_end = min(col_end, scan->range_end);
if (check_free_hole(adj_start, adj_end,
scan->size, scan->alignment)) {