linux.git/tools/power/cpupower/utils, branch v3.19

tools / cpupower: Fix no idle state information return value

2014-12-19T22:01:12+00:00

sysfs_get_idlestate_count() returns an unsigned int.  Returning -ENODEV
is not the right thing to do here, and in any case is handled the same
way as if there are no states found.

Signed-off-by: Prarit Bhargava 
Acked-by: Thomas Renninger 
Signed-off-by: Rafael J. Wysocki

tools / cpupower: Correctly detect if running as root

2014-12-19T22:01:03+00:00

Some operations, like frequency-set, need root privileges. However,
the way that this is detected is not correct. The getuid() is called,
while in fact geteuid() should be. This way we can allow
distributions or users to set SETUID flags on the cpupower binary if
they want to and let regular users change the cpu frequency governor.

Signed-off-by: Michal Privoznik 
Acked-by: Thomas Renninger 
Signed-off-by: Rafael J. Wysocki

Revert "tools: cpupower: fix return checks for sysfs_get_idlestate_count()"

2014-12-14T21:14:16+00:00

This reverts commit 16b7c275c055cc36218404b5d147be7f76575087.

My previous commit 16b7c275c055 ("tools: cpupower: fix return checks for
sysfs_get_idlestate_count()") was not correct.  After looking
at the changelog for cpupower I noticed that Thomas had changed the return of
sysfs_get_idlestate_count() to an unsigned int to simplify the code.  The
problem is really that both he (in his original change) and I (in my new
change) missed the obvious that sysfs_get_idlestate_count()
can't return -ENODEV.  It should just return 0 for "no c-states".

Fixes: 16b7c275c055 (tools: cpupower: fix return checks for ...)
Signed-off-by: Prarit Bhargava 
Signed-off-by: Rafael J. Wysocki

tools: cpupower: fix return checks for sysfs_get_idlestate_count()

2014-12-05T02:12:34+00:00

Red Hat and Fedora use a bug reporting tool that gathers data about
"broken" systems called sosreport.  Among other things, it includes the
output of 'cpupower idle-info'.  Executing 'cpupower idle-info' on a
system that has cpuidle disabled via 'cpuidle.off=1' results in a 300
second hang in the cpupower application.

ie)
[root@intel-brickland-05]# cpupower idle-info
Could not determine cpuidle driver

Analyzing CPU 0:
Number of idle states: -19
[hang]

The problem is that the cpupower code only checks for a zero return from
sysfs_get_idlestate_count().  The function can return -ENODEV (-19) as
above.  This patch fixes callers to sysfs_get_idlestate_count() to check
the right return values.

Signed-off-by: Prarit Bhargava 
Signed-off-by: Thomas Renninger 
Signed-off-by: Rafael J. Wysocki

cpupower: Remove redundant error check

2014-07-29T23:57:13+00:00

Remove double checks, and move the call to print_error to the
first check. Replace break by return, and return 0 on success.
The simplified version of the coccinelle semantic patch that
fixes this issue is as follows:

// 
@@
expression E; identifier pr; expression list es;
@@
for(...;...;...){
...
-	if (E) break;
+	if (E){
+		pr(es);
+		break;
+	}
...
}
- if(E) pr(es);
// 

Signed-off-by: Peter Senna Tschudin 
Signed-off-by: Thomas Renninger 
Signed-off-by: Rafael J. Wysocki

cpupower: mperf monitor: Correct use of ! and &

2014-07-29T23:57:12+00:00

In commit ae91d60ba88ef0bdb1b5e9b2363bd52fc45d2af7, a bug was fixed that
involved converting !x & y to !(x & y).  The code below shows the same
pattern, and thus should perhaps be fixed in the same way.

The Coccinelle semantic patch that makes this change is as follows:

// 
@@ expression E1,E2; @@
(
  !E1 & !E2
|
- !E1 & E2
+ !(E1 & E2)
)
// 

Signed-off-by: Himangi Saraogi 
Signed-off-by: Thomas Renninger 
Signed-off-by: Rafael J. Wysocki

PM / tools: cpupower: drop negativity check on unsigned value

2014-07-19T19:41:17+00:00

Link: https://bugzilla.kernel.org/show_bug.cgi?id=80621
Reported-by: David Binderman 
Signed-off-by: Andrey Utkin 
Signed-off-by: Rafael J. Wysocki

cpupower: Remove mc and smt power aware scheduler info/settings

2014-05-16T22:36:37+00:00

These kernel interfaces got removed by:

commit 8e7fbcbc22c12414bcc9dfdd683637f58fb32759
Author: Peter Zijlstra 
Date:   Mon Jan 9 11:28:35 2012 +0100

    sched: Remove stale power aware scheduling remnants and dysfunctional knobs

No need to further keep them as userspace configurations.

Signed-off-by: Thomas Renninger 
Signed-off-by: Rafael J. Wysocki

cpupower: cpupower info -b should return 0 on success, not the perf bias value

2014-05-16T22:36:37+00:00

Signed-off-by: Thomas Renninger 
Signed-off-by: Rafael J. Wysocki

cpupower: If root, try to load msr driver on x86 if /dev/cpu/0/msr is not available

2014-05-16T22:36:36+00:00

Signed-off-by: Thomas Renninger 
Signed-off-by: Rafael J. Wysocki