a43db197f3
checkstack() runs at the end of ramstage to warn about stack overflows, and it assumes that CONFIG_STACK_SIZE is always the size of the stack to check. This is only true for systems that bring up multiprocessing in ramstage and assign a separate stack for each core, like x86 and ARM64. Other architectures like ARM and MIPS (for now) don't touch secondary CPUs at all and currently don't look like they'll ever need to, so they generally stay on the same (SRAM-based) stack they have been on since their bootblock. This patch tries to model that difference by making these architectures explicitly set CONFIG_STACK_SIZE to zero, and using that as a cue to assume the whole (_estack - _stack) area in checkstack() instead. Also adds a BUG() to the stack overflow check, since that is currently just as non-fatal as the BIOS_ERR message (despite the incorrect "SYSTEM HALTED" output) but a little more easy to spot. Such a serious failure should not drown out in all the normal random pieces of lower case boot spam (also, I was intending to eventually have a look at assert() and BUG() to hopefully make them a little more useful/noticeable if I ever find the time for it). BRANCH=None BUG=None TEST=Booted Pinky, noticed it no longer complains about stack overflows. Built Falco, Ryu and Urara. Change-Id: I6826e0ec24201d4d83c5929b281828917bc9abf4 Signed-off-by: Patrick Georgi <pgeorgi@chromium.org> Original-Commit-Id: 54229a725e8907b84a105c04ecea33b8f9b91dd4 Original-Change-Id: I49f70bb7ad192bd1c48e077802085dc5ecbfd58b Original-Signed-off-by: Julius Werner <jwerner@chromium.org> Original-Reviewed-on: https://chromium-review.googlesource.com/235894 Original-Reviewed-by: Aaron Durbin <adurbin@chromium.org> Reviewed-on: http://review.coreboot.org/9610 Tested-by: build bot (Jenkins) Reviewed-by: Stefan Reinauer <stefan.reinauer@coreboot.org>
57 lines
2 KiB
C
57 lines
2 KiB
C
/*
|
|
This software and ancillary information (herein called SOFTWARE )
|
|
called LinuxBIOS is made available under the terms described
|
|
here. The SOFTWARE has been approved for release with associated
|
|
LA-CC Number 00-34 . Unless otherwise indicated, this SOFTWARE has
|
|
been authored by an employee or employees of the University of
|
|
California, operator of the Los Alamos National Laboratory under
|
|
Contract No. W-7405-ENG-36 with the U.S. Department of Energy. The
|
|
U.S. Government has rights to use, reproduce, and distribute this
|
|
SOFTWARE. The public may copy, distribute, prepare derivative works
|
|
and publicly display this SOFTWARE without charge, provided that this
|
|
Notice and any statement of authorship are reproduced on all copies.
|
|
Neither the Government nor the University makes any warranty, express
|
|
or implied, or assumes any liability or responsibility for the use of
|
|
this SOFTWARE. If SOFTWARE is modified to produce derivative works,
|
|
such modified SOFTWARE should be clearly marked, so as not to confuse
|
|
it with the version available from LANL.
|
|
*/
|
|
/* Copyright 2000, Ron Minnich, Advanced Computing Lab, LANL
|
|
* rminnich@lanl.gov
|
|
*/
|
|
|
|
#include <assert.h>
|
|
#include <lib.h>
|
|
#include <console/console.h>
|
|
#include <symbols.h>
|
|
|
|
int checkstack(void *top_of_stack, int core)
|
|
{
|
|
/* Not all archs use CONFIG_STACK_SIZE, those who don't set it to 0. */
|
|
size_t stack_size = CONFIG_STACK_SIZE ? CONFIG_STACK_SIZE : _stack_size;
|
|
int i;
|
|
u32 *stack = (u32 *) (top_of_stack - stack_size);
|
|
|
|
if (stack[0] != 0xDEADBEEF){
|
|
printk(BIOS_ERR, "Stack overrun on CPU%d. "
|
|
"Increase stack from current %zu bytes\n",
|
|
core, stack_size);
|
|
BUG();
|
|
return -1;
|
|
}
|
|
|
|
for(i = 1; i < stack_size/sizeof(stack[0]); i++){
|
|
if (stack[i] == 0xDEADBEEF)
|
|
continue;
|
|
printk(BIOS_SPEW, "CPU%d: stack: %p - %p, ",
|
|
core, stack, &stack[stack_size/sizeof(stack[0])]);
|
|
printk(BIOS_SPEW, "lowest used address %p, ", &stack[i]);
|
|
printk(BIOS_SPEW, "stack used: %ld bytes\n",
|
|
(unsigned long)&stack[stack_size / sizeof(stack[0])]
|
|
- (unsigned long)&stack[i]);
|
|
return 0;
|
|
}
|
|
|
|
return 0;
|
|
|
|
}
|