drm/i915: export color_differs
authorMatthew Auld <matthew.auld@intel.com>
Mon, 9 Sep 2019 12:40:50 +0000 (13:40 +0100)
committerChris Wilson <chris@chris-wilson.co.uk>
Mon, 9 Sep 2019 20:00:04 +0000 (21:00 +0100)
Export color_differs so that we can use it elsewhere.

Signed-off-by: Matthew Auld <matthew.auld@intel.com>
Cc: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
Cc: Rodrigo Vivi <rodrigo.vivi@intel.com>
Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Link: https://patchwork.freedesktop.org/patch/msgid/20190909124052.22900-1-matthew.auld@intel.com
drivers/gpu/drm/i915/i915_gem_gtt.c
drivers/gpu/drm/i915/i915_vma.c
drivers/gpu/drm/i915/i915_vma.h

index 906dc6f..095f5e3 100644 (file)
@@ -2552,7 +2552,7 @@ static void i915_gtt_color_adjust(const struct drm_mm_node *node,
                                  u64 *start,
                                  u64 *end)
 {
-       if (node->allocated && node->color != color)
+       if (i915_node_color_differs(node, color))
                *start += I915_GTT_PAGE_SIZE;
 
        /* Also leave a space between the unallocated reserved node after the
index e0e677b..a90bd26 100644 (file)
@@ -477,11 +477,6 @@ void __i915_vma_set_map_and_fenceable(struct i915_vma *vma)
                vma->flags &= ~I915_VMA_CAN_FENCE;
 }
 
-static bool color_differs(struct drm_mm_node *node, unsigned long color)
-{
-       return node->allocated && node->color != color;
-}
-
 bool i915_gem_valid_gtt_space(struct i915_vma *vma, unsigned long cache_level)
 {
        struct drm_mm_node *node = &vma->node;
@@ -502,11 +497,13 @@ bool i915_gem_valid_gtt_space(struct i915_vma *vma, unsigned long cache_level)
        GEM_BUG_ON(list_empty(&node->node_list));
 
        other = list_prev_entry(node, node_list);
-       if (color_differs(other, cache_level) && !drm_mm_hole_follows(other))
+       if (i915_node_color_differs(other, cache_level) &&
+           !drm_mm_hole_follows(other))
                return false;
 
        other = list_next_entry(node, node_list);
-       if (color_differs(other, cache_level) && !drm_mm_hole_follows(node))
+       if (i915_node_color_differs(other, cache_level) &&
+           !drm_mm_hole_follows(node))
                return false;
 
        return true;
index 889fc7c..5b1e0cf 100644 (file)
@@ -373,6 +373,12 @@ static inline bool i915_vma_is_bound(const struct i915_vma *vma,
        return vma->flags & where;
 }
 
+static inline bool i915_node_color_differs(const struct drm_mm_node *node,
+                                          unsigned long color)
+{
+       return node->allocated && node->color != color;
+}
+
 /**
  * i915_vma_pin_iomap - calls ioremap_wc to map the GGTT VMA via the aperture
  * @vma: VMA to iomap