drm/i915/userptr: Keep spin_lock/unlock in the same block
authorChris Wilson <chris@chris-wilson.co.uk>
Thu, 24 Jul 2014 12:28:44 +0000 (13:28 +0100)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Fri, 25 Jul 2014 07:39:03 +0000 (09:39 +0200)
Move the code around in order to acquire and release the spinlock in the
same function and in the same block. This keeps static analysers happy
and the reader sane.

Suggested-by: Julia Lawall <julia.lawall@lip6.fr>
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Julia Lawall <julia.lawall@lip6.fr>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
drivers/gpu/drm/i915/i915_gem_userptr.c

index 74c45da8a1ba5aeb05b0e071df00a8a707ea7e11..fe69fc837d9ee25f33b3dbd15664c2c33b9ca721 100644 (file)
@@ -91,10 +91,10 @@ static unsigned long cancel_userptr(struct drm_i915_gem_object *obj)
        return end;
 }
 
-static void invalidate_range__linear(struct i915_mmu_notifier *mn,
-                                    struct mm_struct *mm,
-                                    unsigned long start,
-                                    unsigned long end)
+static void *invalidate_range__linear(struct i915_mmu_notifier *mn,
+                                     struct mm_struct *mm,
+                                     unsigned long start,
+                                     unsigned long end)
 {
        struct i915_mmu_object *mmu;
        unsigned long serial;
@@ -118,7 +118,7 @@ restart:
                        goto restart;
        }
 
-       spin_unlock(&mn->lock);
+       return NULL;
 }
 
 static void i915_gem_userptr_mn_invalidate_range_start(struct mmu_notifier *_mn,
@@ -133,13 +133,12 @@ static void i915_gem_userptr_mn_invalidate_range_start(struct mmu_notifier *_mn,
 
        end--; /* interval ranges are inclusive, but invalidate range is exclusive */
        while (next < end) {
-               struct drm_i915_gem_object *obj;
+               struct drm_i915_gem_object *obj = NULL;
 
-               obj = NULL;
                spin_lock(&mn->lock);
                if (mn->has_linear)
-                       return invalidate_range__linear(mn, mm, start, end);
-               if (serial == mn->serial)
+                       it = invalidate_range__linear(mn, mm, start, end);
+               else if (serial == mn->serial)
                        it = interval_tree_iter_next(it, next, end);
                else
                        it = interval_tree_iter_first(&mn->objects, start, end);