On Sun, 2011-10-02 at 10:25 -0400, Christoph Hellwig wrote: > Directories are only updated transactionally, which means fsync only > needs to flush the log the inode is currently dirty, but not bother > with checking for dirty data, non-transaction updates, an most importanly > doesn't have to flush disk caches except as part of a transaction commit. > > While the first two optimizations can't easily be measured the latter > actually make a difference when doing lots of fsync that do not actually > have to commit the inode, e.g. becase an ealier fsync already pushed > the log far enough. > > The new xfs_dir_fsync is identifical to xfs_nfs_commit_metadata except > for the prototype, but I'm not sure creating a common helper for the > two is worth it given how simple the functions are. > > Signed-off-by: Christoph Hellwig <hch@xxxxxx> Looks good. Reviewed-by: Alex Elder <aelder@xxxxxxx> _______________________________________________ xfs mailing list xfs@xxxxxxxxxxx http://oss.sgi.com/mailman/listinfo/xfs