Changeset 111b9b9 in mainline for kernel/generic/src/synch/semaphore.c

Timestamp:

2023-02-11T19:13:44Z (2 years ago)

Author:

Jiří Zárevúcky <zarevucky.jiri@…>

Branches:

master, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export

Children:

4777e02

Parents:

76e17d7c

git-author:

Jiří Zárevúcky <zarevucky.jiri@…> (2022-08-15 17:46:39)

git-committer:

Jiří Zárevúcky <zarevucky.jiri@…> (2023-02-11 19:13:44)

Message:

Reimplement waitq using thread_wait/wakeup

This adds a few functions to the thread API which can be
summarized as "stop running until woken up by others".
The ordering and context-switching concerns are thus yeeted
to this abstraction and waitq only deals with maintaining
the queues. Overall, this makes the control flow in waitq
much easier to navigate.

File:

• kernel/generic/src/synch/semaphore.c (modified) (2 diffs)

kernel/generic/src/synch/semaphore.c

@@ -70 +70 @@
 errno_t semaphore_down_timeout(semaphore_t *sem, uint32_t usec)
 {
-        errno_t rc = waitq_sleep_timeout(&sem->wq, usec, SYNCH_FLAGS_NON_BLOCKING, NULL);
+        errno_t rc = waitq_sleep_timeout(&sem->wq, usec);
         assert(rc == EOK || rc == ETIMEOUT || rc == EAGAIN);
         return rc;
@@ -90 +90 @@
 void semaphore_up(semaphore_t *sem)
 {
-        waitq_wakeup(&sem->wq, WAKEUP_FIRST);
+        waitq_wake_one(&sem->wq);
 }