summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorSubrata Banik <subrata.banik@intel.com>2018-05-15 13:31:28 +0530
committerSubrata Banik <subrata.banik@intel.com>2018-05-17 07:33:57 +0000
commit1c9d8632fae58349af91bdda63b1b40c7e169f2a (patch)
treeda4cb0ba605d73c245be0553569f2f059aa04282
parent13f89980263fc546848478e97f20eab6e4b303a1 (diff)
downloadcoreboot-1c9d8632fae58349af91bdda63b1b40c7e169f2a.tar.xz
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 <subrata.banik@intel.com> Reviewed-on: https://review.coreboot.org/26286 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Caveh Jalali <caveh@google.com>
-rw-r--r--src/soc/intel/skylake/cpu.c5
1 files 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 <soc/ramstage.h>
#include <soc/smm.h>
#include <soc/systemagent.h>
+#include <timer.h>
/* 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 = {