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/02/03 07:32:19 UTC

[GitHub] [incubator-nuttx] anchao opened a new pull request #5405: net/tcp/netdev/mld: correct the netlock handling

anchao opened a new pull request #5405:
URL: https://github.com/apache/incubator-nuttx/pull/5405


   ## Summary
   
   arch/xtensa: correct the netlock handling
   arch/arm: correct the sched_lock/spin_lock handling
   net/tcp/netdev/mld: correct the netlock handling
   
   ## Impact
   
   N/A
   
   ## Testing
   
   CI check


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

[GitHub] [incubator-nuttx] masayuki2009 commented on pull request #5405: net/tcp/netdev/mld: correct the netlock handling

Posted by GitBox <gi...@apache.org>.
masayuki2009 commented on pull request #5405:
URL: https://github.com/apache/incubator-nuttx/pull/5405#issuecomment-1029495938


   @anchao 
   Can you explain why you added the following commit to this PR?
   I thought this PR relates to the net_lock handling.
   
   ```
   commit ed4f852073ca71e53de785d3a8e082c382de03bb
   Author: chao.an <an...@xiaomi.com>
   Date:   Thu Feb 3 15:29:14 2022 +0800
   
       arch/arm: correct the sched_lock/spin_lock handling
       
       Signed-off-by: chao.an <an...@xiaomi.com>
   ```


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

[GitHub] [incubator-nuttx] masayuki2009 edited a comment on pull request #5405: net/tcp/netdev/mld: correct the netlock handling

Posted by GitBox <gi...@apache.org>.
masayuki2009 edited a comment on pull request #5405:
URL: https://github.com/apache/incubator-nuttx/pull/5405#issuecomment-1029495938


   @anchao 
   Could you explain why you added the following commit to this PR?
   I thought this PR relates to the net_lock handling.
   
   ```
   commit ed4f852073ca71e53de785d3a8e082c382de03bb
   Author: chao.an <an...@xiaomi.com>
   Date:   Thu Feb 3 15:29:14 2022 +0800
   
       arch/arm: correct the sched_lock/spin_lock handling
       
       Signed-off-by: chao.an <an...@xiaomi.com>
   ```


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

[GitHub] [incubator-nuttx] acassis merged pull request #5405: net/tcp/netdev/mld: correct the netlock handling

Posted by GitBox <gi...@apache.org>.
acassis merged pull request #5405:
URL: https://github.com/apache/incubator-nuttx/pull/5405


   


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

[GitHub] [incubator-nuttx] anchao commented on pull request #5405: net/tcp/netdev/mld: correct the netlock handling

Posted by GitBox <gi...@apache.org>.
anchao commented on pull request #5405:
URL: https://github.com/apache/incubator-nuttx/pull/5405#issuecomment-1031655926


   > @anchao Could you explain why you added the following commit to this PR? I thought this PR relates to the net_lock handling.
   
   @masayuki2009 san,
   
   Sorry for confuse. This pull request is mainly to solve the issue of net_lock handling, 
   the sched_lock change only completes the unlock handling, the fix is minor,
   Anyway, I will split similar commit at next time.
   
   


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