[coreboot-gerrit] Patch merged into coreboot/master: e1be5ae rmodule: put all code/data bits in one section

gerrit at coreboot.org gerrit at coreboot.org
Wed May 1 03:25:52 CEST 2013


the following patch was just integrated into master:
commit e1be5ae2f485989f88ae9af92a97e0577b033155
Author: Aaron Durbin <adurbin at chromium.org>
Date:   Mon Apr 29 13:53:41 2013 -0500

    rmodule: put all code/data bits in one section
    
    While debugging a crash it was discovered that ld was inserting
    address space for sections that were empty depending on section
    address boundaries. This led to the assumption breaking down that
    on-disk payload (code/data bits) was contiguous with the address
    space. When that assumption breaks down relocation updates change
    the wrong memory. Fix this by making the rmodule.ld linker script
    put all code/data bits into a payload section.
    
    Change-Id: Ib5df7941bbd64662090136e49d15a570a1c3e041
    Signed-off-by: Aaron Durbin <adurbin at chromium.org>
    Reviewed-on: http://review.coreboot.org/3149
    Reviewed-by: Stefan Reinauer <stefan.reinauer at coreboot.org>
    Reviewed-by: Paul Menzel <paulepanter at users.sourceforge.net>
    Tested-by: build bot (Jenkins)

Build-Tested: build bot (Jenkins) at Tue Apr 30 02:15:25 2013, giving +1
Reviewed-By: Stefan Reinauer <stefan.reinauer at coreboot.org> at Mon Apr 29 23:55:01 2013, giving +2
See http://review.coreboot.org/3149 for details.

-gerrit



More information about the coreboot-gerrit mailing list