From d6630d1165c844913325a0c36ab6352a33622c7e Mon Sep 17 00:00:00 2001 From: Furquan Shaikh Date: Thu, 29 Mar 2018 00:10:02 -0700 Subject: x86: Increase time out for parking APs to 250ms Change f43adf0 (intel/common/block/cpu: Change post_cpus_init after BS_DEV RESOURCES) moved post_cpus_init to BS_OS_RESUME for S3 path. This results in BSP timing out waiting for APs to be parked. This change increases the time out value for APs to be parked to 250ms. This value was chosen after running suspend-resume stress test and capturing the maximum time taken for APs to be parked for 100 iterations. Typical values observed were ~150ms. Maximum value observed was 152ms. BUG=b:76442753 TEST=Verified for 100 iterations that suspend-resume does not run into any AP park time out. Change-Id: Id3e59db4fe7a5a2fb60357b05565bba89be1e00e Signed-off-by: Furquan Shaikh Reviewed-on: https://review.coreboot.org/25422 Tested-by: build bot (Jenkins) Reviewed-by: Aaron Durbin Reviewed-by: Paul Menzel Reviewed-by: Duncan Laurie --- src/arch/x86/cpu.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) (limited to 'src/arch') diff --git a/src/arch/x86/cpu.c b/src/arch/x86/cpu.c index cfcfdbd5d2..142f2b1400 100644 --- a/src/arch/x86/cpu.c +++ b/src/arch/x86/cpu.c @@ -309,6 +309,5 @@ void arch_bootstate_coreboot_exit(void) return; /* APs are waiting for work. Last thing to do is park them. */ - if (mp_park_aps()) - printk(BIOS_ERR, "Parking APs failed.\n"); + mp_park_aps(); } -- cgit v1.2.3