Changeset 1871118 in mainline for kernel/generic/src/main

Timestamp:

2023-02-10T22:59:11Z (3 years ago)

Author:

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

Branches:

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

Children:

11d2c983

Parents:

daadfa6

git-author:

Jiří Zárevúcky <zarevucky.jiri@…> (2023-02-10 22:53:12)

git-committer:

Jiří Zárevúcky <zarevucky.jiri@…> (2023-02-10 22:59:11)

Message:

Make thread_t reference counted

This simplifies interaction between various locks and thread
lifespan, which simplifies things. For example, threads_lock can
now simply be a mutex protecting the global it was made for, and
nothing more.

Location:

kernel/generic/src/main

Files:

• kinit.c (modified) (2 diffs)
• uinit.c (modified) (1 diff)

kernel/generic/src/main/kinit.c

@@ -102 +102 @@
         thread_t *thread;
 
-        /*
-         * Detach kinit as nobody will call thread_join_timeout() on it.
-         */
-        thread_detach(THREAD);
-
         interrupts_disable();
 
@@ -125 +120 @@
 
                 thread_wire(thread, &cpus[0]);
-                thread_ready(thread);
+                thread_ready(thread_ref(thread));
                 thread_join(thread);
-                thread_detach(thread);
+                thread_put(thread);
 
                 /*

kernel/generic/src/main/uinit.c

@@ -56 +56 @@
 void uinit(void *arg)
 {
-        /*
-         * So far, we don't have a use for joining userspace threads so we
-         * immediately detach each uinit thread. If joining of userspace threads
-         * is required, some userspace API based on the kernel mechanism will
-         * have to be implemented. Moreover, garbage collecting of threads that
-         * didn't detach themselves and nobody else joined them will have to be
-         * deployed for the event of forceful task termination.
-         */
-        thread_detach(THREAD);
-
 #ifdef CONFIG_UDEBUG
         udebug_stoppable_end();