[coreboot] Patch merged into coreboot/master: f7fa218 x86: improve lb_cleanup_memory_ranges

gerrit at coreboot.org gerrit at coreboot.org
Thu Mar 14 20:13:21 CET 2013


the following patch was just integrated into master:
commit f7fa218359cdfa981a2e6ea8c8eba32cb0567693
Author: Aaron Durbin <adurbin at chromium.org>
Date:   Tue Dec 18 17:01:57 2012 -0600

    x86: improve lb_cleanup_memory_ranges
    
    There are 2 issues in lb_cleanup_memory_ranges(). The first
    is that during sort there is a neighbor comparison that initially
    starts with the current entry. The second issue is that merging
    has an off by one comparison for adjacent entries.
    
    Before:
    	coreboot memory table:
    	 0. 0000000000000000-0000000000000fff: CONFIGURATION TABLES
    	 1. 0000000000001000-000000000009ffff: RAM
    	 2. 00000000000a0000-00000000000fffff: RESERVED
    	 3. 0000000000100000-0000000000efffff: RAM
    	 4. 0000000000f00000-0000000000ffffff: RESERVED
    	 5. 0000000001000000-00000000acebffff: RAM
    	 6. 00000000acec0000-00000000acffffff: CONFIGURATION TABLES
    	 7. 00000000ad000000-00000000af9fffff: RESERVED
    	 8. 00000000f0000000-00000000f3ffffff: RESERVED
    	 9. 00000000fed10000-00000000fed17fff: RESERVED
    	10. 00000000fed18000-00000000fed18fff: RESERVED
    	11. 00000000fed19000-00000000fed19fff: RESERVED
    	12. 00000000fed84000-00000000fed84fff: RESERVED
    	13. 0000000100000000-000000018f5fffff: RAM
    
    After:
    	coreboot memory table:
    	 0. 0000000000000000-0000000000000fff: CONFIGURATION TABLES
    	 1. 0000000000001000-000000000009ffff: RAM
    	 2. 00000000000a0000-00000000000fffff: RESERVED
    	 3. 0000000000100000-0000000000efffff: RAM
    	 4. 0000000000f00000-0000000000ffffff: RESERVED
    	 5. 0000000001000000-00000000acebffff: RAM
    	 6. 00000000acec0000-00000000acffffff: CONFIGURATION TABLES
    	 7. 00000000ad000000-00000000af9fffff: RESERVED
    	 8. 00000000f0000000-00000000f3ffffff: RESERVED
    	 9. 00000000fed10000-00000000fed19fff: RESERVED
    	10. 00000000fed84000-00000000fed84fff: RESERVED
    	11. 0000000100000000-000000018f5fffff: RAM
    
    Change-Id: I656aab61b0ed4711c9dceaedb81c290d040ffdec
    Signed-off-by: Aaron Durbin <adurbin at chromium.org>
    Reviewed-on: http://review.coreboot.org/2671
    Tested-by: build bot (Jenkins)
    Reviewed-by: Marc Jones <marc.jones at se-eng.com>
    Reviewed-by: Ronald G. Minnich <rminnich at gmail.com>

Build-Tested: build bot (Jenkins) at Tue Mar 12 23:21:01 2013, giving +1
Reviewed-By: Marc Jones <marc.jones at se-eng.com> at Wed Mar 13 06:27:44 2013, giving +1
Reviewed-By: Ronald G. Minnich <rminnich at gmail.com> at Thu Mar 14 20:13:19 2013, giving +2
See http://review.coreboot.org/2671 for details.

-gerrit



More information about the coreboot mailing list