summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorShilpasri G Bhat <shilpa.bhat@linux.vnet.ibm.com>2015-07-23 12:20:27 +0530
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>2015-07-23 20:40:51 +0200
commit2e5e8fd1ff9d802ab74df1ceaa5243da30491a45 (patch)
tree17ef07278e16b9ad1fd0ce440725788e05a52db8
parent52721d9d3334c1cb1f76219a161084094ec634dc (diff)
downloadop-kernel-dev-2e5e8fd1ff9d802ab74df1ceaa5243da30491a45.zip
op-kernel-dev-2e5e8fd1ff9d802ab74df1ceaa5243da30491a45.tar.gz
cpupower: Do not change the frequency of offline cpu
Check if the cpu is online before changing the frequency/governor of the cpu. Reported-by: Pavaman Subramaniyam <pavsubra@linux.vnet.ibm.com> Signed-off-by: Shilpasri G Bhat <shilpa.bhat@linux.vnet.ibm.com> Reviewed-by: Gautham R. Shenoy <ego@linux.vnet.ibm.com> Acked-by: Thomas Renninger <trenn@suse.com> Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
-rw-r--r--tools/power/cpupower/utils/cpufreq-set.c4
1 files changed, 4 insertions, 0 deletions
diff --git a/tools/power/cpupower/utils/cpufreq-set.c b/tools/power/cpupower/utils/cpufreq-set.c
index f656e58..4e21357 100644
--- a/tools/power/cpupower/utils/cpufreq-set.c
+++ b/tools/power/cpupower/utils/cpufreq-set.c
@@ -17,6 +17,7 @@
#include "cpufreq.h"
#include "helpers/helpers.h"
+#include "helpers/sysfs.h"
#define NORM_FREQ_LEN 32
@@ -318,6 +319,9 @@ int cmd_freq_set(int argc, char **argv)
cpufreq_cpu_exists(cpu))
continue;
+ if (sysfs_is_cpu_online(cpu) != 1)
+ continue;
+
printf(_("Setting cpu: %d\n"), cpu);
ret = do_one_cpu(cpu, &new_pol, freq, policychange);
if (ret) {
OpenPOWER on IntegriCloud