Changeset a1026da in mainline for kernel/generic/src/ipc/sysipc.c

Timestamp:

2017-12-22T22:43:25Z (8 years ago)

Author:

Jakub Jermar <jakub@…>

Branches:

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

Children:

1569a9b

Parents:

f04b5b3

git-author:

Jakub Jermar <jakub@…> (2017-12-22 21:47:09)

git-committer:

Jakub Jermar <jakub@…> (2017-12-22 22:43:25)

Message:

Introduce IPC_CALL_AUTO_REPLY call flag

Calls that never make it to userspace due to error are answered by the
kernel and decorated with the newly introduced IPC_CALL_AUTO_REPLY. This
gives the sender a way to reliably figure out that the recipient is
still waiting for the call.

File:

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

kernel/generic/src/ipc/sysipc.c

@@ -761 +761 @@
 
         if (!call) {
-                ipc_data_t data = {0};
+                ipc_data_t data = {};
                 data.cap_handle = CAP_NIL;
                 STRUCT_TO_USPACE(calldata, &data);
@@ -767 +767 @@
         }
         
+        call->data.flags = call->flags;
         if (call->flags & IPC_CALL_NOTIF) {
                 /* Set in_phone_hash to the interrupt counter */
                 call->data.phone = (void *) call->priv;
                 
-                call->data.flags = IPC_CALL_NOTIF;
                 call->data.cap_handle = CAP_NIL;
 
@@ -788 +788 @@
                 }
 
-                call->data.flags = IPC_CALL_ANSWERED;
                 call->data.cap_handle = CAP_NIL;
                 
@@ -826 +825 @@
         /*
          * The callee will not receive this call and no one else has a chance to
-         * answer it. Reply with the EPARTY error code.
+         * answer it. Set the IPC_CALL_AUTO_REPLY flag and return the EPARTY
+         * error code.
          */
         ipc_data_t saved_data;
@@ -839 +839 @@
         IPC_SET_RETVAL(call->data, EPARTY);
         (void) answer_preprocess(call, saved ? &saved_data : NULL);
+        call->flags |= IPC_CALL_AUTO_REPLY;
         ipc_answer(&TASK->answerbox, call);