xiaoxiang781216 commented on a change in pull request #3517:
URL: https://github.com/apache/incubator-nuttx/pull/3517#discussion_r612096107
##########
File path: arch/avr/src/avr/up_createstack.c
##########
@@ -86,10 +86,6 @@
int up_create_stack(FAR struct tcb_s *tcb, size_t stack_size, uint8_t ttype)
{
- /* Add the size of the TLS information structure */
-
- stack_size += sizeof(struct tls_info_s);
-
#ifdef CONFIG_TLS_ALIGNED
/* The allocated stack size must not exceed the maximum possible for the
* TLS feature.
Review comment:
Yes, TLS_MAXSTACK is still valid because it's key invariant of the
aligned TLS. The real difference is that the stack size is always equals to
caller supplied value without adjustion(+ sizeof(struct tls_info_s)).
--
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.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
