Changeset 40abf56 in mainline for uspace/lib/c/generic/thread.c

Timestamp:

2018-07-18T19:42:28Z (7 years ago)

Author:

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

Branches:

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

Children:

9bde0d5

Parents:

0b05082

git-author:

Jiří Zárevúcky <jiri.zarevucky@…> (2018-07-18 19:05:08)

git-committer:

Jiří Zárevúcky <jiri.zarevucky@…> (2018-07-18 19:42:28)

Message:

Make sure that a thread with uninitialized TLS does not need to call malloc()
to initialize it.

For threads and tasks created by loader, we create TLS beforehand and pass
it to the child. For tasks spawned directly by the kernel, we require it is
a static executable and allocate the initial TLS using as_area_create() instead
of the libc allocator.

File:

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

uspace/lib/c/generic/thread.c

@@ -58 +58 @@
 void __thread_main(uspace_arg_t *uarg)
 {
-        fibril_t *fibril = fibril_setup();
-        if (fibril == NULL)
-                thread_exit(0);
+        assert(!__tcb_is_set());
+
+        fibril_t *fibril = uarg->uspace_thread_arg;
+        assert(fibril);
 
         __tcb_set(fibril->tcb);
 
-        uarg->uspace_thread_function(uarg->uspace_thread_arg);
+        uarg->uspace_thread_function(fibril->arg);
         /*
          * XXX: we cannot free the userspace stack while running on it
@@ -95 +96 @@
     thread_id_t *tid)
 {
-        uspace_arg_t *uarg =
-            (uspace_arg_t *) malloc(sizeof(uspace_arg_t));
+        uspace_arg_t *uarg = calloc(1, sizeof(uspace_arg_t));
         if (!uarg)
                 return ENOMEM;
+
+        fibril_t *fibril = fibril_alloc();
+        if (!fibril) {
+                free(uarg);
+                return ENOMEM;
+        }
 
         size_t stack_size = stack_size_get();
@@ -105 +111 @@
             AS_AREA_LATE_RESERVE, AS_AREA_UNPAGED);
         if (stack == AS_MAP_FAILED) {
+                fibril_teardown(fibril, false);
                 free(uarg);
                 return ENOMEM;
@@ -112 +119 @@
         malloc_enable_multithreaded();
 
+        fibril->arg = arg;
         uarg->uspace_entry = (void *) FADDR(__thread_entry);
         uarg->uspace_stack = stack;
         uarg->uspace_stack_size = stack_size;
         uarg->uspace_thread_function = function;
-        uarg->uspace_thread_arg = arg;
+        uarg->uspace_thread_arg = fibril;
         uarg->uspace_uarg = uarg;