[GitHub] [incubator-nuttx] v01d commented on a change in pull request #3517: arch: Allocate the space from the beginning in up_stack_frame

[GitBox <gi...@apache.org>]

v01d commented on a change in pull request #3517:
URL: https://github.com/apache/incubator-nuttx/pull/3517#discussion_r611982253



##########
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:
       are the lines below (checking stack_size against TLS_MAXSTACK) still valid? I understand that stack_size no longer includes TLS size, right?




-- 
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:
users@infra.apache.org