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

[John Baldwin]

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

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

-- 
John Baldwin <jhb at FreeBSD.org>  <><  http://www.FreeBSD.org/~jhb/
"Power Users Use the Power to Serve"  =  http://www.FreeBSD.org