2024-05-30 05:54:13

by srinivas pandruvada

[permalink] [raw]
Subject: [PATCH] cpufreq: intel_pstate: Support Emerald Rapids OOB mode

Prevent intel_pstate to load when OOB (Out Of Band) P-states mode is
enabled in Emerald Rapids. The OOB identifying bits are same as the
prior generation CPUs like Sapphire Rapids servers. So, also add Emerald
Rapids to intel_pstate_cpu_oob_ids list.

Signed-off-by: Srinivas Pandruvada <[email protected]>
---
drivers/cpufreq/intel_pstate.c | 1 +
1 file changed, 1 insertion(+)

diff --git a/drivers/cpufreq/intel_pstate.c b/drivers/cpufreq/intel_pstate.c
index 4b986c044741..6b6d47d3ec02 100644
--- a/drivers/cpufreq/intel_pstate.c
+++ b/drivers/cpufreq/intel_pstate.c
@@ -2404,6 +2404,7 @@ static const struct x86_cpu_id intel_pstate_cpu_oob_ids[] __initconst = {
X86_MATCH(SKYLAKE_X, core_funcs),
X86_MATCH(ICELAKE_X, core_funcs),
X86_MATCH(SAPPHIRERAPIDS_X, core_funcs),
+ X86_MATCH(EMERALDRAPIDS_X, core_funcs),
{}
};
#endif
--
2.40.1



2024-06-07 19:26:27

by Rafael J. Wysocki

[permalink] [raw]
Subject: Re: [PATCH] cpufreq: intel_pstate: Support Emerald Rapids OOB mode

On Thu, May 30, 2024 at 7:53 AM Srinivas Pandruvada
<[email protected]> wrote:
>
> Prevent intel_pstate to load when OOB (Out Of Band) P-states mode is
> enabled in Emerald Rapids. The OOB identifying bits are same as the
> prior generation CPUs like Sapphire Rapids servers. So, also add Emerald
> Rapids to intel_pstate_cpu_oob_ids list.
>
> Signed-off-by: Srinivas Pandruvada <[email protected]>
> ---
> drivers/cpufreq/intel_pstate.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/drivers/cpufreq/intel_pstate.c b/drivers/cpufreq/intel_pstate.c
> index 4b986c044741..6b6d47d3ec02 100644
> --- a/drivers/cpufreq/intel_pstate.c
> +++ b/drivers/cpufreq/intel_pstate.c
> @@ -2404,6 +2404,7 @@ static const struct x86_cpu_id intel_pstate_cpu_oob_ids[] __initconst = {
> X86_MATCH(SKYLAKE_X, core_funcs),
> X86_MATCH(ICELAKE_X, core_funcs),
> X86_MATCH(SAPPHIRERAPIDS_X, core_funcs),
> + X86_MATCH(EMERALDRAPIDS_X, core_funcs),
> {}
> };
> #endif
> --

Rebased on top of the Tony's series:

https://lore.kernel.org/linux-acpi/[email protected]/

and applied as 6.11 material.

Thanks!