2007-12-18 19:28:23

by J. Bruce Fields

[permalink] [raw]
Subject: Re: [PATCH 01/38] svc: Add an svc transport class

On Tue, Dec 18, 2007 at 01:30:38PM -0600, Tom Tucker wrote:
>
> On Tue, 2007-12-18 at 09:04 -0600, Tom Tucker wrote:
> > On Tue, 2007-12-18 at 09:54 -0500, J. Bruce Fields wrote:
> > > On Mon, Dec 17, 2007 at 06:26:24PM -0600, Tom Tucker wrote:
> > > > I've coded all of the changes that you suggested and built a new
> > > > patchset. Besides the tests I outlined in my "test matrix" are there any
> > > > other tests you would like to see performed?
> > >
> > > krb5p testing might still be interesting.
> > >
> >
> > I'll add it to my matrix. It will take some time to set up though since
> > I don't have a KDC or anything set up. Are we under any time
> > constraints?
> >
> > > >
> > > > I could not reproduce the BUG_ON from Bull. I'm wondering if it's a PPC
> > > > issue :-\
> > >
> > > Hm, too bad, I'd like to understand that one.
> > >
>
> FYI, I have now reproduced this bug on an AMD64 box with the
> CITI_NFS4_ALL-1 tree.
>
> I am regression testing it on the new build have been running it about
> 2hrs without failure.

OK, so probably either something got fixed in the server-xprt-switch
patches since then, or something got fixed in mainline, or the problem
was in one of the other CITI_NFS4_ALL patches. Hm.

--b.