It needs to maintain dirty file inodes only if DATA_FLUSH is set.
Otherwise, let's avoid its overhead.

Signed-off-by: Jaegeuk Kim <jaeg...@kernel.org>
---
 fs/f2fs/checkpoint.c | 11 ++++++++---
 1 file changed, 8 insertions(+), 3 deletions(-)

diff --git a/fs/f2fs/checkpoint.c b/fs/f2fs/checkpoint.c
index 3da6499..d04113b 100644
--- a/fs/f2fs/checkpoint.c
+++ b/fs/f2fs/checkpoint.c
@@ -785,9 +785,11 @@ void update_dirty_page(struct inode *inode, struct page 
*page)
                        !S_ISLNK(inode->i_mode))
                return;
 
-       spin_lock(&sbi->inode_lock[type]);
-       __add_dirty_inode(inode, type);
-       spin_unlock(&sbi->inode_lock[type]);
+       if (type != FILE_INODE || test_opt(sbi, DATA_FLUSH)) {
+               spin_lock(&sbi->inode_lock[type]);
+               __add_dirty_inode(inode, type);
+               spin_unlock(&sbi->inode_lock[type]);
+       }
 
        inode_inc_dirty_pages(inode);
        SetPagePrivate(page);
@@ -803,6 +805,9 @@ void remove_dirty_inode(struct inode *inode)
                        !S_ISLNK(inode->i_mode))
                return;
 
+       if (type == FILE_INODE && !test_opt(sbi, DATA_FLUSH))
+               return;
+
        spin_lock(&sbi->inode_lock[type]);
        __remove_dirty_inode(inode, type);
        spin_unlock(&sbi->inode_lock[type]);
-- 
2.6.3

Reply via email to