mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
synced 2026-05-13 18:49:22 -04:00
btrfs: remove pointless out label from remove_free_space_extent()
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>
This commit is contained in:
committed by
David Sterba
parent
ffb7068f16
commit
e3ecf6f164
@@ -727,7 +727,7 @@ static int remove_free_space_extent(struct btrfs_trans_handle *trans,
|
||||
|
||||
ret = btrfs_search_prev_slot(trans, root, &key, path, -1, 1);
|
||||
if (ret)
|
||||
goto out;
|
||||
return ret;
|
||||
|
||||
btrfs_item_key_to_cpu(path->nodes[0], &key, path->slots[0]);
|
||||
|
||||
@@ -759,7 +759,7 @@ static int remove_free_space_extent(struct btrfs_trans_handle *trans,
|
||||
/* Delete the existing key (cases 1-4). */
|
||||
ret = btrfs_del_item(trans, root, path);
|
||||
if (ret)
|
||||
goto out;
|
||||
return ret;
|
||||
|
||||
/* Add a key for leftovers at the beginning (cases 3 and 4). */
|
||||
if (start > found_start) {
|
||||
@@ -770,7 +770,7 @@ static int remove_free_space_extent(struct btrfs_trans_handle *trans,
|
||||
btrfs_release_path(path);
|
||||
ret = btrfs_insert_empty_item(trans, root, path, &key, 0);
|
||||
if (ret)
|
||||
goto out;
|
||||
return ret;
|
||||
new_extents++;
|
||||
}
|
||||
|
||||
@@ -783,16 +783,12 @@ static int remove_free_space_extent(struct btrfs_trans_handle *trans,
|
||||
btrfs_release_path(path);
|
||||
ret = btrfs_insert_empty_item(trans, root, path, &key, 0);
|
||||
if (ret)
|
||||
goto out;
|
||||
return ret;
|
||||
new_extents++;
|
||||
}
|
||||
|
||||
btrfs_release_path(path);
|
||||
ret = update_free_space_extent_count(trans, block_group, path,
|
||||
new_extents);
|
||||
|
||||
out:
|
||||
return ret;
|
||||
return update_free_space_extent_count(trans, block_group, path, new_extents);
|
||||
}
|
||||
|
||||
EXPORT_FOR_TESTS
|
||||
|
||||
Reference in New Issue
Block a user