From: Dave Chinner <david@fromorbit.com>
To: "Darrick J. Wong" <darrick.wong@oracle.com>
Cc: Christoph Hellwig <hch@infradead.org>,
fstests@vger.kernel.org, linux-ext4@vger.kernel.org,
Anna.Schumaker@netapp.com, linux-btrfs@vger.kernel.org,
xfs@oss.sgi.com
Subject: Re: [RFCv3 00/12] xfstests: test the btrfs/xfs reflink/dedupe ioctls
Date: Thu, 12 Nov 2015 11:41:14 +1100 [thread overview]
Message-ID: <20151112004114.GO19199@dastard> (raw)
In-Reply-To: <20151109184913.GB3255@birch.djwong.org>
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@fromorbit.com
prev parent reply other threads:[~2015-11-12 0:41 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-10-07 5:12 [RFCv3 00/12] xfstests: test the btrfs/xfs reflink/dedupe ioctls Darrick J. Wong
2015-10-07 5:13 ` [PATCH 01/12] xfs: fix merge errors in fuzzer tests Darrick J. Wong
2015-10-07 5:13 ` [PATCH 02/12] btrfs: move btrfs reflink tests to generic Darrick J. Wong
2015-10-07 5:13 ` [PATCH 03/12] generic/80[0-2]: support xfs in addition to btrfs Darrick J. Wong
2015-10-15 9:19 ` Christoph Hellwig
2015-10-15 14:41 ` Darrick J. Wong
2015-10-07 5:13 ` [PATCH 04/12] reflink: basic tests of the reflink and dedupe ioctls Darrick J. Wong
2015-10-07 5:13 ` [PATCH 05/12] reflink: test CoW behaviors of reflinked files Darrick J. Wong
2015-10-07 5:13 ` [PATCH 06/12] reflink: test the various fallocate modes Darrick J. Wong
2015-10-07 5:13 ` [PATCH 07/12] reflink: concurrent operations tests Darrick J. Wong
2015-10-07 5:13 ` [PATCH 08/12] reflink: test accuracy of free block counts Darrick J. Wong
2015-10-07 5:14 ` [PATCH 09/12] reflink: test error conditions due to bad inputs Darrick J. Wong
2015-10-07 5:14 ` [PATCH 10/12] xfs: test xfs-specific reflink pieces Darrick J. Wong
2015-10-07 5:14 ` [PATCH 11/12] reflink: test what happens when we hit resource limits Darrick J. Wong
2015-10-07 5:14 ` [PATCH 12/12] reflink: test that CoW writes fail when we're out of space Darrick J. Wong
[not found] ` <9163f8156b2742a0b003ba9fa0a26258@nebula-exfe-01.nebula.local>
2015-10-08 12:33 ` [PATCH 03/12] generic/80[0-2]: support xfs in addition to btrfs Ari Sundholm
2015-10-09 18:58 ` Darrick J. Wong
2015-11-09 7:59 ` [RFCv3 00/12] xfstests: test the btrfs/xfs reflink/dedupe ioctls Christoph Hellwig
2015-11-09 18:49 ` Darrick J. Wong
2015-11-11 13:56 ` Christoph Hellwig
2015-11-12 0:41 ` Dave Chinner [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20151112004114.GO19199@dastard \
--to=david@fromorbit.com \
--cc=Anna.Schumaker@netapp.com \
--cc=darrick.wong@oracle.com \
--cc=fstests@vger.kernel.org \
--cc=hch@infradead.org \
--cc=linux-btrfs@vger.kernel.org \
--cc=linux-ext4@vger.kernel.org \
--cc=xfs@oss.sgi.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).