Skip to content

Commit 54ab7d7

Browse files
SSapkalsuperm1
authored andcommitted
amd-pstate: Switch to amd-pstate by default on some Server platforms
Currently the default cpufreq driver for all the AMD EPYC servers is acpi-cpufreq. Going forward, switch to amd-pstate as the default driver on the AMD EPYC server platforms with CPU family 0x1A or higher. The default mode will be active mode. Testing shows that amd-pstate with active mode and performance governor provides comparable or better performance per-watt against acpi-cpufreq + performance governor. Likewise, amd-pstate with active mode and powersave governor with the energy_performance_preference=power (EPP=255) provides comparable or better performance per-watt against acpi-cpufreq + schedutil governor for a wide range of workloads. Users can still revert to using acpi-cpufreq driver on these platforms with the "amd_pstate=disable" kernel commandline parameter. Signed-off-by: Swapnil Sapkal <[email protected]> Signed-off-by: Gautham R. Shenoy <[email protected]> Reviewed-by: Mario Limonciello <[email protected]> Link: https://lore.kernel.org/r/[email protected] Signed-off-by: Mario Limonciello <[email protected]>
1 parent 0c411b3 commit 54ab7d7

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

drivers/cpufreq/amd-pstate.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1832,10 +1832,10 @@ static int __init amd_pstate_init(void)
18321832
if (cppc_state == AMD_PSTATE_UNDEFINED) {
18331833
/* Disable on the following configs by default:
18341834
* 1. Undefined platforms
1835-
* 2. Server platforms
1835+
* 2. Server platforms with CPUs older than Family 0x1A.
18361836
*/
18371837
if (amd_pstate_acpi_pm_profile_undefined() ||
1838-
amd_pstate_acpi_pm_profile_server()) {
1838+
(amd_pstate_acpi_pm_profile_server() && boot_cpu_data.x86 < 0x1A)) {
18391839
pr_info("driver load is disabled, boot with specific mode to enable this\n");
18401840
return -ENODEV;
18411841
}

0 commit comments

Comments
 (0)