[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