Hello Patrick Rudolph,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/32795
to look at the new patch set (#2).
Change subject: {arch,cpu}/x86, drivers/intel: Restore cpu_index error handling ......................................................................
{arch,cpu}/x86, drivers/intel: Restore cpu_index error handling
Previously cpu_index() always succeeded, but since commit 095c931 (src/arch/x86: Use core apic id to get cpu_index()) it is now possible for it to indicate an error by returning -1. This commit adds error handling for all calls to cpu_index(), and restores several checks that were removed in commit 7c712bb (Fix code that would trip -Wtype-limits) but are now needed.
Signed-off-by: Jacob Garber jgarber1@ualberta.ca Change-Id: I5436eed4cb5675f916924eb9670db04592a8b927 --- M src/arch/x86/cpu.c M src/arch/x86/exception.c M src/arch/x86/include/arch/cpu.h M src/cpu/x86/mp_init.c M src/cpu/x86/pae/pgtbl.c M src/drivers/intel/fsp2_0/ppi/mp_service_ppi.c 6 files changed, 28 insertions(+), 6 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/95/32795/2