diff options
| author | Subrata Banik <subrata.banik@intel.com> | 2018-05-15 13:31:28 +0530 |
|---|---|---|
| committer | Subrata Banik <subrata.banik@intel.com> | 2018-05-17 07:33:57 +0000 |
| commit | 1c9d8632fae58349af91bdda63b1b40c7e169f2a (patch) | |
| tree | da4cb0ba605d73c245be0553569f2f059aa04282 | |
| parent | 13f89980263fc546848478e97f20eab6e4b303a1 (diff) | |
| download | coreboot-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.c | 5 |
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 = { |