Changeset 0f4f1b2 in mainline for kernel/test/mm/slab2.c

Timestamp:

2024-01-15T17:10:27Z (11 months ago)

Author:

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

Branches:

master

Children:

e82879c

Parents:

a064d4f

git-author:

Jiří Zárevúcky <zarevucky.jiri@…> (2024-01-15 16:37:22)

git-committer:

Jiří Zárevúcky <zarevucky.jiri@…> (2024-01-15 17:10:27)

Message:

Add (and use) functions thread_start() and thread_detach()

Mostly cosmetic, with thread_start() replacing calls to thread_ready(),
but not consuming the passed reference, and thread_detach() being
synonym for thread_put(). Makes the code's function more obvious.

Also modify some threaded tests to use thread_join() for waiting,
instead of counting threads with atomics or semaphores.

File:

• kernel/test/mm/slab2.c (modified) (4 diffs)

kernel/test/mm/slab2.c

@@ -127 +127 @@
 
 static slab_cache_t *thr_cache;
-static semaphore_t thr_sem;
 static condvar_t thread_starter;
 static mutex_t starter_mutex;
@@ -188 +187 @@
         if (!test_quiet)
                 slab_print_list();
-
-        semaphore_up(&thr_sem);
 }
 
@@ -198 +195 @@
          * then release everything, then again allocate, then release
          */
-        thread_t *t;
-        int i;
 
         TPRINTF("Running stress test with size %d\n", size);
@@ -207 +202 @@
 
         thr_cache = slab_cache_create("thread_cache", size, 0, NULL, NULL, 0);
-        semaphore_initialize(&thr_sem, 0);
-        for (i = 0; i < THREADS; i++) {
-                if (!(t = thread_create(slabtest, NULL, TASK, THREAD_FLAG_NONE, "slabtest"))) {
+
+        thread_t *threads[THREADS] = { };
+
+        for (int i = 0; i < THREADS; i++) {
+                threads[i] = thread_create(slabtest, NULL,
+                    TASK, THREAD_FLAG_NONE, "slabtest");
+                if (threads[i]) {
+                        thread_start(threads[i]);
+                } else {
                         TPRINTF("Could not create thread %d\n", i);
-                } else
-                        thread_ready(t);
-        }
+                }
+        }
+
         thread_sleep(1);
         condvar_broadcast(&thread_starter);
 
-        for (i = 0; i < THREADS; i++)
-                semaphore_down(&thr_sem);
+        for (int i = 0; i < THREADS; i++) {
+                if (threads[i] != NULL)
+                        thread_join(threads[i]);
+        }
 
         slab_cache_destroy(thr_cache);