super: wait until we passed kill super
authorChristian Brauner <brauner@kernel.org>
Fri, 18 Aug 2023 14:00:51 +0000 (16:00 +0200)
committerChristian Brauner <brauner@kernel.org>
Mon, 21 Aug 2023 16:09:08 +0000 (18:09 +0200)
commit2c18a63b760a0f68f14cb8bb4c3840bb0b63b73e
tree08544495066d5e714d17fb3cb253cda95e37cbf7
parent5e87491415217d6bec0bcae08a3156622be2b177
super: wait until we passed kill super

Recent rework moved block device closing out of sb->put_super() and into
sb->kill_sb() to avoid deadlocks as s_umount is held in put_super() and
blkdev_put() can end up taking s_umount again.

That means we need to move the removal of the superblock from @fs_supers
out of generic_shutdown_super() and into deactivate_locked_super() to
ensure that concurrent mounters don't fail to open block devices that
are still in use because blkdev_put() in sb->kill_sb() hasn't been
called yet.

We can now do this as we can make iterators through @fs_super and
@super_blocks wait without holding s_umount. Concurrent mounts will wait
until a dying superblock is fully dead so until sb->kill_sb() has been
called and SB_DEAD been set. Concurrent iterators can already discard
any SB_DYING superblock.

Reviewed-by: Jan Kara <jack@suse.cz>
Message-Id: <20230818-vfs-super-fixes-v3-v3-4-9f0b1876e46b@kernel.org>
Signed-off-by: Christian Brauner <brauner@kernel.org>
fs/super.c
include/linux/fs.h