Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31545 )
Change subject: acpi: Sort the reported APIC-IDs in the MADT table ......................................................................
Patch Set 7:
(1 comment)
https://review.coreboot.org/#/c/31545/7/src/arch/x86/acpi.c File src/arch/x86/acpi.c:
https://review.coreboot.org/#/c/31545/7/src/arch/x86/acpi.c@166 PS7, Line 166: num_cpus check if > 0. Some older compilers (gcc6) don't like this potentially being 0.