nilfs2: delete mark_inode_dirty in nilfs_new_inode
authorJiro SEKIBA <jir@unicus.jp>
Fri, 27 Nov 2009 10:41:06 +0000 (19:41 +0900)
committerRyusuke Konishi <konishi.ryusuke@lab.ntt.co.jp>
Fri, 27 Nov 2009 11:05:15 +0000 (20:05 +0900)
It is redundant to call mark_inode_dirty() in nilfs_new_inode() because
all caller of nilfs_new_inode() will call mark_inode_dirty()
after calling nilfs_new_inode() directly or indirectly in transaction.

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

index a16c179f2b9ac4ede3d48f736487f9dd4b244f47..ede03752b96fabe8926df58191066943a157d0ec 100644 (file)
@@ -323,7 +323,6 @@ struct inode *nilfs_new_inode(struct inode *dir, int mode)
                                    nilfs_init_acl(), proper cancellation of
                                    above jobs should be considered */
 
-       mark_inode_dirty(inode);
        return inode;
 
  failed_acl:
index ed02e886fa79ba8ad64d8f5180a3d1cc67e31541..01adda8232978be60f59eed92b9e6af8f09054ae 100644 (file)
@@ -188,7 +188,7 @@ static int nilfs_symlink(struct inode *dir, struct dentry *dentry,
                goto out_fail;
 
        /* mark_inode_dirty(inode); */
-       /* nilfs_new_inode() and page_symlink() do this */
+       /* page_symlink() do this */
 
        err = nilfs_add_nondir(dentry, inode);
 out: