cvs commit: src/sys/kern kern_cpu.c

[Nate Lawson]

Nate Lawson wrote:
> njl         2005-08-30 04:45:32 UTC
> 
>   FreeBSD src repository
> 
>   Modified files:
>     sys/kern             kern_cpu.c 
>   Log:
>   Eliminate cpufreq levels for two cases that are less than optimal:
>   
>   1. Walk the absolute list in reverse to prefer duplicated levels that have
>   a lower absolute setting, i.e. 800 Mhz/50% is better than 1600 Mhz/25% even
>   though both have the same actual frequency.  This also removes the need to
>   check for already-modified levels since by definition, those will be added
>   later in the sorted list.
>   
>   2. Compare the absolute settings for derived levels and don't use the new
>   level if it's higher.  For example, a level of 800 Mhz/75% is preferable to
>   1600 Mhz/25% even though the latter has a lower total frequency.
>   
>   This work is based on a patch from the submitter but reworked by myself.
>   
>   Submitted by:   Tijl Coosemans (tijl/ulyssis.org)
>   
>   Revision  Changes    Path
>   1.20      +48 -47    src/sys/kern/kern_cpu.c

If this commit gives you problems, please set debug.cpufreq.verbose=1 
and send me the output along with sysctl dev.cpu.  It should reduce the 
levels the same as Tijl's patch.  I took a very slightly different 
implementation approach but the code is mostly his.  Thanks!

-- 
Nate