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/12/30 02:07:54 UTC

[GitHub] [incubator-nuttx] zhaoxiu-zeng opened a new pull request #5120: signal: signal handler may cause task's state error

zhaoxiu-zeng opened a new pull request #5120:
URL: https://github.com/apache/incubator-nuttx/pull/5120


   For example, task is blocked by nxsem_wait(sem1), use nxsem_wait(sem2)
   in signal handler, and take sem2 successfully, after exit from signal
   handler to task, nxsem_wait(sem1) returns OK, but the correct result
   should be -EINTR.
   
   Signed-off-by: Zeng Zhaoxiu <zh...@gmail.com>
   
   ## Summary
   
   ## Impact
   
   ## Testing
   
   


-- 
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 merged pull request #5120: signal: signal handler may cause task's state error

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


   


-- 
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] zhaoxiu-zeng commented on pull request #5120: signal: signal handler may cause task's state error

Posted by GitBox <gi...@apache.org>.
zhaoxiu-zeng commented on pull request #5120:
URL: https://github.com/apache/incubator-nuttx/pull/5120#issuecomment-1004597983


   > > For example, task is blocked by nxsem_wait(sem1), use nxsem_wait(sem2)
   > > in signal handler, and take sem2 successfully, after exit from signal
   > > handler to task, nxsem_wait(sem1) returns OK, but the correct result
   > > should be -EINTR.
   > 
   > @zhaoxiu-zeng Can you extend the ostest so that we can test if the PR works correctry?
   
   https://github.com/apache/incubator-nuttx-apps/pull/955


-- 
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 #5120: signal: signal handler may cause task's state error

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


   >For example, task is blocked by nxsem_wait(sem1), use nxsem_wait(sem2)
   >in signal handler, and take sem2 successfully, after exit from signal
   >handler to task, nxsem_wait(sem1) returns OK, but the correct result
   >should be -EINTR.
   
   @zhaoxiu-zeng 
   Can you extend the ostest so that we can test if the PR works correctry?
   


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