fjpanag commented on code in PR #7436: URL: https://github.com/apache/incubator-nuttx/pull/7436#discussion_r1006707962
########## arch/arm/src/armv7-m/arm_stackcheck.c: ########## @@ -69,6 +69,12 @@ void __stack_overflow_trap(void) uint32_t regval; + regval = getreg32(NVIC_DHCSR); Review Comment: @davids5 This is what I am doing till now, but there are two problems: * There are limited breakpoints that the debugger can set. On more for this check means one less for debugging the application. * If for any reason you forget / remove it and the check fails, you lost the opportunity to diagnose the issue. Since this is not going to introduce any overhead, why not add it? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@nuttx.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
