[coreboot-gerrit] Patch set updated for coreboot: 9a08877 asrock/e350m1: reduce default stack size

Paul Menzel (paulepanter@users.sourceforge.net) gerrit at coreboot.org
Tue May 7 15:04:41 CEST 2013


Paul Menzel (paulepanter at users.sourceforge.net) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/3209

-gerrit

commit 9a08877a3c53a17bd101fa251d112119445f0b54
Author: Stefan Reinauer <reinauer at chromium.org>
Date:   Mon May 6 16:16:03 2013 -0700

    asrock/e350m1: reduce default stack size
    
    The stack used on the ASRock E350M1 is significantly less than
    what we currently set (64k per core). In fact, we use about half
    of the default stack size (4k) on core 0 and even less on non
    BSP cores [1]:
    
        $ grep stack coreboot_without_patch_but_monotonic_timer.log
        CPU1: stack_base 002a0000, stack_end 002afff8
        CPU1: stack: 002a0000 - 002b0000, lowest used address 002afda8, stack used: 600 bytes
        CPU0: stack: 002b0000 - 002c0000, lowest used address 002bf75c, stack used: 2212 bytes
    
    Removing the Kconfig variable STACK_SIZE to use the default results
    in the following numbers of stack usage.
    
        $ grep stack coreboot_with_patch.log
        CPU1: stack_base 00287000, stack_end 00287ff8
        CPU1: stack: 00287000 - 00288000, lowest used address 00287da8, stack used: 600 bytes
        CPU0: stack: 00288000 - 00289000, lowest used address 0028875c, stack used: 2212 bytes
    
    [1] http://review.coreboot.org/#/c/3154/
        (comment May 2 10:21 AM)
    
    Change-Id: Ibdb2102c86094fce3787e3b5a162ca8423de205c
    Signed-off-by: Stefan Reinauer <reinauer at google.com>
    Tested-by: Paul Menzel <paulepanter at users.sourceforge.net>
    Signed-off-by: Paul Menzel <paulepanter at users.sourceforge.net>
---
 src/mainboard/asrock/e350m1/Kconfig | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/src/mainboard/asrock/e350m1/Kconfig b/src/mainboard/asrock/e350m1/Kconfig
index a308850..4c05037 100644
--- a/src/mainboard/asrock/e350m1/Kconfig
+++ b/src/mainboard/asrock/e350m1/Kconfig
@@ -84,10 +84,6 @@ config HEAP_SIZE
         hex
         default 0xc0000
 
-config STACK_SIZE
-        hex
-        default 0x10000
-
 config RAMBASE
         hex
         default 0x200000



More information about the coreboot-gerrit mailing list