Changeset cc27c8c5 in mainline for uspace/lib/libc/generic/fibril.c

Timestamp:

2009-06-08T18:13:00Z (16 years ago)

Author:

Jakub Jermar <jakub@…>

Branches:

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

Children:

bac82eeb

Parents:

d9c8c81

Message:

A little bit of cleanup and a remedy for great confusion introduced in revision 2483.

File:

• uspace/lib/libc/generic/fibril.c (modified) (9 diffs)

uspace/lib/libc/generic/fibril.c

@@ -50 +50 @@
 #endif
 
-/** This futex serializes access to ready_list, serialized_list and manage_list.
+/**
+ * This futex serializes access to ready_list, serialized_list and manager_list.
  */ 
 static atomic_t fibril_futex = FUTEX_INITIALIZER;
@@ -60 +61 @@
 static void fibril_main(void);
 
-/** Number of fibrils that are in async_serialized mode */
-static int serialized_fibrils;  /* Protected by async_futex */
-/** Thread-local count of serialization. If >0, we must not preempt */
+/** Number of threads that are executing a manager fibril. */
+static int threads_in_manager;
+/** Number of threads that are executing a manager fibril and are serialized. */
+static int serialized_threads;  /* Protected by async_futex */
+/** Thread-local count of serialization. If > 0, we must not preempt */
 static __thread int serialization_count;
-/** Counter for fibrils residing in async_manager */
-static int fibrils_in_manager;
 
 /** Setup fibril information into TCB structure */
@@ -144 +145 @@
                         goto ret_0;
                 /*
-                 * Do not preempt if there is not sufficient count of fibril 
-                 * managers.
+                 * Do not preempt if there is not enough threads to run the
+                 * ready fibrils, which are not serialized.
                  */
                 if (list_empty(&serialized_list) &&
-                    fibrils_in_manager <= serialized_fibrils) {
+                    threads_in_manager <= serialized_threads) {
                         goto ret_0;
                 }
@@ -195 +196 @@
                 else if (stype == FIBRIL_FROM_MANAGER) {
                         list_append(&srcf->link, &manager_list);
-                        fibrils_in_manager--;
+                        threads_in_manager--;
                 } else {        
                         /*
@@ -209 +210 @@
                 dstf = list_get_instance(manager_list.next, fibril_t, link);
                 if (serialization_count && stype == FIBRIL_TO_MANAGER) {
-                        serialized_fibrils++;
+                        serialized_threads++;
                         srcf->flags |= FIBRIL_SERIALIZED;
                 }
-                fibrils_in_manager++;
+                threads_in_manager++;
 
                 if (stype == FIBRIL_FROM_DEAD) 
@@ -220 +221 @@
                         dstf = list_get_instance(serialized_list.next, fibril_t,
                             link);
-                        serialized_fibrils--;
+                        serialized_threads--;
                 } else {
                         dstf = list_get_instance(ready_list.next, fibril_t,
@@ -270 +271 @@
 /** Add a fibril to the ready list.
  *
- * @param fid           Pinter to the fibril structure of the fibril to be
+ * @param fid           Pointer to the fibril structure of the fibril to be
  *                      added.
  */
@@ -288 +289 @@
 /** Add a fibril to the manager list.
  *
- * @param fid           Pinter to the fibril structure of the fibril to be added.
+ * @param fid           Pointer to the fibril structure of the fibril to be
+ *                      added.
  */
 void fibril_add_manager(fid_t fid)
@@ -315 +317 @@
 /** Return fibril id of the currently running fibril.
  *
- * @return              Fibril ID of the currently running pseudo thread.
+ * @return              Fibril ID of the currently running fibril.
  */
 fid_t fibril_get_id(void)