nilfs2: delete mark_inode_dirty in nilfs_commit_chunk
authorJiro SEKIBA <jir@unicus.jp>
Fri, 27 Nov 2009 10:41:12 +0000 (19:41 +0900)
committerRyusuke Konishi <konishi.ryusuke@lab.ntt.co.jp>
Fri, 27 Nov 2009 11:05:16 +0000 (20:05 +0900)
Delete mark_inode_dirty() in nilfs_commit_chunk(), for callers of
nilfs_commit_chunk() will call equivalent mark_inode_dirty()
after calling nilfs_commit_chunk().

Signed-off-by: Jiro SEKIBA <jir@unicus.jp>
Signed-off-by: Ryusuke Konishi <konishi.ryusuke@lab.ntt.co.jp>
fs/nilfs2/dir.c
fs/nilfs2/namei.c

index 173530d..693539b 100644 (file)
@@ -112,10 +112,8 @@ static void nilfs_commit_chunk(struct page *page,
 
        nr_dirty = nilfs_page_count_clean_buffers(page, from, to);
        copied = block_write_end(NULL, mapping, pos, len, len, page, NULL);
-       if (pos + copied > dir->i_size) {
+       if (pos + copied > dir->i_size)
                i_size_write(dir, pos + copied);
-               mark_inode_dirty(dir);
-       }
        if (IS_DIRSYNC(dir))
                nilfs_set_transaction_flag(NILFS_TI_SYNC);
        err = nilfs_set_file_dirty(sbi, dir, nr_dirty);
index d92e839..d6aa8f4 100644 (file)
@@ -258,7 +258,6 @@ static int nilfs_mkdir(struct inode *dir, struct dentry *dentry, int mode)
        inode->i_mapping->a_ops = &nilfs_aops;
 
        inc_nlink(inode);
-       mark_inode_dirty(inode);
 
        err = nilfs_make_empty(inode, dir);
        if (err)
@@ -268,6 +267,7 @@ static int nilfs_mkdir(struct inode *dir, struct dentry *dentry, int mode)
        if (err)
                goto out_fail;
 
+       mark_inode_dirty(inode);
        d_instantiate(dentry, inode);
 out:
        if (!err)