yamt commented on code in PR #7565:
URL: https://github.com/apache/incubator-nuttx/pull/7565#discussion_r1017677013
##########
include/pthread.h:
##########
@@ -357,6 +357,7 @@ struct pthread_barrier_s
{
sem_t sem;
unsigned int count;
+ bool busy;
Review Comment:
isn't it enough to look at semcount/waitlist?
##########
libs/libc/pthread/pthread_barrierwait.c:
##########
@@ -143,6 +145,7 @@ int pthread_barrier_wait(FAR pthread_barrier_t *barrier)
* zero.
*/
+ barrier->busy = false;
Review Comment:
don't you want to wait for all the threads?
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
