Commit 64bc6c2a authored by Nikolay Borisov's avatar Nikolay Borisov Committed by David Sterba

btrfs: Remove superfluous check form btrfs_remove_chunk

It's unnecessary to check map->stripes[i].dev for NULL given its value
is already set and dereferenced above the the check. No functional
changes.
Reviewed-by: default avatarQu Wenruo <wqu@suse.com>
Reviewed-by: default avatarAnand Jain <anand.jain@oracle.com>
Signed-off-by: default avatarNikolay Borisov <nborisov@suse.com>
Reviewed-by: default avatarDavid Sterba <dsterba@suse.com>
Signed-off-by: default avatarDavid Sterba <dsterba@suse.com>
parent f9085abf
...@@ -2848,13 +2848,11 @@ int btrfs_remove_chunk(struct btrfs_trans_handle *trans, u64 chunk_offset) ...@@ -2848,13 +2848,11 @@ int btrfs_remove_chunk(struct btrfs_trans_handle *trans, u64 chunk_offset)
mutex_unlock(&fs_info->chunk_mutex); mutex_unlock(&fs_info->chunk_mutex);
} }
if (map->stripes[i].dev) { ret = btrfs_update_device(trans, device);
ret = btrfs_update_device(trans, map->stripes[i].dev); if (ret) {
if (ret) { mutex_unlock(&fs_devices->device_list_mutex);
mutex_unlock(&fs_devices->device_list_mutex); btrfs_abort_transaction(trans, ret);
btrfs_abort_transaction(trans, ret); goto out;
goto out;
}
} }
} }
mutex_unlock(&fs_devices->device_list_mutex); mutex_unlock(&fs_devices->device_list_mutex);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment