ESP-IDF4+ : crash handler and boot status #902
Merged
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
llange
force-pushed
the
boot-status-esp-idf4+
branch
from
6a39afb
to
964872f
Compare
markwj
reviewed
May 18, 2023
markwj
approved these changes
May 18, 2023
`xt_set_error_handler_callback` is only available in our ESP-IDF 3.3.4 fork. For ESP-IDF >= 4, we implement it a little bit differently by using the linker wrap feature, which let us execute our handler before `esp_panic_handler()`. Signed-off-by: Ludovic LANGE <[email protected]>
`esp_task_wdt_get_trigger_tasknames()` was a custom patch of our ESP-IDF 3.3.4 fork. It is used to list the names of the task(s) responsible for triggering the WDT, and this information is saved in the (persistent) boot_data structure for post-crash analysis. Unfortunately it's not available on upstream ESP-IDF 4+. For the moment we remove the feature, until we find a way to implement it without patching ESP-IDF 4+. Signed-off-by: Ludovic LANGE <[email protected]>
…ata.crash_data` XTensa and RISC V have differences, and `boot_data.crash_data` being low-level is exposed to these differences. We try to handle the CPU registers dump for both architectures. However backtrace is not handled (yet) for RISC V. Signed-off-by: Ludovic LANGE <[email protected]>
llange
force-pushed
the
boot-status-esp-idf4+
branch
from
964872f
to
6babe06
Compare
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.
xt_set_error_handler_callbackis only available in our ESP-IDF 3.3.4 fork.)esp_task_wdt_get_trigger_tasknames()not available (feature removed from ESP-IDF4+ builds) TODOboot_data.crash_data