diff options
| author | Matthias Gazzari <mail@qtux.eu> | 2018-05-19 00:44:20 +0200 |
|---|---|---|
| committer | Martin Roth <martinroth@google.com> | 2018-05-21 13:25:24 +0000 |
| commit | dfa51259ad1434aa75bef6d071ba5dfacbf13cba (patch) | |
| tree | bd264bc9ebd80f77c5ab2be2b61d1ee36efa2d3f /util | |
| parent | 67403ed6e43a0c0f5b2911f68c0f8161d24575b6 (diff) | |
| download | coreboot-dfa51259ad1434aa75bef6d071ba5dfacbf13cba.tar.xz | |
nb/intel/nehalem: Fix smashed stack in romstage
Stack smashing was detected during raminit when not loading from MRC.
Adding CAR_GLOBAL to a struct inside raminit was suggested in
https://mail.coreboot.org/pipermail/coreboot/2018-May/086677.html in
order to fix the problem.
Adding CAR_GLOBAL to the ram timings variable solves the issue (adding
it to the ram_training or raminfo struct had no effect).
This is just a workaround and might need a proper fix in the future.
Tested on Lenovo X201i with 2+2 and 4+4 GB RAM.
Change-Id: I21b380db61be2aedc045201821d83e18e7d07ad1
Signed-off-by: Matthias Gazzari <mail@qtux.eu>
Reviewed-on: https://review.coreboot.org/26388
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Kyösti Mälkki <kyosti.malkki@gmail.com>
Diffstat (limited to 'util')
0 files changed, 0 insertions, 0 deletions