[coreboot-gerrit] Patch merged into coreboot/master: 89be154 Kconfig: Fix incorrect CONFIG_STACK_SIZE values for X86 and ARM64

gerrit at coreboot.org gerrit at coreboot.org
Wed Apr 15 00:22:14 CEST 2015


the following patch was just integrated into master:
commit 89be154f0fffb3fcf06ce3ff9bb5f8c1edd0e02b
Author: Julius Werner <jwerner at chromium.org>
Date:   Thu Dec 18 19:24:48 2014 -0800

    Kconfig: Fix incorrect CONFIG_STACK_SIZE values for X86 and ARM64
    
    Commit 54229a7 (arm: Fix checkstack() to use correct stack size) didn't
    quite hit the mark. Due to the crazy way our Kconfig includes work, It
    accidentally set CONFIG_STACK_SIZE to 0 even on architectures that need
    it.
    
    This patch fixes the issue by moving everything back to a single entry
    in src/Kconfig, making sure we end up with the intended numbers on all
    architectures.
    
    BRANCH=None
    BUG=chrome-os-partner:34750
    TEST=Built for Pinky, Urara, Falco and Ryu. Confirmed that the generated
    .config contained CONFIG_STACK_SIZE=0x0 for the former two, and
    CONFIG_STACK_SIZE=0x1000 for the latter.
    
    Original-Change-Id: Ib18561925aafe7c74e6c4f0b10b55000a785e144
    Original-Signed-off-by: Julius Werner <jwerner at chromium.org>
    Original-Reviewed-on: https://chromium-review.googlesource.com/236753
    Original-Reviewed-by: David Hendricks <dhendrix at chromium.org>
    
    (cherry picked from commit c64b127e163f98162f3f7195b6ed09bd5a4b77c4)
    Signed-off-by: Aaron Durbin <adurbin at chromium.org>
    
    Change-Id: I2c747b04760bc97f43523596640bfb15317e5730
    Reviewed-on: http://review.coreboot.org/9696
    Reviewed-by: David Hendricks <dhendrix at chromium.org>
    Reviewed-by: Edward O'Callaghan <edward.ocallaghan at koparo.com>
    Tested-by: build bot (Jenkins)


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

-gerrit



More information about the coreboot-gerrit mailing list