Changes in / [0453261:b5143bd] in mainline

Location:

uspace/lib/c

Files:

• generic/fibril.c (modified) (5 diffs)
• generic/libc.c (modified) (1 diff)
• generic/thread.c (modified) (1 diff)
• include/fibril.h (modified) (1 diff)

uspace/lib/c/generic/fibril.c

@@ -57 +57 @@
 /**
  * This futex serializes access to ready_list,
- * serialized_list, manager_list and fibril_list.
+ * serialized_list and manager_list.
  */
 static futex_t fibril_futex = FUTEX_INITIALIZER;
@@ -126 +126 @@
         
         fibril->waits_for = NULL;
-
-        futex_lock(&fibril_futex);
         list_append(&fibril->all_link, &fibril_list);
-        futex_unlock(&fibril_futex);
         
         return fibril;
 }
 
-void fibril_teardown(fibril_t *fibril, bool locked)
-{       
-        if (!locked)
-                futex_lock(&fibril_futex);
+void fibril_teardown(fibril_t *fibril)
+{
         list_remove(&fibril->all_link);
-        if (!locked)
-                futex_unlock(&fibril_futex);
         tls_free(fibril->tcb);
         free(fibril);
@@ -215 +208 @@
                                         as_area_destroy(stack);
                                 }
-                                fibril_teardown(srcf->clean_after_me, true);
+                                fibril_teardown(srcf->clean_after_me);
                                 srcf->clean_after_me = NULL;
                         }
@@ -301 +294 @@
             AS_AREA_LATE_RESERVE);
         if (fibril->stack == (void *) -1) {
-                fibril_teardown(fibril, false);
+                fibril_teardown(fibril);
                 return 0;
         }
@@ -328 +321 @@
         
         as_area_destroy(fibril->stack);
-        fibril_teardown(fibril, false);
+        fibril_teardown(fibril);
 }
 

uspace/lib/c/generic/libc.c

@@ -122 +122 @@
                 __stdio_done();
                 task_retval(status);
-                fibril_teardown(__tcb_get()->fibril_data, false);
+                fibril_teardown(__tcb_get()->fibril_data);
         }
         

uspace/lib/c/generic/thread.c

@@ -88 +88 @@
 #endif
         
-        fibril_teardown(fibril, false);
+        fibril_teardown(fibril);
         
         thread_exit(0);

uspace/lib/c/include/fibril.h

@@ -94 +94 @@
 extern void fibril_destroy(fid_t fid);
 extern fibril_t *fibril_setup(void);
-extern void fibril_teardown(fibril_t *f, bool locked);
+extern void fibril_teardown(fibril_t *f);
 extern int fibril_switch(fibril_switch_type_t stype);
 extern void fibril_add_ready(fid_t fid);