nilfs2: stop marking metadata inode dirty within btree operations
authorRyusuke Konishi <konishi.ryusuke@lab.ntt.co.jp>
Thu, 5 Nov 2009 06:53:27 +0000 (15:53 +0900)
committerRyusuke Konishi <konishi.ryusuke@lab.ntt.co.jp>
Fri, 20 Nov 2009 01:05:47 +0000 (10:05 +0900)
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>
fs/nilfs2/bmap.c

index 08834df6ec686911198ab4cc65d91b167109acc1..afc060abd5ab559cd84e3010d2e98337bbd7ed7f 100644 (file)
@@ -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);
 }