Changeset a064d4f in mainline for kernel/generic/src/proc

Timestamp:

2024-01-15T15:39:09Z (19 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.

Location:

kernel/generic/src/proc

Files:

• scheduler.c (modified) (1 diff)
• thread.c (modified) (2 diffs)

kernel/generic/src/proc/scheduler.c

@@ -449 +449 @@
         THREAD->kcycles += get_cycle() - THREAD->last_cycle;
 
+        /*
+         * On Sparc, this saves some extra userspace state that's not
+         * covered by context_save()/context_restore().
+         */
         after_thread_ran_arch();
 

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