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/07/03 12:44:32 UTC

[GitHub] [incubator-nuttx] acassis commented on issue #1354: Bug in can driver: wrong use of nxsem_get_value

acassis commented on issue #1354:
URL: https://github.com/apache/incubator-nuttx/issues/1354#issuecomment-653530715


   Hi Olev,
   Yes, you are right: https://pubs.opengroup.org/onlinepubs/9699919799/functions/sem_getvalue.html
   Please submit a patch to fix the text.
   
   The DEBUGASSERT to cd_nrxwaiters < 255 is there to guarantee we will have less than 255 threads reading the data.
   
   Then can_poll receives a signal it will return the same why it returns when you get data, but then you need to check the errno var to verify if it returned with a value of EINTR. I think this is what the awkward means in this context.


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