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 2020/04/25 17:19:46 UTC

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #872: nuttx: Change all default stack size to DEFAULT_TASK_STACKSIZE

xiaoxiang781216 commented on pull request #872:
URL: https://github.com/apache/incubator-nuttx/pull/872#issuecomment-619412558


   > After reviewing an trying to comment on all of the problems and risks. I come to the conclusion that this change is wrong and must not be merged.
   
   The change make the default value bigger than the old one, so the side effect is the stack memory consumption will a little bigger than before. But the benefit is that user can change all default stack value by CONFIG_DEFAULT_STACKSIZE for the normal task and still can adjust the stack size in the fine tune phase.


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