[coreboot-gerrit] Patch merged into coreboot/master: f0cd03c exynos5420: Don't map low addresses that lead nowhere
gerrit at coreboot.org
gerrit at coreboot.org
Mon Aug 18 19:04:10 CEST 2014
the following patch was just integrated into master:
commit f0cd03c142013e0f50705a1c8741c0f02bea43de
Author: Julius Werner <jwerner at chromium.org>
Date: Thu Sep 19 20:15:45 2013 -0700
exynos5420: Don't map low addresses that lead nowhere
I just spent half a day (including the time to implement a stack dumper)
to figure out that I am reading from a NULL pointer. A problem this
simple should be more easy to catch. Let's mark the address range below
SRAM as uncached so that the MMU can yell at you right away for being
the bad programmer you are when you access a NULL pointer.
Change-Id: I4a3a13f75bf21b25732be2ecb69d47503eff1b53
Signed-off-by: Julius Werner <jwerner at chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/170112
Reviewed-by: Ronald Minnich <rminnich at chromium.org>
(cherry picked from commit 7316732ea0ccdc0d607bde81dbb38ca9abd29fa9)
Signed-off-by: Isaac Christensen <isaac.christensen at se-eng.com>
Reviewed-on: http://review.coreboot.org/6650
Tested-by: build bot (Jenkins)
Reviewed-by: Ronald G. Minnich <rminnich at gmail.com>
See http://review.coreboot.org/6650 for details.
-gerrit
More information about the coreboot-gerrit
mailing list