Andrey Petrov has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/36662 )
Change subject: arch/x86: Correctly determine number of enabled cores ......................................................................
arch/x86: Correctly determine number of enabled cores
Do not use CONFIG_MAX_CPUs in calculation, just assume we are enabling all the cores.
TEST=tested with dmidecode
Change-Id: Id0935f48e73c037bb7c0e1cf36f94d98a40a499c Signed-off-by: Andrey Petrov anpetrov@fb.com --- M src/arch/x86/smbios.c 1 file changed, 2 insertions(+), 2 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/62/36662/1
diff --git a/src/arch/x86/smbios.c b/src/arch/x86/smbios.c index 261888f..789f8e1 100644 --- a/src/arch/x86/smbios.c +++ b/src/arch/x86/smbios.c @@ -658,8 +658,8 @@ t->processor_family = (res.eax > 0) ? 0x0c : 0x6; t->processor_type = 3; /* System Processor */ t->core_count = (res.ebx >> 16) & 0xff; - /* Assume we enable all the cores always, capped only by MAX_CPUS */ - t->core_enabled = MAX(t->core_count, CONFIG_MAX_CPUS); + /* Assume we enable all the cores always */ + t->core_enabled = t->core_count; t->l1_cache_handle = 0xffff; t->l2_cache_handle = 0xffff; t->l3_cache_handle = 0xffff;