aboutsummaryrefslogtreecommitdiffstats
path: root/fs/btrfs/dev-replace.c
diff options
context:
space:
mode:
authorAnand Jain <anand.jain@oracle.com>2018-11-11 09:22:24 -0500
committerDavid Sterba <dsterba@suse.com>2018-12-17 08:51:35 -0500
commit53e62fb5a4aedeff05aa1efd277de32dc765f4bf (patch)
tree108451ec04b515f5e429a4a48b8d1ff2a5fbfd66 /fs/btrfs/dev-replace.c
parentfe97e2e173afb61f1bb889c45f9597102c9f4849 (diff)
btrfs: dev-replace: add explicit check for replace result "no error"
We recast the replace return status BTRFS_IOCTL_DEV_REPLACE_RESULT_SCRUB_INPROGRESS to 0, to indicate no error. And since BTRFS_IOCTL_DEV_REPLACE_RESULT_NO_ERROR should also return 0, which is also declared as 0, so we just return. Instead add it to the if statement so that there is enough clarity while reading the code. Signed-off-by: Anand Jain <anand.jain@oracle.com> Reviewed-by: David Sterba <dsterba@suse.com> Signed-off-by: David Sterba <dsterba@suse.com>
Diffstat (limited to 'fs/btrfs/dev-replace.c')
-rw-r--r--fs/btrfs/dev-replace.c5
1 files changed, 3 insertions, 2 deletions
diff --git a/fs/btrfs/dev-replace.c b/fs/btrfs/dev-replace.c
index a84affa425d4..85d93bd3b27a 100644
--- a/fs/btrfs/dev-replace.c
+++ b/fs/btrfs/dev-replace.c
@@ -539,8 +539,9 @@ int btrfs_dev_replace_by_ioctl(struct btrfs_fs_info *fs_info,
539 args->start.cont_reading_from_srcdev_mode); 539 args->start.cont_reading_from_srcdev_mode);
540 args->result = ret; 540 args->result = ret;
541 /* don't warn if EINPROGRESS, someone else might be running scrub */ 541 /* don't warn if EINPROGRESS, someone else might be running scrub */
542 if (ret == BTRFS_IOCTL_DEV_REPLACE_RESULT_SCRUB_INPROGRESS) 542 if (ret == BTRFS_IOCTL_DEV_REPLACE_RESULT_SCRUB_INPROGRESS ||
543 ret = 0; 543 ret == BTRFS_IOCTL_DEV_REPLACE_RESULT_NO_ERROR)
544 return 0;
544 545
545 return ret; 546 return ret;
546} 547}