[coreboot-gerrit] Change in coreboot[master]: soc/intel/xeon_sp/cpx: Calculate number of threads based on sockets

[Andrey Petrov (Code Review)]

Andrey Petrov has submitted this change. ( 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 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/40671
Tested-by: build bot (Jenkins) 
Reviewed-by: Angel Pons 
Reviewed-by: Maxim Polyakov 
---
M src/soc/intel/xeon_sp/cpx/cpu.c
1 file changed, 8 insertions(+), 1 deletion(-)

Approvals:
  build bot (Jenkins): Verified
  Angel Pons: Looks good to me, approved
  Maxim Polyakov: Looks good to me, approved

diff --git a/src/soc/intel/xeon_sp/cpx/cpu.c b/src/soc/intel/xeon_sp/cpx/cpu.c
index a62c2f0..a7a4c54 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. 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 = {