[coreboot-gerrit] Patch merged into coreboot/master: libpayload: head.S: Avoid clearing BSS (and heap) again

gerrit at coreboot.org gerrit at coreboot.org
Sat Aug 13 02:46:21 CEST 2016


the following patch was just integrated into master:
commit 1143d08f7e0a5c203312f054a08301445cafdec8
Author: Julius Werner <jwerner at chromium.org>
Date:   Fri Aug 5 21:27:47 2016 -0700

    libpayload: head.S: Avoid clearing BSS (and heap) again
    
    3 out of 4 architectures currently zero out the payload BSS in early
    assembly code, which is pointless since the code loading the payload has
    already done that (with a more efficient memset). ARM64 has never had
    any code like this and can run just fine without it. This also defeats
    the new optimization of moving the heap out of the BSS, since all three
    implementations assume that everything between _edata and _end is BSS.
    We should just take this out.
    
    Change-Id: I45cd2dabd94da43ff0f77e990f11c877cee6cda1
    Signed-off-by: Julius Werner <jwerner at chromium.org>
    Reviewed-on: https://review.coreboot.org/16091
    Tested-by: build bot (Jenkins)
    Reviewed-by: Aaron Durbin <adurbin at chromium.org>
    Reviewed-by: Stefan Reinauer <stefan.reinauer at coreboot.org>


See https://review.coreboot.org/16091 for details.

-gerrit



More information about the coreboot-gerrit mailing list