nilfs2: delete mark_inode_dirty in nilfs_new_inode

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>

diff --git a/fs/nilfs2/inode.c b/fs/nilfs2/inode.c
index a16c179f2b9ac4ede3d48f736487f9dd4b244f47..ede03752b96fabe8926df58191066943a157d0ec 100644 (file)
--- a/fs/nilfs2/inode.c
+++ b/fs/nilfs2/inode.c
@@ -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:

diff --git a/fs/nilfs2/namei.c b/fs/nilfs2/namei.c
index ed02e886fa79ba8ad64d8f5180a3d1cc67e31541..01adda8232978be60f59eed92b9e6af8f09054ae 100644 (file)
--- a/fs/nilfs2/namei.c
+++ b/fs/nilfs2/namei.c
@@ -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: