Re: [PATCH v2] vfs: prevent copy_file_range to copy across devices

[Steve French]

On Thu, Feb 18, 2021 at 4:03 AM Amir Goldstein <amir73il@xxxxxxxxx> wrote:
>
> On Thu, Feb 18, 2021 at 9:42 AM Christoph Hellwig <hch@xxxxxxxxxxxxx> wrote:
> >
> > Looks good:
> >
> > Reviewed-by: Christoph Hellwig <hch@xxxxxx>
> >
> > This whole idea of cross-device copie has always been a horrible idea,
> > and I've been arguing against it since the patches were posted.
>
> Ok. I'm good with this v2 as well, but need to add the fallback to
> do_splice_direct()
> in nfsd_copy_file_range(), because this patch breaks it.

Interestingly, for ksmbd (cifsd) looks like they already do splice not
copy_file_range


-- 
Thanks,

Steve