Kyösti Mälkki has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/37487 )
Change subject: program.ld: Qualify .bss linking with ENV_EARLY_RAM
......................................................................
Patch Set 2:
(1 comment)
https://review.coreboot.org/c/coreboot/+/37487/1//COMMIT_MSG
Commit Message:
https://review.coreboot.org/c/coreboot/+/37487/1//COMMIT_MSG@10
PS1, Line 10: location across early stages, and not necessarily within the
The semantics of .bss is that it is not shared between stages and therefore its location has not been fixed. Why do you need to change this?
--
To view, visit
https://review.coreboot.org/c/coreboot/+/37487
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Icde1cfd02611b8cce634aa76f5c192d5b637cf44
Gerrit-Change-Number: 37487
Gerrit-PatchSet: 2
Gerrit-Owner: Marshall Dawson
marshalldawson3rd@gmail.com
Gerrit-Reviewer: Julius Werner
jwerner@chromium.org
Gerrit-Reviewer: Marshall Dawson
marshalldawson3rd@gmail.com
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-CC: Kyösti Mälkki
kyosti.malkki@gmail.com
Gerrit-Comment-Date: Fri, 06 Dec 2019 03:31:58 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment