From: Peter Zijlstra <pet...@infradead.org>

Try a speculative fault before acquiring mmap_sem, if it returns with
VM_FAULT_RETRY continue with the mmap_sem acquisition and do the
traditional fault.

Signed-off-by: Peter Zijlstra (Intel) <pet...@infradead.org>

[Clearing of FAULT_FLAG_ALLOW_RETRY is now done in
 handle_speculative_fault()]
[Retry with usual fault path in the case VM_ERROR is returned by
 handle_speculative_fault(). This allows signal to be delivered]
[Don't build SPF call if !CONFIG_SPF]
Signed-off-by: Laurent Dufour <lduf...@linux.vnet.ibm.com>
---
 arch/x86/mm/fault.c | 21 +++++++++++++++++++++
 1 file changed, 21 insertions(+)

diff --git a/arch/x86/mm/fault.c b/arch/x86/mm/fault.c
index e2baeaa053a5..583e19d10b21 100644
--- a/arch/x86/mm/fault.c
+++ b/arch/x86/mm/fault.c
@@ -1364,6 +1364,24 @@ __do_page_fault(struct pt_regs *regs, unsigned long 
error_code,
        if (error_code & PF_INSTR)
                flags |= FAULT_FLAG_INSTRUCTION;
 
+#ifdef CONFIG_SPF
+       if (error_code & PF_USER) {
+               fault = handle_speculative_fault(mm, address, flags);
+
+               /*
+                * We also check against VM_FAULT_ERROR because we have to
+                * raise a signal by calling later mm_fault_error() which
+                * requires the vma pointer to be set. So in that case,
+                * we fall through the normal path.
+                */
+               if (!(fault & VM_FAULT_RETRY || fault & VM_FAULT_ERROR)) {
+                       perf_sw_event(PERF_COUNT_SW_SPF, 1,
+                                     regs, address);
+                       goto done;
+               }
+       }
+#endif /* CONFIG_SPF */
+
        /*
         * When running in the kernel we expect faults to occur only to
         * addresses in user space.  All other faults represent errors in
@@ -1474,6 +1492,9 @@ __do_page_fault(struct pt_regs *regs, unsigned long 
error_code,
                return;
        }
 
+#ifdef CONFIG_SPF
+done:
+#endif
        /*
         * Major/minor page fault accounting. If any of the events
         * returned VM_FAULT_MAJOR, we account it as a major fault.
-- 
2.7.4

Reply via email to