From: Filipe Manana Date: Tue, 20 Jan 2026 20:06:57 +0000 (+0000) Subject: btrfs: remove out label in btrfs_init_space_info() X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=5eb01bf4a9407e8d825ac9ee5b5a1ef2c1972e61;p=thirdparty%2Fkernel%2Flinux.git btrfs: remove out label in btrfs_init_space_info() There is no point in having the label since all it does is return the value in the 'ret' variable. Instead make every goto return directly and remove the label. Reviewed-by: Johannes Thumshirn Signed-off-by: Filipe Manana Reviewed-by: David Sterba Signed-off-by: David Sterba --- diff --git a/fs/btrfs/space-info.c b/fs/btrfs/space-info.c index bc493243f777d..bb5aac7ee9d25 100644 --- a/fs/btrfs/space-info.c +++ b/fs/btrfs/space-info.c @@ -329,7 +329,7 @@ int btrfs_init_space_info(struct btrfs_fs_info *fs_info) struct btrfs_super_block *disk_super; u64 features; u64 flags; - int mixed = 0; + bool mixed = false; int ret; disk_super = fs_info->super_copy; @@ -338,28 +338,28 @@ int btrfs_init_space_info(struct btrfs_fs_info *fs_info) features = btrfs_super_incompat_flags(disk_super); if (features & BTRFS_FEATURE_INCOMPAT_MIXED_GROUPS) - mixed = 1; + mixed = true; flags = BTRFS_BLOCK_GROUP_SYSTEM; ret = create_space_info(fs_info, flags); if (ret) - goto out; + return ret; if (mixed) { flags = BTRFS_BLOCK_GROUP_METADATA | BTRFS_BLOCK_GROUP_DATA; ret = create_space_info(fs_info, flags); if (ret) - goto out; + return ret; } else { flags = BTRFS_BLOCK_GROUP_METADATA; ret = create_space_info(fs_info, flags); if (ret) - goto out; + return ret; flags = BTRFS_BLOCK_GROUP_DATA; ret = create_space_info(fs_info, flags); if (ret) - goto out; + return ret; } if (features & BTRFS_FEATURE_INCOMPAT_REMAP_TREE) { @@ -367,7 +367,6 @@ int btrfs_init_space_info(struct btrfs_fs_info *fs_info) ret = create_space_info(fs_info, flags); } -out: return ret; }