summaryrefslogtreecommitdiff
path: root/fs/btrfs/dev-replace.c
diff options
context:
space:
mode:
authorJosef Bacik <josef@toxicpanda.com>2021-10-05 23:12:41 +0300
committerDavid Sterba <dsterba@suse.com>2021-10-26 20:08:07 +0300
commit8b41393fe7c3b180abadc26856fb653014733bb9 (patch)
treed9c63ffcdd0109adffa37db7d0df4b4eb41b743c /fs/btrfs/dev-replace.c
parentadd9745adc2fa13e6a6e2c26c78dcb05da031a44 (diff)
downloadlinux-8b41393fe7c3b180abadc26856fb653014733bb9.tar.xz
btrfs: do not call close_fs_devices in btrfs_rm_device
There's a subtle case where if we're removing the seed device from a file system we need to free its private copy of the fs_devices. However we do not need to call close_fs_devices(), because at this point there are no devices left to close as we've closed the last one. The only thing that close_fs_devices() does is decrement ->opened, which should be 1. We want to avoid calling close_fs_devices() here because it has a lockdep_assert_held(&uuid_mutex), and we are going to stop holding the uuid_mutex in this path. So simply decrement the ->opened counter like we should, and then clean up like normal. Also add a comment explaining what we're doing here as I initially removed this code erroneously. Reviewed-by: Anand Jain <anand.jain@oracle.com> Signed-off-by: Josef Bacik <josef@toxicpanda.com> Signed-off-by: David Sterba <dsterba@suse.com>
Diffstat (limited to 'fs/btrfs/dev-replace.c')
0 files changed, 0 insertions, 0 deletions