summaryrefslogtreecommitdiff
path: root/fs/nilfs2/bmap.c
diff options
context:
space:
mode:
authorRyusuke Konishi <konishi.ryusuke@lab.ntt.co.jp>2009-11-05 09:53:27 +0300
committerRyusuke Konishi <konishi.ryusuke@lab.ntt.co.jp>2009-11-20 04:05:47 +0300
commit09bf4aae0a3c471b721c43e7bdb6132200d907b2 (patch)
treeb2f8a8507abfd1a443a9a0965650899ce68f923f /fs/nilfs2/bmap.c
parent30db4e6c3d51a89e4923e525303f714e6508bbd0 (diff)
downloadlinux-09bf4aae0a3c471b721c43e7bdb6132200d907b2.tar.xz
nilfs2: stop marking metadata inode dirty within btree operations
Since metadata file routines mark the inode dirty after they successfully changed bmap objects, nilfs_mdt_mark_dirty() calls in nilfs_bmap_add_blocks() and nilfs_bmap_sub_blocks() are redundant. This removes these overlapping calls from the bmap routines. Signed-off-by: Ryusuke Konishi <konishi.ryusuke@lab.ntt.co.jp>
Diffstat (limited to 'fs/nilfs2/bmap.c')
-rw-r--r--fs/nilfs2/bmap.c8
1 files changed, 2 insertions, 6 deletions
diff --git a/fs/nilfs2/bmap.c b/fs/nilfs2/bmap.c
index 08834df6ec68..afc060abd5ab 100644
--- a/fs/nilfs2/bmap.c
+++ b/fs/nilfs2/bmap.c
@@ -402,18 +402,14 @@ int nilfs_bmap_test_and_clear_dirty(struct nilfs_bmap *bmap)
void nilfs_bmap_add_blocks(const struct nilfs_bmap *bmap, int n)
{
inode_add_bytes(bmap->b_inode, (1 << bmap->b_inode->i_blkbits) * n);
- if (NILFS_MDT(bmap->b_inode))
- nilfs_mdt_mark_dirty(bmap->b_inode);
- else
+ if (!NILFS_MDT(bmap->b_inode))
mark_inode_dirty(bmap->b_inode);
}
void nilfs_bmap_sub_blocks(const struct nilfs_bmap *bmap, int n)
{
inode_sub_bytes(bmap->b_inode, (1 << bmap->b_inode->i_blkbits) * n);
- if (NILFS_MDT(bmap->b_inode))
- nilfs_mdt_mark_dirty(bmap->b_inode);
- else
+ if (!NILFS_MDT(bmap->b_inode))
mark_inode_dirty(bmap->b_inode);
}