Skip to content

Commit 918cdf4

Browse files
fdmananakdave
authored andcommitted
btrfs: remove useless 'fail_unlock' label from btrfs_csum_file_blocks()
The label 'fail_unlock' is pointless, all it does is to jump to the label 'out', so just remove it. Signed-off-by: Filipe Manana <[email protected]> Reviewed-by: David Sterba <[email protected]> Signed-off-by: David Sterba <[email protected]>
1 parent 7e4a3f7 commit 918cdf4

File tree

1 file changed

+4
-7
lines changed

1 file changed

+4
-7
lines changed

fs/btrfs/file-item.c

Lines changed: 4 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -871,7 +871,7 @@ int btrfs_csum_file_blocks(struct btrfs_trans_handle *trans,
871871
}
872872
ret = PTR_ERR(item);
873873
if (ret != -EFBIG && ret != -ENOENT)
874-
goto fail_unlock;
874+
goto out;
875875

876876
if (ret == -EFBIG) {
877877
u32 item_size;
@@ -929,7 +929,7 @@ int btrfs_csum_file_blocks(struct btrfs_trans_handle *trans,
929929
ret = btrfs_search_slot(trans, root, &file_key, path,
930930
csum_size, 1);
931931
if (ret < 0)
932-
goto fail_unlock;
932+
goto out;
933933

934934
if (ret > 0) {
935935
if (path->slots[0] == 0)
@@ -996,9 +996,9 @@ int btrfs_csum_file_blocks(struct btrfs_trans_handle *trans,
996996
ins_size);
997997
path->leave_spinning = 0;
998998
if (ret < 0)
999-
goto fail_unlock;
999+
goto out;
10001000
if (WARN_ON(ret != 0))
1001-
goto fail_unlock;
1001+
goto out;
10021002
leaf = path->nodes[0];
10031003
csum:
10041004
item = btrfs_item_ptr(leaf, path->slots[0], struct btrfs_csum_item);
@@ -1028,9 +1028,6 @@ int btrfs_csum_file_blocks(struct btrfs_trans_handle *trans,
10281028
out:
10291029
btrfs_free_path(path);
10301030
return ret;
1031-
1032-
fail_unlock:
1033-
goto out;
10341031
}
10351032

10361033
void btrfs_extent_item_to_extent_map(struct btrfs_inode *inode,

0 commit comments

Comments
 (0)