Changeset a064d4f in mainline for kernel/generic/src/proc/thread.c

Timestamp:

2024-01-15T15:39:09Z (4 months ago)

Author:

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

Branches:

master

Children:

0f4f1b2

Parents:

5861b60

Message:

Make thread_join() imply thread_put()

This makes the function more similar to traditional threading APIs.

File:

• kernel/generic/src/proc/thread.c (modified) (2 diffs)

kernel/generic/src/proc/thread.c

@@ -685 +685 @@
 
 /** Wait for another thread to exit.
- * This function does not destroy the thread. Reference counting handles that.
+ * After successful wait, the thread reference is destroyed.
  *
  * @param thread Thread to join on exit.
@@ -703 +703 @@
         irq_spinlock_unlock(&thread->lock, true);
 
-        if (state == Exiting) {
-                return EOK;
-        } else {
-                return _waitq_sleep_timeout(&thread->join_wq, usec, flags);
-        }
+        errno_t rc = EOK;
+
+        if (state != Exiting)
+                rc = _waitq_sleep_timeout(&thread->join_wq, usec, flags);
+
+        if (rc == EOK)
+                thread_put(thread);
+
+        return rc;
 }