ARM: Remove use of struct kprobe from generic probes code
authorDavid A. Long <dave.long@linaro.org>
Fri, 7 Mar 2014 16:19:32 +0000 (11:19 -0500)
committerDavid A. Long <dave.long@linaro.org>
Tue, 18 Mar 2014 20:39:37 +0000 (16:39 -0400)
Change the generic ARM probes code to pass in the opcode and architecture-specific
structure separately instead of using struct kprobe, so we do not pollute
code being used only for uprobes or other non-kprobes instruction
interpretation.

Signed-off-by: David A. Long <dave.long@linaro.org>
Acked-by: Jon Medhurst <tixy@linaro.org>
arch/arm/include/asm/probes.h
arch/arm/kernel/kprobes-arm.c
arch/arm/kernel/kprobes-common.c
arch/arm/kernel/kprobes-thumb.c
arch/arm/kernel/kprobes.c
arch/arm/kernel/probes-arm.c
arch/arm/kernel/probes-arm.h
arch/arm/kernel/probes-thumb.c
arch/arm/kernel/probes.c
arch/arm/kernel/probes.h

index 737a9b3..4d014c4 100644 (file)
 
 struct kprobe;
 
-typedef void (kprobe_insn_handler_t)(struct kprobe *, struct pt_regs *);
+struct arch_specific_insn;
+typedef void (kprobe_insn_handler_t)(kprobe_opcode_t,
+                                    struct arch_specific_insn *,
+                                    struct pt_regs *);
 typedef unsigned long (kprobe_check_cc)(unsigned long);
-typedef void (kprobe_insn_singlestep_t)(struct kprobe *, struct pt_regs *);
+typedef void (kprobe_insn_singlestep_t)(kprobe_opcode_t,
+                                       struct arch_specific_insn *,
+                                       struct pt_regs *);
 typedef void (kprobe_insn_fn_t)(void);
 
 /* Architecture specific copy of original instruction. */
index 8ebd84c..4a232e6 100644 (file)
                        "mov    pc, "reg"       \n\t"
 #endif
 
-
 static void __kprobes
-emulate_ldrdstrd(struct kprobe *p, struct pt_regs *regs)
+emulate_ldrdstrd(kprobe_opcode_t insn,
+       struct arch_specific_insn *asi, struct pt_regs *regs)
 {
-       kprobe_opcode_t insn = p->opcode;
-       unsigned long pc = (unsigned long)p->addr + 8;
+       unsigned long pc = regs->ARM_pc + 4;
        int rt = (insn >> 12) & 0xf;
        int rn = (insn >> 16) & 0xf;
        int rm = insn & 0xf;
@@ -92,7 +91,7 @@ emulate_ldrdstrd(struct kprobe *p, struct pt_regs *regs)
                BLX("%[fn]")
                : "=r" (rtv), "=r" (rt2v), "=r" (rnv)
                : "0" (rtv), "1" (rt2v), "2" (rnv), "r" (rmv),
-                 [fn] "r" (p->ainsn.insn_fn)
+                 [fn] "r" (asi->insn_fn)
                : "lr", "memory", "cc"
        );
 
@@ -103,10 +102,10 @@ emulate_ldrdstrd(struct kprobe *p, struct pt_regs *regs)
 }
 
 static void __kprobes
-emulate_ldr(struct kprobe *p, struct pt_regs *regs)
+emulate_ldr(kprobe_opcode_t insn,
+       struct arch_specific_insn *asi, struct pt_regs *regs)
 {
-       kprobe_opcode_t insn = p->opcode;
-       unsigned long pc = (unsigned long)p->addr + 8;
+       unsigned long pc = regs->ARM_pc + 4;
        int rt = (insn >> 12) & 0xf;
        int rn = (insn >> 16) & 0xf;
        int rm = insn & 0xf;
@@ -119,7 +118,7 @@ emulate_ldr(struct kprobe *p, struct pt_regs *regs)
        __asm__ __volatile__ (
                BLX("%[fn]")
                : "=r" (rtv), "=r" (rnv)
-               : "1" (rnv), "r" (rmv), [fn] "r" (p->ainsn.insn_fn)
+               : "1" (rnv), "r" (rmv), [fn] "r" (asi->insn_fn)
                : "lr", "memory", "cc"
        );
 
@@ -133,11 +132,11 @@ emulate_ldr(struct kprobe *p, struct pt_regs *regs)
 }
 
 static void __kprobes
-emulate_str(struct kprobe *p, struct pt_regs *regs)
+emulate_str(kprobe_opcode_t insn,
+       struct arch_specific_insn *asi, struct pt_regs *regs)
 {
-       kprobe_opcode_t insn = p->opcode;
-       unsigned long rtpc = (unsigned long)p->addr + str_pc_offset;
-       unsigned long rnpc = (unsigned long)p->addr + 8;
+       unsigned long rtpc = regs->ARM_pc - 4 + str_pc_offset;
+       unsigned long rnpc = regs->ARM_pc + 4;
        int rt = (insn >> 12) & 0xf;
        int rn = (insn >> 16) & 0xf;
        int rm = insn & 0xf;
@@ -151,7 +150,7 @@ emulate_str(struct kprobe *p, struct pt_regs *regs)
        __asm__ __volatile__ (
                BLX("%[fn]")
                : "=r" (rnv)
-               : "r" (rtv), "0" (rnv), "r" (rmv), [fn] "r" (p->ainsn.insn_fn)
+               : "r" (rtv), "0" (rnv), "r" (rmv), [fn] "r" (asi->insn_fn)
                : "lr", "memory", "cc"
        );
 
@@ -160,10 +159,10 @@ emulate_str(struct kprobe *p, struct pt_regs *regs)
 }
 
 static void __kprobes
-emulate_rd12rn16rm0rs8_rwflags(struct kprobe *p, struct pt_regs *regs)
+emulate_rd12rn16rm0rs8_rwflags(kprobe_opcode_t insn,
+       struct arch_specific_insn *asi, struct pt_regs *regs)
 {
-       kprobe_opcode_t insn = p->opcode;
-       unsigned long pc = (unsigned long)p->addr + 8;
+       unsigned long pc = regs->ARM_pc + 4;
        int rd = (insn >> 12) & 0xf;
        int rn = (insn >> 16) & 0xf;
        int rm = insn & 0xf;
@@ -183,7 +182,7 @@ emulate_rd12rn16rm0rs8_rwflags(struct kprobe *p, struct pt_regs *regs)
                "mrs    %[cpsr], cpsr           \n\t"
                : "=r" (rdv), [cpsr] "=r" (cpsr)
                : "0" (rdv), "r" (rnv), "r" (rmv), "r" (rsv),
-                 "1" (cpsr), [fn] "r" (p->ainsn.insn_fn)
+                 "1" (cpsr), [fn] "r" (asi->insn_fn)
                : "lr", "memory", "cc"
        );
 
@@ -195,9 +194,9 @@ emulate_rd12rn16rm0rs8_rwflags(struct kprobe *p, struct pt_regs *regs)
 }
 
 static void __kprobes
-emulate_rd12rn16rm0_rwflags_nopc(struct kprobe *p, struct pt_regs *regs)
+emulate_rd12rn16rm0_rwflags_nopc(kprobe_opcode_t insn,
+       struct arch_specific_insn *asi, struct pt_regs *regs)
 {
-       kprobe_opcode_t insn = p->opcode;
        int rd = (insn >> 12) & 0xf;
        int rn = (insn >> 16) & 0xf;
        int rm = insn & 0xf;
@@ -213,7 +212,7 @@ emulate_rd12rn16rm0_rwflags_nopc(struct kprobe *p, struct pt_regs *regs)
                "mrs    %[cpsr], cpsr           \n\t"
                : "=r" (rdv), [cpsr] "=r" (cpsr)
                : "0" (rdv), "r" (rnv), "r" (rmv),
-                 "1" (cpsr), [fn] "r" (p->ainsn.insn_fn)
+                 "1" (cpsr), [fn] "r" (asi->insn_fn)
                : "lr", "memory", "cc"
        );
 
@@ -222,9 +221,10 @@ emulate_rd12rn16rm0_rwflags_nopc(struct kprobe *p, struct pt_regs *regs)
 }
 
 static void __kprobes
-emulate_rd16rn12rm0rs8_rwflags_nopc(struct kprobe *p, struct pt_regs *regs)
+emulate_rd16rn12rm0rs8_rwflags_nopc(kprobe_opcode_t insn,
+       struct arch_specific_insn *asi,
+       struct pt_regs *regs)
 {
-       kprobe_opcode_t insn = p->opcode;
        int rd = (insn >> 16) & 0xf;
        int rn = (insn >> 12) & 0xf;
        int rm = insn & 0xf;
@@ -242,7 +242,7 @@ emulate_rd16rn12rm0rs8_rwflags_nopc(struct kprobe *p, struct pt_regs *regs)
                "mrs    %[cpsr], cpsr           \n\t"
                : "=r" (rdv), [cpsr] "=r" (cpsr)
                : "0" (rdv), "r" (rnv), "r" (rmv), "r" (rsv),
-                 "1" (cpsr), [fn] "r" (p->ainsn.insn_fn)
+                 "1" (cpsr), [fn] "r" (asi->insn_fn)
                : "lr", "memory", "cc"
        );
 
@@ -251,9 +251,9 @@ emulate_rd16rn12rm0rs8_rwflags_nopc(struct kprobe *p, struct pt_regs *regs)
 }
 
 static void __kprobes
-emulate_rd12rm0_noflags_nopc(struct kprobe *p, struct pt_regs *regs)
+emulate_rd12rm0_noflags_nopc(kprobe_opcode_t insn,
+       struct arch_specific_insn *asi, struct pt_regs *regs)
 {
-       kprobe_opcode_t insn = p->opcode;
        int rd = (insn >> 12) & 0xf;
        int rm = insn & 0xf;
 
@@ -263,7 +263,7 @@ emulate_rd12rm0_noflags_nopc(struct kprobe *p, struct pt_regs *regs)
        __asm__ __volatile__ (
                BLX("%[fn]")
                : "=r" (rdv)
-               : "0" (rdv), "r" (rmv), [fn] "r" (p->ainsn.insn_fn)
+               : "0" (rdv), "r" (rmv), [fn] "r" (asi->insn_fn)
                : "lr", "memory", "cc"
        );
 
@@ -271,9 +271,10 @@ emulate_rd12rm0_noflags_nopc(struct kprobe *p, struct pt_regs *regs)
 }
 
 static void __kprobes
-emulate_rdlo12rdhi16rn0rm8_rwflags_nopc(struct kprobe *p, struct pt_regs *regs)
+emulate_rdlo12rdhi16rn0rm8_rwflags_nopc(kprobe_opcode_t insn,
+       struct arch_specific_insn *asi,
+       struct pt_regs *regs)
 {
-       kprobe_opcode_t insn = p->opcode;
        int rdlo = (insn >> 12) & 0xf;
        int rdhi = (insn >> 16) & 0xf;
        int rn = insn & 0xf;
@@ -291,7 +292,7 @@ emulate_rdlo12rdhi16rn0rm8_rwflags_nopc(struct kprobe *p, struct pt_regs *regs)
                "mrs    %[cpsr], cpsr           \n\t"
                : "=r" (rdlov), "=r" (rdhiv), [cpsr] "=r" (cpsr)
                : "0" (rdlov), "1" (rdhiv), "r" (rnv), "r" (rmv),
-                 "2" (cpsr), [fn] "r" (p->ainsn.insn_fn)
+                 "2" (cpsr), [fn] "r" (asi->insn_fn)
                : "lr", "memory", "cc"
        );
 
index 029b79c..abe0389 100644 (file)
 #include "kprobes.h"
 
 
-static void __kprobes simulate_ldm1stm1(struct kprobe *p, struct pt_regs *regs)
+static void __kprobes simulate_ldm1stm1(kprobe_opcode_t insn,
+               struct arch_specific_insn *asi,
+               struct pt_regs *regs)
 {
-       kprobe_opcode_t insn = p->opcode;
        int rn = (insn >> 16) & 0xf;
        int lbit = insn & (1 << 20);
        int wbit = insn & (1 << 21);
@@ -58,24 +59,31 @@ static void __kprobes simulate_ldm1stm1(struct kprobe *p, struct pt_regs *regs)
        }
 }
 
-static void __kprobes simulate_stm1_pc(struct kprobe *p, struct pt_regs *regs)
+static void __kprobes simulate_stm1_pc(kprobe_opcode_t insn,
+       struct arch_specific_insn *asi,
+       struct pt_regs *regs)
 {
-       regs->ARM_pc = (long)p->addr + str_pc_offset;
-       simulate_ldm1stm1(p, regs);
-       regs->ARM_pc = (long)p->addr + 4;
+       unsigned long addr = regs->ARM_pc - 4;
+
+       regs->ARM_pc = (long)addr + str_pc_offset;
+       simulate_ldm1stm1(insn, asi, regs);
+       regs->ARM_pc = (long)addr + 4;
 }
 
-static void __kprobes simulate_ldm1_pc(struct kprobe *p, struct pt_regs *regs)
+static void __kprobes simulate_ldm1_pc(kprobe_opcode_t insn,
+       struct arch_specific_insn *asi,
+       struct pt_regs *regs)
 {
-       simulate_ldm1stm1(p, regs);
+       simulate_ldm1stm1(insn, asi, regs);
        load_write_pc(regs->ARM_pc, regs);
 }
 
 static void __kprobes
-emulate_generic_r0_12_noflags(struct kprobe *p, struct pt_regs *regs)
+emulate_generic_r0_12_noflags(kprobe_opcode_t insn,
+       struct arch_specific_insn *asi, struct pt_regs *regs)
 {
        register void *rregs asm("r1") = regs;
-       register void *rfn asm("lr") = p->ainsn.insn_fn;
+       register void *rfn asm("lr") = asi->insn_fn;
 
        __asm__ __volatile__ (
                "stmdb  sp!, {%[regs], r11}     \n\t"
@@ -99,15 +107,19 @@ emulate_generic_r0_12_noflags(struct kprobe *p, struct pt_regs *regs)
 }
 
 static void __kprobes
-emulate_generic_r2_14_noflags(struct kprobe *p, struct pt_regs *regs)
+emulate_generic_r2_14_noflags(kprobe_opcode_t insn,
+       struct arch_specific_insn *asi, struct pt_regs *regs)
 {
-       emulate_generic_r0_12_noflags(p, (struct pt_regs *)(regs->uregs+2));
+       emulate_generic_r0_12_noflags(insn, asi,
+               (struct pt_regs *)(regs->uregs+2));
 }
 
 static void __kprobes
-emulate_ldm_r3_15(struct kprobe *p, struct pt_regs *regs)
+emulate_ldm_r3_15(kprobe_opcode_t insn,
+       struct arch_specific_insn *asi, struct pt_regs *regs)
 {
-       emulate_generic_r0_12_noflags(p, (struct pt_regs *)(regs->uregs+3));
+       emulate_generic_r0_12_noflags(insn, asi,
+               (struct pt_regs *)(regs->uregs+3));
        load_write_pc(regs->ARM_pc, regs);
 }
 
index d83f609..adc08f8 100644 (file)
 #define t32_emulate_rd8rn16rm0ra12_noflags \
                t32_emulate_rdlo12rdhi8rn16rm0_noflags
 
-/*
- * Return the PC value for a probe in thumb code.
- * This is the address of the probed instruction plus 4.
- * We subtract one because the address will have bit zero set to indicate
- * a pointer to thumb code.
- */
-static inline unsigned long __kprobes thumb_probe_pc(struct kprobe *p)
-{
-       return (unsigned long)p->addr - 1 + 4;
-}
-
 /* t32 thumb actions */
 
 static void __kprobes
-t32_simulate_table_branch(struct kprobe *p, struct pt_regs *regs)
+t32_simulate_table_branch(kprobe_opcode_t insn,
+               struct arch_specific_insn *asi, struct pt_regs *regs)
 {
-       kprobe_opcode_t insn = p->opcode;
-       unsigned long pc = thumb_probe_pc(p);
+       unsigned long pc = regs->ARM_pc;
        int rn = (insn >> 16) & 0xf;
        int rm = insn & 0xf;
 
@@ -54,19 +43,19 @@ t32_simulate_table_branch(struct kprobe *p, struct pt_regs *regs)
 }
 
 static void __kprobes
-t32_simulate_mrs(struct kprobe *p, struct pt_regs *regs)
+t32_simulate_mrs(kprobe_opcode_t insn,
+               struct arch_specific_insn *asi, struct pt_regs *regs)
 {
-       kprobe_opcode_t insn = p->opcode;
        int rd = (insn >> 8) & 0xf;
        unsigned long mask = 0xf8ff03df; /* Mask out execution state */
        regs->uregs[rd] = regs->ARM_cpsr & mask;
 }
 
 static void __kprobes
-t32_simulate_cond_branch(struct kprobe *p, struct pt_regs *regs)
+t32_simulate_cond_branch(kprobe_opcode_t insn,
+               struct arch_specific_insn *asi, struct pt_regs *regs)
 {
-       kprobe_opcode_t insn = p->opcode;
-       unsigned long pc = thumb_probe_pc(p);
+       unsigned long pc = regs->ARM_pc;
 
        long offset = insn & 0x7ff;             /* imm11 */
        offset += (insn & 0x003f0000) >> 5;     /* imm6 */
@@ -88,10 +77,10 @@ t32_decode_cond_branch(kprobe_opcode_t insn, struct arch_specific_insn *asi,
 }
 
 static void __kprobes
-t32_simulate_branch(struct kprobe *p, struct pt_regs *regs)
+t32_simulate_branch(kprobe_opcode_t insn,
+                   struct arch_specific_insn *asi, struct pt_regs *regs)
 {
-       kprobe_opcode_t insn = p->opcode;
-       unsigned long pc = thumb_probe_pc(p);
+       unsigned long pc = regs->ARM_pc;
 
        long offset = insn & 0x7ff;             /* imm11 */
        offset += (insn & 0x03ff0000) >> 5;     /* imm10 */
@@ -104,7 +93,7 @@ t32_simulate_branch(struct kprobe *p, struct pt_regs *regs)
 
        if (insn & (1 << 14)) {
                /* BL or BLX */
-               regs->ARM_lr = (unsigned long)p->addr + 4;
+               regs->ARM_lr = regs->ARM_pc | 1;
                if (!(insn & (1 << 12))) {
                        /* BLX so switch to ARM mode */
                        regs->ARM_cpsr &= ~PSR_T_BIT;
@@ -116,10 +105,10 @@ t32_simulate_branch(struct kprobe *p, struct pt_regs *regs)
 }
 
 static void __kprobes
-t32_simulate_ldr_literal(struct kprobe *p, struct pt_regs *regs)
+t32_simulate_ldr_literal(kprobe_opcode_t insn,
+               struct arch_specific_insn *asi, struct pt_regs *regs)
 {
-       kprobe_opcode_t insn = p->opcode;
-       unsigned long addr = thumb_probe_pc(p) & ~3;
+       unsigned long addr = regs->ARM_pc & ~3;
        int rt = (insn >> 12) & 0xf;
        unsigned long rtv;
 
@@ -168,10 +157,10 @@ t32_decode_ldmstm(kprobe_opcode_t insn, struct arch_specific_insn *asi,
 }
 
 static void __kprobes
-t32_emulate_ldrdstrd(struct kprobe *p, struct pt_regs *regs)
+t32_emulate_ldrdstrd(kprobe_opcode_t insn,
+               struct arch_specific_insn *asi, struct pt_regs *regs)
 {
-       kprobe_opcode_t insn = p->opcode;
-       unsigned long pc = thumb_probe_pc(p) & ~3;
+       unsigned long pc = regs->ARM_pc & ~3;
        int rt1 = (insn >> 12) & 0xf;
        int rt2 = (insn >> 8) & 0xf;
        int rn = (insn >> 16) & 0xf;
@@ -184,7 +173,7 @@ t32_emulate_ldrdstrd(struct kprobe *p, struct pt_regs *regs)
        __asm__ __volatile__ (
                "blx    %[fn]"
                : "=r" (rt1v), "=r" (rt2v), "=r" (rnv)
-               : "0" (rt1v), "1" (rt2v), "2" (rnv), [fn] "r" (p->ainsn.insn_fn)
+               : "0" (rt1v), "1" (rt2v), "2" (rnv), [fn] "r" (asi->insn_fn)
                : "lr", "memory", "cc"
        );
 
@@ -195,9 +184,9 @@ t32_emulate_ldrdstrd(struct kprobe *p, struct pt_regs *regs)
 }
 
 static void __kprobes
-t32_emulate_ldrstr(struct kprobe *p, struct pt_regs *regs)
+t32_emulate_ldrstr(kprobe_opcode_t insn,
+               struct arch_specific_insn *asi, struct pt_regs *regs)
 {
-       kprobe_opcode_t insn = p->opcode;
        int rt = (insn >> 12) & 0xf;
        int rn = (insn >> 16) & 0xf;
        int rm = insn & 0xf;
@@ -209,7 +198,7 @@ t32_emulate_ldrstr(struct kprobe *p, struct pt_regs *regs)
        __asm__ __volatile__ (
                "blx    %[fn]"
                : "=r" (rtv), "=r" (rnv)
-               : "0" (rtv), "1" (rnv), "r" (rmv), [fn] "r" (p->ainsn.insn_fn)
+               : "0" (rtv), "1" (rnv), "r" (rmv), [fn] "r" (asi->insn_fn)
                : "lr", "memory", "cc"
        );
 
@@ -221,9 +210,9 @@ t32_emulate_ldrstr(struct kprobe *p, struct pt_regs *regs)
 }
 
 static void __kprobes
-t32_emulate_rd8rn16rm0_rwflags(struct kprobe *p, struct pt_regs *regs)
+t32_emulate_rd8rn16rm0_rwflags(kprobe_opcode_t insn,
+               struct arch_specific_insn *asi, struct pt_regs *regs)
 {
-       kprobe_opcode_t insn = p->opcode;
        int rd = (insn >> 8) & 0xf;
        int rn = (insn >> 16) & 0xf;
        int rm = insn & 0xf;
@@ -239,7 +228,7 @@ t32_emulate_rd8rn16rm0_rwflags(struct kprobe *p, struct pt_regs *regs)
                "mrs    %[cpsr], cpsr           \n\t"
                : "=r" (rdv), [cpsr] "=r" (cpsr)
                : "0" (rdv), "r" (rnv), "r" (rmv),
-                 "1" (cpsr), [fn] "r" (p->ainsn.insn_fn)
+                 "1" (cpsr), [fn] "r" (asi->insn_fn)
                : "lr", "memory", "cc"
        );
 
@@ -248,10 +237,10 @@ t32_emulate_rd8rn16rm0_rwflags(struct kprobe *p, struct pt_regs *regs)
 }
 
 static void __kprobes
-t32_emulate_rd8pc16_noflags(struct kprobe *p, struct pt_regs *regs)
+t32_emulate_rd8pc16_noflags(kprobe_opcode_t insn,
+               struct arch_specific_insn *asi, struct pt_regs *regs)
 {
-       kprobe_opcode_t insn = p->opcode;
-       unsigned long pc = thumb_probe_pc(p);
+       unsigned long pc = regs->ARM_pc;
        int rd = (insn >> 8) & 0xf;
 
        register unsigned long rdv asm("r1") = regs->uregs[rd];
@@ -260,7 +249,7 @@ t32_emulate_rd8pc16_noflags(struct kprobe *p, struct pt_regs *regs)
        __asm__ __volatile__ (
                "blx    %[fn]"
                : "=r" (rdv)
-               : "0" (rdv), "r" (rnv), [fn] "r" (p->ainsn.insn_fn)
+               : "0" (rdv), "r" (rnv), [fn] "r" (asi->insn_fn)
                : "lr", "memory", "cc"
        );
 
@@ -268,9 +257,9 @@ t32_emulate_rd8pc16_noflags(struct kprobe *p, struct pt_regs *regs)
 }
 
 static void __kprobes
-t32_emulate_rd8rn16_noflags(struct kprobe *p, struct pt_regs *regs)
+t32_emulate_rd8rn16_noflags(kprobe_opcode_t insn,
+               struct arch_specific_insn *asi, struct pt_regs *regs)
 {
-       kprobe_opcode_t insn = p->opcode;
        int rd = (insn >> 8) & 0xf;
        int rn = (insn >> 16) & 0xf;
 
@@ -280,7 +269,7 @@ t32_emulate_rd8rn16_noflags(struct kprobe *p, struct pt_regs *regs)
        __asm__ __volatile__ (
                "blx    %[fn]"
                : "=r" (rdv)
-               : "0" (rdv), "r" (rnv), [fn] "r" (p->ainsn.insn_fn)
+               : "0" (rdv), "r" (rnv), [fn] "r" (asi->insn_fn)
                : "lr", "memory", "cc"
        );
 
@@ -288,9 +277,10 @@ t32_emulate_rd8rn16_noflags(struct kprobe *p, struct pt_regs *regs)
 }
 
 static void __kprobes
-t32_emulate_rdlo12rdhi8rn16rm0_noflags(struct kprobe *p, struct pt_regs *regs)
+t32_emulate_rdlo12rdhi8rn16rm0_noflags(kprobe_opcode_t insn,
+               struct arch_specific_insn *asi,
+               struct pt_regs *regs)
 {
-       kprobe_opcode_t insn = p->opcode;
        int rdlo = (insn >> 12) & 0xf;
        int rdhi = (insn >> 8) & 0xf;
        int rn = (insn >> 16) & 0xf;
@@ -305,7 +295,7 @@ t32_emulate_rdlo12rdhi8rn16rm0_noflags(struct kprobe *p, struct pt_regs *regs)
                "blx    %[fn]"
                : "=r" (rdlov), "=r" (rdhiv)
                : "0" (rdlov), "1" (rdhiv), "r" (rnv), "r" (rmv),
-                 [fn] "r" (p->ainsn.insn_fn)
+                 [fn] "r" (asi->insn_fn)
                : "lr", "memory", "cc"
        );
 
@@ -315,33 +305,33 @@ t32_emulate_rdlo12rdhi8rn16rm0_noflags(struct kprobe *p, struct pt_regs *regs)
 /* t16 thumb actions */
 
 static void __kprobes
-t16_simulate_bxblx(struct kprobe *p, struct pt_regs *regs)
+t16_simulate_bxblx(kprobe_opcode_t insn,
+               struct arch_specific_insn *asi, struct pt_regs *regs)
 {
-       kprobe_opcode_t insn = p->opcode;
-       unsigned long pc = thumb_probe_pc(p);
+       unsigned long pc = regs->ARM_pc + 2;
        int rm = (insn >> 3) & 0xf;
        unsigned long rmv = (rm == 15) ? pc : regs->uregs[rm];
 
        if (insn & (1 << 7)) /* BLX ? */
-               regs->ARM_lr = (unsigned long)p->addr + 2;
+               regs->ARM_lr = regs->ARM_pc | 1;
 
        bx_write_pc(rmv, regs);
 }
 
 static void __kprobes
-t16_simulate_ldr_literal(struct kprobe *p, struct pt_regs *regs)
+t16_simulate_ldr_literal(kprobe_opcode_t insn,
+               struct arch_specific_insn *asi, struct pt_regs *regs)
 {
-       kprobe_opcode_t insn = p->opcode;
-       unsigned long* base = (unsigned long *)(thumb_probe_pc(p) & ~3);
+       unsigned long *base = (unsigned long *)((regs->ARM_pc + 2) & ~3);
        long index = insn & 0xff;
        int rt = (insn >> 8) & 0x7;
        regs->uregs[rt] = base[index];
 }
 
 static void __kprobes
-t16_simulate_ldrstr_sp_relative(struct kprobe *p, struct pt_regs *regs)
+t16_simulate_ldrstr_sp_relative(kprobe_opcode_t insn,
+               struct arch_specific_insn *asi, struct pt_regs *regs)
 {
-       kprobe_opcode_t insn = p->opcode;
        unsigned long* base = (unsigned long *)regs->ARM_sp;
        long index = insn & 0xff;
        int rt = (insn >> 8) & 0x7;
@@ -352,20 +342,20 @@ t16_simulate_ldrstr_sp_relative(struct kprobe *p, struct pt_regs *regs)
 }
 
 static void __kprobes
-t16_simulate_reladr(struct kprobe *p, struct pt_regs *regs)
+t16_simulate_reladr(kprobe_opcode_t insn,
+               struct arch_specific_insn *asi, struct pt_regs *regs)
 {
-       kprobe_opcode_t insn = p->opcode;
        unsigned long base = (insn & 0x800) ? regs->ARM_sp
-                                           : (thumb_probe_pc(p) & ~3);
+                                           : ((regs->ARM_pc + 2) & ~3);
        long offset = insn & 0xff;
        int rt = (insn >> 8) & 0x7;
        regs->uregs[rt] = base + offset * 4;
 }
 
 static void __kprobes
-t16_simulate_add_sp_imm(struct kprobe *p, struct pt_regs *regs)
+t16_simulate_add_sp_imm(kprobe_opcode_t insn,
+               struct arch_specific_insn *asi, struct pt_regs *regs)
 {
-       kprobe_opcode_t insn = p->opcode;
        long imm = insn & 0x7f;
        if (insn & 0x80) /* SUB */
                regs->ARM_sp -= imm * 4;
@@ -374,21 +364,22 @@ t16_simulate_add_sp_imm(struct kprobe *p, struct pt_regs *regs)
 }
 
 static void __kprobes
-t16_simulate_cbz(struct kprobe *p, struct pt_regs *regs)
+t16_simulate_cbz(kprobe_opcode_t insn,
+               struct arch_specific_insn *asi, struct pt_regs *regs)
 {
-       kprobe_opcode_t insn = p->opcode;
        int rn = insn & 0x7;
        kprobe_opcode_t nonzero = regs->uregs[rn] ? insn : ~insn;
        if (nonzero & 0x800) {
                long i = insn & 0x200;
                long imm5 = insn & 0xf8;
-               unsigned long pc = thumb_probe_pc(p);
+               unsigned long pc = regs->ARM_pc + 2;
                regs->ARM_pc = pc + (i >> 3) + (imm5 >> 2);
        }
 }
 
 static void __kprobes
-t16_simulate_it(struct kprobe *p, struct pt_regs *regs)
+t16_simulate_it(kprobe_opcode_t insn,
+               struct arch_specific_insn *asi, struct pt_regs *regs)
 {
        /*
         * The 8 IT state bits are split into two parts in CPSR:
@@ -396,7 +387,6 @@ t16_simulate_it(struct kprobe *p, struct pt_regs *regs)
         *      ITSTATE<7:2> are in CPSR<15:10>
         * The new IT state is in the lower byte of insn.
         */
-       kprobe_opcode_t insn = p->opcode;
        unsigned long cpsr = regs->ARM_cpsr;
        cpsr &= ~PSR_IT_MASK;
        cpsr |= (insn & 0xfc) << 8;
@@ -405,10 +395,11 @@ t16_simulate_it(struct kprobe *p, struct pt_regs *regs)
 }
 
 static void __kprobes
-t16_singlestep_it(struct kprobe *p, struct pt_regs *regs)
+t16_singlestep_it(kprobe_opcode_t insn,
+                 struct arch_specific_insn *asi, struct pt_regs *regs)
 {
        regs->ARM_pc += 2;
-       t16_simulate_it(p, regs);
+       t16_simulate_it(insn, asi, regs);
 }
 
 static enum kprobe_insn __kprobes
@@ -420,10 +411,10 @@ t16_decode_it(kprobe_opcode_t insn, struct arch_specific_insn *asi,
 }
 
 static void __kprobes
-t16_simulate_cond_branch(struct kprobe *p, struct pt_regs *regs)
+t16_simulate_cond_branch(kprobe_opcode_t insn,
+               struct arch_specific_insn *asi, struct pt_regs *regs)
 {
-       kprobe_opcode_t insn = p->opcode;
-       unsigned long pc = thumb_probe_pc(p);
+       unsigned long pc = regs->ARM_pc + 2;
        long offset = insn & 0x7f;
        offset -= insn & 0x80; /* Apply sign bit */
        regs->ARM_pc = pc + (offset * 2);
@@ -440,17 +431,18 @@ t16_decode_cond_branch(kprobe_opcode_t insn, struct arch_specific_insn *asi,
 }
 
 static void __kprobes
-t16_simulate_branch(struct kprobe *p, struct pt_regs *regs)
+t16_simulate_branch(kprobe_opcode_t insn,
+                  struct arch_specific_insn *asi, struct pt_regs *regs)
 {
-       kprobe_opcode_t insn = p->opcode;
-       unsigned long pc = thumb_probe_pc(p);
+       unsigned long pc = regs->ARM_pc + 2;
        long offset = insn & 0x3ff;
        offset -= insn & 0x400; /* Apply sign bit */
        regs->ARM_pc = pc + (offset * 2);
 }
 
 static unsigned long __kprobes
-t16_emulate_loregs(struct kprobe *p, struct pt_regs *regs)
+t16_emulate_loregs(kprobe_opcode_t insn,
+                  struct arch_specific_insn *asi, struct pt_regs *regs)
 {
        unsigned long oldcpsr = regs->ARM_cpsr;
        unsigned long newcpsr;
@@ -463,7 +455,7 @@ t16_emulate_loregs(struct kprobe *p, struct pt_regs *regs)
                "mrs    %[newcpsr], cpsr        \n\t"
                : [newcpsr] "=r" (newcpsr)
                : [oldcpsr] "r" (oldcpsr), [regs] "r" (regs),
-                 [fn] "r" (p->ainsn.insn_fn)
+                 [fn] "r" (asi->insn_fn)
                : "r0", "r1", "r2", "r3", "r4", "r5", "r6", "r7",
                  "lr", "memory", "cc"
                );
@@ -472,24 +464,26 @@ t16_emulate_loregs(struct kprobe *p, struct pt_regs *regs)
 }
 
 static void __kprobes
-t16_emulate_loregs_rwflags(struct kprobe *p, struct pt_regs *regs)
+t16_emulate_loregs_rwflags(kprobe_opcode_t insn,
+               struct arch_specific_insn *asi, struct pt_regs *regs)
 {
-       regs->ARM_cpsr = t16_emulate_loregs(p, regs);
+       regs->ARM_cpsr = t16_emulate_loregs(insn, asi, regs);
 }
 
 static void __kprobes
-t16_emulate_loregs_noitrwflags(struct kprobe *p, struct pt_regs *regs)
+t16_emulate_loregs_noitrwflags(kprobe_opcode_t insn,
+               struct arch_specific_insn *asi, struct pt_regs *regs)
 {
-       unsigned long cpsr = t16_emulate_loregs(p, regs);
+       unsigned long cpsr = t16_emulate_loregs(insn, asi, regs);
        if (!in_it_block(cpsr))
                regs->ARM_cpsr = cpsr;
 }
 
 static void __kprobes
-t16_emulate_hiregs(struct kprobe *p, struct pt_regs *regs)
+t16_emulate_hiregs(kprobe_opcode_t insn,
+               struct arch_specific_insn *asi, struct pt_regs *regs)
 {
-       kprobe_opcode_t insn = p->opcode;
-       unsigned long pc = thumb_probe_pc(p);
+       unsigned long pc = regs->ARM_pc + 2;
        int rdn = (insn & 0x7) | ((insn & 0x80) >> 4);
        int rm = (insn >> 3) & 0xf;
 
@@ -505,7 +499,7 @@ t16_emulate_hiregs(struct kprobe *p, struct pt_regs *regs)
                "blx    %[fn]                   \n\t"
                "mrs    %[cpsr], cpsr           \n\t"
                : "=r" (rdnv), [cpsr] "=r" (cpsr)
-               : "0" (rdnv), "r" (rmv), "1" (cpsr), [fn] "r" (p->ainsn.insn_fn)
+               : "0" (rdnv), "r" (rmv), "1" (cpsr), [fn] "r" (asi->insn_fn)
                : "lr", "memory", "cc"
        );
 
@@ -528,7 +522,8 @@ t16_decode_hiregs(kprobe_opcode_t insn, struct arch_specific_insn *asi,
 }
 
 static void __kprobes
-t16_emulate_push(struct kprobe *p, struct pt_regs *regs)
+t16_emulate_push(kprobe_opcode_t insn,
+               struct arch_specific_insn *asi, struct pt_regs *regs)
 {
        __asm__ __volatile__ (
                "ldr    r9, [%[regs], #13*4]    \n\t"
@@ -537,7 +532,7 @@ t16_emulate_push(struct kprobe *p, struct pt_regs *regs)
                "blx    %[fn]                   \n\t"
                "str    r9, [%[regs], #13*4]    \n\t"
                :
-               : [regs] "r" (regs), [fn] "r" (p->ainsn.insn_fn)
+               : [regs] "r" (regs), [fn] "r" (asi->insn_fn)
                : "r0", "r1", "r2", "r3", "r4", "r5", "r6", "r7", "r8", "r9",
                  "lr", "memory", "cc"
                );
@@ -559,7 +554,8 @@ t16_decode_push(kprobe_opcode_t insn, struct arch_specific_insn *asi,
 }
 
 static void __kprobes
-t16_emulate_pop_nopc(struct kprobe *p, struct pt_regs *regs)
+t16_emulate_pop_nopc(kprobe_opcode_t insn,
+               struct arch_specific_insn *asi, struct pt_regs *regs)
 {
        __asm__ __volatile__ (
                "ldr    r9, [%[regs], #13*4]    \n\t"
@@ -568,14 +564,15 @@ t16_emulate_pop_nopc(struct kprobe *p, struct pt_regs *regs)
                "stmia  %[regs], {r0-r7}        \n\t"
                "str    r9, [%[regs], #13*4]    \n\t"
                :
-               : [regs] "r" (regs), [fn] "r" (p->ainsn.insn_fn)
+               : [regs] "r" (regs), [fn] "r" (asi->insn_fn)
                : "r0", "r1", "r2", "r3", "r4", "r5", "r6", "r7", "r9",
                  "lr", "memory", "cc"
                );
 }
 
 static void __kprobes
-t16_emulate_pop_pc(struct kprobe *p, struct pt_regs *regs)
+t16_emulate_pop_pc(kprobe_opcode_t insn,
+               struct arch_specific_insn *asi, struct pt_regs *regs)
 {
        register unsigned long pc asm("r8");
 
@@ -586,7 +583,7 @@ t16_emulate_pop_pc(struct kprobe *p, struct pt_regs *regs)
                "stmia  %[regs], {r0-r7}        \n\t"
                "str    r9, [%[regs], #13*4]    \n\t"
                : "=r" (pc)
-               : [regs] "r" (regs), [fn] "r" (p->ainsn.insn_fn)
+               : [regs] "r" (regs), [fn] "r" (asi->insn_fn)
                : "r0", "r1", "r2", "r3", "r4", "r5", "r6", "r7", "r9",
                  "lr", "memory", "cc"
                );
index a757c3c..b4a3028 100644 (file)
@@ -204,7 +204,7 @@ singlestep_skip(struct kprobe *p, struct pt_regs *regs)
 static inline void __kprobes
 singlestep(struct kprobe *p, struct pt_regs *regs, struct kprobe_ctlblk *kcb)
 {
-       p->ainsn.insn_singlestep(p, regs);
+       p->ainsn.insn_singlestep(p->opcode, &p->ainsn, regs);
 }
 
 /*
index 496e0e9..36002e5 100644 (file)
@@ -19,9 +19,8 @@
 #include <linux/module.h>
 #include <linux/stddef.h>
 #include <linux/ptrace.h>
-#include <linux/kprobes.h>
 
-#include "kprobes.h"
+#include "probes.h"
 #include "probes-arm.h"
 
 #define sign_extend(x, signbit) ((x) | (0 - ((x) & (1 << (signbit)))))
  * read and write of flags.
  */
 
-void __kprobes simulate_bbl(struct kprobe *p, struct pt_regs *regs)
+void __kprobes simulate_bbl(kprobe_opcode_t insn,
+               struct arch_specific_insn *asi, struct pt_regs *regs)
 {
-       kprobe_opcode_t insn = p->opcode;
-       long iaddr = (long)p->addr;
+       long iaddr = (long) regs->ARM_pc - 4;
        int disp  = branch_displacement(insn);
 
        if (insn & (1 << 24))
@@ -70,10 +69,10 @@ void __kprobes simulate_bbl(struct kprobe *p, struct pt_regs *regs)
        regs->ARM_pc = iaddr + 8 + disp;
 }
 
-void __kprobes simulate_blx1(struct kprobe *p, struct pt_regs *regs)
+void __kprobes simulate_blx1(kprobe_opcode_t insn,
+               struct arch_specific_insn *asi, struct pt_regs *regs)
 {
-       kprobe_opcode_t insn = p->opcode;
-       long iaddr = (long)p->addr;
+       long iaddr = (long) regs->ARM_pc - 4;
        int disp = branch_displacement(insn);
 
        regs->ARM_lr = iaddr + 4;
@@ -81,14 +80,14 @@ void __kprobes simulate_blx1(struct kprobe *p, struct pt_regs *regs)
        regs->ARM_cpsr |= PSR_T_BIT;
 }
 
-void __kprobes simulate_blx2bx(struct kprobe *p, struct pt_regs *regs)
+void __kprobes simulate_blx2bx(kprobe_opcode_t insn,
+               struct arch_specific_insn *asi, struct pt_regs *regs)
 {
-       kprobe_opcode_t insn = p->opcode;
        int rm = insn & 0xf;
        long rmv = regs->uregs[rm];
 
        if (insn & (1 << 5))
-               regs->ARM_lr = (long)p->addr + 4;
+               regs->ARM_lr = (long) regs->ARM_pc;
 
        regs->ARM_pc = rmv & ~0x1;
        regs->ARM_cpsr &= ~PSR_T_BIT;
@@ -96,15 +95,16 @@ void __kprobes simulate_blx2bx(struct kprobe *p, struct pt_regs *regs)
                regs->ARM_cpsr |= PSR_T_BIT;
 }
 
-void __kprobes simulate_mrs(struct kprobe *p, struct pt_regs *regs)
+void __kprobes simulate_mrs(kprobe_opcode_t insn,
+               struct arch_specific_insn *asi, struct pt_regs *regs)
 {
-       kprobe_opcode_t insn = p->opcode;
        int rd = (insn >> 12) & 0xf;
        unsigned long mask = 0xf8ff03df; /* Mask out execution state */
        regs->uregs[rd] = regs->ARM_cpsr & mask;
 }
 
-void __kprobes simulate_mov_ipsp(struct kprobe *p, struct pt_regs *regs)
+void __kprobes simulate_mov_ipsp(kprobe_opcode_t insn,
+               struct arch_specific_insn *asi, struct pt_regs *regs)
 {
        regs->uregs[12] = regs->uregs[13];
 }
@@ -704,10 +704,11 @@ const union decode_item kprobe_decode_arm_table[] = {
 EXPORT_SYMBOL_GPL(kprobe_decode_arm_table);
 #endif
 
-static void __kprobes arm_singlestep(struct kprobe *p, struct pt_regs *regs)
+static void __kprobes arm_singlestep(kprobe_opcode_t insn,
+               struct arch_specific_insn *asi, struct pt_regs *regs)
 {
        regs->ARM_pc += 4;
-       p->ainsn.insn_handler(p, regs);
+       asi->insn_handler(insn, asi, regs);
 }
 
 /* Return:
index ef30894..d0ac8a4 100644 (file)
@@ -53,10 +53,15 @@ enum probes_arm_action {
        NUM_PROBES_ARM_ACTIONS
 };
 
-void __kprobes simulate_bbl(struct kprobe *p, struct pt_regs *regs);
-void __kprobes simulate_blx1(struct kprobe *p, struct pt_regs *regs);
-void __kprobes simulate_blx2bx(struct kprobe *p, struct pt_regs *regs);
-void __kprobes simulate_mrs(struct kprobe *p, struct pt_regs *regs);
-void __kprobes simulate_mov_ipsp(struct kprobe *p, struct pt_regs *regs);
+void __kprobes simulate_bbl(kprobe_opcode_t opcode,
+       struct arch_specific_insn *asi, struct pt_regs *regs);
+void __kprobes simulate_blx1(kprobe_opcode_t opcode,
+       struct arch_specific_insn *asi, struct pt_regs *regs);
+void __kprobes simulate_blx2bx(kprobe_opcode_t opcode,
+       struct arch_specific_insn *asi, struct pt_regs *regs);
+void __kprobes simulate_mrs(kprobe_opcode_t opcode,
+       struct arch_specific_insn *asi, struct pt_regs *regs);
+void __kprobes simulate_mov_ipsp(kprobe_opcode_t opcode,
+       struct arch_specific_insn *asi, struct pt_regs *regs);
 
 #endif
index 2abe8ce..aa3176d 100644 (file)
 
 #include <linux/stddef.h>
 #include <linux/kernel.h>
-#include <linux/kprobes.h>
 #include <linux/module.h>
 
-#include "kprobes.h"
+#include "probes.h"
 #include "probes-thumb.h"
 
 
@@ -844,17 +843,21 @@ static unsigned long __kprobes thumb_check_cc(unsigned long cpsr)
        return true;
 }
 
-static void __kprobes thumb16_singlestep(struct kprobe *p, struct pt_regs *regs)
+static void __kprobes thumb16_singlestep(kprobe_opcode_t opcode,
+               struct arch_specific_insn *asi,
+               struct pt_regs *regs)
 {
        regs->ARM_pc += 2;
-       p->ainsn.insn_handler(p, regs);
+       asi->insn_handler(opcode, asi, regs);
        regs->ARM_cpsr = it_advance(regs->ARM_cpsr);
 }
 
-static void __kprobes thumb32_singlestep(struct kprobe *p, struct pt_regs *regs)
+static void __kprobes thumb32_singlestep(kprobe_opcode_t opcode,
+               struct arch_specific_insn *asi,
+               struct pt_regs *regs)
 {
        regs->ARM_pc += 4;
-       p->ainsn.insn_handler(p, regs);
+       asi->insn_handler(opcode, asi, regs);
        regs->ARM_cpsr = it_advance(regs->ARM_cpsr);
 }
 
index efd92c5..f2ab885 100644 (file)
 
 #include <linux/kernel.h>
 #include <linux/types.h>
-#include <linux/kprobes.h>
 #include <asm/system_info.h>
 #include <asm/ptrace.h>
 #include <linux/bug.h>
 
-#include "kprobes.h"
+#include "probes.h"
 
 
 #ifndef find_str_pc_offset
@@ -176,13 +175,17 @@ kprobe_check_cc * const kprobe_condition_checks[16] = {
 };
 
 
-void __kprobes kprobe_simulate_nop(struct kprobe *p, struct pt_regs *regs)
+void __kprobes kprobe_simulate_nop(kprobe_opcode_t opcode,
+       struct arch_specific_insn *asi,
+       struct pt_regs *regs)
 {
 }
 
-void __kprobes kprobe_emulate_none(struct kprobe *p, struct pt_regs *regs)
+void __kprobes kprobe_emulate_none(kprobe_opcode_t opcode,
+       struct arch_specific_insn *asi,
+       struct pt_regs *regs)
 {
-       p->ainsn.insn_fn();
+       asi->insn_fn();
 }
 
 /*
index 5554f16..efea63c 100644 (file)
@@ -22,6 +22,7 @@
 #include <linux/types.h>
 #include <linux/stddef.h>
 #include <linux/kprobes.h>
+#include "kprobes.h"
 
 #if __LINUX_ARM_ARCH__ >= 7
 
@@ -37,6 +38,7 @@ void __init find_str_pc_offset(void);
 
 #endif
 
+struct decode_header;
 
 /*
  * Update ITSTATE after normal execution of an IT block instruction.
@@ -129,8 +131,10 @@ static inline void __kprobes alu_write_pc(long pcv, struct pt_regs *regs)
 }
 
 
-void __kprobes kprobe_simulate_nop(struct kprobe *p, struct pt_regs *regs);
-void __kprobes kprobe_emulate_none(struct kprobe *p, struct pt_regs *regs);
+void __kprobes kprobe_simulate_nop(kprobe_opcode_t, struct arch_specific_insn *,
+               struct pt_regs *regs);
+void __kprobes kprobe_emulate_none(kprobe_opcode_t, struct arch_specific_insn *,
+               struct pt_regs *regs);
 
 enum kprobe_insn __kprobes
 kprobe_decode_ldmstm(kprobe_opcode_t insn, struct arch_specific_insn *asi,