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

-gerrit

commit 2c9b877bc3c935a3ec4e7e392405f29d088a76b4
Author: Aaron Durbin <[email protected]>
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 <[email protected]>
---
 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
                *(COMMON);
                . = 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=<>

-- 
coreboot mailing list: [email protected]
http://www.coreboot.org/mailman/listinfo/coreboot

Reply via email to