x86, kexec, 64bit: Only set ident mapping for ram.
authorYinghai Lu <yinghai@kernel.org>
Thu, 24 Jan 2013 20:20:05 +0000 (12:20 -0800)
committerH. Peter Anvin <hpa@linux.intel.com>
Tue, 29 Jan 2013 23:26:35 +0000 (15:26 -0800)
We should set mappings only for usable memory ranges under max_pfn
Otherwise causes same problem that is fixed by

x86, mm: Only direct map addresses that are marked as E820_RAM

This patch exposes pfn_mapped array, and only sets ident mapping for ranges
in that array.

This patch relies on new kernel_ident_mapping_init that could handle existing
pgd/pud between different calls.

Signed-off-by: Yinghai Lu <yinghai@kernel.org>
Link: http://lkml.kernel.org/r/1359058816-7615-25-git-send-email-yinghai@kernel.org
Cc: Alexander Duyck <alexander.h.duyck@intel.com>
Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>
arch/x86/include/asm/page.h
arch/x86/kernel/machine_kexec_64.c
arch/x86/mm/init.c

index 8ca82839288a35e326f43fbac24a8384eb497652..100a20c7b98dbebc90f68fe6abc935e57c028552 100644 (file)
 
 struct page;
 
+#include <linux/range.h>
+extern struct range pfn_mapped[];
+extern int nr_pfn_mapped;
+
 static inline void clear_user_page(void *page, unsigned long vaddr,
                                   struct page *pg)
 {
index d2d7e023a8c81d017b96ad8bc22bee193679d956..4eabc160696f510ec5ffaddc7939dd71856463d5 100644 (file)
@@ -100,10 +100,15 @@ static int init_pgtable(struct kimage *image, unsigned long start_pgtable)
 
        level4p = (pgd_t *)__va(start_pgtable);
        clear_page(level4p);
-       result = kernel_ident_mapping_init(&info, level4p,
-                                               0, max_pfn << PAGE_SHIFT);
-       if (result)
-               return result;
+       for (i = 0; i < nr_pfn_mapped; i++) {
+               mstart = pfn_mapped[i].start << PAGE_SHIFT;
+               mend   = pfn_mapped[i].end << PAGE_SHIFT;
+
+               result = kernel_ident_mapping_init(&info,
+                                                level4p, mstart, mend);
+               if (result)
+                       return result;
+       }
 
        /*
         * segments's mem ranges could be outside 0 ~ max_pfn,
index 3364a7643a4cc99dba1c51ee87e074a0f42bd4c1..d41815265a0b250d35e1d03834d14439faa4e679 100644 (file)
@@ -302,8 +302,8 @@ static int __meminit split_mem_range(struct map_range *mr, int nr_range,
        return nr_range;
 }
 
-static struct range pfn_mapped[E820_X_MAX];
-static int nr_pfn_mapped;
+struct range pfn_mapped[E820_X_MAX];
+int nr_pfn_mapped;
 
 static void add_pfn_range_mapped(unsigned long start_pfn, unsigned long end_pfn)
 {