
Hi Simon,
On Wed, Mar 22, 2017 at 4:06 AM, Simon Glass sjg@chromium.org wrote:
Hi Bin,
On 16 March 2017 at 08:26, Bin Meng bmeng.cn@gmail.com wrote:
U-Boot itself as well as everything that is consumed by U-Boot (like heap, stack, dtb, etc) needs to be reserved and reported in the E820 table when S3 resume is on.
Signed-off-by: Bin Meng bmeng.cn@gmail.com
arch/x86/Kconfig | 8 ++++++++ arch/x86/lib/fsp/fsp_dram.c | 12 ++++++++++++ 2 files changed, 20 insertions(+)
Reviewed-by: Simon Glass sjg@chromium.org
Can you detect if the stack space is too small?
It should be possible to measure the stack size easily enough - e.g. using cpu_get_sp().
I suspect we can fill the stack memory with some magic number, and then after U-Boot boots to shell check the stack to see where the pattern ends. This practice should be done periodically as U-Boot's code base changes very rapidly.
Regards, Bin