diff options
| author |   Nikolay Borisov <nborisov@suse.com> | 2021-10-14 10:03:11 +0300 |
|---|---|---|
| committer |   David Sterba <dsterba@suse.com> | 2022-01-03 15:09:42 +0100 |
| commit | 17130a65f0cd71f9c26bec8f0f097fc61013b6f8 (patch) | |
| tree | 18930c5c8847301b870c3e620e1987b2af20b119 /fs/btrfs/block-group.c | |
| parent | btrfs: fix deadlock between quota enable and other quota operations (diff) | |
| download | linux-dev-17130a65f0cd71f9c26bec8f0f097fc61013b6f8.tar.xz linux-dev-17130a65f0cd71f9c26bec8f0f097fc61013b6f8.zip | |
btrfs: remove spurious unlock/lock of unused_bgs_lock
Since both unused block groups and reclaim bgs lists are protected by
unused_bgs_lock then free them in the same critical section without
doing an extra unlock/lock pair.
Reviewed-by: Josef Bacik <josef@toxicpanda.com>
Signed-off-by: Nikolay Borisov <nborisov@suse.com>
Reviewed-by: David Sterba <dsterba@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
Diffstat (limited to 'fs/btrfs/block-group.c')
| -rw-r--r-- | fs/btrfs/block-group.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/fs/btrfs/block-group.c b/fs/btrfs/block-group.c index 444e9c89ff3e..6ab864655090 100644 --- a/fs/btrfs/block-group.c +++ b/fs/btrfs/block-group.c @@ -3911,9 +3911,7 @@ int btrfs_free_block_groups(struct btrfs_fs_info *info) list_del_init(&block_group->bg_list); btrfs_put_block_group(block_group); } - spin_unlock(&info->unused_bgs_lock); - spin_lock(&info->unused_bgs_lock); while (!list_empty(&info->reclaim_bgs)) { block_group = list_first_entry(&info->reclaim_bgs, struct btrfs_block_group, |