From owner-cvs-all Wed Feb 13 19:41: 8 2002 Delivered-To: cvs-all@freebsd.org Received: from mail.rpi.edu (mail.rpi.edu [128.113.22.40]) by hub.freebsd.org (Postfix) with ESMTP id DCEA437B400; Wed, 13 Feb 2002 19:41:02 -0800 (PST) Received: from [128.113.24.47] (gilead.acs.rpi.edu [128.113.24.47]) by mail.rpi.edu (8.11.3/8.11.3) with ESMTP id g1E3f1G76666; Wed, 13 Feb 2002 22:41:01 -0500 Mime-Version: 1.0 X-Sender: drosih@mail.rpi.edu Message-Id: In-Reply-To: <20020213000544.A11749@dragon.nuxi.com> References: <200202130744.g1D7ixN41503@freefall.freebsd.org> <20020213000544.A11749@dragon.nuxi.com> Date: Wed, 13 Feb 2002 22:41:00 -0500 To: obrien@FreeBSD.org, Robert Watson From: Garance A Drosihn Subject: Re: cvs commit: src/sys/ia64/conf GENERIC src/sys/i386/conf GENERIC src/sys/alpha/conf GENERIC Cc: cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Content-Type: text/plain; charset="us-ascii" ; format="flowed" X-Scanned-By: MIMEDefang 2.3 (www dot roaringpenguin dot com slash mimedefang) 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 At 12:05 AM -0800 2/13/02, David O'Brien wrote: >On Tue, Feb 12, 2002 at 11:44:59PM -0800, Robert Watson wrote: >> rwatson 2002/02/12 23:44:59 PST >> >> Modified files: >> sys/ia64/conf GENERIC >> sys/i386/conf GENERIC >> sys/alpha/conf GENERIC >> Log: > > Remove WITNESS from GENERIC by default: as we grow more locks, > > this gets slower, and may be impeding adoption of -CURRENT by > > developers. We > >Back this out. This was discussed in the lists just this past week >and it was agreed to leave it. If you disagreed you should have >mentioned it in a public forum. It still might be a good idea to add one or two comment lines to the GENERIC kernel, right at the WITNESS define point, that adding WITNESS will more than double the CPU time it takes to do some operations, but that it would be appreciated if people would keep that option turned on if they can stand the performance hit. Say something which includes the magnitude of the performance hit. So, leave it turned on, but make it a lot easier for a developer to realize where the extra CPU time is going, so they don't blame it on current per se. -- Garance Alistair Drosehn = gad@eclipse.acs.rpi.edu Senior Systems Programmer or gad@freebsd.org Rensselaer Polytechnic Institute or drosih@rpi.edu To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message