cvs commit: src/sys/dev/acpica acpi.c acpi_cpu.c

[Nate Lawson]

On Thu, 20 Nov 2003, Marcel Moolenaar wrote:
> On Thu, Nov 20, 2003 at 03:40:58PM -0800, Nate Lawson wrote:
> > No good ideas over here.  Took me a while to even come up with
> > hw.acpi.cpu.cx_lowest.  You want to indicate higher states will be also
> > used as determined by the scheduling algorithm and that states with higher
> > indices provide more power savings but also more latency.  cx_lowest_idx
> > seems kinda lame.  Suggestions?
>
> Maybe make the Cx state type an "unimportant" attribute?
>
> MIB-like:
>
> hw.acpi.cpu.cx.count=4
> hw.acpi.cpu.cx.lowest=0
> hw.acpi.cpu.cx.0.type=C1
> hw.acpi.cpu.cx.0.latency=1
> hw.acpi.cpu.cx.0.longsleep=x
> hw.acpi.cpu.cx.0.shortsleep=y
> hw.acpi.cpu.cx.1.type=C2
> 	:
> hw.acpu.cpu.cx.2.type=C3
> 	:
> hw.acpu.cpu.cx.3.type=C3
> 	:
>
> Something like this makes the sysctl variables less magical, better
> to parse mechanically and puts the emphasis on the index. It's more
> verbose though...

Nice, but too verbose for my tastes.  Since this is something that will
only be looked at by someone while tuning the system, I think more compact
is better.  Also it's really good to be able to do sysctl hw.acpi.cpu and
get only 5-10 lines.  I plan to reduce the verboseness of the throttling
entries as well once more of the power control/devd stuff is committed.

-Nate