diff options
author | Jeremy Fitzhardinge <jeremy.fitzhardinge@citrix.com> | 2009-02-18 02:05:19 -0500 |
---|---|---|
committer | Jeremy Fitzhardinge <jeremy.fitzhardinge@citrix.com> | 2009-03-30 02:35:38 -0400 |
commit | b8bcfe997e46150fedcc3f5b26b846400122fdd9 (patch) | |
tree | 435f1bdfa3317e13ddcd3b87176602b597639c1b /arch/x86/mm/fault.c | |
parent | a8a93f3f03b7a8008d720e8d91798efe599d416c (diff) |
x86/paravirt: remove lazy mode in interrupts
Impact: simplification, robustness
Make paravirt_lazy_mode() always return PARAVIRT_LAZY_NONE
when in an interrupt. This prevents interrupt code from
accidentally inheriting an outer lazy state, and instead
does everything synchronously. Outer batched operations
are left deferred.
Signed-off-by: Jeremy Fitzhardinge <jeremy.fitzhardinge@citrix.com>
Acked-by: Peter Zijlstra <a.p.zijlstra@chello.nl>
Cc: Thomas Gleixner <tglx@linutronix.de>
Diffstat (limited to 'arch/x86/mm/fault.c')
-rw-r--r-- | arch/x86/mm/fault.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/arch/x86/mm/fault.c b/arch/x86/mm/fault.c index a03b7279efa0..cfbb4a738011 100644 --- a/arch/x86/mm/fault.c +++ b/arch/x86/mm/fault.c | |||
@@ -225,12 +225,10 @@ static inline pmd_t *vmalloc_sync_one(pgd_t *pgd, unsigned long address) | |||
225 | if (!pmd_present(*pmd_k)) | 225 | if (!pmd_present(*pmd_k)) |
226 | return NULL; | 226 | return NULL; |
227 | 227 | ||
228 | if (!pmd_present(*pmd)) { | 228 | if (!pmd_present(*pmd)) |
229 | set_pmd(pmd, *pmd_k); | 229 | set_pmd(pmd, *pmd_k); |
230 | arch_flush_lazy_mmu_mode(); | 230 | else |
231 | } else { | ||
232 | BUG_ON(pmd_page(*pmd) != pmd_page(*pmd_k)); | 231 | BUG_ON(pmd_page(*pmd) != pmd_page(*pmd_k)); |
233 | } | ||
234 | 232 | ||
235 | return pmd_k; | 233 | return pmd_k; |
236 | } | 234 | } |