From owner-cvs-all Wed Mar 20 13: 9:20 2002 Delivered-To: cvs-all@freebsd.org Received: from freefall.freebsd.org (freefall.FreeBSD.org [216.136.204.21]) by hub.freebsd.org (Postfix) with ESMTP id 1FA2637B405; Wed, 20 Mar 2002 13:09:10 -0800 (PST) Received: (from jhb@localhost) by freefall.freebsd.org (8.11.6/8.11.6) id g2KL9Am89620; Wed, 20 Mar 2002 13:09:10 -0800 (PST) (envelope-from jhb) Message-Id: <200203202109.g2KL9Am89620@freefall.freebsd.org> From: John Baldwin Date: Wed, 20 Mar 2002 13:09:09 -0800 (PST) To: cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: cvs commit: src/sys/alpha/alpha trap.c src/sys/i386/i386 trap.c src/sys/ia64/ia64 trap.c src/sys/powerpc/powerpc trap.c src/sys/sparc64/sparc64 trap.c src/sys/sys proc.h X-FreeBSD-CVS-Branch: HEAD Sender: owner-cvs-all@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.ORG jhb 2002/03/20 13:09:09 PST Modified files: sys/alpha/alpha trap.c sys/i386/i386 trap.c sys/ia64/ia64 trap.c sys/powerpc/powerpc trap.c sys/sparc64/sparc64 trap.c sys/sys proc.h Log: Change the way we ensure td_ucred is NULL if DIAGNOSTIC is defined. Instead of caching the ucred reference, just go ahead and eat the decerement and increment of the refcount. Now that Giant is pushed down into crfree(), we no longer have to get Giant in the common case. In the case when we are actually free'ing the ucred, we would normally free it on the next kernel entry, so the cost there is not new, just in a different place. This also removse td_cache_ucred from struct thread. This is still only done #ifdef DIAGNOSTIC. Tested on: i386, alpha Revision Changes Path 1.88 +6 -25 src/sys/alpha/alpha/trap.c 1.219 +6 -25 src/sys/i386/i386/trap.c 1.47 +6 -25 src/sys/ia64/ia64/trap.c 1.13 +3 -13 src/sys/powerpc/powerpc/trap.c 1.29 +6 -25 src/sys/sparc64/sparc64/trap.c 1.211 +0 -5 src/sys/sys/proc.h To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message