[src/netbsd-9]: src/sys/arch/arm/arm32 Pull up following revision(s) (request...

[martin <martin%NetBSD.org@localhost>]

details:   https://anonhg.NetBSD.org/src/rev/1fd5f8511e26
branches:  netbsd-9
changeset: 843629:1fd5f8511e26
user:      martin <martin%NetBSD.org@localhost>
date:      Sun Dec 08 14:31:57 2019 +0000

description:
Pull up following revision(s) (requested by ryo in ticket #510):

        sys/arch/arm/arm32/fault.c: revision 1.109

if Thumb-32 bit instruction located on a page boundariy, also need to consider the pc + 2 address.
Fix PR/54720. more detail and PoC are descrived in the PR.

diffstat:

 sys/arch/arm/arm32/fault.c |  19 +++++++++++++++++--
 1 files changed, 17 insertions(+), 2 deletions(-)

diffs (47 lines):

diff -r ff8f11ecfd75 -r 1fd5f8511e26 sys/arch/arm/arm32/fault.c
--- a/sys/arch/arm/arm32/fault.c        Sun Dec 08 14:29:36 2019 +0000
+++ b/sys/arch/arm/arm32/fault.c        Sun Dec 08 14:31:57 2019 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: fault.c,v 1.108 2019/04/06 03:06:25 thorpej Exp $      */
+/*     $NetBSD: fault.c,v 1.108.4.1 2019/12/08 14:31:57 martin Exp $   */
 
 /*
  * Copyright 2003 Wasabi Systems, Inc.
@@ -81,7 +81,7 @@
 #include "opt_kgdb.h"
 
 #include <sys/types.h>
-__KERNEL_RCSID(0, "$NetBSD: fault.c,v 1.108 2019/04/06 03:06:25 thorpej Exp $");
+__KERNEL_RCSID(0, "$NetBSD: fault.c,v 1.108.4.1 2019/12/08 14:31:57 martin Exp $");
 
 #include <sys/param.h>
 #include <sys/systm.h>
@@ -838,6 +838,9 @@
        UVMHIST_LOG(maphist, " (pc=0x%jx, l=0x%#jx, tf=0x%#jx)",
            fault_pc, (uintptr_t)l, (uintptr_t)tf, 0);
 
+#ifdef THUMB_CODE
+ recheck:
+#endif
        /* Ok validate the address, can only execute in USER space */
        if (__predict_false(fault_pc >= VM_MAXUSER_ADDRESS ||
            (fault_pc < VM_MIN_ADDRESS && vector_page == ARM_VECTORS_LOW))) {
@@ -897,6 +900,18 @@
        call_trapsignal(l, tf, &ksi);
 
 out:
+
+#ifdef THUMB_CODE
+#define THUMB_32BIT(hi) (((hi) & 0xe000) == 0xe000 && ((hi) & 0x1800))
+       /* thumb-32 instruction was located on page boundary? */
+       if ((tf->tf_spsr & PSR_T_bit) &&
+           ((fault_pc & PAGE_MASK) == (PAGE_SIZE - THUMB_INSN_SIZE)) &&
+           THUMB_32BIT(*(uint16_t *)tf->tf_pc)) {
+               fault_pc = tf->tf_pc + THUMB_INSN_SIZE;
+               goto recheck;
+       }
+#endif /* THUMB_CODE */
+
        KASSERT(!TRAP_USERMODE(tf) || VALID_R15_PSR(tf->tf_pc, tf->tf_spsr));
        userret(l);
 }