[coreboot] Patch merged into coreboot/master: 75dbc38 Clean up stack checking code

gerrit at coreboot.org gerrit at coreboot.org
Tue Nov 13 18:25:18 CET 2012


the following patch was just integrated into master:
commit 75dbc389ec62d9ec85178eb5364850c7f9a77997
Author: Stefan Reinauer <reinauer at chromium.org>
Date:   Mon Oct 15 15:19:43 2012 -0700

    Clean up stack checking code
    
    Several small improvements of the stack checking code:
    - move the CPU0 stack check right before jumping to the payload
      and out of hardwaremain (that file is too crowded anyways)
    - fix prototype in lib.h
    - print size of used stack
    - use checkstack function both on CPU0 and CPU1-x
    - print amount of stack used per core
    
    Signed-off-by: Stefan Reinauer <reinauer at google.com>
    
    Test: Boot coreboot on Link, see the following output:
         ...
         CPU1: stack: 00156000 - 00157000, lowest used address 00156c68,
               stack used: 920 bytes
         CPU2: stack: 00155000 - 00156000, lowest used address 00155c68,
               stack used: 920 bytes
         CPU3: stack: 00154000 - 00155000, lowest used address 00154c68,
               stack used: 920 bytes
         ...
         Jumping to boot code at 1110008
         CPU0: stack: 00157000 - 00158000, lowest used address 00157af8,
               stack used: 1288 bytes
    
    Change-Id: I7b83eeee0186559a0a62daa12e3f7782990fd2df
    Reviewed-on: http://review.coreboot.org/1787
    Tested-by: build bot (Jenkins)
    Reviewed-by: Ronald G. Minnich <rminnich at gmail.com>

Build-Tested: build bot (Jenkins) at Tue Nov 13 01:23:40 2012, giving +1
See http://review.coreboot.org/1787 for details.

-gerrit




More information about the coreboot mailing list