Re: [PATCH RFC 01/11] vfs: pull btrfs clone API to vfs layer

[Dave Chinner <david@fromorbit.com>]

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@vger.kernel.org
> Cc: linux-fsdevel@vger.kernel.org

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....

Cheers,

Dave.

PS: CC list spraying also plays badly with dup filters and per-list
procmail sorting that a lot of people use to manage incoming
feeds....
-- 
Dave Chinner
david@fromorbit.com