Commit 09bf4aae authored by Ryusuke Konishi's avatar Ryusuke Konishi

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: default avatarRyusuke Konishi <konishi.ryusuke@lab.ntt.co.jp>
parent 30db4e6c
...@@ -402,18 +402,14 @@ int nilfs_bmap_test_and_clear_dirty(struct nilfs_bmap *bmap) ...@@ -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) 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); inode_add_bytes(bmap->b_inode, (1 << bmap->b_inode->i_blkbits) * n);
if (NILFS_MDT(bmap->b_inode)) if (!NILFS_MDT(bmap->b_inode))
nilfs_mdt_mark_dirty(bmap->b_inode);
else
mark_inode_dirty(bmap->b_inode); mark_inode_dirty(bmap->b_inode);
} }
void nilfs_bmap_sub_blocks(const struct nilfs_bmap *bmap, int n) 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); inode_sub_bytes(bmap->b_inode, (1 << bmap->b_inode->i_blkbits) * n);
if (NILFS_MDT(bmap->b_inode)) if (!NILFS_MDT(bmap->b_inode))
nilfs_mdt_mark_dirty(bmap->b_inode);
else
mark_inode_dirty(bmap->b_inode); mark_inode_dirty(bmap->b_inode);
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment