Kyösti Mälkki has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/46834 )
Change subject: x86: Put bootblock startup code into .text._start section ......................................................................
Patch Set 3:
(2 comments)
https://review.coreboot.org/c/coreboot/+/46834/1//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/46834/1//COMMIT_MSG@14 PS1, Line 14: _start16bit
It needs to be within 64KiB of reset vector which is the end of the bootblock program for dealing wi […]
CB:47599 enforces this too.
https://review.coreboot.org/c/coreboot/+/46834/1//COMMIT_MSG@16 PS1, Line 16: lot of space if it didn't.)
The 4KiB alignment requirement is only for certain platforms where the APs need to be SIPI'd towards […]
CB:47599 drops static C_ENV_BOOTBLOCK_SIZE. Due the different arrangement of sections, the problematic bootblock intel/d945gclf actually increases to 19KiB, but log2 size requirement was already dropped with preceding works on topic:x86-bootblock.
I have certainty with mPGA604 and LGA755 sockets, but I am not sure if SIPI_VECTOR_IN_ROM is a requirement for any CPU model intel/d945gclf is compatible with. We go back to P4 NetBurst microarchitecture and cirka year 2009 silicons.