x86: fix trimming e820 with MTRR holes. - fix
authorYinghai Lu <yhlu.kernel.send@gmail.com>
Wed, 30 Apr 2008 03:25:16 +0000 (20:25 -0700)
committerThomas Gleixner <tglx@linutronix.de>
Sun, 25 May 2008 08:55:10 +0000 (10:55 +0200)
v2: process hole then end_pfn
    fix update_memory_range with whole cover comparing

Signed-off-by: Yinghai Lu <yinghai.lu@gmail.com>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
arch/x86/kernel/cpu/mtrr/main.c
arch/x86/kernel/e820_32.c
arch/x86/kernel/e820_64.c

index 9ab5c16b0d527e533a2fed2f461a66b8a915ea4f..5a2a4c146333b7db4ae506e850836c2d708c8815 100644 (file)
@@ -1098,11 +1098,12 @@ int __init amd_special_default_mtrr(void)
 static u64 __init real_trim_memory(unsigned long start_pfn, unsigned long limit_pfn)
 {
        u64 trim_start, trim_size;
-       trim_start =  start_pfn;
+       trim_start = start_pfn;
        trim_start <<= PAGE_SHIFT;
        trim_size = limit_pfn;
        trim_size <<= PAGE_SHIFT;
        trim_size -= trim_start;
+
        return update_memory_range(trim_start, trim_size, E820_RAM,
                                E820_RESERVED);
 }
@@ -1124,7 +1125,6 @@ int __init mtrr_trim_uncached_memory(unsigned long end_pfn)
        struct res_range range[RANGE_NUM];
        int nr_range;
        u64 total_real_trim_size;
-       int changed;
 
        /* extra one for all 0 */
        int num[MTRR_NUM_TYPES + 1];
@@ -1189,49 +1189,35 @@ int __init mtrr_trim_uncached_memory(unsigned long end_pfn)
        }
        nr_range = x86_get_mtrr_mem_range(range, nr_range, 0, 0);
 
-       changed = 0;
-       total_real_trim_size = 0;
-
-       /* check the top at first */
-       i = nr_range - 1;
-       if (range[i].end + 1 < end_pfn) {
-                       total_real_trim_size += real_trim_memory(range[i].end + 1, end_pfn);
-       }
-
-       if (total_real_trim_size) {
-               printk(KERN_WARNING "WARNING: BIOS bug: CPU MTRRs don't cover"
-                       " all of memory, losing %lluMB of RAM.\n",
-                       total_real_trim_size >> 20);
-
-               WARN_ON(1);
-
-               printk(KERN_INFO "update e820 for mtrr -- end_pfn\n");
-               update_e820();
-               changed = 1;
-       }
-
        total_real_trim_size = 0;
+       /* check the head */
        if (range[0].start)
                total_real_trim_size += real_trim_memory(0, range[0].start);
-
-       for (i = 0; i < nr_range - 1; i--) {
+       /* check the holes */
+       for (i = 0; i < nr_range - 1; i++) {
                if (range[i].end + 1 < range[i+1].start)
                        total_real_trim_size += real_trim_memory(range[i].end + 1, range[i+1].start);
        }
+       /* check the top */
+       i = nr_range - 1;
+       if (range[i].end + 1 < end_pfn)
+               total_real_trim_size += real_trim_memory(range[i].end + 1, end_pfn);
 
        if (total_real_trim_size) {
                printk(KERN_WARNING "WARNING: BIOS bug: CPU MTRRs don't cover"
                        " all of memory, losing %lluMB of RAM.\n",
                        total_real_trim_size >> 20);
 
-               WARN_ON(1);
+               if (enable_mtrr_cleanup < 1)
+                       WARN_ON(1);
 
-               printk(KERN_INFO "update e820 for mtrr -- holes\n");
+               printk(KERN_INFO "update e820 for mtrr\n");
                update_e820();
-               changed = 1;
+
+               return 1;
        }
 
-       return changed;
+       return 0;
 }
 
 /**
index 857f706273a8473bfdf0c014cbd633acbcddb063..751d517d802cc57794ec54799844c59a61b595d6 100644 (file)
@@ -797,7 +797,8 @@ u64 __init update_memory_range(u64 start, u64 size, unsigned old_type,
                if (ei->type != old_type)
                        continue;
                /* totally covered? */
-               if (ei->addr >= start && ei->size <= size) {
+               if (ei->addr >= start &&
+                   (ei->addr + ei->size) <= (start + size)) {
                        ei->type = new_type;
                        real_updated_size += ei->size;
                        continue;
index 848b2cd2d1dd4f226a5e7f4689674290ffb8c1ed..c45b4dea405543126e9f814e4f4174d2beb84d13 100644 (file)
@@ -843,7 +843,8 @@ u64 __init update_memory_range(u64 start, u64 size, unsigned old_type,
                if (ei->type != old_type)
                        continue;
                /* totally covered? */
-               if (ei->addr >= start && ei->size <= size) {
+               if (ei->addr >= start &&
+                   (ei->addr + ei->size) <= (start + size)) {
                        ei->type = new_type;
                        real_updated_size += ei->size;
                        continue;