Commit 6ed9e363 authored by Srinivas Pandruvada's avatar Srinivas Pandruvada Committed by Hans de Goede

tools/power/x86/intel-speed-select: turbo-freq auto mode with SMT off

When SMT is disabled from kernel command line, sibling CPUs still
appears in the sysfs as offline CPUs. This is a problem when turbo-freq
is enabled in auto mode. They are still assigned to CLOS value
of 3 as they are still in the present CPU list. But they are not in the
sibling list of a CPU. When the CPU is a high priority CPU, because of
sibling it will be still set to CLOS to 3 as CLOS is assigned at core
level not at CPU level.

So, avoid setting CLOS 3 to offline CPU.
Signed-off-by: default avatarSrinivas Pandruvada <srinivas.pandruvada@linux.intel.com>
Signed-off-by: default avatarHans de Goede <hdegoede@redhat.com>
parent cf3b8e8f
...@@ -2066,6 +2066,9 @@ static void set_fact_enable(int arg) ...@@ -2066,6 +2066,9 @@ static void set_fact_enable(int arg)
if (!CPU_ISSET_S(i, present_cpumask_size, present_cpumask)) if (!CPU_ISSET_S(i, present_cpumask_size, present_cpumask))
continue; continue;
if (is_cpu_online(i) != 1)
continue;
set_isst_id(&id, i); set_isst_id(&id, i);
ret = set_clos_param(&id, 0, 0, 0, 0, 0xff); ret = set_clos_param(&id, 0, 0, 0, 0, 0xff);
if (ret) if (ret)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment