nuttx-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #872: nuttx: Change all default stack size to DEFAULT_TASK_STACKSIZE
Date Sat, 25 Apr 2020 17:19:46 GMT

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



Mime
View raw message