On 8/15/2023 01:15, Meng Li wrote:
Drop CPU_SUP_INTE from SCHED_MC_PRIO, and it can be used by X86_AMD_PSTATE.
s/CPU_SUP_INTE/CPU_SUP_INTEL/
Also I think this should have a wider description.
Something like:
"amd-pstate also uses SCHED_MC_PRIO, so decouple the requirement of CPU_SUP_INTEL from the dependencies to allow compilation in kernels without Intel CPU support."
Signed-off-by: Meng Li li.meng@amd.com
arch/x86/Kconfig | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/arch/x86/Kconfig b/arch/x86/Kconfig index e36261b4ea14..03322d2840fa 100644 --- a/arch/x86/Kconfig +++ b/arch/x86/Kconfig @@ -1052,8 +1052,9 @@ config SCHED_MC config SCHED_MC_PRIO bool "CPU core priorities scheduler support"
- depends on SCHED_MC && CPU_SUP_INTEL
- depends on SCHED_MC select X86_INTEL_PSTATE
- select X86_AMD_PSTATE
Thinking about it further I wonder if what we actually want is something like this:
select X86_INTEL_PSTATE if CPU_SUP_INTEL select X86_INTEL_PSTATE if CPU_SUP_AMD
Then it will work functionally the same for Intel systems but now you can compile the kernel with support for one or both and get the right combination included.
select CPU_FREQ default y help