[coreboot-gerrit] [XS] Change in coreboot[master]: [UNDERTESTS] soc/intel/common: Fix creation of non-existant efficient...

[Jérémy Compostella (Code Review)]

Attention is currently required from: Arthur Heymans, Sridhar Siricilla.

Hello Arthur Heymans, Sridhar Siricilla,

I'd like you to reexamine a change. Please visit

https://review.coreboot.org/c/coreboot/+/77615?usp=email

to look at the new patch set (#3).

Change subject: [UNDERTESTS] soc/intel/common: Fix creation of non-existant efficient cores
......................................................................

[UNDERTESTS] soc/intel/common: Fix creation of non-existant efficient cores

commit b793aa3bca5a3f8a6c4ef5a28925a1aeebf426c8 ("soc/intel/common:
Order the CPUs based on their APIC IDs") sort algorithnm leads to the
creation of invalid `Local x2APIC' entries in the MADT (APIC) ACPI
table.

It results in the kernel either:
1. Filtering them out when the random ID >= MAX_LOCAL_APIC (32768)
2. Fail to filter out when because the ID ends up being a negative
   number (0xfxxxxxxx) leading to efficient core(s) being considered
   bad and disabled.

Change-Id: I19c7aa51f232bf48201bd6d28f108e9120a21f7e
Signed-off-by: Jeremy Compostella 
---
M src/arch/x86/cpu.c
M src/soc/intel/common/block/acpi/cpu_hybrid.c
2 files changed, 3 insertions(+), 1 deletion(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/15/77615/3