78 lines
2.6 KiB
Diff
78 lines
2.6 KiB
Diff
|
From aabfc7370a7da9c52be97c79ba70a20201e6864a Mon Sep 17 00:00:00 2001
|
||
|
|
From: Mario Limonciello <mario.limonciello@amd.com>
|
||
|
|
Date: Sat, 31 Aug 2024 21:49:12 -0500
|
||
|
|
Subject: cpufreq/amd-pstate-ut: Add test case for mode switches
|
||
|
|
|
||
|
|
There is a state machine in the amd-pstate driver utilized for
|
||
|
|
switches for all modes. To make sure that cleanup and setup works
|
||
|
|
properly for each mode add a unit test case that tries all
|
||
|
|
combinations.
|
||
|
|
|
||
|
|
Signed-off-by: Mario Limonciello <mario.limonciello@amd.com>
|
||
|
|
---
|
||
|
|
drivers/cpufreq/amd-pstate-ut.c | 41 ++++++++++++++++++++++++++++++++-
|
||
|
|
1 file changed, 40 insertions(+), 1 deletion(-)
|
||
|
|
|
||
|
|
--- a/drivers/cpufreq/amd-pstate-ut.c
|
||
|
|
+++ b/drivers/cpufreq/amd-pstate-ut.c
|
||
|
|
@@ -54,12 +54,14 @@ static void amd_pstate_ut_acpi_cpc_valid
|
||
|
|
static void amd_pstate_ut_check_enabled(u32 index);
|
||
|
|
static void amd_pstate_ut_check_perf(u32 index);
|
||
|
|
static void amd_pstate_ut_check_freq(u32 index);
|
||
|
|
+static void amd_pstate_ut_check_driver(u32 index);
|
||
|
|
|
||
|
|
static struct amd_pstate_ut_struct amd_pstate_ut_cases[] = {
|
||
|
|
{"amd_pstate_ut_acpi_cpc_valid", amd_pstate_ut_acpi_cpc_valid },
|
||
|
|
{"amd_pstate_ut_check_enabled", amd_pstate_ut_check_enabled },
|
||
|
|
{"amd_pstate_ut_check_perf", amd_pstate_ut_check_perf },
|
||
|
|
- {"amd_pstate_ut_check_freq", amd_pstate_ut_check_freq }
|
||
|
|
+ {"amd_pstate_ut_check_freq", amd_pstate_ut_check_freq },
|
||
|
|
+ {"amd_pstate_ut_check_driver", amd_pstate_ut_check_driver }
|
||
|
|
};
|
||
|
|
|
||
|
|
static bool get_shared_mem(void)
|
||
|
|
@@ -257,6 +259,43 @@ skip_test:
|
||
|
|
cpufreq_cpu_put(policy);
|
||
|
|
}
|
||
|
|
|
||
|
|
+static int amd_pstate_set_mode(enum amd_pstate_mode mode)
|
||
|
|
+{
|
||
|
|
+ const char *mode_str = amd_pstate_get_mode_string(mode);
|
||
|
|
+
|
||
|
|
+ pr_debug("->setting mode to %s\n", mode_str);
|
||
|
|
+
|
||
|
|
+ return amd_pstate_update_status(mode_str, strlen(mode_str));
|
||
|
|
+}
|
||
|
|
+
|
||
|
|
+static void amd_pstate_ut_check_driver(u32 index)
|
||
|
|
+{
|
||
|
|
+ enum amd_pstate_mode mode1, mode2;
|
||
|
|
+ int ret;
|
||
|
|
+
|
||
|
|
+ for (mode1 = AMD_PSTATE_DISABLE; mode1 < AMD_PSTATE_MAX; mode1++) {
|
||
|
|
+ ret = amd_pstate_set_mode(mode1);
|
||
|
|
+ if (ret)
|
||
|
|
+ goto out;
|
||
|
|
+ for (mode2 = AMD_PSTATE_DISABLE; mode2 < AMD_PSTATE_MAX; mode2++) {
|
||
|
|
+ if (mode1 == mode2)
|
||
|
|
+ continue;
|
||
|
|
+ ret = amd_pstate_set_mode(mode2);
|
||
|
|
+ if (ret)
|
||
|
|
+ goto out;
|
||
|
|
+ }
|
||
|
|
+ }
|
||
|
|
+out:
|
||
|
|
+ if (ret)
|
||
|
|
+ pr_warn("%s: failed to update status for %s->%s: %d\n", __func__,
|
||
|
|
+ amd_pstate_get_mode_string(mode1),
|
||
|
|
+ amd_pstate_get_mode_string(mode2), ret);
|
||
|
|
+
|
||
|
|
+ amd_pstate_ut_cases[index].result = ret ?
|
||
|
|
+ AMD_PSTATE_UT_RESULT_FAIL :
|
||
|
|
+ AMD_PSTATE_UT_RESULT_PASS;
|
||
|
|
+}
|
||
|
|
+
|
||
|
|
static int __init amd_pstate_ut_init(void)
|
||
|
|
{
|
||
|
|
u32 i = 0, arr_size = ARRAY_SIZE(amd_pstate_ut_cases);
|