Changeset cd529c4 in mainline

Timestamp:

2012-08-13T09:15:57Z (12 years ago)

Author:

Jakub Jermar <jakub@…>

Branches:

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

Children:

4820360, cfaa35a

Parents:

d06c4ae

Message:

In the absence of synchronous IPC calls, call_t's callerbox is just
caching call_t's sender→answerbox. Get rid of this redundancy.

Location:

kernel/generic

Files:

• include/ipc/ipc.h (modified) (1 diff)
• src/ipc/ipc.c (modified) (2 diffs)

kernel/generic/include/ipc/ipc.h

@@ -113 +113 @@
         struct task *sender;
         
-        /*
-         * The caller box is different from sender->answerbox
-         * for synchronous calls.
-         */
-        answerbox_t *callerbox;
-        
         /** Private data to internal IPC. */
         sysarg_t priv;

kernel/generic/src/ipc/ipc.c

@@ -71 +71 @@
 {
         memsetb(call, sizeof(*call), 0);
-        call->callerbox = &TASK->answerbox;
         call->sender = TASK;
         call->buffer = NULL;
@@ -170 +169 @@
 static void _ipc_answer_free_call(call_t *call, bool selflocked)
 {
-        answerbox_t *callerbox = call->callerbox;
+        answerbox_t *callerbox = &call->sender->answerbox;
         bool do_lock = ((!selflocked) || callerbox != (&TASK->answerbox));