raiden00pl commented on PR #18408: URL: https://github.com/apache/nuttx/pull/18408#issuecomment-3932346301
The only thing that puzzles me about this change is how the "init thread" aligns with safety standards. This mechanism creates a thread that is later destroyed, which isn't entirely desirable for safety critical applications. But on the other hand, "init thread" is part of the system initialization process, and during the initialization phase, dynamic memory allocation and deallocation is tolerated. The goal here is that in the event of a board initialization failure, the system is in a known safe state. So, I think it still fits within the acceptable use of dynamic memory for safety critical apps, but not sure. Ultimately, there's always `BOARDIOC_FINALINIT` option, which allows users to get rid of the "init thread" and perform initialization in "main", just like old `BOARDIOC_INIT`. -- 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]
