f2fs: clean up error handling in sanity_check_{compress_,}inode()

In sanity_check_{compress_,}inode(), it doesn't need to set SBI_NEED_FSCK
in each error case, instead, we can set the flag in do_read_inode() only
once when sanity_check_inode() fails.

Signed-off-by: Chao Yu <chao@kernel.org>
Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>

diff --git a/fs/f2fs/inode.c b/fs/f2fs/inode.c
index e81725c922cd4be887a4636096a18d65b0ad22f3..8ed5406ea204a347a3c6ae5691a6e57f25f358c9 100644 (file)
--- a/fs/f2fs/inode.c
+++ b/fs/f2fs/inode.c
@@ -214,7 +214,7 @@ static bool sanity_check_compress_inode(struct inode *inode,
                f2fs_warn(sbi,
                        "%s: inode (ino=%lx) has unsupported compress algorithm: %u, run fsck to fix",
                        __func__, inode->i_ino, ri->i_compress_algorithm);
-               goto err;
+               return false;
        }
        if (le64_to_cpu(ri->i_compr_blocks) >
                        SECTOR_TO_BLOCK(inode->i_blocks)) {
@@ -222,14 +222,14 @@ static bool sanity_check_compress_inode(struct inode *inode,
                        "%s: inode (ino=%lx) has inconsistent i_compr_blocks:%llu, i_blocks:%llu, run fsck to fix",
                        __func__, inode->i_ino, le64_to_cpu(ri->i_compr_blocks),
                        SECTOR_TO_BLOCK(inode->i_blocks));
-               goto err;
+               return false;
        }
        if (ri->i_log_cluster_size < MIN_COMPRESS_LOG_SIZE ||
                ri->i_log_cluster_size > MAX_COMPRESS_LOG_SIZE) {
                f2fs_warn(sbi,
                        "%s: inode (ino=%lx) has unsupported log cluster size: %u, run fsck to fix",
                        __func__, inode->i_ino, ri->i_log_cluster_size);
-               goto err;
+               return false;
        }
 
        clevel = le16_to_cpu(ri->i_compress_flag) >>
@@ -273,8 +273,6 @@ static bool sanity_check_compress_inode(struct inode *inode,
 err_level:
        f2fs_warn(sbi, "%s: inode (ino=%lx) has unsupported compress level: %u, run fsck to fix",
                  __func__, inode->i_ino, clevel);
-err:
-       set_sbi_flag(sbi, SBI_NEED_FSCK);
        return false;
 }
 
@@ -287,14 +285,12 @@ static bool sanity_check_inode(struct inode *inode, struct page *node_page)
 
        iblocks = le64_to_cpu(F2FS_INODE(node_page)->i_blocks);
        if (!iblocks) {
-               set_sbi_flag(sbi, SBI_NEED_FSCK);
                f2fs_warn(sbi, "%s: corrupted inode i_blocks i_ino=%lx iblocks=%llu, run fsck to fix.",
                          __func__, inode->i_ino, iblocks);
                return false;
        }
 
        if (ino_of_node(node_page) != nid_of_node(node_page)) {
-               set_sbi_flag(sbi, SBI_NEED_FSCK);
                f2fs_warn(sbi, "%s: corrupted inode footer i_ino=%lx, ino,nid: [%u, %u] run fsck to fix.",
                          __func__, inode->i_ino,
                          ino_of_node(node_page), nid_of_node(node_page));
@@ -303,7 +299,6 @@ static bool sanity_check_inode(struct inode *inode, struct page *node_page)
 
        if (f2fs_has_extra_attr(inode)) {
                if (!f2fs_sb_has_extra_attr(sbi)) {
-                       set_sbi_flag(sbi, SBI_NEED_FSCK);
                        f2fs_warn(sbi, "%s: inode (ino=%lx) is with extra_attr, but extra_attr feature is off",
                                  __func__, inode->i_ino);
                        return false;
@@ -311,7 +306,6 @@ static bool sanity_check_inode(struct inode *inode, struct page *node_page)
                if (fi->i_extra_isize > F2FS_TOTAL_EXTRA_ATTR_SIZE ||
                        fi->i_extra_isize < F2FS_MIN_EXTRA_ATTR_SIZE ||
                        fi->i_extra_isize % sizeof(__le32)) {
-                       set_sbi_flag(sbi, SBI_NEED_FSCK);
                        f2fs_warn(sbi, "%s: inode (ino=%lx) has corrupted i_extra_isize: %d, max: %zu",
                                  __func__, inode->i_ino, fi->i_extra_isize,
                                  F2FS_TOTAL_EXTRA_ATTR_SIZE);
@@ -321,7 +315,6 @@ static bool sanity_check_inode(struct inode *inode, struct page *node_page)
                        f2fs_has_inline_xattr(inode) &&
                        (!fi->i_inline_xattr_size ||
                        fi->i_inline_xattr_size > MAX_INLINE_XATTR_SIZE)) {
-                       set_sbi_flag(sbi, SBI_NEED_FSCK);
                        f2fs_warn(sbi, "%s: inode (ino=%lx) has corrupted i_inline_xattr_size: %d, max: %zu",
                                  __func__, inode->i_ino, fi->i_inline_xattr_size,
                                  MAX_INLINE_XATTR_SIZE);
@@ -335,7 +328,6 @@ static bool sanity_check_inode(struct inode *inode, struct page *node_page)
                                return false;
                }
        } else if (f2fs_sb_has_flexible_inline_xattr(sbi)) {
-               set_sbi_flag(sbi, SBI_NEED_FSCK);
                f2fs_warn(sbi, "%s: corrupted inode ino=%lx, run fsck to fix.",
                          __func__, inode->i_ino);
                return false;
@@ -343,31 +335,26 @@ static bool sanity_check_inode(struct inode *inode, struct page *node_page)
 
        if (!f2fs_sb_has_extra_attr(sbi)) {
                if (f2fs_sb_has_project_quota(sbi)) {
-                       set_sbi_flag(sbi, SBI_NEED_FSCK);
                        f2fs_warn(sbi, "%s: corrupted inode ino=%lx, wrong feature flag: %u, run fsck to fix.",
                                  __func__, inode->i_ino, F2FS_FEATURE_PRJQUOTA);
                        return false;
                }
                if (f2fs_sb_has_inode_chksum(sbi)) {
-                       set_sbi_flag(sbi, SBI_NEED_FSCK);
                        f2fs_warn(sbi, "%s: corrupted inode ino=%lx, wrong feature flag: %u, run fsck to fix.",
                                  __func__, inode->i_ino, F2FS_FEATURE_INODE_CHKSUM);
                        return false;
                }
                if (f2fs_sb_has_flexible_inline_xattr(sbi)) {
-                       set_sbi_flag(sbi, SBI_NEED_FSCK);
                        f2fs_warn(sbi, "%s: corrupted inode ino=%lx, wrong feature flag: %u, run fsck to fix.",
                                  __func__, inode->i_ino, F2FS_FEATURE_FLEXIBLE_INLINE_XATTR);
                        return false;
                }
                if (f2fs_sb_has_inode_crtime(sbi)) {
-                       set_sbi_flag(sbi, SBI_NEED_FSCK);
                        f2fs_warn(sbi, "%s: corrupted inode ino=%lx, wrong feature flag: %u, run fsck to fix.",
                                  __func__, inode->i_ino, F2FS_FEATURE_INODE_CRTIME);
                        return false;
                }
                if (f2fs_sb_has_compression(sbi)) {
-                       set_sbi_flag(sbi, SBI_NEED_FSCK);
                        f2fs_warn(sbi, "%s: corrupted inode ino=%lx, wrong feature flag: %u, run fsck to fix.",
                                  __func__, inode->i_ino, F2FS_FEATURE_COMPRESSION);
                        return false;
@@ -375,21 +362,18 @@ static bool sanity_check_inode(struct inode *inode, struct page *node_page)
        }
 
        if (f2fs_sanity_check_inline_data(inode)) {
-               set_sbi_flag(sbi, SBI_NEED_FSCK);
                f2fs_warn(sbi, "%s: inode (ino=%lx, mode=%u) should not have inline_data, run fsck to fix",
                          __func__, inode->i_ino, inode->i_mode);
                return false;
        }
 
        if (f2fs_has_inline_dentry(inode) && !S_ISDIR(inode->i_mode)) {
-               set_sbi_flag(sbi, SBI_NEED_FSCK);
                f2fs_warn(sbi, "%s: inode (ino=%lx, mode=%u) should not have inline_dentry, run fsck to fix",
                          __func__, inode->i_ino, inode->i_mode);
                return false;
        }
 
        if ((fi->i_flags & F2FS_CASEFOLD_FL) && !f2fs_sb_has_casefold(sbi)) {
-               set_sbi_flag(sbi, SBI_NEED_FSCK);
                f2fs_warn(sbi, "%s: inode (ino=%lx) has casefold flag, but casefold feature is off",
                          __func__, inode->i_ino);
                return false;
@@ -477,6 +461,7 @@ static int do_read_inode(struct inode *inode)
 
        if (!sanity_check_inode(inode, node_page)) {
                f2fs_put_page(node_page, 1);
+               set_sbi_flag(sbi, SBI_NEED_FSCK);
                f2fs_handle_error(sbi, ERROR_CORRUPTED_INODE);
                return -EFSCORRUPTED;
        }