parisc: Don't call faulthandler_disabled() in do_page_fault()
authorJohn David Anglin <dave.anglin@bell.net>
Tue, 4 Jan 2022 21:34:26 +0000 (21:34 +0000)
committerHelge Deller <deller@gmx.de>
Fri, 7 Jan 2022 00:29:21 +0000 (01:29 +0100)
It is dangerous to call faulthandler_disabled() when user_mode(regs)
is true.  The task pagefault_disabled counter is racy and it is not
updated atomically on parisc. As a result, calling faulthandler_disabled()
may cause erroneous termination.

We now handle execption fixups and termination when user_mode(regs) is
false in handle_interruption(). Thus, we can just remove the
faulthandler_disabled() check from do_page_fault().

Signed-off-by: John David Anglin <dave.anglin@bell.net>
Signed-off-by: Helge Deller <deller@gmx.de>
arch/parisc/mm/fault.c

index 01fd2a3..499e2e8 100644 (file)
@@ -267,9 +267,6 @@ void do_page_fault(struct pt_regs *regs, unsigned long code,
        vm_fault_t fault = 0;
        unsigned int flags;
 
-       if (faulthandler_disabled())
-               goto no_context;
-
        tsk = current;
        mm = tsk->mm;
        if (!mm)