f2fs: avoid wrong count on dirty inodes
authorJaegeuk Kim <jaegeuk@kernel.org>
Thu, 2 Jun 2016 18:08:56 +0000 (11:08 -0700)
committerJaegeuk Kim <jaegeuk@kernel.org>
Tue, 7 Jun 2016 16:45:38 +0000 (09:45 -0700)
The number should be covered by spin_lock. Otherwise we can see wrong count
in f2fs_stat.

Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>
fs/f2fs/super.c

index 27f76819eef14a4819c8dae07f8449e0046720bb..9e754336e4e55e7bb37d9716ce071ff4a80c1011 100644 (file)
@@ -645,8 +645,8 @@ static void f2fs_dirty_inode(struct inode *inode, int flags)
        list_add_tail(&F2FS_I(inode)->gdirty_list,
                                &sbi->inode_list[DIRTY_META]);
        inc_page_count(sbi, F2FS_DIRTY_IMETA);
-       spin_unlock(&sbi->inode_lock[DIRTY_META]);
        stat_inc_dirty_inode(sbi, DIRTY_META);
+       spin_unlock(&sbi->inode_lock[DIRTY_META]);
 }
 
 void f2fs_inode_synced(struct inode *inode)
@@ -662,8 +662,8 @@ void f2fs_inode_synced(struct inode *inode)
        clear_inode_flag(inode, FI_DIRTY_INODE);
        clear_inode_flag(inode, FI_AUTO_RECOVER);
        dec_page_count(sbi, F2FS_DIRTY_IMETA);
-       spin_unlock(&sbi->inode_lock[DIRTY_META]);
        stat_dec_dirty_inode(F2FS_I_SB(inode), DIRTY_META);
+       spin_unlock(&sbi->inode_lock[DIRTY_META]);
 }
 
 static void f2fs_i_callback(struct rcu_head *head)