diff options
author | Yinghai Lu <yinghai@kernel.org> | 2013-01-24 15:20:05 -0500 |
---|---|---|
committer | H. Peter Anvin <hpa@linux.intel.com> | 2013-01-29 18:26:35 -0500 |
commit | 0e691cf824f76adefb4498fe39c300aba2c2575a (patch) | |
tree | 6e61f59e7f100eac02a9aa7e064eaaa34e4e1904 /arch | |
parent | 9ebdc79f7a177d3098b89ba8ef2dd2b235163685 (diff) |
x86, kexec, 64bit: Only set ident mapping for ram.
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>
Diffstat (limited to 'arch')
-rw-r--r-- | arch/x86/include/asm/page.h | 4 | ||||
-rw-r--r-- | arch/x86/kernel/machine_kexec_64.c | 13 | ||||
-rw-r--r-- | arch/x86/mm/init.c | 4 |
3 files changed, 15 insertions, 6 deletions
diff --git a/arch/x86/include/asm/page.h b/arch/x86/include/asm/page.h index 8ca82839288a..100a20c7b98d 100644 --- a/arch/x86/include/asm/page.h +++ b/arch/x86/include/asm/page.h | |||
@@ -17,6 +17,10 @@ | |||
17 | 17 | ||
18 | struct page; | 18 | struct page; |
19 | 19 | ||
20 | #include <linux/range.h> | ||
21 | extern struct range pfn_mapped[]; | ||
22 | extern int nr_pfn_mapped; | ||
23 | |||
20 | static inline void clear_user_page(void *page, unsigned long vaddr, | 24 | static inline void clear_user_page(void *page, unsigned long vaddr, |
21 | struct page *pg) | 25 | struct page *pg) |
22 | { | 26 | { |
diff --git a/arch/x86/kernel/machine_kexec_64.c b/arch/x86/kernel/machine_kexec_64.c index d2d7e023a8c8..4eabc160696f 100644 --- a/arch/x86/kernel/machine_kexec_64.c +++ b/arch/x86/kernel/machine_kexec_64.c | |||
@@ -100,10 +100,15 @@ static int init_pgtable(struct kimage *image, unsigned long start_pgtable) | |||
100 | 100 | ||
101 | level4p = (pgd_t *)__va(start_pgtable); | 101 | level4p = (pgd_t *)__va(start_pgtable); |
102 | clear_page(level4p); | 102 | clear_page(level4p); |
103 | result = kernel_ident_mapping_init(&info, level4p, | 103 | for (i = 0; i < nr_pfn_mapped; i++) { |
104 | 0, max_pfn << PAGE_SHIFT); | 104 | mstart = pfn_mapped[i].start << PAGE_SHIFT; |
105 | if (result) | 105 | mend = pfn_mapped[i].end << PAGE_SHIFT; |
106 | return result; | 106 | |
107 | result = kernel_ident_mapping_init(&info, | ||
108 | level4p, mstart, mend); | ||
109 | if (result) | ||
110 | return result; | ||
111 | } | ||
107 | 112 | ||
108 | /* | 113 | /* |
109 | * segments's mem ranges could be outside 0 ~ max_pfn, | 114 | * segments's mem ranges could be outside 0 ~ max_pfn, |
diff --git a/arch/x86/mm/init.c b/arch/x86/mm/init.c index 3364a7643a4c..d41815265a0b 100644 --- a/arch/x86/mm/init.c +++ b/arch/x86/mm/init.c | |||
@@ -302,8 +302,8 @@ static int __meminit split_mem_range(struct map_range *mr, int nr_range, | |||
302 | return nr_range; | 302 | return nr_range; |
303 | } | 303 | } |
304 | 304 | ||
305 | static struct range pfn_mapped[E820_X_MAX]; | 305 | struct range pfn_mapped[E820_X_MAX]; |
306 | static int nr_pfn_mapped; | 306 | int nr_pfn_mapped; |
307 | 307 | ||
308 | static void add_pfn_range_mapped(unsigned long start_pfn, unsigned long end_pfn) | 308 | static void add_pfn_range_mapped(unsigned long start_pfn, unsigned long end_pfn) |
309 | { | 309 | { |