]> git.karo-electronics.de Git - karo-tx-linux.git/commitdiff
Btrfs: call the ordered free operation without any locks held
authorChris Mason <chris.mason@fusionio.com>
Wed, 25 Jul 2012 19:57:13 +0000 (15:57 -0400)
committerBen Hutchings <ben@decadent.org.uk>
Thu, 2 Aug 2012 13:38:02 +0000 (14:38 +0100)
commit e9fbcb42201c862fd6ab45c48ead4f47bb2dea9d upstream.

Each ordered operation has a free callback, and this was called with the
worker spinlock held.  Josef made the free callback also call iput,
which we can't do with the spinlock.

This drops the spinlock for the free operation and grabs it again before
moving through the rest of the list.  We'll circle back around to this
and find a cleaner way that doesn't bounce the lock around so much.

Signed-off-by: Chris Mason <chris.mason@fusionio.com>
Signed-off-by: Ben Hutchings <ben@decadent.org.uk>
fs/btrfs/async-thread.c

index 0b394580d8603becf8c5c031745687f6ba9e9aeb..03321e5e098f3feff11bde9f52e238008d10ee90 100644 (file)
@@ -206,10 +206,17 @@ static noinline int run_ordered_completions(struct btrfs_workers *workers,
 
                work->ordered_func(work);
 
-               /* now take the lock again and call the freeing code */
+               /* now take the lock again and drop our item from the list */
                spin_lock(&workers->order_lock);
                list_del(&work->order_list);
+               spin_unlock(&workers->order_lock);
+
+               /*
+                * we don't want to call the ordered free functions
+                * with the lock held though
+                */
                work->ordered_free(work);
+               spin_lock(&workers->order_lock);
        }
 
        spin_unlock(&workers->order_lock);