You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@nuttx.apache.org by GitBox <gi...@apache.org> on 2021/06/08 16:54:05 UTC

[GitHub] [incubator-nuttx] gustavonihei commented on pull request #3871: arch: adjust idle stack offset to reserve space of stack info

gustavonihei commented on pull request #3871:
URL: https://github.com/apache/incubator-nuttx/pull/3871#issuecomment-856933479


   > If the user wants a 4096 stack they will not get the value they set. That seam wrong and confusing.
   > 
   > The TLS size should be added the the allocation So that USER+TLS == stack(USER's STACK) + (tls size) (rounded up for alignment)
   
   I think the developer should not create this expectation that the defined stack size will be entirely available for usage.
   
   The Linux Kernel seem to do the same thing:
   https://elixir.bootlin.com/linux/v4.3/source/include/linux/sched.h#L2400


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