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

Nate Lawson nate at root.org
Wed Nov 19 16:34:54 PST 2003


On Wed, 19 Nov 2003, John Baldwin wrote:
> One minor suggestion btw.  Could you fix the hw.acpi.cpu.cpu_cx_lowest
> to use the "C1", "C2", and "C3" strings for its user-visible interface
> instead of the direct integer?  Setting the value to C2 is a bit more
> intuitive than setting it to 1 to get C2.

I considered this before, however Cx is really a type, not a given state.
There can be more than 3 sleep states.  For instance, the IBM R40 has C1,
C2, C3, and C3 (4 states).  An index was the only unique way I could
specify the states.  The transition latency is also unique but setting a
state to 182 to get the last C3 state is even worse from a user
perspective.

-Nate


More information about the cvs-src mailing list