diff options
author | Jeremy Fitzhardinge <jeremy@goop.org> | 2007-11-28 19:21:20 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@woody.linux-foundation.org> | 2007-11-29 12:24:52 -0500 |
commit | 2c80b01beae3db9f99a161ec216405dd694bc4c2 (patch) | |
tree | aea1b56634dad1467eca5a53a81b6a08d7b619b5 /arch/x86/xen/mmu.c | |
parent | c06869d660ceffec7dc0fca19ed9646ed5845c7f (diff) |
xen: mask _PAGE_PCD from ptes
_PAGE_PCD maps a page with caching disabled, which is typically used for
mapping harware registers. Xen never allows it to be set on a mapping, and
unprivileged guests never need it since they can't see the real underlying
hardware. However, some uncached mappings are made early when probing the
(non-existent) APIC, and its OK to mask off the PCD flag in these cases.
This became necessary because Xen started checking for this bit, rather
than silently masking it off.
Signed-off-by: Jeremy Fitzhardinge <jeremy@xensource.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'arch/x86/xen/mmu.c')
-rw-r--r-- | arch/x86/xen/mmu.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/arch/x86/xen/mmu.c b/arch/x86/xen/mmu.c index b2e32f9d0071..0ac6c5dc49ba 100644 --- a/arch/x86/xen/mmu.c +++ b/arch/x86/xen/mmu.c | |||
@@ -244,6 +244,8 @@ pte_t xen_make_pte(unsigned long long pte) | |||
244 | if (pte & 1) | 244 | if (pte & 1) |
245 | pte = phys_to_machine(XPADDR(pte)).maddr; | 245 | pte = phys_to_machine(XPADDR(pte)).maddr; |
246 | 246 | ||
247 | pte &= ~_PAGE_PCD; | ||
248 | |||
247 | return (pte_t){ pte, pte >> 32 }; | 249 | return (pte_t){ pte, pte >> 32 }; |
248 | } | 250 | } |
249 | 251 | ||
@@ -291,6 +293,8 @@ pte_t xen_make_pte(unsigned long pte) | |||
291 | if (pte & _PAGE_PRESENT) | 293 | if (pte & _PAGE_PRESENT) |
292 | pte = phys_to_machine(XPADDR(pte)).maddr; | 294 | pte = phys_to_machine(XPADDR(pte)).maddr; |
293 | 295 | ||
296 | pte &= ~_PAGE_PCD; | ||
297 | |||
294 | return (pte_t){ pte }; | 298 | return (pte_t){ pte }; |
295 | } | 299 | } |
296 | 300 | ||