From: Christian Brauner Date: Tue, 24 Oct 2023 14:53:41 +0000 (+0200) Subject: ext4: simplify device handling X-Git-Tag: v6.8-rc1~214^2~14 X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=218de67764e5c4682892427dd78b382e1709674c;p=thirdparty%2Fkernel%2Flinux.git ext4: simplify device handling We removed all codepaths where s_umount is taken beneath open_mutex and bd_holder_lock so don't make things more complicated than they need to be and hold s_umount over block device opening. Link: https://lore.kernel.org/r/20231024-vfs-super-rework-v1-3-37a8aa697148@kernel.org Reviewed-by: Jan Kara Signed-off-by: Christian Brauner --- diff --git a/fs/ext4/super.c b/fs/ext4/super.c index c5fcf377ab1fa..bd752f3f1dd52 100644 --- a/fs/ext4/super.c +++ b/fs/ext4/super.c @@ -5864,11 +5864,8 @@ static struct bdev_handle *ext4_get_journal_blkdev(struct super_block *sb, struct ext4_super_block *es; int errno; - /* see get_tree_bdev why this is needed and safe */ - up_write(&sb->s_umount); bdev_handle = bdev_open_by_dev(j_dev, BLK_OPEN_READ | BLK_OPEN_WRITE, sb, &fs_holder_ops); - down_write(&sb->s_umount); if (IS_ERR(bdev_handle)) { ext4_msg(sb, KERN_ERR, "failed to open journal device unknown-block(%u,%u) %ld",