[coreboot] New patch to review for coreboot: 2c9b877 rmodule: include heap in bss section

Stefan Reinauer (stefan.reinauer@coreboot.org) gerrit at coreboot.org
Fri Mar 15 20:49:53 CET 2013

Stefan Reinauer (stefan.reinauer at coreboot.org) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/2750


commit 2c9b877bc3c935a3ec4e7e392405f29d088a76b4
Author: Aaron Durbin <adurbin at chromium.org>
Date:   Tue Feb 5 11:09:49 2013 -0600

    rmodule: include heap in bss section
    By including the heap in the bss output section the size is accounted
    for in a elf PT_LOAD segment. Without this change the heap wasn't being
    put into a PT_LOAD segment. The result is a nop w.r.t. functionality,
    but readelf and company will have proper MemSiz fields.
    Change-Id: Ibfe9bb87603dcd4c5ff1c57c6af910bbba96b02b
    Signed-off-by: Aaron Durbin <adurbin at chromium.org>
 src/lib/rmodule.ld | 2 --
 1 file changed, 2 deletions(-)

diff --git a/src/lib/rmodule.ld b/src/lib/rmodule.ld
index 4c13c84..fdee279 100644
--- a/src/lib/rmodule.ld
+++ b/src/lib/rmodule.ld
@@ -74,9 +74,7 @@ SECTIONS
 		. = ALIGN(8);
 		_bss_end = .;
-	}
-	.heap (NOLOAD) : {
 		 * Place the heap after BSS. The heap size is passed in by
 		 * by way of ld --defsym=__heap_size=<>

More information about the coreboot mailing list