[Intel-gfx] [PATCH 5/5] drm/i915: Tidy i915_gem_valid_gtt_space()

[Chris Wilson]

We can replace a couple of tests with an assertion that the passed in
node is already allocated (as matches the existing call convention) and
by a small bit of refactoring we can bring the line lengths to under
80cols.

Signed-off-by: Chris Wilson <ch...@chris-wilson.co.uk>
---
 drivers/gpu/drm/i915/i915_vma.c | 23 +++++++++++++----------
 1 file changed, 13 insertions(+), 10 deletions(-)

diff --git a/drivers/gpu/drm/i915/i915_vma.c b/drivers/gpu/drm/i915/i915_vma.c
index 19f17b02f4a0..da50be488861 100644
--- a/drivers/gpu/drm/i915/i915_vma.c
+++ b/drivers/gpu/drm/i915/i915_vma.c
@@ -297,10 +297,15 @@ 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 *gtt_space = &vma->node;
+       struct drm_mm_node *node = &vma->node;
        struct drm_mm_node *other;
 
        /*
@@ -313,18 +318,16 @@ bool i915_gem_valid_gtt_space(struct i915_vma *vma,
        if (vma->vm->mm.color_adjust == NULL)
                return true;
 
-       if (!drm_mm_node_allocated(gtt_space))
-               return true;
-
-       if (list_empty(&gtt_space->node_list))
-               return true;
+       /* Only valid to be called on an already inserted vma */
+       GEM_BUG_ON(!drm_mm_node_allocated(node));
+       GEM_BUG_ON(list_empty(&node->node_list));
 
-       other = list_entry(gtt_space->node_list.prev, struct drm_mm_node, 
node_list);
-       if (other->allocated && !other->hole_follows && other->color != 
cache_level)
+       other = list_prev_entry(node, node_list);
+       if (color_differs(other, cache_level) && !other->hole_follows)
                return false;
 
-       other = list_entry(gtt_space->node_list.next, struct drm_mm_node, 
node_list);
-       if (other->allocated && !gtt_space->hole_follows && other->color != 
cache_level)
+       other = list_next_entry(node, node_list);
+       if (color_differs(other, cache_level) && !node->hole_follows)
                return false;
 
        return true;
-- 
2.10.2

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx