Date: Wed, 13 Feb 2002 22:41:00 -0500 From: Garance A Drosihn <drosih@rpi.edu> To: obrien@FreeBSD.org, Robert Watson <rwatson@FreeBSD.org> Cc: cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/ia64/conf GENERIC src/sys/i386/conf GENERIC src/sys/alpha/conf GENERIC Message-ID: <p05101400b890de305268@[128.113.24.47]> In-Reply-To: <20020213000544.A11749@dragon.nuxi.com> References: <200202130744.g1D7ixN41503@freefall.freebsd.org> <20020213000544.A11749@dragon.nuxi.com>
next in thread | previous in thread | raw e-mail | index | archive | help
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
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?p05101400b890de305268>