cvs commit: src/sys/sys ktr.h src/sys/kern kern_clock.c kern_switch.c

Scott Long scottl at samsco.org
Sat Dec 17 13:14:03 PST 2005


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



More information about the cvs-src mailing list