pussuw commented on code in PR #6197:
URL: https://github.com/apache/incubator-nuttx/pull/6197#discussion_r868150288


##########
include/nuttx/sched.h:
##########
@@ -470,9 +429,7 @@ struct task_group_s
 
   /* [at|on]exit support ****************************************************/
 
-#ifdef CONFIG_SCHED_EXIT_MAX
-  struct exitinfo_s tg_exit[CONFIG_SCHED_EXIT_MAX];
-#endif
+  FAR struct atexit_list_s *tg_exit;

Review Comment:
   I don't agree on this. In my opinion the group structure is the right place. 
TLS is thread local storage, right ? And every thread has its own?
   
   The exit functions should be on process level, and they are called when the 
process exits, not when a thread exits.



-- 
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

Reply via email to