tools/power/x86/intel-speed-select: v1.22 release

This version has fix for:
- Display of die ID and optimize array size for multi package
systems.
- Fix build warning with cross compiler

Signed-off

tools/power/x86/intel-speed-select: v1.22 release

This version has fix for:
- Display of die ID and optimize array size for multi package
systems.
- Fix build warning with cross compiler

Signed-off-by: Srinivas Pandruvada <[email protected]>

show more ...

tools/power/x86/intel-speed-select: Prefix header search path with sysroot

This helps when using a cross-compiler for building intel-speed-select,
currently, its hardcoded to pick libnl3 headers fro

tools/power/x86/intel-speed-select: Prefix header search path with sysroot

This helps when using a cross-compiler for building intel-speed-select,
currently, its hardcoded to pick libnl3 headers from build host which may
not be same as build target when cross compiling.

cc -print-sysroot will print nothing if compiler is configured without
a sysroot and result in same string as it is now.

Fixes errors with gcc configured with host include poisoning e.g.

cc1: error: include location "/usr/include/libnl3" is unsafe for
cross-compilation [-Werror=poison-system-directories]

Signed-off-by: Khem Raj <[email protected]>
[ srinivas: Changelog edits for checkpatch warning ]
Signed-off-by: Srinivas Pandruvada <[email protected]>

show more ...

tools/power/x86/intel-speed-select: Die ID for IO dies

Instead of displaying "-1" for IO dies, display "IO".

Signed-off-by: Srinivas Pandruvada <[email protected]>

tools/power/x86/intel-speed-select: Fix the condition to check multi die system

Even when there is no die exported by CPUID leaf 0x1F, the kernel version
after 6.9 will show non zero die_id in the s

tools/power/x86/intel-speed-select: Fix the condition to check multi die system

Even when there is no die exported by CPUID leaf 0x1F, the kernel version
after 6.9 will show non zero die_id in the sysfs. In that case maximum
die_id can still match maximum power domain ID. So the condition to check
if the power domain ID is same a die_id to prevent duplicate display
doesn't hold true.

The better condition is to check if the maximum die_id is more than the
maximum package_id. If the die_id is exposed by CPUID leaf 0x1F, the
maximum die_id will be more than maximum package_id.

With this change tracking of max_punit_id is not used, so remove storing
max_punit_id.

Signed-off-by: Srinivas Pandruvada <[email protected]>

show more ...

tools/power/x86/intel-speed-select: Prevent increasing MAX_DIE_PER_PACKAGE

In the function for_each_online_power_domain_in_set() to pick one CPU
from each power domain a three-dimensional array is u

tools/power/x86/intel-speed-select: Prevent increasing MAX_DIE_PER_PACKAGE

In the function for_each_online_power_domain_in_set() to pick one CPU
from each power domain a three-dimensional array is used, which assumes
that a package contains multiple dies, that means the die_id from
/sys/devices/system/cpu/cpu0/topology/die_id is only local to package.

If it is not unique, still there will be no functional issues in the
current generation of products, but the MAX_DIE_PER_PACKAGE will need to
be increased for future products with many packages.

After kernel version 6.9 die ID is unique system wide not per package.
Even if the CPU topology has no dies, the ID will still increment across
package. In this case the die_id in package 0 will be 0 and die_id in
package 1 will be 1 in a 2-package system.

Since the die count must be same for packages, just count the number of
dies in package 0 and calculate die index from
/sys/devices/system/cpu/cpu0/topology/die_id which is only unique within
a package.

In this way the array size
"int cpus[MAX_PACKAGE_COUNT][MAX_DIE_PER_PACKAGE][MAX_PUNIT_PER_DIE]"
doesn't have to increase with increasing package count.

No functional change is expected.

Signed-off-by: Srinivas Pandruvada <[email protected]>

show more ...