btrfs: free cached state outside critical section in wait_extent_bit()

There's no need to free the cached extent state record while holding the
io tree's spinlock, it's just making the critical section longer than it
needs to be. So just do it after unlocking the io tree.

Signed-off-by: Filipe Manana <fdmanana@suse.com>
Reviewed-by: David Sterba <dsterba@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>

diff --git a/fs/btrfs/extent-io-tree.c b/fs/btrfs/extent-io-tree.c
index 0d9437e72bd4f4acb6fa9cdbeb61c20c74f50263..29fcd08b9ca9b64722dc5eedcb8f06da06068f60 100644 (file)
--- a/fs/btrfs/extent-io-tree.c
+++ b/fs/btrfs/extent-io-tree.c
@@ -877,13 +877,13 @@ process_node:
                }
        }
 out:
+       spin_unlock(&tree->lock);
        /* This state is no longer useful, clear it and free it up. */
        if (cached_state && *cached_state) {
                state = *cached_state;
                *cached_state = NULL;
                btrfs_free_extent_state(state);
        }
-       spin_unlock(&tree->lock);
 }
 
 static void cache_state_if_flags(struct extent_state *state,