Commit 9df6d6f9 authored by Zhang Qilong's avatar Zhang Qilong Committed by Jaegeuk Kim

f2fs: remove redundant check in f2fs_sanity_check_cluster

It have checked "compressed" at the entry of
f2fs_sanity_check_cluster, just remove the
redundant check for better performance here.
Signed-off-by: default avatarZhang Qilong <zhangqilong3@huawei.com>
Reviewed-by: default avatarChao Yu <chao@kernel.org>
Signed-off-by: default avatarJaegeuk Kim <jaegeuk@kernel.org>
parent 049ea86c
...@@ -912,17 +912,15 @@ bool f2fs_sanity_check_cluster(struct dnode_of_data *dn) ...@@ -912,17 +912,15 @@ bool f2fs_sanity_check_cluster(struct dnode_of_data *dn)
reason = "[C|*|C|*]"; reason = "[C|*|C|*]";
goto out; goto out;
} }
if (compressed) { if (!__is_valid_data_blkaddr(blkaddr)) {
if (!__is_valid_data_blkaddr(blkaddr)) { if (!cluster_end)
if (!cluster_end) cluster_end = i;
cluster_end = i; continue;
continue; }
} /* [COMPR_ADDR, NULL_ADDR or NEW_ADDR, valid_blkaddr] */
/* [COMPR_ADDR, NULL_ADDR or NEW_ADDR, valid_blkaddr] */ if (cluster_end) {
if (cluster_end) { reason = "[C|N|N|V]";
reason = "[C|N|N|V]"; goto out;
goto out;
}
} }
} }
return false; return false;
......
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