From 1c9d8632fae58349af91bdda63b1b40c7e169f2a Mon Sep 17 00:00:00 2001 From: Subrata Banik Date: Tue, 15 May 2018 13:31:28 +0530 Subject: [PATCH] soc/intel/skylake: Fix AP timeout issue while executing sgx_configure Increase AP timeout limit for sgx_configure function. As per debug log sgx_configure was not successful on all cores with given timeout value. TEST=Ensures no timeout error in AP function execution. Change-Id: Ia83f7a7eb6cd6c4808d55febfebe32724a633173 Signed-off-by: Subrata Banik Reviewed-on: https://review.coreboot.org/26286 Tested-by: build bot (Jenkins) Reviewed-by: Caveh Jalali --- src/soc/intel/skylake/cpu.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/soc/intel/skylake/cpu.c b/src/soc/intel/skylake/cpu.c index 7f19dac072..d9e90d2df4 100644 --- a/src/soc/intel/skylake/cpu.c +++ b/src/soc/intel/skylake/cpu.c @@ -48,6 +48,7 @@ #include #include #include +#include /* Convert time in seconds to POWER_LIMIT_1_TIME MSR value */ static const u8 power_limit_time_sec_to_msr[] = { @@ -477,9 +478,9 @@ static void post_mp_init(void) smm_lock(); #endif - mp_run_on_all_cpus(vmx_configure, NULL, 2000); + mp_run_on_all_cpus(vmx_configure, NULL, 2 * USECS_PER_MSEC); - mp_run_on_all_cpus(sgx_configure, NULL, 2000); + mp_run_on_all_cpus(sgx_configure, NULL, 14 * USECS_PER_MSEC); } static const struct mp_ops mp_ops = {