Re: [PATCH 16/20] xfs: pass a 64-bit count argument to xfs_iomap_write_unwritten

J. Bruce Fields
2015-02-02T19:48:28Z

On Tue, Feb 03, 2015 at 06:43:00AM +1100, Dave Chinner wrote:
> On Tue, Feb 03, 2015 at 06:24:04AM +1100, Dave Chinner wrote:
> > On Mon, Feb 02, 2015 at 08:30:24AM +0100, Christoph Hellwig wrote:
> > > On Thu, Jan 29, 2015 at 03:52:32PM -0500, J. Bruce Fields wrote:
> > > > Who can give us ACKs on these last five fs/xfs patches? (And is it
> > > > going to cause trouble if they go in through the nfsd tree?)
> > >
> > >
> > > We'd need ACKs from Dave. He already has pulled in two patches so
> > > we might run into some conflicts. Maybe the best idea is to add the
> > > exportfs patch to both the XFS and nfsd trees, so each of them can
> > > pull in the rest? Or we could commit the two XFS preparation patches
> > > to both tree and get something that compiles and works in the nfsd
> > > tree.
> >
> > This patch has already been committed to the XFS repo.
>
> And it looks like I missed the sync transaction on growfs patch,
> too, so I'll commit that one later today.
>
> As to the pNFSD specific changes, I haven't really looked them over
> in any great detail yet. My main concern is that there are no
> specific regression tests for this yet, I'm not sure how we go about
> verifying it actually works properly and we don't inadvertantly
> break it in the future. Christoph?

Previously: http://lkml.kernel.org/r/20150106175611.GA16413@lst.de

> - any advice on testing? Is there was some simple
> virtual setup that would allow any loser with no special
> hardware (e.g., me) to check whether they've broken the
> block server?

Run two kvm VMs that share the same disk. Create an XFS
filesystem on the MDS, and export it. If the client has blkmapd
running (on Debian it needs to be started manually) it will use
pNFS for accessing the filesystem. Verify that using the
per-operation counters in /proc/self/mounstats. Repeat with
additional clients as nessecary.

Alternatively set up a simple iSCSI target using tgt or lio and
connect to it from multiple clients.

Which sounds reasonable to me, but I haven't tried to incorporate this
into my regression testing yet.

--b.