f2fs: don't mark compressed inode dirty during f2fs_iget()
authorChao Yu <yuchao0@huawei.com>
Wed, 18 Mar 2020 11:40:45 +0000 (19:40 +0800)
committerJaegeuk Kim <jaegeuk@kernel.org>
Tue, 31 Mar 2020 03:46:23 +0000 (20:46 -0700)
- f2fs_iget
 - do_read_inode
  - set_inode_flag(, FI_COMPRESSED_FILE)
   - __mark_inode_dirty_flag(, true)

It's unnecessary, so let's just mark compressed inode dirty while
compressed inode conversion.

Signed-off-by: Chao Yu <yuchao0@huawei.com>
Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>
fs/f2fs/f2fs.h

index 3e727a6af08eb4b6561bb975772ba2b97404040a..e720054c9062790e057547be0652d9eeca16097e 100644 (file)
@@ -2573,7 +2573,6 @@ static inline void __mark_inode_dirty_flag(struct inode *inode,
        case FI_DATA_EXIST:
        case FI_INLINE_DOTS:
        case FI_PIN_FILE:
-       case FI_COMPRESSED_FILE:
                f2fs_mark_inode_dirty_sync(inode, true);
        }
 }
@@ -3837,6 +3836,7 @@ static inline void set_compress_context(struct inode *inode)
        F2FS_I(inode)->i_flags |= F2FS_COMPR_FL;
        set_inode_flag(inode, FI_COMPRESSED_FILE);
        stat_inc_compr_inode(inode);
+       f2fs_mark_inode_dirty_sync(inode, true);
 }
 
 static inline u64 f2fs_disable_compressed_file(struct inode *inode)
@@ -3853,6 +3853,7 @@ static inline u64 f2fs_disable_compressed_file(struct inode *inode)
        fi->i_flags &= ~F2FS_COMPR_FL;
        stat_dec_compr_inode(inode);
        clear_inode_flag(inode, FI_COMPRESSED_FILE);
+       f2fs_mark_inode_dirty_sync(inode, true);
        return 0;
 }