[coreboot-gerrit] Patch merged into coreboot/master: 7c6e489 arm: Put assembly functions into separate sections

gerrit at coreboot.org gerrit at coreboot.org
Thu Nov 13 06:49:43 CET 2014


the following patch was just integrated into master:
commit 7c6e489b23476ad6149c173b6971327aa2eb7726
Author: Julius Werner <jwerner at chromium.org>
Date:   Fri Jan 24 16:23:08 2014 -0800

    arm: Put assembly functions into separate sections
    
    This patch changes the ENTRY() macro in asm.h to create a new section
    for every assembler function, thus providing dcache_clean/invalidate_all
    and friends with the same --gc-sections goodness that our C functions
    have. This requires a few minor changes of moving around data (to make
    sure it ends up in the right section) and changing some libgcc functions
    (which apparently need to have two names?), but nothing serious.
    
    (You may note that some of our assembly functions have data, sometimes
    even writable, within the same .text section. This has been this way
    before and I'm not looking to change it for now, although it's not
    totally clean. Since we don't enforce read-only sections through paging,
    it doesn't really hurt.)
    
    BUG=None
    TEST=Nyan and Snow still boot. Confirm dcache_invalidate_all is not
    output into any binary anymore since no one actually uses it.
    
    Original-Change-Id: I247b29d6173ba516c8dff59126c93b66f7dc4b8d
    Original-Signed-off-by: Julius Werner <jwerner at chromium.org>
    Original-Reviewed-on: https://chromium-review.googlesource.com/183891
    (cherry picked from commit 4a3f2e45e06cc8592d56c3577f41ff879f10e9cc)
    Signed-off-by: Marc Jones <marc.jones at se-eng.com>
    
    Change-Id: Ieaa4f2ea9d81c5b9e2b36a772ff9610bdf6446f9
    Reviewed-on: http://review.coreboot.org/7451
    Tested-by: build bot (Jenkins)
    Reviewed-by: David Hendricks <dhendrix at chromium.org>


See http://review.coreboot.org/7451 for details.

-gerrit



More information about the coreboot-gerrit mailing list