mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
synced 2026-04-05 07:08:06 -04:00
btrfs: remove out label in btrfs_check_rw_degradable()
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 <johannes.thumshirn@wdc.com> Signed-off-by: Filipe Manana <fdmanana@suse.com> Reviewed-by: David Sterba <dsterba@suse.com> Signed-off-by: David Sterba <dsterba@suse.com>
This commit is contained in:
committed by
David Sterba
parent
61fb7f04ee
commit
cefef3cc12
@@ -7576,10 +7576,9 @@ bool btrfs_check_rw_degradable(struct btrfs_fs_info *fs_info,
|
||||
|
||||
map = btrfs_find_chunk_map(fs_info, 0, U64_MAX);
|
||||
/* No chunk at all? Return false anyway */
|
||||
if (!map) {
|
||||
ret = false;
|
||||
goto out;
|
||||
}
|
||||
if (!map)
|
||||
return false;
|
||||
|
||||
while (map) {
|
||||
int missing = 0;
|
||||
int max_tolerated;
|
||||
@@ -7604,15 +7603,14 @@ bool btrfs_check_rw_degradable(struct btrfs_fs_info *fs_info,
|
||||
"chunk %llu missing %d devices, max tolerance is %d for writable mount",
|
||||
map->start, missing, max_tolerated);
|
||||
btrfs_free_chunk_map(map);
|
||||
ret = false;
|
||||
goto out;
|
||||
return false;
|
||||
}
|
||||
next_start = map->start + map->chunk_len;
|
||||
btrfs_free_chunk_map(map);
|
||||
|
||||
map = btrfs_find_chunk_map(fs_info, next_start, U64_MAX - next_start);
|
||||
}
|
||||
out:
|
||||
|
||||
return ret;
|
||||
}
|
||||
|
||||
|
||||
Reference in New Issue
Block a user