Jonathan Zhang has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/38546 )
Change subject: Make MP init timeout configurable ......................................................................
Make MP init timeout configurable
The current MP init timeout is hardcoded as 1s. To support Intel Xeon scalable processor, the timeout needs to be adjusted.
This patch makes MP init timeout configurable.
Signed-off-by: Jonathan Zhang jonzhang@fb.com Signed-off-by: Reddy Chagam anjaneya.chagam@intel.com Change-Id: Ibc079fc6aa8641d4ac8d8e726899b6c8d055052e --- M src/cpu/x86/Kconfig M src/cpu/x86/mp_init.c 2 files changed, 7 insertions(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/46/38546/1
diff --git a/src/cpu/x86/Kconfig b/src/cpu/x86/Kconfig index 76446a0..eedd2e6 100644 --- a/src/cpu/x86/Kconfig +++ b/src/cpu/x86/Kconfig @@ -184,3 +184,9 @@ help The SoC requires different access methods for reading and writing the MSRs. Use SoC specific routines to handle the MSR access. + +config FLIGHT_PLAN_TIMEOUT_US + int + default 1000000 + help + Time-out (in micro seconds) to wait for APs to check-in diff --git a/src/cpu/x86/mp_init.c b/src/cpu/x86/mp_init.c index 45776f8..e874931 100644 --- a/src/cpu/x86/mp_init.c +++ b/src/cpu/x86/mp_init.c @@ -522,7 +522,7 @@ * could take a longer time for APs to check-in as the number of APs * increases (contention for resources like UART also increases). */ - const int timeout_us = 1000000; + const int timeout_us = CONFIG_FLIGHT_PLAN_TIMEOUT_US; const int step_us = 100; int num_aps = mp_params->num_cpus - 1; struct stopwatch sw;