summaryrefslogtreecommitdiff
path: root/util/genbuild_h/genbuild_h.sh
diff options
context:
space:
mode:
authorPatrick Rudolph <patrick.rudolph@9elements.com>2023-07-14 17:44:33 +0200
committerFelix Held <felix-coreboot@felixheld.de>2023-07-18 13:45:14 +0000
commitac02857b975fdccc84709231088a098baf716c84 (patch)
treee089b70ff5802df2de0fb3fcba9d09f063302791 /util/genbuild_h/genbuild_h.sh
parentc56df92d9018cf6599446a818c1f6443b4de1fa9 (diff)
soc/intel/xeon_sp: Skip empty sockets
The current Sapphire Rapids code assumes that all sockets have working CPUs. On multi-socket platforms a CPU might be missing or was disabled due to an error. The variable PlatformData.numofIIO and the variable SystemStatus.numCpus reflect the working CPUs, but not the actual socket count. Update the code to iterate over sockets until PlatformData.numofIIO IIOs have been found. This is required as FSP doesn't sort IIOs by working/non working status. This resolves invalid ACPI table generation and it fixes a crash as commands were sent to a disabled CPU. TEST: Disabled Socket1 on IBM/SBP1. Change-Id: I237b6392764bbdb3b96013f577a10a4394ba9c6e Signed-off-by: Patrick Rudolph <patrick.rudolph@9elements.com> Reviewed-on: https://review.coreboot.org/c/coreboot/+/76559 Reviewed-by: Arthur Heymans <arthur@aheymans.xyz> Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'util/genbuild_h/genbuild_h.sh')
0 files changed, 0 insertions, 0 deletions