diff options
| author |   Filipe Manana <fdmanana@suse.com> | 2025-06-11 13:06:39 +0100 |
|---|---|---|
| committer |   David Sterba <dsterba@suse.com> | 2025-07-21 23:58:02 +0200 |
| commit | 8bfa3727ea6b852d6e23273cdc8f05f578bc119e (patch) | |
| tree | 44a3167373e17b82f92a98fc39f18329fee75c82 | |
| parent | btrfs: remove pointless out label from load_free_space_bitmaps() (diff) | |
| download | wireguard-linux-8bfa3727ea6b852d6e23273cdc8f05f578bc119e.tar.xz wireguard-linux-8bfa3727ea6b852d6e23273cdc8f05f578bc119e.zip | |
btrfs: remove pointless out label from load_free_space_extents()
All we do under the label is to return, so there's no point in having it,
just return directly whenever we get an error.
Reviewed-by: Boris Burkov <boris@bur.io>
Signed-off-by: Filipe Manana <fdmanana@suse.com>
Reviewed-by: David Sterba <dsterba@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
| -rw-r--r-- | fs/btrfs/free-space-tree.c | 11 |
1 files changed, 4 insertions, 7 deletions
diff --git a/fs/btrfs/free-space-tree.c b/fs/btrfs/free-space-tree.c index 0514b0a04572..d98a927ca079 100644 --- a/fs/btrfs/free-space-tree.c +++ b/fs/btrfs/free-space-tree.c @@ -1636,7 +1636,7 @@ static int load_free_space_extents(struct btrfs_caching_control *caching_ctl, ret = btrfs_next_item(root, path); if (ret < 0) - goto out; + return ret; if (ret) break; @@ -1652,7 +1652,7 @@ static int load_free_space_extents(struct btrfs_caching_control *caching_ctl, key.objectid + key.offset, &space_added); if (ret) - goto out; + return ret; total_found += space_added; if (total_found > CACHING_CTL_WAKE_UP) { total_found = 0; @@ -1667,13 +1667,10 @@ static int load_free_space_extents(struct btrfs_caching_control *caching_ctl, block_group->start, extent_count, expected_extent_count); DEBUG_WARN(); - ret = -EIO; - goto out; + return -EIO; } - ret = 0; -out: - return ret; + return 0; } int load_free_space_tree(struct btrfs_caching_control *caching_ctl) |