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

[GitBox]

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



##########
File path: include/nuttx/sched.h
##########
@@ -402,7 +401,7 @@ struct stackinfo_s
   FAR void *stack_alloc_ptr;             /* Pointer to allocated stack         
 */
                                          /* Needed to deallocate stack         
 */
   FAR void *adj_stack_ptr;               /* Adjusted stack_alloc_ptr for HW    
 */
-                                         /* The initial stack pointer value    
 */
+  FAR void *tls_info_ptr;                /* Thread local storage information   
 */

Review comment:
       This change is wrong.  Please remove tls_info_ptr which must always have 
the same values stack_alloc_ptr.




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