[Debug Helpers] add esp_backtrace_print_all_tasks function (IDFGH-10315) #11575
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
chipweinberger
force-pushed
the
user/chip/add-backtrace-print-all-tasks
branch
2 times, most recently
from
e3a1e5f to
7ebfe30
Compare
github-actions
bot
changed the title
chipweinberger
force-pushed
the
user/chip/add-backtrace-print-all-tasks
branch
3 times, most recently
from
82b40ba to
e721d93
Compare
chipweinberger
force-pushed
the
user/chip/add-backtrace-print-all-tasks
branch
from
e721d93 to
517c822
Compare
|
@o-marshmallow bump |
|
@o-marshmallow bump :) |
|
Great addition. I based some work off of Chip's PR that can be used to print all task backtraces to a JSON string so it can be logged somewhere other than the console. I don't know that it makes sense to add it to IDF, but here's a gist: https://gist.github.com/bdpdx/376d0dab638227556bf7631d9089e005 |
|
@chipweinberger Sorry for the delay of response, I am going to handle it. |
o-marshmallow
added
PR-Sync-Rebase
|
sha=517c822cf12e8ed892d2de3df0e3959c5a0c501c |
o-marshmallow
added
PR-Sync-Rebase
|
bump =) |
|
@Dazza0 I see you're making changes recently about this. can we still halt the scheduler & merge this? |
espressif-bot
added
Status: Selected for Development
espressif-bot
added
Status: In Progress
espressif-bot
added
Status: Done
|
@o-marshmallow awesome!! thank you! =) |
jnross
pushed a commit
to junelife/esp-idf
that referenced
this pull request
Feb 12, 2024
This commit adds esp_backtrace_print_all_tasks() which prints the backtraces of all tasks at runtime. Closes espressif#9708 CLoses espressif#11575 [Omar Chebib: Prevent task switching while printing backtraces of tasks.] [Omar Chebib: Ensure all task stacks are flushed from register to RAM.] [Omar Chebib: Removed esp_task_snapshot_to_backtrace_frame() as task snapshot is private API.] [Omar Chebib: Added test case for esp_backtrace_print_all_tasks().] Signed-off-by: Omar Chebib <[email protected]> (cherry picked from commit 3686689) # Conflicts: # components/esp_system/port/arch/xtensa/debug_helpers.c # components/esp_system/test_apps/esp_system_unity_tests/main/test_backtrace.c # components/freertos/esp_additions/freertos_tasks_c_additions.h # components/freertos/esp_additions/include/esp_private/freertos_debug.h
jnross
pushed a commit
to junelife/esp-idf
that referenced
this pull request
Feb 12, 2024
This commit adds esp_backtrace_print_all_tasks() which prints the backtraces of all tasks at runtime. Closes espressif#9708 CLoses espressif#11575 [Omar Chebib: Prevent task switching while printing backtraces of tasks.] [Omar Chebib: Ensure all task stacks are flushed from register to RAM.] [Omar Chebib: Removed esp_task_snapshot_to_backtrace_frame() as task snapshot is private API.] [Omar Chebib: Added test case for esp_backtrace_print_all_tasks().] Signed-off-by: Omar Chebib <[email protected]> (cherry picked from commit 3686689) # Conflicts: # components/esp_system/port/arch/xtensa/debug_helpers.c # components/esp_system/test_apps/esp_system_unity_tests/main/test_backtrace.c # components/freertos/esp_additions/freertos_tasks_c_additions.h # components/freertos/esp_additions/include/esp_private/freertos_debug.h
jnross
pushed a commit
to junelife/esp-idf
that referenced
this pull request
Feb 12, 2024
This commit adds esp_backtrace_print_all_tasks() which prints the backtraces of all tasks at runtime. Closes espressif#9708 CLoses espressif#11575 [Omar Chebib: Prevent task switching while printing backtraces of tasks.] [Omar Chebib: Ensure all task stacks are flushed from register to RAM.] [Omar Chebib: Removed esp_task_snapshot_to_backtrace_frame() as task snapshot is private API.] [Omar Chebib: Added test case for esp_backtrace_print_all_tasks().] Signed-off-by: Omar Chebib <[email protected]> (cherry picked from commit 3686689) # Conflicts: # components/esp_system/port/arch/xtensa/debug_helpers.c # components/esp_system/test_apps/esp_system_unity_tests/main/test_backtrace.c # components/freertos/esp_additions/freertos_tasks_c_additions.h # components/freertos/esp_additions/include/esp_private/freertos_debug.h
mykmelez
pushed a commit
to junelife/esp-idf
that referenced
this pull request
Aug 9, 2024
This commit adds esp_backtrace_print_all_tasks() which prints the backtraces of all tasks at runtime. Closes espressif#9708 CLoses espressif#11575 [Omar Chebib: Prevent task switching while printing backtraces of tasks.] [Omar Chebib: Ensure all task stacks are flushed from register to RAM.] [Omar Chebib: Removed esp_task_snapshot_to_backtrace_frame() as task snapshot is private API.] [Omar Chebib: Added test case for esp_backtrace_print_all_tasks().] Signed-off-by: Omar Chebib <[email protected]> (cherry picked from commit 3686689) (cherry picked from commit ed19aae)
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.
Related Issue: #9708
Lots of people need the ability to print all tasks in order to debug deadlocks, etc.
Also add
esp_backtrace_frame_t esp_task_snapshot_to_backtrace_frame(TaskSnapshot_t snapshot), as a helper function.