Thomas Renninger
ea05dcce34
- Fix two bugs: memleak in idle-info and wrong idle state count OBS-URL: https://build.opensuse.org/package/show/hardware/cpupower?expand=0&rev=12
46 lines
1.4 KiB
Diff
46 lines
1.4 KiB
Diff
cpupower: Fix number of idle states
|
|
|
|
The number of idle states was wrong.
|
|
The POLL idle state (on X86) was missed out:
|
|
Number of idle states: 4
|
|
Available idle states: C1-NHM C3-NHM C6-NHM
|
|
|
|
While the POLL is not a real idle state, its
|
|
statistics should still be shown. It's now also
|
|
explained in a detailed manpage.
|
|
This should fix a bug of missing the first idle
|
|
state on other archs.
|
|
|
|
Signed-off-by: Thomas Renninger <trenn@suse.de>
|
|
CC: cpufreq@vger.kernel.org
|
|
CC: linux@dominikbrodowski.net
|
|
|
|
---
|
|
a/utils/cpuidle-info.c | 5 ++---
|
|
1 files changed, 2 insertions(+), 3 deletions(-)
|
|
|
|
diff --git a/utils/cpuidle-info.c b/utils/cpuidle-info.c
|
|
index b028267..e076beb 100644
|
|
--- a/utils/cpuidle-info.c
|
|
+++ b/utils/cpuidle-info.c
|
|
@@ -43,9 +43,8 @@ static void cpuidle_cpu_output(unsigned int cpu, int verbose)
|
|
}
|
|
|
|
printf(_("Number of idle states: %d\n"), idlestates);
|
|
-
|
|
printf(_("Available idle states:"));
|
|
- for (idlestate = 1; idlestate < idlestates; idlestate++) {
|
|
+ for (idlestate = 0; idlestate < idlestates; idlestate++) {
|
|
tmp = sysfs_get_idlestate_name(cpu, idlestate);
|
|
if (!tmp)
|
|
continue;
|
|
@@ -57,7 +56,7 @@ static void cpuidle_cpu_output(unsigned int cpu, int verbose)
|
|
if (!verbose)
|
|
return;
|
|
|
|
- for (idlestate = 1; idlestate < idlestates; idlestate++) {
|
|
+ for (idlestate = 0; idlestate < idlestates; idlestate++) {
|
|
tmp = sysfs_get_idlestate_name(cpu, idlestate);
|
|
if (!tmp)
|
|
continue;
|