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 2022/08/30 07:37:04 UTC

[GitHub] [incubator-nuttx] pkarashchenko commented on pull request #6958: sched: Remove volatile from the task list

pkarashchenko commented on PR #6958:
URL: https://github.com/apache/incubator-nuttx/pull/6958#issuecomment-1231268050

   @xiaoxiang781216 the PR description states about "protection by volatile", but volatile just ensures that structure members are always fetched from memory. But in general I agree that if access to the struct is protected by a critical section, then the structure elements can't change its value during protected time. The only question is SMP case where a spin lock might be needed. I will take a look into this change today


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

To unsubscribe, e-mail: commits-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org