Bug 2635
Summary: | no C1 usage/C1 usage not counted on nforce2 | ||
---|---|---|---|
Product: | ACPI | Reporter: | Prakash Punnoor (prakash) |
Component: | Power-Processor | Assignee: | Len Brown (lenb) |
Status: | CLOSED DUPLICATE | ||
Severity: | normal | CC: | acpi-bugzilla, yi.zhu |
Priority: | P2 | ||
Hardware: | i386 | ||
OS: | Linux | ||
Kernel Version: | 2.6.6-rc3-mm1 based | Subsystem: | |
Regression: | --- | Bisected commit-id: | |
Attachments: |
dmidecode
acpidmp use acpi_idle even when only C1 is available |
Description
Prakash Punnoor
2004-05-04 02:57:37 UTC
Created attachment 2783 [details]
dmidecode
Created attachment 2784 [details]
acpidmp
reproduced on non-nforce2 system. Perhaps just to mention: It seems on VIA chipset it works. Checked on MVP3 and KT133. (Both having C1 and C2 state.) I think, this is more of a misinformation from ACPI C-state driver than anything else. C1 state is same as "halt". When a platform only supports C1 state, there is nothing special that ACPI can do to save power. It just lets the normal idle routine to handle it, by calling "halt". If you look at 'cat /proc/acpi/processor/CPU0/info', you should see "no" for power. That means there is nothing more ACPI can do to manage C-states. And we dont use acpi pm_idle in that case. If a platform supports C2 or C3, thats when acpi pm_idle becomes active, by changing across different C states. Oh OK, sound very reasonable. Nevertheless it would be nice if even in just C1 case, acpi would handle it and count the usage, so -like in my case- it would be easier to hunt after drivers which seem to prevent the C1 usage (-> higher idle temps.) by looking at the amount of C1 calls done. Created attachment 2855 [details]
use acpi_idle even when only C1 is available
Ok. Simple, effectively one line, patch attached here does that. Disclaimer - It doesn't save any power. It doesn't improve performance. Only thing it does is to use acpi_idle, even when just C1 state is available. And gives prettier numbers in /proc/acpi/processor/CPU1/power. Not sure whether incorporating this patch in acpi adds good enough value. Len can make that call. Thanks for the patch. Even it just gives statistics, I think it may be usefull for some people. Len, what do you think about the patch? Then we can close the bug. *** This bug has been marked as a duplicate of 4233 *** |