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)
Fri, Oct 25, 10:57 AM
Unknown Object (File)
Fri, Oct 25, 10:55 AM
Unknown Object (File)
Oct 16 2024, 1:00 PM
Unknown Object (File)
Oct 16 2024, 11:57 AM
Unknown Object (File)
Oct 16 2024, 11:57 AM
Unknown Object (File)
Oct 16 2024, 11:56 AM
Unknown Object (File)
Oct 15 2024, 1:57 AM
Unknown Object (File)
Oct 4 2024, 1:58 PM
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 Skipped
Unit
Tests Skipped

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