From: Piotr Kubaj <piotr.kubaj@intel.com>
To: ltp@lists.linux.it
Cc: helena.anna.dubel@intel.com, tomasz.ossowski@intel.com,
rafael.j.wysocki@intel.com, daniel.niestepski@intel.com
Subject: [LTP] [PATCH] cpufreq.c: add new test for cpufreq sysfs interface validation
Date: Wed, 22 Apr 2026 12:35:43 +0200 [thread overview]
Message-ID: <20260422103542.20933-2-piotr.kubaj@intel.com> (raw)
Runs various sanity checks for intel_pstate cpufreq sysfs interface.
Checks whether intel_pstate driver is used, whether it is possible
to switch scaling_governor to powersave and performance,
whether it is possible to disable turbo frequency and
whether it is possible to change scaling_min_freq and scaling_max_freq
to its minimal valid and maximal valid values.
Signed-off-by: Piotr Kubaj <piotr.kubaj@intel.com>
---
runtest/power_management_tests | 1 +
testcases/kernel/power_management/.gitignore | 1 +
testcases/kernel/power_management/cpufreq.c | 355 +++++++++++++++++++
3 files changed, 357 insertions(+)
create mode 100644 testcases/kernel/power_management/cpufreq.c
diff --git a/runtest/power_management_tests b/runtest/power_management_tests
index 884e615cd..f235f8f0b 100644
--- a/runtest/power_management_tests
+++ b/runtest/power_management_tests
@@ -1,4 +1,5 @@
#POWER_MANAGEMENT
+cpufreq cpufreq
runpwtests01 runpwtests01.sh
runpwtests02 runpwtests02.sh
runpwtests03 runpwtests03.sh
diff --git a/testcases/kernel/power_management/.gitignore b/testcases/kernel/power_management/.gitignore
index 0c2a3ed4b..d0d7336ad 100644
--- a/testcases/kernel/power_management/.gitignore
+++ b/testcases/kernel/power_management/.gitignore
@@ -1 +1,2 @@
+cpufreq
pm_get_sched_values
diff --git a/testcases/kernel/power_management/cpufreq.c b/testcases/kernel/power_management/cpufreq.c
new file mode 100644
index 000000000..379aac0fe
--- /dev/null
+++ b/testcases/kernel/power_management/cpufreq.c
@@ -0,0 +1,355 @@
+// SPDX-License-Identifier: GPL-2.0-or-later
+/*
+ * Copyright (c) 2026 Piotr Kubaj <piotr.kubaj@intel.com>
+ */
+
+/*\
+ * Runs various sanity checks for intel_pstate cpufreq sysfs interface.
+ *
+ * Checks whether intel_pstate driver is used, whether it is possible to switch
+ * scaling_governor to powersave and performance, whether it is possible
+ * to disable turbo frequency and whether it is possible to change
+ * scaling_min_freq and scaling_max_freq to its minimal valid and maximal valid
+ * values.
+ */
+
+#include "tst_test.h"
+
+static bool *online;
+static char path[PATH_MAX], intel_pstate_status[16];
+static int no_turbo, nproc;
+static long *previous_scaling_max_freq, *previous_scaling_min_freq;
+
+static void cleanup(void)
+{
+ SAFE_FILE_PRINTF("/sys/devices/system/cpu/intel_pstate/no_turbo", "%d", no_turbo);
+ SAFE_FILE_PRINTF("/sys/devices/system/cpu/intel_pstate/status", intel_pstate_status);
+ for (int i = 0; i < nproc; i++) {
+ if (!online[i])
+ continue;
+
+ snprintf(path, sizeof(path), "/sys/devices/system/cpu/cpu%d/cpufreq/scaling_max_freq", i);
+ SAFE_FILE_PRINTF(path, "%ld", previous_scaling_max_freq[i]);
+
+ snprintf(path, sizeof(path), "/sys/devices/system/cpu/cpu%d/cpufreq/scaling_min_freq", i);
+ SAFE_FILE_PRINTF(path, "%ld", previous_scaling_min_freq[i]);
+ }
+ free(online);
+ free(previous_scaling_max_freq);
+ free(previous_scaling_min_freq);
+}
+
+static void setup(void)
+{
+ SAFE_FILE_SCANF("/sys/devices/system/cpu/intel_pstate/no_turbo", "%d", &no_turbo);
+ SAFE_FILE_SCANF("/sys/devices/system/cpu/intel_pstate/status", "%15s", intel_pstate_status);
+ SAFE_FILE_PRINTF("/sys/devices/system/cpu/intel_pstate/status", "active");
+
+ nproc = tst_ncpus_conf();
+ online = SAFE_CALLOC(nproc, sizeof(*online));
+ previous_scaling_max_freq = SAFE_CALLOC(nproc, sizeof(*previous_scaling_max_freq));
+ previous_scaling_min_freq = SAFE_CALLOC(nproc, sizeof(*previous_scaling_min_freq));
+ online[0] = true;
+ for (int i = 1; i < nproc; i++) {
+ int tmp;
+
+ snprintf(path, sizeof(path), "/sys/devices/system/cpu/cpu%d/online", i);
+ SAFE_FILE_SCANF(path, "%d", &tmp);
+ online[i] = (bool)tmp;
+ }
+}
+
+static void run(void)
+{
+ const char * const cpufreq_nodes[] = {
+ "affected_cpus",
+ "cpuinfo_max_freq",
+ "cpuinfo_min_freq",
+ "cpuinfo_transition_latency",
+ "related_cpus",
+ "scaling_available_governors",
+ "scaling_cur_freq",
+ "scaling_driver",
+ "scaling_governor",
+ "scaling_max_freq",
+ "scaling_min_freq",
+ "scaling_setspeed",
+ NULL
+ };
+ char contents[256] = {0}, path_cpuinfo_min_freq[PATH_MAX];
+ int fd;
+ long cpuinfo_max_freq = 0, cpuinfo_min_freq = 0, scaling_max_freq, scaling_min_freq;
+
+ for (int i = 0; i < nproc; i++) {
+ if (!online[i])
+ continue;
+
+ snprintf(path, sizeof(path), "/sys/devices/system/cpu/cpu%d/cpufreq/scaling_driver", i);
+
+ SAFE_FILE_SCANF(path, "%255s", contents);
+ tst_res(TDEBUG, "Checking whether %s is \"intel_pstate\"", path);
+ if (strstr(contents, "intel_pstate"))
+ tst_res(TPASS, "%s is intel_pstate", path);
+ else {
+ tst_res(TINFO, "%s contains: %s", path, contents);
+ tst_res(TFAIL, "%s is not intel_pstate", path);
+ }
+
+ for (int j = 0; cpufreq_nodes[j] != NULL; j++) {
+ struct stat stats;
+
+ snprintf(path, sizeof(path), "/sys/devices/system/cpu/cpu%d/cpufreq/%s", i, cpufreq_nodes[j]);
+ tst_res(TDEBUG, "Checking whether %s is a regular file", path);
+ SAFE_STAT(path, &stats);
+ if (!S_ISREG(stats.st_mode)) {
+ tst_res(TINFO, "%s mode: %o\n", path, stats.st_mode);
+ tst_res(TFAIL, "%s is not a regular file", path);
+ } else
+ tst_res(TPASS, "%s is a regular file", path);
+ }
+
+ snprintf(path, sizeof(path), "/sys/devices/system/cpu/cpu%d/cpufreq/scaling_available_governors", i);
+ memset(contents, 0, sizeof(contents));
+ fd = SAFE_OPEN(path, O_RDONLY);
+ SAFE_READ(0, fd, contents, sizeof(contents));
+ SAFE_CLOSE(fd);
+
+ tst_res(TDEBUG, "Checking whether %s is \"performance powersave\"", path);
+ if (strstr(contents, "performance powersave"))
+ tst_res(TPASS, "%s contains: %s", path, contents);
+ else {
+ tst_res(TINFO, "%s contains: %s", path, contents);
+ tst_res(TFAIL, "%s is not \"performance powersave\"", path);
+ }
+
+ snprintf(path, sizeof(path), "/sys/devices/system/cpu/cpu%d/cpufreq/scaling_governor", i);
+ tst_res(TDEBUG, "Checking %s", path);
+ memset(contents, 0, sizeof(contents));
+ SAFE_FILE_SCANF(path, "%255s", contents);
+ if (strstr(contents, "performance")) {
+ tst_res(TDEBUG, "%s is \"performance\"", path);
+ tst_res(TDEBUG, "Checking whether %s can be switched to \"powersave\"", path);
+ SAFE_FILE_PRINTF(path, "powersave");
+ memset(contents, 0, sizeof(contents));
+ SAFE_FILE_SCANF(path, "%255s", contents);
+ if (strstr(contents, "powersave"))
+ tst_res(TPASS, "Changing scaling_governor from performance to powersave succeeded");
+ else
+ tst_res(TFAIL, "%s: failed to change scaling_governor from performance to powersave, current scaling_governor: %s", path, contents);
+
+ } else if (strstr(contents, "powersave")) {
+ tst_res(TDEBUG, "%s is \"powersave\"", path);
+ tst_res(TDEBUG, "Checking whether %s can be switched to \"performance\"", path);
+ SAFE_FILE_PRINTF(path, "performance");
+ memset(contents, 0, sizeof(contents));
+ SAFE_FILE_SCANF(path, "%255s", contents);
+ if (strstr(contents, "performance"))
+ tst_res(TPASS, "Changing scaling_governor from powersave to performance succeeded");
+ else
+ tst_res(TFAIL, "%s: failed to change scaling_governor from powersave to performance, current scaling_governor: %s", path, contents);
+
+ } else
+ tst_brk(TBROK, "Unknown scaling_governor: %s", contents);
+
+ snprintf(path, sizeof(path), "/sys/devices/system/cpu/cpu%d/cpufreq/cpuinfo_max_freq", i);
+ snprintf(path_cpuinfo_min_freq, sizeof(path_cpuinfo_min_freq), "/sys/devices/system/cpu/cpu%d/cpufreq/cpuinfo_min_freq", i);
+ tst_res(TDEBUG, "Checking whether %s is bigger than cpuinfo_min_freq", path);
+ SAFE_FILE_SCANF(path, "%ld", &cpuinfo_max_freq);
+
+ SAFE_FILE_SCANF(path_cpuinfo_min_freq, "%ld", &cpuinfo_min_freq);
+
+ if (cpuinfo_min_freq > cpuinfo_max_freq)
+ tst_res(TFAIL, "Value in %s: %ld, should be bigger than cpuinfo_min_freq: %ld", path, cpuinfo_max_freq, cpuinfo_min_freq);
+ else
+ tst_res(TPASS, "Value in %s: %ld, is bigger than cpuinfo_min_freq: %ld", path, cpuinfo_max_freq, cpuinfo_min_freq);
+ }
+
+ snprintf(path, sizeof(path), "/sys/devices/system/cpu/intel_pstate/status");
+ tst_res(TDEBUG, "Checking whether %s contains \"active\"", path);
+ memset(contents, 0, sizeof(contents));
+ SAFE_FILE_SCANF(path, "%255s", contents);
+
+ if (strstr(contents, "active"))
+ tst_res(TPASS, "%s is \"active\"", path);
+ else
+ tst_res(TFAIL, "%s is not \"active\", but %s", path, contents);
+
+ tst_res(TDEBUG, "Checking whether %s can be switched to \"passive\"", path);
+ SAFE_FILE_PRINTF(path, "passive");
+ memset(contents, 0, sizeof(contents));
+ SAFE_FILE_SCANF(path, "%255s", contents);
+
+ if (strstr(contents, "passive"))
+ tst_res(TPASS, "%s is \"passive\"", path);
+ else
+ tst_res(TFAIL, "%s is not \"passive\", but %s", path, contents);
+
+ tst_res(TDEBUG, "Checking whether %s can be switched back to \"active\"", path);
+ SAFE_FILE_PRINTF(path, "active");
+ memset(contents, 0, sizeof(contents));
+ SAFE_FILE_SCANF(path, "%255s", contents);
+
+ if (strstr(contents, "active"))
+ tst_res(TPASS, "%s is \"active\"", path);
+ else
+ tst_res(TFAIL, "%s is not \"active\", but %s", path, contents);
+
+ snprintf(path, sizeof(path), "/sys/devices/system/cpu/intel_pstate/no_turbo");
+ tst_res(TDEBUG, "Checking whether %s can be switched to 1", path);
+ SAFE_FILE_PRINTF(path, "1");
+ memset(contents, 0, sizeof(contents));
+ SAFE_FILE_SCANF(path, "%255s", contents);
+
+ if (strstr(contents, "1"))
+ tst_res(TPASS, "%s is \"1\"", path);
+ else
+ tst_res(TFAIL, "%s is not \"1\", but %s", path, contents);
+
+ tst_res(TDEBUG, "Checking whether %s can be switched back to 0", path);
+ SAFE_FILE_PRINTF(path, "0");
+ memset(contents, 0, sizeof(contents));
+ SAFE_FILE_SCANF(path, "%255s", contents);
+
+ if (strstr(contents, "0"))
+ tst_res(TPASS, "%s is \"0\"", path);
+ else
+ tst_res(TFAIL, "%s is not \"0\", but %s", path, contents);
+
+ tst_res(TDEBUG, "Checking whether scaling_available_governors changes to \"performance schedutil \" after switching /sys/devices/system/cpu/intel_pstate/status to \"passive\"");
+ snprintf(path, sizeof(path), "/sys/devices/system/cpu/intel_pstate/status");
+ SAFE_FILE_PRINTF(path, "passive");
+
+ for (int i = 0; i < nproc; i++) {
+ if (!online[i])
+ continue;
+
+ snprintf(path, sizeof(path), "/sys/devices/system/cpu/cpu%d/cpufreq/scaling_available_governors", i);
+ memset(contents, 0, sizeof(contents));
+ fd = SAFE_OPEN(path, O_RDONLY);
+ SAFE_READ(0, fd, contents, sizeof(contents));
+ SAFE_CLOSE(fd);
+
+ tst_res(TDEBUG, "Checking whether %s is \"performance schedutil\"", path);
+ if (strstr(contents, "performance schedutil"))
+ tst_res(TPASS, "%s contains: %s", path, contents);
+ else {
+ tst_res(TINFO, "%s contains: %s", path, contents);
+ tst_res(TFAIL, "%s is not \"performance schedutil \"", path);
+ }
+
+ snprintf(path, sizeof(path), "/sys/devices/system/cpu/cpu%d/cpufreq/scaling_governor", i);
+ tst_res(TDEBUG, "Checking whether %s can be changed to performance", path);
+ SAFE_FILE_PRINTF(path, "performance");
+ memset(contents, 0, sizeof(contents));
+ SAFE_FILE_SCANF(path, "%255s", contents);
+
+ if (strstr(contents, "performance"))
+ tst_res(TPASS, "%s is \"performance\"", path);
+ else
+ tst_res(TFAIL, "%s is not \"performance\", but %s", path, contents);
+
+ tst_res(TDEBUG, "Checking whether %s can be changed to schedutil", path);
+ SAFE_FILE_PRINTF(path, "schedutil");
+ memset(contents, 0, sizeof(contents));
+ SAFE_FILE_SCANF(path, "%255s", contents);
+
+ if (strstr(contents, "schedutil"))
+ tst_res(TPASS, "%s is \"schedutil\"", path);
+ else
+ tst_res(TFAIL, "%s is not \"schedutil\", but %s", path, contents);
+ }
+
+ for (int i = 0; i < nproc; i++) {
+ if (!online[i])
+ continue;
+
+ snprintf(path, sizeof(path), "/sys/devices/system/cpu/cpu%d/cpufreq/cpuinfo_max_freq", i);
+ tst_res(TDEBUG, "Checking whether %s can be assigned to scaling_max_freq and scaling_min_freq", path);
+ SAFE_FILE_SCANF(path, "%ld", &cpuinfo_max_freq);
+
+ snprintf(path, sizeof(path), "/sys/devices/system/cpu/cpu%d/cpufreq/cpuinfo_min_freq", i);
+ SAFE_FILE_SCANF(path, "%ld", &cpuinfo_min_freq);
+
+ snprintf(path, sizeof(path), "/sys/devices/system/cpu/cpu%d/cpufreq/scaling_max_freq", i);
+ SAFE_FILE_SCANF(path, "%ld", &previous_scaling_max_freq[i]);
+ SAFE_FILE_PRINTF(path, "%ld", cpuinfo_max_freq);
+ SAFE_FILE_SCANF(path, "%ld", &scaling_max_freq);
+
+ if (cpuinfo_max_freq < scaling_max_freq) {
+ tst_res(TINFO, "cpuinfo_max_freq: %ld", cpuinfo_max_freq);
+ tst_res(TINFO, "scaling_max_freq: %ld", scaling_max_freq);
+ tst_res(TFAIL, "Failure setting %s", path);
+ } else
+ tst_res(TPASS, "Successfully set up %s", path);
+
+ snprintf(path, sizeof(path), "/sys/devices/system/cpu/cpu%d/cpufreq/scaling_min_freq", i);
+ SAFE_FILE_SCANF(path, "%ld", &previous_scaling_min_freq[i]);
+ SAFE_FILE_PRINTF(path, "%ld", cpuinfo_max_freq);
+ SAFE_FILE_SCANF(path, "%ld", &scaling_min_freq);
+
+ if (cpuinfo_max_freq < scaling_min_freq) {
+ tst_res(TINFO, "cpuinfo_max_freq: %ld", cpuinfo_max_freq);
+ tst_res(TINFO, "scaling_min_freq: %ld", scaling_min_freq);
+ tst_res(TFAIL, "Failure setting %s", path);
+ } else
+ tst_res(TPASS, "Successfully set up %s", path);
+
+ snprintf(path, sizeof(path), "/sys/devices/system/cpu/cpu%d/cpufreq/scaling_min_freq", i);
+ SAFE_FILE_PRINTF(path, "%ld", previous_scaling_min_freq[i]);
+
+ snprintf(path, sizeof(path), "/sys/devices/system/cpu/cpu%d/cpufreq/scaling_max_freq", i);
+ SAFE_FILE_PRINTF(path, "%ld", previous_scaling_max_freq[i]);
+ }
+
+ for (int i = 0; i < nproc; i++) {
+ if (!online[i])
+ continue;
+
+ snprintf(path, sizeof(path), "/sys/devices/system/cpu/cpu%d/cpufreq/cpuinfo_max_freq", i);
+ tst_res(TDEBUG, "Checking whether %s can be assigned to scaling_max_freq and scaling_min_freq", path);
+ SAFE_FILE_SCANF(path, "%ld", &cpuinfo_max_freq);
+
+ snprintf(path, sizeof(path), "/sys/devices/system/cpu/cpu%d/cpufreq/cpuinfo_min_freq", i);
+ SAFE_FILE_SCANF(path, "%ld", &cpuinfo_min_freq);
+
+ snprintf(path, sizeof(path), "/sys/devices/system/cpu/cpu%d/cpufreq/scaling_min_freq", i);
+ SAFE_FILE_SCANF(path, "%ld", &previous_scaling_min_freq[i]);
+ SAFE_FILE_PRINTF(path, "%ld", cpuinfo_min_freq);
+ SAFE_FILE_SCANF(path, "%ld", &scaling_min_freq);
+
+ if (cpuinfo_min_freq > scaling_min_freq) {
+ tst_res(TINFO, "cpuinfo_min_freq: %ld", cpuinfo_min_freq);
+ tst_res(TINFO, "scaling_min_freq: %ld", scaling_min_freq);
+ tst_res(TFAIL, "Failure setting %s", path);
+ } else
+ tst_res(TPASS, "Successfully set up %s", path);
+
+ snprintf(path, sizeof(path), "/sys/devices/system/cpu/cpu%d/cpufreq/scaling_max_freq", i);
+ SAFE_FILE_SCANF(path, "%ld", &previous_scaling_max_freq[i]);
+ SAFE_FILE_PRINTF(path, "%ld", cpuinfo_min_freq);
+ SAFE_FILE_SCANF(path, "%ld", &scaling_max_freq);
+
+ if (cpuinfo_min_freq > scaling_max_freq) {
+ tst_res(TINFO, "cpuinfo_min_freq: %ld", cpuinfo_min_freq);
+ tst_res(TINFO, "scaling_max_freq: %ld", scaling_max_freq);
+ tst_res(TFAIL, "Failure setting %s", path);
+ } else
+ tst_res(TPASS, "Successfully set up %s", path);
+ }
+}
+
+static struct tst_test test = {
+ .cleanup = cleanup,
+ .needs_kconfigs = (const char *const []) {
+ "CONFIG_X86_INTEL_PSTATE",
+ NULL
+ },
+ .needs_root = 1,
+ .setup = setup,
+ .supported_archs = (const char *const []) {
+ "x86",
+ "x86_64",
+ NULL
+ },
+ .test_all = run
+};
--
2.47.3
---------------------------------------------------------------------
Intel Technology Poland sp. z o.o.
ul. Slowackiego 173 | 80-298 Gdansk | Sad Rejonowy Gdansk Polnoc | VII Wydzial Gospodarczy Krajowego Rejestru Sadowego - KRS 101882 | NIP 957-07-52-316 | Kapital zakladowy 200.000 PLN.
Spolka oswiadcza, ze posiada status duzego przedsiebiorcy w rozumieniu ustawy z dnia 8 marca 2013 r. o przeciwdzialaniu nadmiernym opoznieniom w transakcjach handlowych.
Ta wiadomosc wraz z zalacznikami jest przeznaczona dla okreslonego adresata i moze zawierac informacje poufne. W razie przypadkowego otrzymania tej wiadomosci, prosimy o powiadomienie nadawcy oraz trwale jej usuniecie; jakiekolwiek przegladanie lub rozpowszechnianie jest zabronione.
This e-mail and any attachments may contain confidential material for the sole use of the intended recipient(s). If you are not the intended recipient, please contact the sender and delete all copies; any review or distribution by others is strictly prohibited.
--
Mailing list info: https://lists.linux.it/listinfo/ltp
next reply other threads:[~2026-04-22 10:37 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-04-22 10:35 Piotr Kubaj [this message]
2026-04-22 11:11 ` [LTP] cpufreq.c: add new test for cpufreq sysfs interface validation linuxtestproject.agent
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20260422103542.20933-2-piotr.kubaj@intel.com \
--to=piotr.kubaj@intel.com \
--cc=daniel.niestepski@intel.com \
--cc=helena.anna.dubel@intel.com \
--cc=ltp@lists.linux.it \
--cc=rafael.j.wysocki@intel.com \
--cc=tomasz.ossowski@intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox