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/09/04 11:11:37 UTC

[GitHub] [incubator-nuttx] sebastianene07 edited a comment on pull request #1709: arch/sim: Fix the up_cpu_start initialisation on OSX with SMP config enabled

sebastianene07 edited a comment on pull request #1709:
URL: https://github.com/apache/incubator-nuttx/pull/1709#issuecomment-687069166


   > After more reviewing, I think we can remove the semphare from the code since the synchronization isn't really needed.
   
   ~~Ok I will remove it, in that case I will also drop the changes from `nuttx-names.in`~~ I think we need this synchronization after a careful look I ended up in a bad situation where :  I received a signal(SIGUSR1) in a thread that didn't end up setting the g_cpu_key. 


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