On Sat, Oct 12, 2013 at 12:55:06AM -0700, Christoph Hellwig wrote: > Currently fallocate always holds the iolock when calling into > xfs_change_file_space, while the ioctl path lets some of the lower level > functions take it, but leave it out in others. > > This patch makes sure the ioctl path also always holds the iolock and > thus introduces consistent locking for the preallocation operations while > simplifying the code and allowing to kill the now unused XFS_ATTR_NOLOCK > flag. > > Signed-off-by: Christoph Hellwig <hch@xxxxxx> Looks good to me. Reviewed-by: Dave Chinner <dchinner@xxxxxxxxxx> -- Dave Chinner david@xxxxxxxxxxxxx _______________________________________________ xfs mailing list xfs@xxxxxxxxxxx http://oss.sgi.com/mailman/listinfo/xfs
