drm: Apply range restriction after color adjustment when allocation
authorChris Wilson <chris@chris-wilson.co.uk>
Thu, 22 Dec 2016 08:36:38 +0000 (08:36 +0000)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Wed, 28 Dec 2016 12:50:34 +0000 (13:50 +0100)
mm->color_adjust() compares the hole with its neighbouring nodes. They
only abutt before we restrict the hole, so we have to apply color_adjust
before we apply the range restriction.

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-36-chris@chris-wilson.co.uk
drivers/gpu/drm/drm_mm.c

index c0024719f32b1e995d9285cc5b59358bf5eb2231..e279ac7c7aafb46fcf56b1a3eada90411e5a0193 100644 (file)
@@ -410,14 +410,12 @@ static void drm_mm_insert_helper_range(struct drm_mm_node *hole_node,
 
        DRM_MM_BUG_ON(!drm_mm_hole_follows(hole_node) || node->allocated);
 
-       if (adj_start < start)
-               adj_start = start;
-       if (adj_end > end)
-               adj_end = end;
-
        if (mm->color_adjust)
                mm->color_adjust(hole_node, color, &adj_start, &adj_end);
 
+       adj_start = max(adj_start, start);
+       adj_end = min(adj_end, end);
+
        if (flags & DRM_MM_CREATE_TOP)
                adj_start = adj_end - size;
 
@@ -625,17 +623,15 @@ static struct drm_mm_node *drm_mm_search_free_in_range_generic(const struct drm_
                               flags & DRM_MM_SEARCH_BELOW) {
                u64 hole_size = adj_end - adj_start;
 
-               if (adj_start < start)
-                       adj_start = start;
-               if (adj_end > end)
-                       adj_end = end;
-
                if (mm->color_adjust) {
                        mm->color_adjust(entry, color, &adj_start, &adj_end);
                        if (adj_end <= adj_start)
                                continue;
                }
 
+               adj_start = max(adj_start, start);
+               adj_end = min(adj_end, end);
+
                if (!check_free_hole(adj_start, adj_end, size, alignment))
                        continue;