Andrey Petrov has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/40671 )
Change subject: soc/intel/xeon_sp/cpx: Calculate number of threads based on sockets ......................................................................
soc/intel/xeon_sp/cpx: Calculate number of threads based on sockets
Assuming given system is populated with multiple CPUs of same SKUs, calculate number of threads based on MAX_SOCKET.
This is a stop gap solution until proper way of identifying total number of sockets is determined.
Change-Id: I7ebad3d57c47b9eeb7d727ffb21bc0a1a84734fd Signed-off-by: Andrey Petrov anpetrov@fb.com --- M src/soc/intel/xeon_sp/cpx/cpu.c 1 file changed, 8 insertions(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/71/40671/1
diff --git a/src/soc/intel/xeon_sp/cpx/cpu.c b/src/soc/intel/xeon_sp/cpx/cpu.c index a62c2f0..f5530df 100644 --- a/src/soc/intel/xeon_sp/cpx/cpu.c +++ b/src/soc/intel/xeon_sp/cpx/cpu.c @@ -63,7 +63,14 @@
cpu_read_topology(&num_phys, &num_virts); printk(BIOS_SPEW, "Detected %u cores and %u threads\n", num_phys, num_virts); - return num_virts; + /* + * Currently we do not know a way to figure out how many CPUs we have total + * on multi-socketed boards are used. So we pretend all sockets are populated + * with CPUs with same thread/core fusing. + * TODO: properly figure out number of active sockets OR refactor MPinit code + * to remove requirements of having to know total number of CPUs in advance. + */ + return num_virts * CONFIG_MAX_SOCKET; }
static const struct mp_ops mp_ops = {