94xhn commented on PR #19398: URL: https://github.com/apache/nuttx/pull/19398#issuecomment-4965649353
You're right. I missed two constraints in this path: _exit() enters a critical section before nxtask_exithook(), and any thread-context scheduler change that moves the ready-to-run head must complete its SVC context switch before execution continues. g_running_tasks[] is maintained for interrupt-level switching and assertion reporting on these Cortex-M paths, so it is not a valid generic replacement in thread context. The hardware breadcrumb needs a different root-cause analysis; this patch does not establish that this_task() is wrong in nxtask_exit(). Closing this rather than keeping an unproven scheduler change. Thanks for the correction. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
