Attention is currently required from: Angel Pons, Arthur Heymans, Nico Huber.
Thierry Laurion has posted comments on this change by Angel Pons. ( https://review.coreboot.org/c/coreboot/+/81897?usp=email )
Change subject: Haswell NRI: Implement fast boot path ......................................................................
Patch Set 1:
(1 comment)
Patchset:
PS1:
False alarm, it was a problem with the board I was testing things with: CB:68188
There would be desire and willingness from Heads side to test this and switch to NRI (MRC based ram init cannot be used anymore until https://ticket.coreboot.org/issues/457 is fixed)
I would suggest organizing under https://github.com/linuxboot/heads/issues/1711 if some collaboration moving this forward is desired as well for Heads platforms testers to provide proper bug reports. I could handle triage and facilitate communications upstream/downstream until this gets merged.
This patchset is more than needed. Otherwise Heads now takes >50s to boot as of now, toolstack having grown to ~7.8Mb and no contiguous space available with MRC blob, having required Heads to augment CBFS >8mb CBFS limit needed to use MRC without boot time penalties.
To go forward, I would need an upstream coreboot commit + patch needed to create a downstream PR for willing testers CircleCI stitched roms and report in said PR until functional satisfaction to merge downstream.
-
Thierry Laurion (Code Review)