[coreboot-gerrit] Patch merged into coreboot/master: 0135702 x86: mark .textfirst as allocatable and executable
gerrit at coreboot.org
gerrit at coreboot.org
Sat Mar 23 19:38:54 CET 2013
the following patch was just integrated into master:
commit 0135702802601c19937eec57513f3a6e2f4d1e00
Author: Aaron Durbin <adurbin at chromium.org>
Date: Fri Mar 22 20:23:17 2013 -0500
x86: mark .textfirst as allocatable and executable
When the linking of ramstage was changed to use an intermeidate
object with all ramstage objects in it the .textfirst section
was introduced to keep the entry point at 0. However, the
section was not marked allocatable or executable. Nor was it
marked as @progbits. That didn't cause an issue on its own since
.textfirst was directly called out in the linker script. However,
the rmodule infrastructure relies on all the relocation entries
being included in the rmodule. Without the proper section attributes
the .rel.textfirst section entries were not being included in
the final ramstage rmodule.
Change-Id: I54e7055a19bee6c86e269eba047d9a560702afde
Signed-off-by: Aaron Durbin <adurbin at chromium.org>
Reviewed-on: http://review.coreboot.org/2885
Tested-by: build bot (Jenkins)
Reviewed-by: Stefan Reinauer <stefan.reinauer at coreboot.org>
Build-Tested: build bot (Jenkins) at Sat Mar 23 04:05:02 2013, giving +1
Reviewed-By: Stefan Reinauer <stefan.reinauer at coreboot.org> at Sat Mar 23 19:38:52 2013, giving +2
See http://review.coreboot.org/2885 for details.
-gerrit
More information about the coreboot-gerrit
mailing list