From owner-freebsd-arch Fri Feb 22 17:11:44 2002 Delivered-To: freebsd-arch@freebsd.org Received: from mail6.speakeasy.net (mail6.speakeasy.net [216.254.0.206]) by hub.freebsd.org (Postfix) with ESMTP id D0F3337B400 for ; Fri, 22 Feb 2002 17:11:40 -0800 (PST) Received: (qmail 2997 invoked from network); 23 Feb 2002 01:11:39 -0000 Received: from unknown (HELO laptop.baldwin.cx) ([65.91.137.227]) (envelope-sender ) by mail6.speakeasy.net (qmail-ldap-1.03) with SMTP for ; 23 Feb 2002 01:11:39 -0000 Message-ID: X-Mailer: XFMail 1.4.0 on FreeBSD X-Priority: 3 (Normal) Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 8bit MIME-Version: 1.0 In-Reply-To: Date: Fri, 22 Feb 2002 20:11:38 -0500 (EST) From: John Baldwin To: Julian Elischer Subject: Re: RE: that INVARIANT/ucred freeing stuff. Cc: arch@FreeBSD.org, Matthew Dillon Sender: owner-freebsd-arch@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.ORG On 22-Feb-02 Julian Elischer wrote: > OK here is my suggestion: > > We add extra code under DIAGNOSTIC > the code does: Hang on. The only reason this stuff is under DIAGNOSTIC is because of the performance. If you put it under DIAGNOSTIC, please don't make it all gross and complicated, just leave it simple. If you want to optimize it put it under INVARIANTS. Geez. Secondly, did anyone try pushing down Giant into crfree() for the case where we actually call free() and see if that helped the performacne? Giant thrashing is probably what the big problem was. In almost every case you would be just decrementing the refcount using the same logic used to justify this caching scheme. Oh well, for my testing I'll just stick this all back under INVARIANTS and push down Giant in my branch. Much cleaner and useful to me personally. If no one else wants to test this (td_ucred) code more power to them. -- John Baldwin <>< http://www.FreeBSD.org/~jhb/ "Power Users Use the Power to Serve!" - http://www.FreeBSD.org/ To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-arch" in the body of the message