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.

Reviewed-by: Perry Yuan <perry.yuan@amd.com>
Signed-off-by: Mario Limonciello <mario.limonciello@amd.com>

diff --git a/drivers/cpufreq/amd-pstate-ut.c b/drivers/cpufreq/amd-pstate-ut.c
index 66b73c308ce67292f99fa2fb5697a50d831db397..24432bd78a5a395068e9c7caabf6a31fbbb8b980 100644 (file)
--- 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(u32 index);
 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)
@@ -254,6 +256,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);