diff options
| author | Filipe Manana <fdmanana@suse.com> | 2023-12-04 19:20:28 +0300 |
|---|---|---|
| committer | David Sterba <dsterba@suse.com> | 2023-12-16 00:59:01 +0300 |
| commit | d224d2ef959a768fc88242224d8527e5f88789b6 (patch) | |
| tree | c85b3b73cb64b72a084f7f16187a64313438b0fe /fs/btrfs/extent_map.c | |
| parent | db9d94464a7acb149d014de1b0aa982b8c3856a6 (diff) | |
| download | linux-d224d2ef959a768fc88242224d8527e5f88789b6.tar.xz | |
btrfs: remove redundant value assignment at btrfs_add_extent_mapping()
At btrfs_add_extent_mapping(), in case add_extent_mapping() returned
-EEXIST, it's pointless to assign 0 to 'ret' since we will assign a value
to it shortly after, without 'ret' being used before that. So remove that
pointless assignment.
Signed-off-by: Filipe Manana <fdmanana@suse.com>
Reviewed-by: David Sterba <dsterba@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
Diffstat (limited to 'fs/btrfs/extent_map.c')
| -rw-r--r-- | fs/btrfs/extent_map.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/fs/btrfs/extent_map.c b/fs/btrfs/extent_map.c index 18a5c4332ed6..a3d69c943eec 100644 --- a/fs/btrfs/extent_map.c +++ b/fs/btrfs/extent_map.c @@ -586,8 +586,6 @@ int btrfs_add_extent_mapping(struct btrfs_fs_info *fs_info, if (ret == -EEXIST) { struct extent_map *existing; - ret = 0; - existing = search_extent_mapping(em_tree, start, len); trace_btrfs_handle_em_exist(fs_info, existing, em, start, len); |