Changeset 7c7aae16 in mainline for generic/src/ipc/ipc.c

Timestamp:

2006-03-19T19:42:00Z (19 years ago)

Author:

Ondrej Palkovsky <ondrap@…>

Branches:

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

Children:

ad64a2d

Parents:

9f22213

Message:

Reduced unnecessary IPC system calls.
Allow everything to be sync & async, everything is handled using messages.

File:

• generic/src/ipc/ipc.c (modified) (4 diffs)

generic/src/ipc/ipc.c

@@ -146 +146 @@
         answerbox_t *callerbox = call->callerbox;
 
-        call->flags &= ~IPC_CALL_DISPATCHED;
         call->flags |= IPC_CALL_ANSWERED;
 
@@ -167 +166 @@
         spinlock_unlock(&box->lock);
         /* Send back answer */
+        _ipc_answer_free_call(call);
+}
+
+/** Simulate sending back a message
+ *
+ * Most errors are better handled by forming a normal backward
+ * message and sending it as a normal answer.
+ */
+void ipc_backsend_err(phone_t *phone, call_t *call, __native err)
+{
+        call->data.phone = phone;
+        atomic_inc(&phone->active_calls);
+        if (phone->busy == IPC_BUSY_CONNECTED)
+                IPC_SET_RETVAL(call->data, EHANGUP);
+        else
+                IPC_SET_RETVAL(call->data, ENOENT);
+
         _ipc_answer_free_call(call);
 }
@@ -201 +217 @@
                 if (call->flags & IPC_CALL_FORWARDED) {
                         IPC_SET_RETVAL(call->data, EFORWARD);
-                } else { /* Simulate sending a message */
-                        call->data.phone = phone;
-                        atomic_inc(&phone->active_calls);
+                        _ipc_answer_free_call(call);
+                } else { /* Simulate sending back a message */
                         if (phone->busy == IPC_BUSY_CONNECTED)
-                                IPC_SET_RETVAL(call->data, EHANGUP);
+                                ipc_backsend_err(phone, call, EHANGUP);
                         else
-                                IPC_SET_RETVAL(call->data, ENOENT);
+                                ipc_backsend_err(phone, call, ENOENT);
                 }
 
-                _ipc_answer_free_call(call);
                 return ENOENT;
         }
@@ -308 +322 @@
                 /* Append request to dispatch queue */
                 list_append(&request->list, &box->dispatched_calls);
-                request->flags |= IPC_CALL_DISPATCHED;
         } else {
                 printf("WARNING: Spurious IPC wakeup.\n");