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/10/17 14:40:10 UTC

[GitHub] [incubator-nuttx] hartmannathan commented on pull request #4591: net/devif: Remove the unnecessary net lock/unlock in devif_callback

hartmannathan commented on pull request #4591:
URL: https://github.com/apache/incubator-nuttx/pull/4591#issuecomment-945137192


   > > since the caller already hold the net lock before.
   > 
   > @xiaoxiang781216 can we be sure of this? I tried to follow the calls, but in some cases I couldn't find a lock.
   
   Also I think we should consider future inadvertent changes. That is, even if all code paths today have a lock, what happens if in the future one of those code paths is changed and doesn't have a lock anymore?


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