On Thu, Jan 31, 2019 at 11:13 AM Kyösti Mälkki (Code Review) < [email protected]> wrote:
> Aaron, Julius; there is a bit of dilemma with car.ld. > > 1) We need consistent layout across PRE_RAM stages > 2) We want RO bootblock, unaware of (future) RW romstage requirements > 3) I don't like the semi-random size reserve, like done here for usbdebug > > Any ideas how to from improve this? Looking at CONFIG_COMMONLIB_STORAGE > and CONFIG_PAGING_IN_CACHE_AS_RAM, I am not sure if the fixed locations are > always maintained properly. There is some strong assumption at least, that > bootblock and romstage are built with same set of Kconfig options set. > That definitely is the assumption. Kconfigs are global and there shouldn't be guarding of entries in the car.ld linker script based on stage w.r.t. to order and size of the allocated space. > View Change <https://review.coreboot.org/c/coreboot/+/31174> > > 1 comment: > > - > > File src/arch/x86/car.ld: > <https://review.coreboot.org/#/c/31174/1/src/arch/x86/car.ld> > - > > Patch Set #1, Line 70: > <https://review.coreboot.org/#/c/31174/1/src/arch/x86/car.ld@70> . > += 0x60; > > yes > > Done > > To view, visit change 31174 > <https://review.coreboot.org/c/coreboot/+/31174>. To unsubscribe, or for > help writing mail filters, visit settings > <https://review.coreboot.org/settings>. > Gerrit-Project: coreboot > Gerrit-Branch: master > Gerrit-Change-Id: Ib49fca781e55619179aa8888e2d859560e050876 > Gerrit-Change-Number: 31174 > Gerrit-PatchSet: 2 > Gerrit-Owner: Kyösti Mälkki <[email protected]> > Gerrit-Reviewer: Arthur Heymans <[email protected]> > Gerrit-Reviewer: Julius Werner <[email protected]> > Gerrit-Reviewer: Kyösti Mälkki <[email protected]> > Gerrit-Reviewer: Nico Huber <[email protected]> > Gerrit-Reviewer: Patrick Rudolph <[email protected]> > Gerrit-Reviewer: build bot (Jenkins) <[email protected]> > Gerrit-CC: Paul Menzel <[email protected]> > Gerrit-Comment-Date: Thu, 31 Jan 2019 18:13:01 +0000 > Gerrit-HasComments: Yes > Gerrit-Has-Labels: No > Comment-In-Reply-To: Patrick Rudolph <[email protected]> > Comment-In-Reply-To: Arthur Heymans <[email protected]> > Gerrit-MessageType: comment > _______________________________________________ > coreboot-gerrit mailing list -- [email protected] > To unsubscribe send an email to [email protected] >
_______________________________________________ coreboot mailing list -- [email protected] To unsubscribe send an email to [email protected]
