On Wed, Aug 26, 2015 at 11:50:34AM +1000, Dave Chinner wrote: > On Tue, Aug 25, 2015 at 11:33:39PM +0800, Peng Tao wrote: > > Now that a few file systems are adding clone functionality, namingly > > btrfs, NFS (later in the series) and XFS > > (ttp://oss.sgi.com/archives/xfs/2015-06/msg00407.html), it makes sense > > to pull the ioctl to common code. > > > > Add vfs_file_clone_range() helper and .clone_range file operation interface > > to allow underlying filesystems to clone between regular files. > > > > The change in do_vfs_ioctl() is defered to next patch where btrfs > > .clone_range is added, just so that we don't break btrfs CLONE ioctl > > with this patch. > > > > Cc: linux-btrfs@xxxxxxxxxxxxxxx > > Cc: linux-fsdevel@xxxxxxxxxxxxxxx > > Can you please cc the entire patch series to linux-fsdevel? > > Spraying random patches from a larger series to different lists is > not very nice - I can't really comment on this patch because I have > no idea what context it is being proposed in, what the problem being > solved is, how it is being used by existing filesystems, how it will > be used by your new changes, etc. > > You may have explained all this in patch 0 for the series, but > that hasn't been cc'd to linux-fsdevel.... Also, proposed test infrastructure for the clone_range functionality that Darrick is working on for XFS (and to validate existing btrfs functionality): http://oss.sgi.com/archives/xfs/2015-06/msg00479.html Cheers, Dave. -- Dave Chinner david@xxxxxxxxxxxxx -- 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