Attention is currently required from: Intel coreboot Reviewers, Vladimir Serbinenko.
Patrick Rudolph has posted comments on this change by Vladimir Serbinenko. ( https://review.coreboot.org/c/coreboot/+/86247?usp=email )
Change subject: intel/acpi: Put BSP as the first entry ......................................................................
Patch Set 3:
(2 comments)
File src/soc/intel/common/block/acpi/cpu_hybrid.c:
https://review.coreboot.org/c/coreboot/+/86247/comment/f6eec103_491dd148?usp... : PS3, Line 40: The The comment is no longer true as the BSP/first entry might be an efficient core. Not sure if that break assumptions made within other OS.
https://review.coreboot.org/c/coreboot/+/86247/comment/fd6c9505_66b83f7c?usp... : PS3, Line 53: /* While technically correct I wonder how it can happen. It looks like the BSP runs on the efficient core instead of performance core at boot?