diff options
author | Felix Held <felix-coreboot@felixheld.de> | 2021-07-21 16:50:10 +0200 |
---|---|---|
committer | Felix Held <felix-coreboot@felixheld.de> | 2021-07-27 14:00:32 +0000 |
commit | 98fb72fa3fcf399c790d24580b1021fecfda778f (patch) | |
tree | 3748ea0d1a5038614ceea527af3258779b2869de /src/cpu/x86/Kconfig | |
parent | 3136424e48b533a29258d5b2412439db066ff744 (diff) | |
download | coreboot-98fb72fa3fcf399c790d24580b1021fecfda778f.tar.gz coreboot-98fb72fa3fcf399c790d24580b1021fecfda778f.tar.bz2 coreboot-98fb72fa3fcf399c790d24580b1021fecfda778f.zip |
cpu/x86/mp_init: don't wait between INIT and SIPI for X86_AMD_INIT_SIPI
Since current AMD SoCs don't need some wait time between INIT and SIPI,
we can skip the 10ms wait there, which improves the boot time a bit.
before: CPU_CLUSTER: 0 init finished in 632 msecs
after: CPU_CLUSTER: 0 init finished in 619 msecs
mpinit still works on Mandolin and all CPU cores show up and are usable.
This also doesn't change the binary in a timeless build for boards/SoCs
that don't select X86_AMD_INIT_SIPI which I verified for lenovo/x230.
BUG=b:193885336
Signed-off-by: Felix Held <felix-coreboot@felixheld.de>
Change-Id: I1e044776f45021742a88a5e369a74383c1baaab6
Reviewed-on: https://review.coreboot.org/c/coreboot/+/56533
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Marshall Dawson <marshalldawson3rd@gmail.com>
Diffstat (limited to 'src/cpu/x86/Kconfig')
-rw-r--r-- | src/cpu/x86/Kconfig | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/src/cpu/x86/Kconfig b/src/cpu/x86/Kconfig index 6a3bbd8c0b1e..a5c2a4df47c2 100644 --- a/src/cpu/x86/Kconfig +++ b/src/cpu/x86/Kconfig @@ -173,7 +173,9 @@ config X86_AMD_INIT_SIPI help This option limits the number of SIPI signals sent during during the common AP setup. Intel documentation specifies an INIT SIPI SIPI - sequence, however this doesn't work on some AMD platforms. + sequence, however this doesn't work on some AMD platforms. These + newer AMD platforms don't need the 10ms wait between INIT and SIPI, + so skip that too to save some time. config SOC_SETS_MSRS bool |