Re: [RFCv3.1 00/11] xfstests: test the nfs/cifs/btrfs/xfs reflink/dedupe ioctls

2015-11-13 Thread Darrick J. Wong
On Fri, Nov 13, 2015 at 06:12:28AM -0800, Christoph Hellwig wrote: > On Fri, Nov 13, 2015 at 01:08:22AM -0800, Darrick J. Wong wrote: > > I think I got all the tests renumbered correctly and fixed all the other > > bugs. > > I'd like to send the updated patchpile to Dave tomorrow, but if you have

Re: [RFCv3.1 00/11] xfstests: test the nfs/cifs/btrfs/xfs reflink/dedupe ioctls

2015-11-13 Thread Christoph Hellwig
On Fri, Nov 13, 2015 at 01:08:22AM -0800, Darrick J. Wong wrote: > I think I got all the tests renumbered correctly and fixed all the other bugs. > I'd like to send the updated patchpile to Dave tomorrow, but if you have a few > spare cycles would you mind giving this a quick test to make sure I fi

Re: [RFCv3.1 00/11] xfstests: test the nfs/cifs/btrfs/xfs reflink/dedupe ioctls

2015-11-13 Thread Darrick J. Wong
On Thu, Nov 12, 2015 at 09:36:35AM -0800, Christoph Hellwig wrote: > On Thu, Nov 12, 2015 at 09:34:27AM -0800, Darrick J. Wong wrote: > > Bleargh, _require_*_dedupe forgot to check for ENOTTY output, so all the > > dedupe > > tests should have _notrun. > > > > Also, generic/806 was calling the wr

Re: [RFCv3.1 00/11] xfstests: test the nfs/cifs/btrfs/xfs reflink/dedupe ioctls

2015-11-12 Thread Christoph Hellwig
On Thu, Nov 12, 2015 at 09:34:27AM -0800, Darrick J. Wong wrote: > Bleargh, _require_*_dedupe forgot to check for ENOTTY output, so all the > dedupe > tests should have _notrun. > > Also, generic/806 was calling the wrong _require. > > I'll start renumbering tests; Christoph, did you see anythin

Re: [RFCv3.1 00/11] xfstests: test the nfs/cifs/btrfs/xfs reflink/dedupe ioctls

2015-11-12 Thread Darrick J. Wong
On Thu, Nov 12, 2015 at 04:51:15AM -0800, Christoph Hellwig wrote: > On Thu, Nov 12, 2015 at 01:07:56AM -0800, Christoph Hellwig wrote: > > Looks fine: > > > > Acked-by: Christoph Hellwig > > Actually I take this back. I had though this was the existing series > with my fixes, but this one stil

Re: [RFCv3.1 00/11] xfstests: test the nfs/cifs/btrfs/xfs reflink/dedupe ioctls

2015-11-12 Thread Christoph Hellwig
On Thu, Nov 12, 2015 at 01:07:56AM -0800, Christoph Hellwig wrote: > Looks fine: > > Acked-by: Christoph Hellwig Actually I take this back. I had though this was the existing series with my fixes, but this one still incorrectly assumes that if reflink works dedup works as well, leading to lots

Re: [RFCv3.1 00/11] xfstests: test the nfs/cifs/btrfs/xfs reflink/dedupe ioctls

2015-11-12 Thread Christoph Hellwig
Looks fine: Acked-by: Christoph Hellwig some of the free block range stuff might need minor fixups for certain file systems later, but I'd rather do this once all the patches are merged. -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majord.

[RFCv3.1 00/11] xfstests: test the nfs/cifs/btrfs/xfs reflink/dedupe ioctls

2015-11-11 Thread Darrick J. Wong
Hi all, This is part of the third revision of an RFC for adding to XFS support for tracking reverse-mappings of physical blocks to file and metadata; and support for mapping multiple file logical blocks to the same physical block, more commonly known as reflinking. This patchset aims to make xfst