[M] Change in coreboot[master]: soc/intel/meteorlake: Allow to drop redundant CPU feature programming

[Subrata Banik (Code Review)]

Attention is currently required from: Tarun Tuli, Paul Menzel, Kapil Porwal, Ivy Jian, Eric Lai, Ronak Kanabar.

Hello build bot (Jenkins), Tarun Tuli, Kapil Porwal, Ivy Jian, Eric Lai, Ronak Kanabar,

I'd like you to reexamine a change. Please visit

https://review.coreboot.org/c/coreboot/+/74167

to look at the new patch set (#2).

Change subject: soc/intel/meteorlake: Allow to drop redundant CPU feature programming
......................................................................

soc/intel/meteorlake: Allow to drop redundant CPU feature programming

This patch introduces a new config named
`DROP_CPU_FEATURE_PROGRAM_IN_FSP` to avoid FSP running basic CPU
feature programming on BSP and on APs using the "CpuFeaturesPei.efi"
module.

Most of this feature programming is getting performed today in scope
of coreboot doing MP Init. Running this redundant programming in
scope of FSP (when `USE_FSP_FEATURE_PROGRAM_ON_APS` config is enabled)
results in CPU exception (for example: attempting to reprogram CPU
feature lock MSR is causing CPU exception).

SoC users should select this config after dropping "CpuFeaturesPei.ffs"
module from FSP-S Firmware Volume (FV). Upon selection, coreboot runs
those additional feature programming on BSP and APs.

This feature is by default enabled, in case of "coreboot running MP
init" aka `MP_SERVICES_PPI_V2_NOOP` config is selected.

At present, this option does not do anything unless any platform
eventually decides to drop FSP feature programming module and choose
coreboot CPU feature programming over it.

Signed-off-by: Subrata Banik 
Change-Id: I3be5329390401024d7ec9eed85a5afc35ab1b776
---
M src/soc/intel/meteorlake/Kconfig
1 file changed, 53 insertions(+), 0 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/67/74167/2