From owner-freebsd-fs@FreeBSD.ORG Wed Mar 24 06:40:07 2010 Return-Path: Delivered-To: freebsd-fs@hub.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 92D96106564A for ; Wed, 24 Mar 2010 06:40:07 +0000 (UTC) (envelope-from gnats@FreeBSD.org) Received: from freefall.freebsd.org (freefall.freebsd.org [IPv6:2001:4f8:fff6::28]) by mx1.freebsd.org (Postfix) with ESMTP id 67CFE8FC13 for ; Wed, 24 Mar 2010 06:40:07 +0000 (UTC) Received: from freefall.freebsd.org (localhost [127.0.0.1]) by freefall.freebsd.org (8.14.4/8.14.4) with ESMTP id o2O6e7tg007547 for ; Wed, 24 Mar 2010 06:40:07 GMT (envelope-from gnats@freefall.freebsd.org) Received: (from gnats@localhost) by freefall.freebsd.org (8.14.4/8.14.4/Submit) id o2O6e7YG007546; Wed, 24 Mar 2010 06:40:07 GMT (envelope-from gnats) Date: Wed, 24 Mar 2010 06:40:07 GMT Message-Id: <201003240640.o2O6e7YG007546@freefall.freebsd.org> To: freebsd-fs@FreeBSD.org From: Kai Kockro Cc: Subject: Re: kern/144330: [nfs] mbuf leakage in nfsd with zfs X-BeenThere: freebsd-fs@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list Reply-To: Kai Kockro List-Id: Filesystems List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 24 Mar 2010 06:40:07 -0000 The following reply was made to PR kern/144330; it has been noted by GNATS. From: Kai Kockro To: Rick Macklem Cc: Daniel Braniss , Mikolaj Golub , Jeremy Chadwick , freebsd-fs@freebsd.org, bug-followup@freebsd.org, gerrit@pmp.uni-hannover.de Subject: Re: kern/144330: [nfs] mbuf leakage in nfsd with zfs Date: Wed, 24 Mar 2010 07:39:04 +0100 Hi, after 3 days with the first patch ( FreeBSD 8-STABLE AMD64, old nfsd ): netstat -m 5732/10528/16260 mbufs in use (current/cache/total) 4825/10131/14956/262144 mbuf clusters in use (current/cache/total/max) It looks very fine. I'll patch with the final corrections and then test aga= in. But why i have the same issues? I dont use UDP connects, only TCP?! Greetings, Kai Am Dienstag, 23. M=E4rz 2010 14:34:39 schrieb Rick Macklem: > On Tue, 23 Mar 2010, Daniel Braniss wrote: > >> I only have a FreeBSD client at this point, and it doesn't cause the > >> leak for nfsv3,udp for me here. > > > > my client is also FreeBSD 8.0, strange >=20 > I was already using the patch below when I tested and couldn't see it, > so I guess it now appears that the patch works. >=20 > >> Doug Rabson pointed out that there would be a leak for the "default:" > >> case too, although didn't know if that would occur in practice. > > > > it does! :-) > > > >> So, maybe you could test this variant of the patch (just in case that > >> was the slow leak...): > >> --- rpc/svc.c.sav 2010-03-21 18:46:20.000000000 -0400 > >> +++ rpc/svc.c 2010-03-22 19:00:17.000000000 -0400 > >> @@ -819,9 +819,11 @@ > >> free(r->rq_addr, M_SONAME); > >> r->rq_addr =3D NULL; > >> } > >> + m_freem(args); > >> goto call_done; > >> > >> default: > >> + m_freem(args); > >> goto call_done; > >> } > >> } > > > > that plugged it! > > see > > ftp://ftp.cs.huji.ac.il/users/danny/freebsd/mbuf-leak/store-02+++.ps >=20 > Good work with the testing. I'll get it committed and put it up on the > nfs patches page I have under http://people.freebsd.org/~rmacklem >=20 > > thanks to you for taking time off of your retirement :-) >=20 > I plan on doing quite a bit of FreeBSD/NFS stuff during it, rick >=20