On Friday, March 29, 2013 07:56:39 PM Viresh Kumar wrote:
Earlier definitions of affected and related cpus were: Related_cpus: CPUs which run at the same hardware frequency. Affected_cpus: CPUs which need to have their frequency coordinated by software.
These definitions were very confusing as they don't communicate the real difference between them.
Following are the new definitions of these variables: Related_cpus: All (Online & Offline) CPUs that run at the same hardware frequency. Affected_cpus: Online CPUs that run at the same hardware frequency.
Above definitions are more consistent with latest cpufreq core code.
Signed-off-by: Viresh Kumar viresh.kumar@linaro.org
Thomas Renninger is maintaining cpupower nowadays (added to CC). I won't get any cpupower changes without his ACK.
Thanks, Rafael
tools/power/cpupower/utils/cpufreq-info.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/tools/power/cpupower/utils/cpufreq-info.c b/tools/power/cpupower/utils/cpufreq-info.c index 28953c9..a81d4ec 100644 --- a/tools/power/cpupower/utils/cpufreq-info.c +++ b/tools/power/cpupower/utils/cpufreq-info.c @@ -247,7 +247,7 @@ static void debug_output_one(unsigned int cpu) cpus = cpufreq_get_related_cpus(cpu); if (cpus) {
printf(_(" CPUs which run at the same hardware frequency: "));
printf(_(" All (Online & Offline) CPUs that run at the same hardware frequency: "));@@ -258,7 +258,7 @@ static void debug_output_one(unsigned int cpu) cpus = cpufreq_get_affected_cpus(cpu); if (cpus) {
printf(_(" CPUs which need to have their frequency coordinated by software: "));
printf(_(" Online CPUs that run at the same hardware frequency: "));