Skip to content

Fix/panic decoding#398

Merged
hfudev merged 3 commits intomainfrom
fix/panic-decoding
Dec 30, 2025
Merged

Fix/panic decoding#398
hfudev merged 3 commits intomainfrom
fix/panic-decoding

Conversation

@hfudev
Copy link
Copy Markdown
Member

@hfudev hfudev commented Dec 15, 2025

No description provided.

…ption

also remove the gdb_panic_server.py in tests fixtures
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@hfudev hfudev force-pushed the fix/panic-decoding branch from 57b37fb to 2a958d9 Compare December 16, 2025 13:25
@hfudev hfudev force-pushed the fix/panic-decoding branch from 2a958d9 to db11168 Compare December 16, 2025 13:28
@hfudev hfudev marked this pull request as ready for review December 16, 2025 13:29
@hfudev hfudev requested review from Copilot and igrr December 16, 2025 13:29
@hfudev
Copy link
Copy Markdown
Member Author

hfudev commented Dec 16, 2025

@igrr PTAL

now by default, the panic will be decoded on riscv chips. --skip-decode-panic would skip. the output will be in logdir/panic_decoded.log

the coredump output will be in logdir/coredump.log

alongside the dut.log.

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 28 out of 42 changed files in this pull request and generated 2 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread tests/fixtures/hello_world_esp32c3_panic/build/gdbinit/prefix_map
Comment thread tests/fixtures/hello_world_esp32c3_panic/build/gdbinit/prefix_map
@hfudev hfudev merged commit 4c44d9d into main Dec 30, 2025
10 of 11 checks passed
@hfudev hfudev deleted the fix/panic-decoding branch December 30, 2025 07:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants