On Fri, 27 Nov 2009 19:41:05 +0900, Jiro SEKIBA <[email protected]> wrote:
> Hi,
> 
> This is a revised series of patches to reduce redandunt mark_inode_dirty 
> calls.
> 
> Many functions call mark_inode_dirty() whenever they change inode object.
> However, it is redundant to call mark_inode_dirty() to same inode object
> within a transaction.
> 
> Last patch also replaces all mark_inode_dirty() calls as
> nilfs_mark_inode_dirty(), to reduce deep function call tree.
> 
> Changes since v2 is:
> - deleted mark_inode_dirty(dir) in nilfs_mkdir()
> - moved mark_inod_dirty(dir) from nilfs_do_unlink() to nilfs_unlink()
> - correct diff stat ;-)
> 
>  fs/nilfs2/dir.c   |   24 ++++++---------
>  fs/nilfs2/inode.c |    7 ++--
>  fs/nilfs2/namei.c |   83 +++++++++++++++++++++++++++++++++-------------------
>  3 files changed, 66 insertions(+), 48 deletions(-)

Looks good to me. Applied on top of the for-next branch.

Thank you,
Ryusuke Konishi
_______________________________________________
users mailing list
[email protected]
https://www.nilfs.org/mailman/listinfo/users

Reply via email to