diff options
author | Jiro SEKIBA <jir@unicus.jp> | 2009-11-27 05:41:13 -0500 |
---|---|---|
committer | Ryusuke Konishi <konishi.ryusuke@lab.ntt.co.jp> | 2009-11-27 06:05:16 -0500 |
commit | 3534573b58fd7576d3dc8dd66a9973592ac08b2d (patch) | |
tree | 336143c737901da8612baac5e0434d80ee4aade8 /fs/nilfs2/namei.c | |
parent | 58d55471cb1911f7493aba7bf3b6b87ca91e4314 (diff) |
nilfs2: delete mark_inode_dirty in nilfs_delete_entry
Delete mark_inode_dirty() in nilfs_delete_entry() to reduce duplicate
mark_inode_dirty() calls both in nilfs_rename() and nilfs_delete_entry().
Signed-off-by: Jiro SEKIBA <jir@unicus.jp>
Signed-off-by: Ryusuke Konishi <konishi.ryusuke@lab.ntt.co.jp>
Diffstat (limited to 'fs/nilfs2/namei.c')
-rw-r--r-- | fs/nilfs2/namei.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/fs/nilfs2/namei.c b/fs/nilfs2/namei.c index d6aa8f4c804b..4237722b549c 100644 --- a/fs/nilfs2/namei.c +++ b/fs/nilfs2/namei.c | |||
@@ -334,6 +334,7 @@ static int nilfs_unlink(struct inode *dir, struct dentry *dentry) | |||
334 | err = nilfs_do_unlink(dir, dentry); | 334 | err = nilfs_do_unlink(dir, dentry); |
335 | 335 | ||
336 | if (!err) { | 336 | if (!err) { |
337 | mark_inode_dirty(dir); | ||
337 | mark_inode_dirty(dentry->d_inode); | 338 | mark_inode_dirty(dentry->d_inode); |
338 | err = nilfs_transaction_commit(dir->i_sb); | 339 | err = nilfs_transaction_commit(dir->i_sb); |
339 | } else | 340 | } else |
@@ -450,8 +451,8 @@ static int nilfs_rename(struct inode *old_dir, struct dentry *old_dentry, | |||
450 | if (dir_de) { | 451 | if (dir_de) { |
451 | nilfs_set_link(old_inode, dir_de, dir_page, new_dir); | 452 | nilfs_set_link(old_inode, dir_de, dir_page, new_dir); |
452 | drop_nlink(old_dir); | 453 | drop_nlink(old_dir); |
453 | mark_inode_dirty(old_dir); | ||
454 | } | 454 | } |
455 | mark_inode_dirty(old_dir); | ||
455 | mark_inode_dirty(old_inode); | 456 | mark_inode_dirty(old_inode); |
456 | 457 | ||
457 | err = nilfs_transaction_commit(old_dir->i_sb); | 458 | err = nilfs_transaction_commit(old_dir->i_sb); |