Kyösti Mälkki has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/74399 )
Change subject: cpu,soc/intel: Separate single SSDT CPU entry ......................................................................
Patch Set 1:
(1 comment)
File src/soc/intel/common/block/acpi/acpi.c:
https://review.coreboot.org/c/coreboot/+/74399/comment/f2ede8e2_71d60ef8 PS1, Line 422: generate_cpu_entry(device, cpu_id, core_id, num_virt); This loop order determines UUIDs for each CPU Device().
Now, as long as we create a symmetrical entry for each CPU, the order does not really matter. But the UUIDs need/should sync with MADT LAPIC entries?
Is it really okay that we use the BSP CPU to create also the AP CPU's entries on some platforms?