[coreboot-gerrit] Patch merged into coreboot/master: libpayload: arm64: Fix MMU range overlap check

gerrit at coreboot.org gerrit at coreboot.org
Mon Aug 8 17:35:41 CEST 2016


the following patch was just integrated into master:
commit 41ddd4fcc1d2f7f2eccfa2a5795da5d49e09f18d
Author: Julius Werner <jwerner at chromium.org>
Date:   Fri Aug 5 10:37:52 2016 -0700

    libpayload: arm64: Fix MMU range overlap check
    
    The ARM64 MMU code maintains a list of used ranges, to avoid mapping the
    DMA buffer over the coreboot tables and things like that. Unfortunately,
    the overlap with ranges in that list is checked with
    
     (start1 >= start2 && start1 <= end2) || (end1 >= start2 && end1 <= end2)
    
    which is not a full overlap check and misses the case where the second
    region is completely contained within the first. This patch replaces
    that code with a properly vetted primitive from Stack Overflow.
    
    BRANCH=none
    BUG=chrome-os-partner:54416
    TEST=Observe how Kevin recovery screen now gets drawn at 10x the speed.
    
    Change-Id: I7e2706426762794e160d743bbfc40da1e26eee12
    Signed-off-by: Julius Werner <jwerner at chromium.org>
    Reviewed-on: https://review.coreboot.org/16075
    Tested-by: build bot (Jenkins)
    Reviewed-by: Aaron Durbin <adurbin at chromium.org>


See https://review.coreboot.org/16075 for details.

-gerrit



More information about the coreboot-gerrit mailing list