summaryrefslogtreecommitdiffstats
path: root/arch/x86
diff options
context:
space:
mode:
authorPeter Zijlstra <peterz@infradead.org>2009-01-29 16:02:12 +0100
committerIngo Molnar <mingo@elte.hu>2009-01-29 16:03:34 +0100
commit010060741ad35eacb504414bc6fb9bb575b15f62 (patch)
tree91c9a6f497e6a03635e73386bbce50cd56cfcacb /arch/x86
parentfb746d0e1365b7472ccc4c3d5b0672b34a092d0b (diff)
downloadop-kernel-dev-010060741ad35eacb504414bc6fb9bb575b15f62.zip
op-kernel-dev-010060741ad35eacb504414bc6fb9bb575b15f62.tar.gz
x86: add might_sleep() to do_page_fault()
Impact: widen debug checks VirtualBox calls do_page_fault() from an atomic context but runs into a might_sleep() way pas this point, cure that. Signed-off-by: Peter Zijlstra <a.p.zijlstra@chello.nl> Signed-off-by: Ingo Molnar <mingo@elte.hu>
Diffstat (limited to 'arch/x86')
-rw-r--r--arch/x86/mm/fault.c6
1 files changed, 6 insertions, 0 deletions
diff --git a/arch/x86/mm/fault.c b/arch/x86/mm/fault.c
index 8f4b859..eb4d7fe 100644
--- a/arch/x86/mm/fault.c
+++ b/arch/x86/mm/fault.c
@@ -888,6 +888,12 @@ void __kprobes do_page_fault(struct pt_regs *regs, unsigned long error_code)
return;
}
down_read(&mm->mmap_sem);
+ } else {
+ /*
+ * The above down_read_trylock() might have succeeded in which
+ * case we'll have missed the might_sleep() from down_read().
+ */
+ might_sleep();
}
vma = find_vma(mm, address);
OpenPOWER on IntegriCloud