Changes in kernel/generic/include/ipc/ipc.h [c33f39f:cd529c4] in mainline

File:

• kernel/generic/include/ipc/ipc.h (modified) (5 diffs)

kernel/generic/include/ipc/ipc.h

@@ -72 +72 @@
 typedef struct answerbox {
         IRQ_SPINLOCK_DECLARE(lock);
-
-        /** Answerbox is active until it enters cleanup. */
-        bool active;
         
         struct task *task;
@@ -109 +106 @@
 
 typedef struct {
-        /**
-         * Task link.
-         * Valid only when the call is not forgotten.
-         * Protected by the task's active_calls_lock.
-         */
-        link_t ta_link;
-
-        /** Answerbox link. */
-        link_t ab_link;
+        link_t link;
         
         unsigned int flags;
-
-        /** Protects the forget member. */
-        SPINLOCK_DECLARE(forget_lock);
-
-        /**
-         * True if the caller 'forgot' this call and donated it to the callee.
-         * Forgotten calls are discarded upon answering (the answer is not
-         * delivered) and answered calls cannot be forgotten. Forgotten calls
-         * also do not figure on the task's active call list.
-         *
-         * We keep this separate from the flags so that it is not necessary
-         * to take a lock when accessing them.
-         */
-        bool forget;
-
-        /** True if the call is in the active list. */
-        bool active;
         
-        /**
-         * Identification of the caller.
-         * Valid only when the call is not forgotten.
-         */
+        /** Identification of the caller. */
         struct task *sender;
-        
-        /** Phone which was used to send the call. */
-        phone_t *caller_phone;
         
         /** Private data to internal IPC. */
@@ -152 +118 @@
         /** Data passed from/to userspace. */
         ipc_data_t data;
-
-        /** Method as it was sent in the request. */
-        sysarg_t request_method;
-
+        
         /** Buffer for IPC_M_DATA_WRITE and IPC_M_DATA_READ. */
         uint8_t *buffer;
+        
+        /*
+         * The forward operation can masquerade the caller phone. For those
+         * cases, we must keep it aside so that the answer is processed
+         * correctly.
+         */
+        phone_t *caller_phone;
 } call_t;
 
@@ -171 +141 @@
 extern int ipc_forward(call_t *, phone_t *, answerbox_t *, unsigned int);
 extern void ipc_answer(answerbox_t *, call_t *);
-extern void _ipc_answer_free_call(call_t *, bool);
 
 extern void ipc_phone_init(phone_t *);
-extern bool ipc_phone_connect(phone_t *, answerbox_t *);
+extern void ipc_phone_connect(phone_t *, answerbox_t *);
 extern int ipc_phone_hangup(phone_t *);
 
@@ -182 +151 @@
 extern void ipc_backsend_err(phone_t *, call_t *, sysarg_t);
 extern void ipc_answerbox_slam_phones(answerbox_t *, bool);
-extern void ipc_cleanup_call_list(answerbox_t *, list_t *);
+extern void ipc_cleanup_call_list(list_t *);
 
 extern void ipc_print_task(task_id_t);