yf13 commented on code in PR #12320: URL: https://github.com/apache/nuttx/pull/12320#discussion_r1623186467
########## include/nuttx/tls.h: ########## @@ -219,6 +219,7 @@ struct tls_info_s int16_t tl_cpcount; /* Nested cancellation point count */ #endif + uint16_t tl_size; /* Actual size with alignments */ Review Comment: If the size is the same for all threads, putting it in `task_info_s` is fine as it is not thread specific. Putting it in-place is also fine as we don't need worry whether in the future if the size is thread-specific. -- 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
