[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20151112004114.GO19199@dastard>
Date: Thu, 12 Nov 2015 11:41:14 +1100
From: Dave Chinner <david@...morbit.com>
To: "Darrick J. Wong" <darrick.wong@...cle.com>
Cc: Christoph Hellwig <hch@...radead.org>, fstests@...r.kernel.org,
linux-ext4@...r.kernel.org, Anna.Schumaker@...app.com,
linux-btrfs@...r.kernel.org, xfs@....sgi.com
Subject: Re: [RFCv3 00/12] xfstests: test the btrfs/xfs reflink/dedupe ioctls
On Mon, Nov 09, 2015 at 10:49:13AM -0800, Darrick J. Wong wrote:
> On Sun, Nov 08, 2015 at 11:59:26PM -0800, Christoph Hellwig wrote:
> > On Tue, Oct 06, 2015 at 10:12:57PM -0700, Darrick J. Wong wrote:
> > > * I don't have any interesting NFS/CIFS setups for test. :(
> >
> > I have a banrch with client and server support for NFSv4.2 CLONE
> > support:
> >
> > http://git.infradead.org/users/hch/pnfs.git/shortlog/refs/heads/reflink+clone
> >
> > For now you want to use btrfs on the server, as using reflinks on XFS
> > seems to be a little unstable over NFS.
>
> I found a few more bugs in the kernel-side implementation, which might explain
> that. I'm about to start working on making CoW less crappy, but I'll push all
> the patches out to github. (I wasn't planning on patchbombing again until
> December.)
>
> > > If you're going to start using this mess, you probably ought to just
> > > pull from my github trees for kernel[1], xfsprogs[2], and xfstests[3].
> > > They should just work with the btrfs that's in 4.3.
> > >
> > > Comments and questions are, as always, welcome.
> >
> > Any reason the groups are called clone? I don't really have an opinion
> > on clone vs reflink but given that the xfs_io command is reflink I'd
> > rather be consistent.
>
> The existing btrfs reflink tests were tagged in the 'clone' group prior to my
> patchset.
>
> > Otherwise I'd say get it merged ASAP, we can still fix up various
> > details later.
>
> I'll merge your patch and repost the whole pile of tests. I'm almost ready to
> send a pile of updates for the XFS on-disk structure document which add stuff
> about the v5 format, rmapbt, and reflink.
Darrick, can you renumber the xfstests against what is currently at
the head of the repo? If both you an Christoph need them working,
you may as well both patch against the main xfstests repo...
Cheers,
Dave.
--
Dave Chinner
david@...morbit.com
--
To unsubscribe from this list: send the line "unsubscribe linux-ext4" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists