diff options
author | Jan Beulich <jbeulich@novell.com> | 2007-10-17 12:04:33 -0400 |
---|---|---|
committer | Thomas Gleixner <tglx@inhelltoy.tec.linutronix.de> | 2007-10-17 14:15:28 -0400 |
commit | aa506dc7b12d03fbf8fd11aab752aed1aadd9c07 (patch) | |
tree | a4f60e7fbbaf1782ceb958ef3e91c44a849336e1 | |
parent | d72b1b4f41b5159d2d0e54e54c794d500197572e (diff) |
i386: avoid temporarily inconsistent pte-s
One more of these issues (which were considered fixed a few releases
back): other than on x86-64, i386 allows set_fixmap() to replace
already present mappings. Consequently, on PAE, care must be taken to
not update the high half of a pte while the low half is still holding
the old value.
[ tglx: arch/x86 adaptation ]
Signed-off-by: Jan Beulich <jbeulich@novell.com>
Signed-off-by: Andi Kleen <ak@suse.de>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
arch/x86/mm/pgtable_32.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
-rw-r--r-- | arch/x86/mm/pgtable_32.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/arch/x86/mm/pgtable_32.c b/arch/x86/mm/pgtable_32.c index ef1f6cd3ea66..bbaa2b153934 100644 --- a/arch/x86/mm/pgtable_32.c +++ b/arch/x86/mm/pgtable_32.c | |||
@@ -97,8 +97,7 @@ static void set_pte_pfn(unsigned long vaddr, unsigned long pfn, pgprot_t flags) | |||
97 | } | 97 | } |
98 | pte = pte_offset_kernel(pmd, vaddr); | 98 | pte = pte_offset_kernel(pmd, vaddr); |
99 | if (pgprot_val(flags)) | 99 | if (pgprot_val(flags)) |
100 | /* <pfn,flags> stored as-is, to permit clearing entries */ | 100 | set_pte_present(&init_mm, vaddr, pte, pfn_pte(pfn, flags)); |
101 | set_pte(pte, pfn_pte(pfn, flags)); | ||
102 | else | 101 | else |
103 | pte_clear(&init_mm, vaddr, pte); | 102 | pte_clear(&init_mm, vaddr, pte); |
104 | 103 | ||