Page MenuHomeFreeBSD

dtrace: handle NOP instructions in RISC-V
ClosedPublic

Authored by christos on Apr 10 2023, 12:38 PM.
Tags
Referenced Files
Unknown Object (File)
Tue, Oct 7, 5:04 PM
Unknown Object (File)
Tue, Sep 23, 4:27 AM
Unknown Object (File)
Sat, Sep 13, 7:01 AM
Unknown Object (File)
Sep 10 2025, 10:58 AM
Unknown Object (File)
Sep 10 2025, 2:42 AM
Unknown Object (File)
Sep 6 2025, 3:52 AM
Unknown Object (File)
Aug 31 2025, 9:11 PM
Unknown Object (File)
Aug 4 2025, 12:07 AM
Subscribers

Details

Summary

dtrace_invop_start() was panicking if the provider returned
MATCH_C_NOP (DTRACE_INVOP_NOP).

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

sys/cddl/dev/dtrace/riscv/dtrace_subr.c
275

It looks like fbt_invop() returns the original instruction, rather than using the DTRACE_INVOP_* constants. In particular, fbt->fbt_rval is not used anywhere.

We should thus return a nop instruction and match it the way other instructions are matched, rather than having multiple return types.

christos marked an inline comment as done.
christos retitled this revision from dtrace: handle DTRACE_INVOP_NOP in RISC-V to dtrace: handle NOP instructions in RISC-V.
christos edited the summary of this revision. (Show Details)
This revision is now accepted and ready to land.Apr 10 2023, 3:03 PM