diff options
author | Jiro SEKIBA <jir@unicus.jp> | 2009-11-27 05:41:06 -0500 |
---|---|---|
committer | Ryusuke Konishi <konishi.ryusuke@lab.ntt.co.jp> | 2009-11-27 06:05:15 -0500 |
commit | 9ca941d4b62e72571948efe5a73c563b4cacc98d (patch) | |
tree | 8c4a5cde6c2e626758c0aed85fe9cd7759747890 | |
parent | 0234576d041b9b2cc7043691ea61d2c2ca597aaa (diff) |
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>
-rw-r--r-- | fs/nilfs2/inode.c | 1 | ||||
-rw-r--r-- | fs/nilfs2/namei.c | 2 |
2 files changed, 1 insertions, 2 deletions
diff --git a/fs/nilfs2/inode.c b/fs/nilfs2/inode.c index a16c179f2b9a..ede03752b96f 100644 --- a/fs/nilfs2/inode.c +++ b/fs/nilfs2/inode.c | |||
@@ -323,7 +323,6 @@ struct inode *nilfs_new_inode(struct inode *dir, int mode) | |||
323 | nilfs_init_acl(), proper cancellation of | 323 | nilfs_init_acl(), proper cancellation of |
324 | above jobs should be considered */ | 324 | above jobs should be considered */ |
325 | 325 | ||
326 | mark_inode_dirty(inode); | ||
327 | return inode; | 326 | return inode; |
328 | 327 | ||
329 | failed_acl: | 328 | failed_acl: |
diff --git a/fs/nilfs2/namei.c b/fs/nilfs2/namei.c index ed02e886fa79..01adda823297 100644 --- a/fs/nilfs2/namei.c +++ b/fs/nilfs2/namei.c | |||
@@ -188,7 +188,7 @@ static int nilfs_symlink(struct inode *dir, struct dentry *dentry, | |||
188 | goto out_fail; | 188 | goto out_fail; |
189 | 189 | ||
190 | /* mark_inode_dirty(inode); */ | 190 | /* mark_inode_dirty(inode); */ |
191 | /* nilfs_new_inode() and page_symlink() do this */ | 191 | /* page_symlink() do this */ |
192 | 192 | ||
193 | err = nilfs_add_nondir(dentry, inode); | 193 | err = nilfs_add_nondir(dentry, inode); |
194 | out: | 194 | out: |