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