From owner-freebsd-current Tue Jun 22 13:58: 6 1999 Delivered-To: freebsd-current@freebsd.org Received: from helmholtz.salk.edu (helmholtz.salk.edu [198.202.70.34]) by hub.freebsd.org (Postfix) with ESMTP id DF74715344 for ; Tue, 22 Jun 1999 13:57:52 -0700 (PDT) (envelope-from bartol@salk.edu) Received: from eccles.salk.edu (eccles [198.202.70.120]) by helmholtz.salk.edu (8.7.5/8.7.3) with ESMTP id NAA03639; Tue, 22 Jun 1999 13:57:28 -0700 (PDT) Date: Tue, 22 Jun 1999 13:57:29 -0700 (PDT) From: Tom Bartol To: Julian Elischer Cc: Andrew Gallatin , dillon@backplane.com, freebsd-current@FreeBSD.ORG Subject: Re: NFS Test patch.. Anyone tested it? In-Reply-To: Message-ID: MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: owner-freebsd-current@FreeBSD.ORG Precedence: bulk X-Loop: FreeBSD.ORG I'll be happy to test a 3.2-STABLE version of the patches as soon as they become available. Tom On Tue, 22 Jun 1999, Julian Elischer wrote: > Ok I 've incorporated that into the patch set I have ready to commit.. > I've also been playing in 3.x > > The patches apply almost cleanly there, except for 3 small problems that > I'm fixing by hand.. > > I'll then make a 3.x version of the patches available. > Hopefully people can start testing that.. > > julian > > > On Tue, 22 Jun 1999, Andrew Gallatin wrote: > > > > > Julian Elischer writes: > > > Matthew Dillon wrote: > > > > > > > > Here's the location: > > > > > > > > http://www.backplane.com/FreeBSD4/ > > > > > > > > It's in the 'NFS bugs first found by David E. Cross' section. > > > > > > > > > > > > > I've looked it over a bit and can't see any major problems.. > > > and it definitly does fix some problems.. > > > > > > Has anyone tried it in a heavy server environment? > > > if so, any problems? It's running ok here but it's only > > > very lightly tested in our environment. > > > > > > I'd like to get it checked in if it really is an improvement.... > > > > > > > OK. First minor problem & fix: > > > > Solaris 2.7 (5.7 Generic_106541-04 sun4u sparc) hosts cannot do a V3 > > mount of an nfs server with these patches applied. V2 mounts still > > seem to work. > > > > It turns out that this is because when a solaris 2.7 host does an > > NFSv3 mount, it calls the null procedure & Matt's diffs expose a bug > > in nfssvc_nfsd. The following patch, applied in addtion to the diffs > > above, corrects the problem: > > > > Index: nfs_syscalls.c > > =================================================================== > > RCS file: /home/ncvs/src/sys/nfs/nfs_syscalls.c,v > > retrieving revision 1.49 > > diff -u -b -B -r1.49 nfs_syscalls.c > > --- nfs_syscalls.c 1999/04/27 11:17:52 1.49 > > +++ nfs_syscalls.c 1999/06/22 19:32:26 > > @@ -651,7 +651,7 @@ > > slp, nfsd->nfsd_procp, &mreq); > > if (mreq == NULL) > > break; > > - if (error) { > > + if (error != 0 && error != NFSERR_RETVOID) { > > if (nd->nd_procnum != NQNFSPROC_VACATED) > > nfsstats.srv_errs++; > > nfsrv_updatecache(nd, FALSE, mreq); > > > > > > > > Cheers, > > > > Drew > > ------------------------------------------------------------------------------ > > Andrew Gallatin, Sr Systems Programmer http://www.cs.duke.edu/~gallatin > > Duke University Email: gallatin@cs.duke.edu > > Department of Computer Science Phone: (919) 660-6590 > > > > > > To Unsubscribe: send mail to majordomo@FreeBSD.org > with "unsubscribe freebsd-current" in the body of the message > To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-current" in the body of the message