Re: [PATCH v2] cifs: fix that return -EINVAL when do dedupe operation

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



пт, 15 мар. 2019 г. в 22:39, Steve French <smfrench@xxxxxxxxx>:

>
> updated and tentatively placed in for-next - running the automated tests now
>
> On Fri, Mar 15, 2019 at 11:12 PM XiaoLi Feng <xifeng@xxxxxxxxxx> wrote:
> >
> > From: Xiaoli Feng <fengxiaoli0714@xxxxxxxxx>
> >
> > dedupe_file_range operations is combiled into remap_file_range.
> > But it's always skipped for dedupe operations in function
> > cifs_remap_file_range.
> >
> > Example to test:
> > Before this patch:
> >   # dd if=/dev/zero of=cifs/file bs=1M count=1
> >   # xfs_io -c "dedupe cifs/file 4k 64k 4k" cifs/file
> >   XFS_IOC_FILE_EXTENT_SAME: Invalid argument
> >
> > After this patch:
> >   # dd if=/dev/zero of=cifs/file bs=1M count=1
> >   # xfs_io -c "dedupe cifs/file 4k 64k 4k" cifs/file
> >   XFS_IOC_FILE_EXTENT_SAME: Operation not supported
> >
> > Influence for xfstests:
> > generic/091
> > generic/112
> > generic/127
> > generic/263
> > These tests report this error "do_copy_range:: Invalid
> > argument" instead of "FIDEDUPERANGE: Invalid argument".
> > Because there are still two bugs cause these test failed.
> > https://bugzilla.kernel.org/show_bug.cgi?id=202935
> > https://bugzilla.kernel.org/show_bug.cgi?id=202785

Hi Xiaoli,

About https://bugzilla.kernel.org/show_bug.cgi?id=202785: which kernel
version did you use for testing?

There is a fix merged recently that fixes a similar problem:
https://patchwork.ozlabs.org/patch/1052067/.

--
Best regards,
Pavel Shilovsky




[Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux