It have checked "compressed" at the entry of
f2fs_sanity_check_cluster, just remove the
redundant check for better performance here.

Signed-off-by: Zhang Qilong <zhangqilo...@huawei.com>
---
 fs/f2fs/compress.c | 20 +++++++++-----------
 1 file changed, 9 insertions(+), 11 deletions(-)

diff --git a/fs/f2fs/compress.c b/fs/f2fs/compress.c
index 6baaff4c52ba..c16bab5bd600 100644
--- a/fs/f2fs/compress.c
+++ b/fs/f2fs/compress.c
@@ -912,17 +912,15 @@ bool f2fs_sanity_check_cluster(struct dnode_of_data *dn)
                        reason = "[C|*|C|*]";
                        goto out;
                }
-               if (compressed) {
-                       if (!__is_valid_data_blkaddr(blkaddr)) {
-                               if (!cluster_end)
-                                       cluster_end = i;
-                               continue;
-                       }
-                       /* [COMPR_ADDR, NULL_ADDR or NEW_ADDR, valid_blkaddr] */
-                       if (cluster_end) {
-                               reason = "[C|N|N|V]";
-                               goto out;
-                       }
+               if (!__is_valid_data_blkaddr(blkaddr)) {
+                       if (!cluster_end)
+                               cluster_end = i;
+                       continue;
+               }
+               /* [COMPR_ADDR, NULL_ADDR or NEW_ADDR, valid_blkaddr] */
+               if (cluster_end) {
+                       reason = "[C|N|N|V]";
+                       goto out;
                }
        }
        return false;
-- 
2.25.1



_______________________________________________
Linux-f2fs-devel mailing list
Linux-f2fs-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel

Reply via email to