Changes in kernel/generic/src/proc/thread.c [aae365bc:abf6c01] in mainline

File:

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

kernel/generic/src/proc/thread.c

@@ -165 +165 @@
 
 #ifdef CONFIG_FPU
-#ifdef CONFIG_FPU_LAZY
-        thread->saved_fpu_context = NULL;
-#else /* CONFIG_FPU_LAZY */
-        thread->saved_fpu_context = slab_alloc(fpu_context_cache, kmflags);
+        thread->saved_fpu_context = slab_alloc(fpu_context_cache,
+            FRAME_ATOMIC | kmflags);
         if (!thread->saved_fpu_context)
                 return ENOMEM;
-#endif /* CONFIG_FPU_LAZY */
 #endif /* CONFIG_FPU */
 
@@ -200 +197 @@
         if (!stack_phys) {
 #ifdef CONFIG_FPU
-                if (thread->saved_fpu_context)
-                        slab_free(fpu_context_cache, thread->saved_fpu_context);
+                assert(thread->saved_fpu_context);
+                slab_free(fpu_context_cache, thread->saved_fpu_context);
 #endif
                 return ENOMEM;
@@ -226 +223 @@
 
 #ifdef CONFIG_FPU
-        if (thread->saved_fpu_context)
-                slab_free(fpu_context_cache, thread->saved_fpu_context);
+        assert(thread->saved_fpu_context);
+        slab_free(fpu_context_cache, thread->saved_fpu_context);
 #endif
 
@@ -342 +339 @@
     thread_flags_t flags, const char *name)
 {
-        thread_t *thread = (thread_t *) slab_alloc(thread_cache, 0);
+        thread_t *thread = (thread_t *) slab_alloc(thread_cache, FRAME_ATOMIC);
         if (!thread)
                 return NULL;
+
+        if (thread_create_arch(thread, flags) != EOK) {
+                slab_free(thread_cache, thread);
+                return NULL;
+        }
 
         /* Not needed, but good for debugging */
@@ -407 +409 @@
         udebug_thread_initialize(&thread->udebug);
 #endif
-
-        /* Might depend on previous initialization */
-        thread_create_arch(thread);
 
         if ((flags & THREAD_FLAG_NOATTACH) != THREAD_FLAG_NOATTACH)
@@ -654 +653 @@
 
         return waitq_sleep_timeout(&thread->join_wq, usec, flags, NULL);
+
+        // FIXME: join should deallocate the thread.
+        //        Current code calls detach after join, that's contrary to how
+        //        join is used in other threading APIs.
 }