Skip to content

Commit 50daa7e

Browse files
committed
platform.stack-dump-enabled feature is added.
1 parent 57ed4f3 commit 50daa7e

File tree

2 files changed

+44
-0
lines changed

2 files changed

+44
-0
lines changed

platform/mbed_lib.json

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -90,6 +90,12 @@
9090
"value": null
9191
},
9292

93+
"stack-dump-enabled": {
94+
"macro_name": "MBED_STACK_DUMP_ENABLED",
95+
"help": "Set to 1 or true to enable stack dump.",
96+
"value": null
97+
},
98+
9399
"cpu-stats-enabled": {
94100
"macro_name": "MBED_CPU_STATS_ENABLED",
95101
"help": "Set to 1 to enable cpu stats. When enabled the function mbed_stats_cpu_get returns non-zero data. See mbed_stats.h for more information",

platform/source/mbed_error.c

Lines changed: 38 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -575,6 +575,44 @@ static void print_error_report(const mbed_error_ctx *ctx, const char *error_msg,
575575
mbed_stats_sys_get(&sys_stats);
576576
mbed_error_printf("\nFor more info, visit: https://mbed.com/s/error?error=0x%08X&osver=%" PRId32 "&core=0x%08" PRIX32 "&comp=%d&ver=%" PRIu32 "&tgt=" GET_TARGET_NAME(TARGET_NAME), ctx->error_status, sys_stats.os_version, sys_stats.cpu_id, sys_stats.compiler_id, sys_stats.compiler_version);
577577
#endif
578+
579+
#if MBED_STACK_DUMP_ENABLED && defined(MBED_CONF_RTOS_PRESENT)
580+
/** The internal threshold to detect the end of the stack.
581+
* The stack is filled with osRtxStackFillPattern at the end.
582+
* However, it is possible that the call stack parameters can theoretically have consecutive osRtxStackFillPattern instances.
583+
* For the best effort stack end detection, we will consider STACK_END_MARK_CNT consecutive osRtxStackFillPattern instances as the stack end. */
584+
#define STACK_END_MARK_CNT 3
585+
#define STACK_DUMP_WIDTH 8
586+
mbed_error_printf("\n\nStack Dump:");
587+
// Find the stack end.
588+
int stack_end_cnt = 0;
589+
uint32_t st_end = ctx->thread_current_sp;
590+
for (; st_end >= ctx->thread_stack_mem; st_end -= sizeof(int)) {
591+
uint32_t st_val = *((uint32_t *)st_end);
592+
if (st_val == osRtxStackFillPattern) {
593+
stack_end_cnt++;
594+
} else {
595+
stack_end_cnt = 0;
596+
}
597+
if (stack_end_cnt >= STACK_END_MARK_CNT) {
598+
st_end += (STACK_END_MARK_CNT - 1) * sizeof(int);
599+
break;
600+
}
601+
}
602+
for (uint32_t st = st_end; st <= ctx->thread_current_sp; st += sizeof(int) * STACK_DUMP_WIDTH) {
603+
mbed_error_printf("\n0x%08" PRIX32 ":", st);
604+
for (int i = 0; i < STACK_DUMP_WIDTH; i++) {
605+
uint32_t st_cur = st + i * sizeof(int);
606+
if (st_cur > ctx->thread_current_sp) {
607+
break;
608+
}
609+
uint32_t st_val = *((uint32_t *)st_cur);
610+
mbed_error_printf("0x%08" PRIX32 " ", st_val);
611+
}
612+
}
613+
mbed_error_printf("\n");
614+
#endif // MBED_STACK_DUMP_ENABLED
615+
578616
mbed_error_printf("\n-- MbedOS Error Info --\n");
579617
}
580618
#endif //ifndef NDEBUG

0 commit comments

Comments
 (0)