On Thu, Feb 04, 2010 at 11:20:16AM -0600, bpm@xxxxxxx wrote: > > I think this is incorrect. datasync = 1 means to only write out > > metadata that is required to access the file data. > > My thinking there was that the XFS wsync mount option would have ensured > that the metadata that changed is already in the log. I wonder if when > using the wsync xfs mount option, could one drop the vfs_fsync > altogether? Looks to me like O_SYNC would also have taken care of any > metadata that is required to access the file data. Indeed, both write_inode_now and vfs_fsync will also cause data to be written. But my understanding of nfsd is that we manage the data writeout separately anyway and we care about the metadata here, which the placement of these calls would suggest: - nfsd_setattr for attribute updates - nfsd_create for creating a new file (of any type) - nfsd_link for adding a new link interestingly we use nfsd_sync_dir in all those same places, just for the parent directory. So if we want to stick to also writing data out in the child (which most of them time probably won't be dirty anyway) we really should make sure to use the same method for both, be that ->fsync or a new export operation. -- To unsubscribe from this list: send the line "unsubscribe linux-nfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
