Jonathan Zhang has uploaded this change for review.
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;
To view, visit change 38546. To unsubscribe, or for help writing mail filters, visit settings.