diff options
author | Ryusuke Konishi <konishi.ryusuke@lab.ntt.co.jp> | 2009-11-05 01:53:27 -0500 |
---|---|---|
committer | Ryusuke Konishi <konishi.ryusuke@lab.ntt.co.jp> | 2009-11-19 20:05:47 -0500 |
commit | 09bf4aae0a3c471b721c43e7bdb6132200d907b2 (patch) | |
tree | b2f8a8507abfd1a443a9a0965650899ce68f923f /fs | |
parent | 30db4e6c3d51a89e4923e525303f714e6508bbd0 (diff) |
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')
-rw-r--r-- | fs/nilfs2/bmap.c | 8 |
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) | |||
402 | void nilfs_bmap_add_blocks(const struct nilfs_bmap *bmap, int n) | 402 | void nilfs_bmap_add_blocks(const struct nilfs_bmap *bmap, int n) |
403 | { | 403 | { |
404 | inode_add_bytes(bmap->b_inode, (1 << bmap->b_inode->i_blkbits) * n); | 404 | inode_add_bytes(bmap->b_inode, (1 << bmap->b_inode->i_blkbits) * n); |
405 | if (NILFS_MDT(bmap->b_inode)) | 405 | if (!NILFS_MDT(bmap->b_inode)) |
406 | nilfs_mdt_mark_dirty(bmap->b_inode); | ||
407 | else | ||
408 | mark_inode_dirty(bmap->b_inode); | 406 | mark_inode_dirty(bmap->b_inode); |
409 | } | 407 | } |
410 | 408 | ||
411 | void nilfs_bmap_sub_blocks(const struct nilfs_bmap *bmap, int n) | 409 | void nilfs_bmap_sub_blocks(const struct nilfs_bmap *bmap, int n) |
412 | { | 410 | { |
413 | inode_sub_bytes(bmap->b_inode, (1 << bmap->b_inode->i_blkbits) * n); | 411 | inode_sub_bytes(bmap->b_inode, (1 << bmap->b_inode->i_blkbits) * n); |
414 | if (NILFS_MDT(bmap->b_inode)) | 412 | if (!NILFS_MDT(bmap->b_inode)) |
415 | nilfs_mdt_mark_dirty(bmap->b_inode); | ||
416 | else | ||
417 | mark_inode_dirty(bmap->b_inode); | 413 | mark_inode_dirty(bmap->b_inode); |
418 | } | 414 | } |
419 | 415 | ||