Re: [PATCH] Btrfs: scrub: remove unnecessary variable set
On Fri, Mar 02, 2018 at 04:10:40PM -0700, Liu Bo wrote: > Variable "success" is only checked when !sctx->is_dev_replace. Though it's right, the code becomes less obvious at least to me that it's not missing something. There are several conditions and branches, one more explicit variable setting will not kill the performance but helps to understand the code flow. -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
[PATCH] Btrfs: scrub: remove unnecessary variable set
Variable "success" is only checked when !sctx->is_dev_replace. Signed-off-by: Liu Bo--- fs/btrfs/scrub.c | 1 - 1 file changed, 1 deletion(-) diff --git a/fs/btrfs/scrub.c b/fs/btrfs/scrub.c index e3203a1..1b5ce2f 100644 --- a/fs/btrfs/scrub.c +++ b/fs/btrfs/scrub.c @@ -1444,7 +1444,6 @@ static int scrub_handle_errored_block(struct scrub_block *sblock_to_check) page_num) != 0) { btrfs_dev_replace_stats_inc( _info->dev_replace.num_write_errors); - success = 0; } } else if (sblock_other) { ret = scrub_repair_page_from_good_copy(sblock_bad, -- 2.9.4 -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html