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

[Nate Lawson]

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).

Ah, I misread your email.  I see now what you said.

> The hardclock trace can probably just be axed entirely as it isn't all that 
> useful.

Ok, done.

-- 
Nate