Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

log_rpc: replace crash log with crash dump #21420

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

Damian-Nordic
Copy link
Contributor

@Damian-Nordic Damian-Nordic commented Apr 4, 2025

First commit shared with #21418

log_rpc: replace crash log with crash dump

Replace a custom solution for storing crash log in retained RAM and exposing it in as a string over RPC with Zephyr Core Dump module. In the latter solution RPC is only used to fetch the crash dump to the remote device.

Add Zephyr core dump backend that saves a core dump to the
internal flash or RRAM partition named "coredump_partition".

This backend is an alternative to BACKEND_FLASH_PARTITION
provided by Zephyr but it bypasses Zephyr flash device layer
and uses nrfx directly, which offers the following benefits:

1. Bypasses synchronization primitives used by Zephyr flash
   or RRAM drivers. Currently, Zephyr flash drivers cannot
   be used to write to flash from a fault handler because
   of this.
2. Works with Partition Manager.
3. Minimizes the dependencies needed to successfully write
   a core dump, which is important as the core dump often
   needs to be written when the system is in the corrupted
   state.

Only flash and RRAM are supported for now (no MRAM support).
Replace a custom solution for storing crash log in retained
RAM and exposing it in as a string over RPC with Zephyr
Core Dump module. In the latter solution RPC is only used
to fetch the crash dump to the remote device.
@github-actions github-actions bot added the doc-required PR must not be merged without tech writer approval. label Apr 4, 2025
@NordicBuilder
Copy link
Contributor

CI Information

To view the history of this post, clich the 'edited' button above
Build number: 1

Inputs:

Sources:

more details

Github labels

Enabled Name Description
ci-disabled Disable the ci execution
ci-all-test Run all of ci, no test spec filtering will be done
ci-force-downstream Force execution of downstream even if twister fails
ci-run-twister Force run twister
ci-run-zephyr-twister Force run zephyr twister
List of changed files detected by CI (0)

Outputs:

Toolchain

Version:
Build docker image:

Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped; ⚠️ Quarantine

  • ❌ Toolchain
  • ❌ Build twister
  • ❌ Integration tests

Note: This message is automatically posted and updated by the CI

Copy link

github-actions bot commented Apr 4, 2025

You can find the documentation preview for this PR here.

Preview links for modified nRF Connect SDK documents:

https://ncsdoc.z6.web.core.windows.net/PR-21420/nrf/releases_and_maturity/releases/release-notes-changelog.html

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc-required PR must not be merged without tech writer approval.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants