diff options
author | Linus Torvalds <torvalds@linux-foundation.org> | 2018-01-06 12:48:27 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2018-01-06 12:48:27 -0500 |
commit | 3219e264b984ec0a13923aa66385819c2e98d582 (patch) | |
tree | 4d344c40928cadce611504a439f33473338984a7 | |
parent | 65c64d1845fbc6420f8836e8731ea6c3012e8e7d (diff) | |
parent | ecb101aed86156ec7cd71e5dca668e09146e6994 (diff) |
Merge tag 'powerpc-4.15-6' of git://git.kernel.org/pub/scm/linux/kernel/git/powerpc/linux
Pull powerpc fix from Michael Ellerman:
"Just one fix to correctly return SEGV_ACCERR when we take a SEGV on a
mapped region. The bug was introduced in the refactoring of the page
fault handler we did in the previous release.
Thanks to John Sperbeck"
* tag 'powerpc-4.15-6' of git://git.kernel.org/pub/scm/linux/kernel/git/powerpc/linux:
powerpc/mm: Fix SEGV on mapped region to return SEGV_ACCERR
-rw-r--r-- | arch/powerpc/mm/fault.c | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/arch/powerpc/mm/fault.c b/arch/powerpc/mm/fault.c index 4797d08581ce..6e1e39035380 100644 --- a/arch/powerpc/mm/fault.c +++ b/arch/powerpc/mm/fault.c | |||
@@ -145,6 +145,11 @@ static noinline int bad_area(struct pt_regs *regs, unsigned long address) | |||
145 | return __bad_area(regs, address, SEGV_MAPERR); | 145 | return __bad_area(regs, address, SEGV_MAPERR); |
146 | } | 146 | } |
147 | 147 | ||
148 | static noinline int bad_access(struct pt_regs *regs, unsigned long address) | ||
149 | { | ||
150 | return __bad_area(regs, address, SEGV_ACCERR); | ||
151 | } | ||
152 | |||
148 | static int do_sigbus(struct pt_regs *regs, unsigned long address, | 153 | static int do_sigbus(struct pt_regs *regs, unsigned long address, |
149 | unsigned int fault) | 154 | unsigned int fault) |
150 | { | 155 | { |
@@ -490,7 +495,7 @@ retry: | |||
490 | 495 | ||
491 | good_area: | 496 | good_area: |
492 | if (unlikely(access_error(is_write, is_exec, vma))) | 497 | if (unlikely(access_error(is_write, is_exec, vma))) |
493 | return bad_area(regs, address); | 498 | return bad_access(regs, address); |
494 | 499 | ||
495 | /* | 500 | /* |
496 | * If for any reason at all we couldn't handle the fault, | 501 | * If for any reason at all we couldn't handle the fault, |