[coreboot] Patch merged into coreboot/master: 5dac6d8 Intel and GFXUMA: fix MTRR and use uma_resource()

gerrit at coreboot.org gerrit at coreboot.org
Fri Jul 27 15:54:09 CEST 2012


the following patch was just integrated into master:
commit 5dac6d8ac870bcac7959ad24893161bf6a133ab6
Author: Kyösti Mälkki <kyosti.malkki at gmail.com>
Date:   Fri Jul 27 08:42:20 2012 +0300

    Intel and GFXUMA: fix MTRR and use uma_resource()
    
    Commit 2d42b340034ff005693482ef9ca34ce3e0f08371 changed the
    variable MTRR setup and removed compensation of uma_memory_size in
    the cacheable memory resources.
    
    Since the cacheable region size was no longer divisible by a large
    power of 2, like 256 MB, this caused excessive use of MTRRs.
    As first symptoms, slow boot with grub and poor user response.
    
    As a solution, register the actual top of low ram with ram_resource(),
    and do not subtract the UMA/TSEG regions from it.
    
    TSEG may require further work as the original did not appear exactly
    right to begin with. To have UMA as un-cacheable, use uma_resource().
    
    Change-Id: I4ca99b5c2ca4e474296590b3d0c6ef5d09550d80
    Signed-off-by: Kyösti Mälkki <kyosti.malkki at gmail.com>

Build-Tested: build bot (Jenkins) at Fri Jul 27 15:50:22 2012, giving +1
Reviewed-By: Anton Kochkov <anton.kochkov at gmail.com> at Fri Jul 27 14:08:05 2012, giving +2
See http://review.coreboot.org/1239 for details.

-gerrit




More information about the coreboot mailing list