diff options
| author |   Len Brown <len.brown@intel.com> | 2025-08-09 16:31:31 -0400 |
|---|---|---|
| committer | Len Brown <len.brown@intel.com> | 2025-08-09 21:24:46 -0400 |
| commit | e60a13bcef206795d3ddf82f130fe8f570176d06 (patch) | |
| tree | 61d064ca4bec2792434f42c176750f22ac183577 | |
| parent | tools/power turbostat: standardize PER_THREAD_PARAMS (diff) | |
| download | wireguard-linux-e60a13bcef206795d3ddf82f130fe8f570176d06.tar.xz wireguard-linux-e60a13bcef206795d3ddf82f130fe8f570176d06.zip | |
tools/power turbostat: Handle non-root legacy-uncore sysfs permissions
/sys/devices/system/cpu/intel_uncore_frequency/package_X_die_Y/
may be readable by all, but
/sys/devices/system/cpu/intel_uncore_frequency/package_X_die_Y/current_freq_khz
may be readable only by root.
Non-root turbostat users see complaints in this scenario.
Fail probe of the interface if we can't read current_freq_khz.
Reported-by: Artem Bityutskiy <artem.bityutskiy@linux.intel.com>
Original-patch-by: Zhang Rui <rui.zhang@intel.com>
Signed-off-by: Len Brown <len.brown@intel.com>
| -rw-r--r-- | tools/power/x86/turbostat/turbostat.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/tools/power/x86/turbostat/turbostat.c b/tools/power/x86/turbostat/turbostat.c index d65a504a6c5f..76f4093959d5 100644 --- a/tools/power/x86/turbostat/turbostat.c +++ b/tools/power/x86/turbostat/turbostat.c @@ -7002,7 +7002,8 @@ static void probe_intel_uncore_frequency_legacy(void) sprintf(path_base, "/sys/devices/system/cpu/intel_uncore_frequency/package_%02d_die_%02d", i, j); - if (access(path_base, R_OK)) + sprintf(path, "%s/current_freq_khz", path_base); + if (access(path, R_OK)) continue; BIC_PRESENT(BIC_UNCORE_MHZ); |