Coredump: adding an In memory backend an various improvements #87565
+909
−353
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
zephyrbot
requested review from
carlescufi,
dcpleung,
jakub-uC,
mrkhldn and
nashif
tbursztyka
force-pushed
the
in_memory_coredump
branch
2 times, most recently
from
5f8b4e0 to
93d635b
Compare
mrkhldn
reviewed
Mar 24, 2025
mrkhldn
reviewed
Mar 24, 2025
tbursztyka
force-pushed
the
in_memory_coredump
branch
from
93d635b to
f136777
Compare
mrkhldn
previously approved these changes
Mar 25, 2025
added 6 commits
March 27, 2025 10:23
This might come in handy for other coredump backends too, not only flash one. Obviously: changing the logic to use the coredump generic API instead of the flash backend's internal functions. Adding a better output, following how the coredump is actually created: parsing and printing headers, and their respective content. Signed-off-by: Tomasz Bursztyka <tobu@bang-olufsen.dk>
To enable/disable any coredump logging while debugging the kernel. Signed-off-by: Tomasz Bursztyka <tobu@bang-olufsen.dk>
Only meant to be used on platforms that would not wipe the RAM on reboot. This won't work if the platform is halted and restarted as RAM is volatile, obviously. A relevant fatal error handler needs to be provided to reboot. Signed-off-by: Tomasz Bursztyka <tobu@bang-olufsen.dk>
For testing purposes only. Testing the fatal error handler or a coredump backend. Or just out of curiosity. Signed-off-by: Tomasz Bursztyka <tobu@bang-olufsen.dk>
A nano fix in the header. Signed-off-by: Tomasz Bursztyka <tobu@bang-olufsen.dk>
Providing the right config options and a new test scenario, coredump_backends test will run under the in-memory backend. Signed-off-by: Tomasz Bursztyka <tobu@bang-olufsen.dk>
tbursztyka
force-pushed
the
in_memory_coredump
branch
from
f136777 to
f26d788
Compare
mrkhldn
approved these changes
Mar 27, 2025
dcpleung
approved these changes
Mar 27, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR makes a few improvements in coredump sub-system, the commit titles are self-explanatory.
About the shell generalization, it gives an output like: