Page MenuHomeFreeBSD
Files F177583574

D56987.1794798897.diff
No OneTemporary
Actions

Authored By
Unknown
Size
1 KB
Referenced Files
None
Subscribers
None

D56987.1794798897.diff
View Options

diff --git a/sys/cddl/dev/kinst/aarch64/kinst_isa.c b/sys/cddl/dev/kinst/aarch64/kinst_isa.c
--- a/sys/cddl/dev/kinst/aarch64/kinst_isa.c
+++ b/sys/cddl/dev/kinst/aarch64/kinst_isa.c
@@ -18,7 +18,7 @@
DPCPU_DEFINE_STATIC(struct kinst_cpu_state, kinst_state);
-static int
+static void
kinst_emulate(struct trapframe *frame, const struct kinst_probe *kp)
{
kinst_patchval_t instr = kp->kp_savedval;
@@ -132,17 +132,13 @@
else
frame->tf_elr += INSN_SIZE;
}
-
- return (0);
}
static int
kinst_jump_next_instr(struct trapframe *frame, const struct kinst_probe *kp)
{
- frame->tf_elr = (register_t)((const uint8_t *)kp->kp_patchpoint +
- INSN_SIZE);
-
- return (0);
+ frame->tf_elr = (register_t)(uintptr_t)kp->kp_patchpoint;
+ return (NOP_INSTR);
}
static void
@@ -215,21 +211,24 @@
dtrace_probe(kp->kp_id, 0, 0, 0, 0, 0);
cpu->cpu_dtrace_caller = 0;
- if (kp->kp_md.emulate)
- return (kinst_emulate(frame, kp));
-
- ks->state = KINST_PROBE_FIRED;
- ks->kp = kp;
+ if (kp->kp_md.emulate) {
+ kinst_emulate(frame, kp);
+ } else {
+ ks->state = KINST_PROBE_FIRED;
+ ks->kp = kp;
- /*
- * Cache the current SPSR and clear interrupts for the duration
- * of the double breakpoint.
- */
- ks->status = frame->tf_spsr;
- frame->tf_spsr |= PSR_I;
- frame->tf_elr = (register_t)kp->kp_tramp;
+ /*
+ * Cache the current SPSR and clear interrupts for the duration
+ * of the double breakpoint.
+ */
+ ks->status = frame->tf_spsr;
+ frame->tf_spsr |= PSR_I;
+ frame->tf_elr = (register_t)kp->kp_tramp;
+ }
- return (0);
+ /* Offset to compensate for NOP handling in dtrace_invop_start(). */
+ frame->tf_elr -= INSN_SIZE;
+ return (NOP_INSTR);
}
void

File Metadata

Mime Type
text/plain
Expires
Mon, Nov 16, 3:14 AM (19 h, 51 m)
Storage Engine
blob
Storage Format
Raw Data
Storage Handle
41757084
Default Alt Text
D56987.1794798897.diff (1 KB)

Event Timeline