From: "J. Bruce Fields" Subject: Re: [PATCH 01/38] svc: Add an svc transport class Date: Tue, 18 Dec 2007 14:28:20 -0500 Message-ID: <20071218192820.GU14783@fieldses.org> References: <20071213184506.GA29496@fieldses.org> <20071217174535.GA13515@fieldses.org> <1197937584.13463.44.camel@trinity.ogc.int> <20071218145419.GC14783@fieldses.org> <1197990256.13463.54.camel@trinity.ogc.int> <1198006238.13463.79.camel@trinity.ogc.int> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: NeilBrown , linux-nfs@vger.kernel.org To: Tom Tucker Return-path: Received: from mail.fieldses.org ([66.93.2.214]:54799 "EHLO fieldses.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750738AbXLRT2X (ORCPT ); Tue, 18 Dec 2007 14:28:23 -0500 In-Reply-To: <1198006238.13463.79.camel-SMNkleLxa3ZimH42XvhXlA@public.gmane.org> Sender: linux-nfs-owner@vger.kernel.org List-ID: 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.