xfs
[Top] [All Lists]

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

To: Christoph Hellwig <hch@xxxxxxxxxxxxx>
Subject: Re: [RFCv3.1 00/11] xfstests: test the nfs/cifs/btrfs/xfs reflink/dedupe ioctls
From: "Darrick J. Wong" <darrick.wong@xxxxxxxxxx>
Date: Fri, 13 Nov 2015 01:08:22 -0800
Cc: david@xxxxxxxxxxxxx, fstests@xxxxxxxxxxxxxxx, xfs@xxxxxxxxxxx, tao.peng@xxxxxxxxxxxxxxx, linux-ext4@xxxxxxxxxxxxxxx, Anna.Schumaker@xxxxxxxxxx, linux-btrfs@xxxxxxxxxxxxxxx
Delivered-to: xfs@xxxxxxxxxxx
In-reply-to: <20151112173635.GA9247@xxxxxxxxxxxxx>
References: <20151111192628.15056.6451.stgit@xxxxxxxxxxxxxxxx> <20151112090756.GA25685@xxxxxxxxxxxxx> <20151112125115.GA28822@xxxxxxxxxxxxx> <20151112173427.GC2217@xxxxxxxxxxxxxxxx> <20151112173635.GA9247@xxxxxxxxxxxxx>
User-agent: Mutt/1.5.21 (2010-09-15)
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 wrong _require.
> > 
> > I'll start renumbering tests; Christoph, did you see anything else?
> 
> Looks fine so far otherwise.

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 fixed
everything?

https://github.com/djwong/xfstests/tree/for-dave

--D

> --
> To unsubscribe from this list: send the line "unsubscribe fstests" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

<Prev in Thread] Current Thread [Next in Thread>