Changes in uspace/lib/c/generic/thread.c [47b7006:63f8966] in mainline

File:

• uspace/lib/c/generic/thread.c (modified) (4 diffs)

uspace/lib/c/generic/thread.c

@@ -31 +31 @@
  */
 /** @file
- */
+ */ 
 
 #include <thread.h>
@@ -41 +41 @@
 #include <str.h>
 #include <async.h>
-#include "private/thread.h"
 
 #ifndef THREAD_INITIAL_STACK_PAGES_NO
@@ -51 +50 @@
  * This function is called from __thread_entry() and is used
  * to call the thread's implementing function and perform cleanup
- * and exit when thread returns back.
+ * and exit when thread returns back. Do not call this function
+ * directly.
  *
  * @param uarg Pointer to userspace argument structure.
- *
  */
 void __thread_main(uspace_arg_t *uarg)
 {
-        fibril_t *fibril = fibril_setup();
-        if (fibril == NULL)
-                thread_exit(0);
-        
-        __tcb_set(fibril->tcb);
-        
+        fibril_t *f;
+
+        f = fibril_setup();
+        __tcb_set(f->tcb);
+
         uarg->uspace_thread_function(uarg->uspace_thread_arg);
-        /* XXX: we cannot free the userspace stack while running on it
-                free(uarg->uspace_stack);
-                free(uarg);
-        */
-        
+        /* XXX: we cannot free the userspace stack while running on it */
+//      free(uarg->uspace_stack);
+//      free(uarg);
+
         /* If there is a manager, destroy it */
         async_destroy_manager();
-        fibril_teardown(fibril);
-        
+        fibril_teardown(f);
+
         thread_exit(0);
 }
@@ -130 +127 @@
  *
  * @param status Exit status. Currently not used.
- *
  */
 void thread_exit(int status)
 {
         __SYSCALL1(SYS_THREAD_EXIT, (sysarg_t) status);
-        
-        /* Unreachable */
-        while (1);
+        for (;;)
+                ;
 }