diff options
| author | Dan Carpenter <[email protected]> | 2025-03-05 15:01:57 +0000 |
|---|---|---|
| committer | David Sterba <[email protected]> | 2025-03-18 19:35:49 +0000 |
| commit | c01b7114b809bbd3804d8a5b7e397f87f4ce52cb (patch) | |
| tree | 243ffc9e10fa4b88f03bbe6b8641515fc2320384 /fs/btrfs/dev-replace.c | |
| parent | btrfs: move btrfs_cleanup_bio() code into its single caller (diff) | |
| download | kernel-c01b7114b809bbd3804d8a5b7e397f87f4ce52cb.tar.gz kernel-c01b7114b809bbd3804d8a5b7e397f87f4ce52cb.zip | |
btrfs: return a literal instead of a variable in btrfs_init_dev_replace()
This is just a small clean up, it doesn't change how the code works.
Originally this code had a goto so we needed to set "ret = 0;" but now
it returns directly and so we can simplify it a bit by doing a
"return 0;" and removing the assignment.
Reviewed-by: Anand Jain <[email protected]>
Signed-off-by: Dan Carpenter <[email protected]>
Reviewed-by: David Sterba <[email protected]>
Signed-off-by: David Sterba <[email protected]>
Diffstat (limited to 'fs/btrfs/dev-replace.c')
| -rw-r--r-- | fs/btrfs/dev-replace.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/fs/btrfs/dev-replace.c b/fs/btrfs/dev-replace.c index 1a82e88ec5c1..53d7d85cb4be 100644 --- a/fs/btrfs/dev-replace.c +++ b/fs/btrfs/dev-replace.c @@ -103,7 +103,6 @@ no_valid_dev_replace_entry_found: "found replace target device without a valid replace item"); return -EUCLEAN; } - ret = 0; dev_replace->replace_state = BTRFS_IOCTL_DEV_REPLACE_STATE_NEVER_STARTED; dev_replace->cont_reading_from_srcdev_mode = @@ -120,7 +119,7 @@ no_valid_dev_replace_entry_found: dev_replace->tgtdev = NULL; dev_replace->is_valid = 0; dev_replace->item_needs_writeback = 0; - return ret; + return 0; } slot = path->slots[0]; eb = path->nodes[0]; |