Changes in kernel/generic/src/ipc/kbox.c [716185d:c0699467] in mainline

File:

• kernel/generic/src/ipc/kbox.c (modified) (5 diffs)

kernel/generic/src/ipc/kbox.c

@@ -48 +48 @@
 {
         /*
-         * Not really needed, just to be consistent with the meaning of
-         * answerbox_t.active.
-         */
-        irq_spinlock_lock(&TASK->kb.box.lock, true);
-        TASK->kb.box.active = false;
-        irq_spinlock_unlock(&TASK->kb.box.lock, true);
-
-        /*
          * Only hold kb.cleanup_lock while setting kb.finished -
          * this is enough.
@@ -96 +88 @@
         
         /* Answer all messages in 'calls' and 'dispatched_calls' queues. */
-        ipc_cleanup_call_list(&TASK->kb.box, &TASK->kb.box.calls);
-        ipc_cleanup_call_list(&TASK->kb.box, &TASK->kb.box.dispatched_calls);
+        irq_spinlock_lock(&TASK->kb.box.lock, true);
+        ipc_cleanup_call_list(&TASK->kb.box.dispatched_calls);
+        ipc_cleanup_call_list(&TASK->kb.box.calls);
+        irq_spinlock_unlock(&TASK->kb.box.lock, true);
 }
 
@@ -168 +162 @@
         while (!done) {
                 call_t *call = ipc_wait_for_call(&TASK->kb.box, SYNCH_NO_TIMEOUT,
-                    SYNCH_FLAGS_NONE);
+                        SYNCH_FLAGS_NONE);
                 
                 if (call == NULL)
@@ -242 +236 @@
         
         /* Connect the newly allocated phone to the kbox */
-        (void) ipc_phone_connect(&TASK->phones[newphid], &task->kb.box);
+        ipc_phone_connect(&TASK->phones[newphid], &task->kb.box);
         
         if (task->kb.thread != NULL) {
@@ -250 +244 @@
         
         /* Create a kbox thread */
-        thread_t *kb_thread = thread_create(kbox_thread_proc, NULL, task,
-            THREAD_FLAG_NONE, "kbox");
+        thread_t *kb_thread = thread_create(kbox_thread_proc, NULL, task, 0,
+            "kbox", false);
         if (!kb_thread) {
                 mutex_unlock(&task->kb.cleanup_lock);