Page MenuHomeFreeBSD

riscv: Print stval in dump_regs for fatal exceptions
ClosedPublic

Authored by jhb on Sep 2 2023, 10:17 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Nov 4, 5:03 AM
Unknown Object (File)
Sat, Oct 26, 1:07 AM
Unknown Object (File)
Thu, Oct 24, 10:49 AM
Unknown Object (File)
Sep 27 2024, 1:41 AM
Unknown Object (File)
Sep 27 2024, 1:41 AM
Unknown Object (File)
Sep 27 2024, 1:41 AM
Unknown Object (File)
Sep 26 2024, 11:21 AM
Unknown Object (File)
Sep 16 2024, 11:49 AM
Subscribers

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 53497
Build 50388: arc lint + arc unit

Event Timeline

jhb requested review of this revision.Sep 2 2023, 10:17 PM
mhorne added inline comments.
sys/riscv/riscv/trap.c
344

We do print sepc in dump_regs() as well. To me it seems more useful to have both values in this message, or neither.

This revision is now accepted and ready to land.Sep 3 2023, 7:42 PM
sys/riscv/riscv/trap.c
344

Ok. I'm a bit torn actually on whether it's useful to duplicate info. If you only had the panic message without dump_regs for some reason then that might be a reason, or if you are categorizing panics based on the panic message (but the addresses are likely to not have enough of a pattern for that to be useful?)

Also, the load and store access fault's above should also be consistent presumably and print the address if this one does. Perhaps for now it would be best to just add the new line to dump_regs in this diff and leave the other changes alone?

Drop panic message changes

This revision now requires review to proceed.Sep 8 2023, 1:42 AM
This revision is now accepted and ready to land.Sep 8 2023, 6:36 PM