diff options
| author | Naohiro Aota <naohiro.aota@wdc.com> | 2022-06-21 15:41:02 +0900 |
|---|---|---|
| committer | David Sterba <dsterba@suse.com> | 2022-07-25 17:45:38 +0200 |
| commit | aaafa1ebd6dbbe57d8b7587815b12e647b8b7b4a (patch) | |
| tree | 3dec6c555b91d5c1a8aacb1ed1fa5f0abc8ec117 /fs/btrfs/inode.c | |
| parent | 71aa147b4d9d81fa65afa6016f50d7818b64a54f (diff) | |
btrfs: replace unnecessary goto with direct return at cow_file_range()
The 'goto out' in cow_file_range() in the exit block are not necessary
and jump back. Replace them with return, while still keeping 'goto out'
in the main code.
Reviewed-by: Filipe Manana <fdmanana@suse.com>
Signed-off-by: Naohiro Aota <naohiro.aota@wdc.com>
Reviewed-by: David Sterba <dsterba@suse.com>
[ keep goto in the main code, update changelog ]
Signed-off-by: David Sterba <dsterba@suse.com>
Diffstat (limited to 'fs/btrfs/inode.c')
| -rw-r--r-- | fs/btrfs/inode.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/fs/btrfs/inode.c b/fs/btrfs/inode.c index 90b1c1df0943..fae0e8457edd 100644 --- a/fs/btrfs/inode.c +++ b/fs/btrfs/inode.c @@ -1413,7 +1413,7 @@ out_unlock: page_ops); start += cur_alloc_size; if (start >= end) - goto out; + return ret; } /* @@ -1425,7 +1425,7 @@ out_unlock: extent_clear_unlock_delalloc(inode, start, end, locked_page, clear_bits | EXTENT_CLEAR_DATA_RESV, page_ops); - goto out; + return ret; } /* |