From owner-cvs-src@FreeBSD.ORG Sat Dec 17 21:14:03 2005 Return-Path: X-Original-To: cvs-src@FreeBSD.org Delivered-To: cvs-src@FreeBSD.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 757BD16A41F; Sat, 17 Dec 2005 21:14:03 +0000 (GMT) (envelope-from scottl@samsco.org) Received: from pooker.samsco.org (pooker.samsco.org [168.103.85.57]) by mx1.FreeBSD.org (Postfix) with ESMTP id D99AE43D46; Sat, 17 Dec 2005 21:14:00 +0000 (GMT) (envelope-from scottl@samsco.org) Received: from [192.168.254.14] (imini.samsco.home [192.168.254.14]) (authenticated bits=0) by pooker.samsco.org (8.13.4/8.13.4) with ESMTP id jBHLDwjW035687; Sat, 17 Dec 2005 14:13:59 -0700 (MST) (envelope-from scottl@samsco.org) Message-ID: <43A47F96.5040304@samsco.org> Date: Sat, 17 Dec 2005 14:13:58 -0700 From: Scott Long User-Agent: Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.7.7) Gecko/20050416 X-Accept-Language: en-us, en MIME-Version: 1.0 To: John Baldwin References: <200512170357.jBH3vAhh030893@repoman.freebsd.org> <200512171445.04475.jhb@freebsd.org> In-Reply-To: <200512171445.04475.jhb@freebsd.org> Content-Type: text/plain; charset=us-ascii; format=flowed Content-Transfer-Encoding: 7bit X-Spam-Status: No, score=-1.4 required=3.8 tests=ALL_TRUSTED autolearn=failed version=3.1.0 X-Spam-Checker-Version: SpamAssassin 3.1.0 (2005-09-13) on pooker.samsco.org Cc: cvs-src@FreeBSD.org, Nate Lawson , src-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/sys ktr.h src/sys/kern kern_clock.c kern_switch.c X-BeenThere: cvs-src@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: CVS commit messages for the src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 17 Dec 2005 21:14:03 -0000 John Baldwin wrote: > On Friday 16 December 2005 10:57 pm, Nate Lawson wrote: > >>njl 2005-12-17 03:57:10 UTC >> >> FreeBSD src repository >> >> Modified files: >> sys/sys ktr.h >> sys/kern kern_clock.c kern_switch.c >> Log: >> Clean up unused or poorly utilized KTR values. Remove KTR_FS, KTR_KGDB, >> and KTR_IO as they were never used. Remove KTR_CLK since it was only >> used for hardclock firing and use KTR_INTR there instead. Remove >> KTR_CRITICAL since it was only used for crit enter/exit and use >> KTR_CONTENTION instead. > > > Actually, I thought I had mentioned that KTR_CRITICAL should stay as it is > (well, and the larger thought about doing away with the entire bitmask > concept which no one responded to). critical_enter/exit are not related in > the least to KTR_CONTENTION which is used for MUTEX_PROFILING, nor do they > have anything at all to do with contention of any sort. If you must stick > them somewhere, put them in KTR_SCHED instead. I think scottl@ recently > added support to schedgraph for those traces anyway (though I'm not sure if > they are in his local tree or CVS). > Yes, it's in CVS. In the big scheme of things, it's probably a good idea to put KTR_CRITICAL in the same domain as KTR_SCHED. And yeah, it has nothing to do with KTR_CONTENTION. It was this part of the drive-by commit that irritated me most. A simple email saying, "I'm about to do this, please review," would have have been welcomed and likely not even ignored. Scott