jlaitine commented on code in PR #16194:
URL: https://github.com/apache/nuttx/pull/16194#discussion_r2076136934
##########
sched/semaphore/sem_post.c:
##########
@@ -116,7 +153,10 @@ int nxsem_post_slow(FAR sem_t *sem)
* initialized if the semaphore is to used for signaling purposes.
*/
- nxsem_release_holder(sem);
+ if (!mutex || blocking)
Review Comment:
Can you please explain what you mean. You need to add the holder to
tcb->holdsem list for the priority inheritance / restoration.
I am definitely not going to remove those. A thread can hold several mutexes
and priority can be boosted many times. To restore the priority to correct
level you need to have the semaphores (holder structures) added to the tcb:s
list (holdsem list).
--
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-unsubscr...@nuttx.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
