Commit 44a0a59c authored by David A. Long's avatar David A. Long

ARM: Rename the shared kprobes/uprobe return value enum

Change the name of kprobes_insn to probes_insn so it can be shared between
kprobes and uprobes without confusion.
Signed-off-by: default avatarDavid A. Long <dave.long@linaro.org>
Acked-by: default avatarJon Medhurst <tixy@linaro.org>
parent eb73ea97
...@@ -123,7 +123,7 @@ emulate_ldm_r3_15(probes_opcode_t insn, ...@@ -123,7 +123,7 @@ emulate_ldm_r3_15(probes_opcode_t insn,
load_write_pc(regs->ARM_pc, regs); load_write_pc(regs->ARM_pc, regs);
} }
enum kprobe_insn __kprobes enum probes_insn __kprobes
kprobe_decode_ldmstm(probes_opcode_t insn, struct arch_specific_insn *asi, kprobe_decode_ldmstm(probes_opcode_t insn, struct arch_specific_insn *asi,
const struct decode_header *h) const struct decode_header *h)
{ {
......
...@@ -66,7 +66,7 @@ t32_simulate_cond_branch(probes_opcode_t insn, ...@@ -66,7 +66,7 @@ t32_simulate_cond_branch(probes_opcode_t insn,
regs->ARM_pc = pc + (offset * 2); regs->ARM_pc = pc + (offset * 2);
} }
static enum kprobe_insn __kprobes static enum probes_insn __kprobes
t32_decode_cond_branch(probes_opcode_t insn, struct arch_specific_insn *asi, t32_decode_cond_branch(probes_opcode_t insn, struct arch_specific_insn *asi,
const struct decode_header *d) const struct decode_header *d)
{ {
...@@ -142,11 +142,11 @@ t32_simulate_ldr_literal(probes_opcode_t insn, ...@@ -142,11 +142,11 @@ t32_simulate_ldr_literal(probes_opcode_t insn,
regs->uregs[rt] = rtv; regs->uregs[rt] = rtv;
} }
static enum kprobe_insn __kprobes static enum probes_insn __kprobes
t32_decode_ldmstm(probes_opcode_t insn, struct arch_specific_insn *asi, t32_decode_ldmstm(probes_opcode_t insn, struct arch_specific_insn *asi,
const struct decode_header *d) const struct decode_header *d)
{ {
enum kprobe_insn ret = kprobe_decode_ldmstm(insn, asi, d); enum probes_insn ret = kprobe_decode_ldmstm(insn, asi, d);
/* Fixup modified instruction to have halfwords in correct order...*/ /* Fixup modified instruction to have halfwords in correct order...*/
insn = asi->insn[0]; insn = asi->insn[0];
...@@ -402,7 +402,7 @@ t16_singlestep_it(probes_opcode_t insn, ...@@ -402,7 +402,7 @@ t16_singlestep_it(probes_opcode_t insn,
t16_simulate_it(insn, asi, regs); t16_simulate_it(insn, asi, regs);
} }
static enum kprobe_insn __kprobes static enum probes_insn __kprobes
t16_decode_it(probes_opcode_t insn, struct arch_specific_insn *asi, t16_decode_it(probes_opcode_t insn, struct arch_specific_insn *asi,
const struct decode_header *d) const struct decode_header *d)
{ {
...@@ -420,7 +420,7 @@ t16_simulate_cond_branch(probes_opcode_t insn, ...@@ -420,7 +420,7 @@ t16_simulate_cond_branch(probes_opcode_t insn,
regs->ARM_pc = pc + (offset * 2); regs->ARM_pc = pc + (offset * 2);
} }
static enum kprobe_insn __kprobes static enum probes_insn __kprobes
t16_decode_cond_branch(probes_opcode_t insn, struct arch_specific_insn *asi, t16_decode_cond_branch(probes_opcode_t insn, struct arch_specific_insn *asi,
const struct decode_header *d) const struct decode_header *d)
{ {
...@@ -510,7 +510,7 @@ t16_emulate_hiregs(probes_opcode_t insn, ...@@ -510,7 +510,7 @@ t16_emulate_hiregs(probes_opcode_t insn,
regs->ARM_cpsr = (regs->ARM_cpsr & ~APSR_MASK) | (cpsr & APSR_MASK); regs->ARM_cpsr = (regs->ARM_cpsr & ~APSR_MASK) | (cpsr & APSR_MASK);
} }
static enum kprobe_insn __kprobes static enum probes_insn __kprobes
t16_decode_hiregs(probes_opcode_t insn, struct arch_specific_insn *asi, t16_decode_hiregs(probes_opcode_t insn, struct arch_specific_insn *asi,
const struct decode_header *d) const struct decode_header *d)
{ {
...@@ -538,7 +538,7 @@ t16_emulate_push(probes_opcode_t insn, ...@@ -538,7 +538,7 @@ t16_emulate_push(probes_opcode_t insn,
); );
} }
static enum kprobe_insn __kprobes static enum probes_insn __kprobes
t16_decode_push(probes_opcode_t insn, struct arch_specific_insn *asi, t16_decode_push(probes_opcode_t insn, struct arch_specific_insn *asi,
const struct decode_header *d) const struct decode_header *d)
{ {
...@@ -591,7 +591,7 @@ t16_emulate_pop_pc(probes_opcode_t insn, ...@@ -591,7 +591,7 @@ t16_emulate_pop_pc(probes_opcode_t insn,
bx_write_pc(pc, regs); bx_write_pc(pc, regs);
} }
static enum kprobe_insn __kprobes static enum probes_insn __kprobes
t16_decode_pop(probes_opcode_t insn, struct arch_specific_insn *asi, t16_decode_pop(probes_opcode_t insn, struct arch_specific_insn *asi,
const struct decode_header *d) const struct decode_header *d)
{ {
......
...@@ -30,28 +30,22 @@ ...@@ -30,28 +30,22 @@
struct decode_header; struct decode_header;
union decode_action; union decode_action;
enum kprobe_insn { typedef enum probes_insn (kprobe_decode_insn_t)(probes_opcode_t,
INSN_REJECTED,
INSN_GOOD,
INSN_GOOD_NO_SLOT
};
typedef enum kprobe_insn (kprobe_decode_insn_t)(probes_opcode_t,
struct arch_specific_insn *, struct arch_specific_insn *,
const union decode_action *); const union decode_action *);
#ifdef CONFIG_THUMB2_KERNEL #ifdef CONFIG_THUMB2_KERNEL
enum kprobe_insn thumb16_kprobe_decode_insn(probes_opcode_t, enum probes_insn thumb16_kprobe_decode_insn(probes_opcode_t,
struct arch_specific_insn *, struct arch_specific_insn *,
const union decode_action *); const union decode_action *);
enum kprobe_insn thumb32_kprobe_decode_insn(probes_opcode_t, enum probes_insn thumb32_kprobe_decode_insn(probes_opcode_t,
struct arch_specific_insn *, struct arch_specific_insn *,
const union decode_action *); const union decode_action *);
#else /* !CONFIG_THUMB2_KERNEL */ #else /* !CONFIG_THUMB2_KERNEL */
enum kprobe_insn arm_kprobe_decode_insn(probes_opcode_t, enum probes_insn arm_kprobe_decode_insn(probes_opcode_t,
struct arch_specific_insn *, struct arch_specific_insn *,
const union decode_action *); const union decode_action *);
......
...@@ -723,7 +723,7 @@ static void __kprobes arm_singlestep(probes_opcode_t insn, ...@@ -723,7 +723,7 @@ static void __kprobes arm_singlestep(probes_opcode_t insn,
* if the work was put into it, but low return considering they * if the work was put into it, but low return considering they
* should also be very rare. * should also be very rare.
*/ */
enum kprobe_insn __kprobes enum probes_insn __kprobes
arm_kprobe_decode_insn(probes_opcode_t insn, struct arch_specific_insn *asi, arm_kprobe_decode_insn(probes_opcode_t insn, struct arch_specific_insn *asi,
const union decode_action *actions) const union decode_action *actions)
{ {
......
...@@ -861,7 +861,7 @@ static void __kprobes thumb32_singlestep(probes_opcode_t opcode, ...@@ -861,7 +861,7 @@ static void __kprobes thumb32_singlestep(probes_opcode_t opcode,
regs->ARM_cpsr = it_advance(regs->ARM_cpsr); regs->ARM_cpsr = it_advance(regs->ARM_cpsr);
} }
enum kprobe_insn __kprobes enum probes_insn __kprobes
thumb16_kprobe_decode_insn(probes_opcode_t insn, struct arch_specific_insn *asi, thumb16_kprobe_decode_insn(probes_opcode_t insn, struct arch_specific_insn *asi,
const union decode_action *actions) const union decode_action *actions)
{ {
...@@ -871,7 +871,7 @@ thumb16_kprobe_decode_insn(probes_opcode_t insn, struct arch_specific_insn *asi, ...@@ -871,7 +871,7 @@ thumb16_kprobe_decode_insn(probes_opcode_t insn, struct arch_specific_insn *asi,
actions); actions);
} }
enum kprobe_insn __kprobes enum probes_insn __kprobes
thumb32_kprobe_decode_insn(probes_opcode_t insn, struct arch_specific_insn *asi, thumb32_kprobe_decode_insn(probes_opcode_t insn, struct arch_specific_insn *asi,
const union decode_action *actions) const union decode_action *actions)
{ {
......
...@@ -138,7 +138,7 @@ void __kprobes probes_simulate_nop(probes_opcode_t, struct arch_specific_insn *, ...@@ -138,7 +138,7 @@ void __kprobes probes_simulate_nop(probes_opcode_t, struct arch_specific_insn *,
void __kprobes probes_emulate_none(probes_opcode_t, struct arch_specific_insn *, void __kprobes probes_emulate_none(probes_opcode_t, struct arch_specific_insn *,
struct pt_regs *regs); struct pt_regs *regs);
enum kprobe_insn __kprobes enum probes_insn __kprobes
kprobe_decode_ldmstm(probes_opcode_t insn, struct arch_specific_insn *asi, kprobe_decode_ldmstm(probes_opcode_t insn, struct arch_specific_insn *asi,
const struct decode_header *h); const struct decode_header *h);
...@@ -313,7 +313,7 @@ union decode_item { ...@@ -313,7 +313,7 @@ union decode_item {
int action; int action;
}; };
typedef enum kprobe_insn (probes_custom_decode_t)(probes_opcode_t, typedef enum probes_insn (probes_custom_decode_t)(probes_opcode_t,
struct arch_specific_insn *, struct arch_specific_insn *,
const struct decode_header *); const struct decode_header *);
...@@ -391,6 +391,11 @@ struct decode_or { ...@@ -391,6 +391,11 @@ struct decode_or {
#define DECODE_OR(_mask, _value) \ #define DECODE_OR(_mask, _value) \
DECODE_HEADER(DECODE_TYPE_OR, _mask, _value, 0) DECODE_HEADER(DECODE_TYPE_OR, _mask, _value, 0)
enum probes_insn {
INSN_REJECTED,
INSN_GOOD,
INSN_GOOD_NO_SLOT
};
struct decode_reject { struct decode_reject {
struct decode_header header; struct decode_header header;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment