Changeset 503ffce in mainline for uspace

Timestamp:

2017-11-23T23:52:59Z (8 years ago)

Author:

Jakub Jermar <jakub@…>

Branches:

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

Children:

f571ca49

Parents:

b1f36e3

Message:

Return IPC_CALLID_* in call data instead of callid

Callid will be replaced by capability handles soon so the API needs
to be cleanup up and any flags passed together with callid must be
passed using some other way.

Location:

uspace

Files:

• app/trace/ipcp.c (modified) (2 diffs)
• lib/c/generic/async.c (modified) (2 diffs)
• lib/c/generic/ipc.c (modified) (4 diffs)
• lib/c/include/ipc/common.h (modified) (1 diff)

uspace/app/trace/ipcp.c

@@ -323 +323 @@
         pending_call_t *pcall;
         
-        if ((hash & IPC_CALLID_ANSWERED) == 0 && hash != IPCP_CALLID_SYNC) {
+        if ((call->flags & IPC_CALLID_ANSWERED) == 0 &&
+            hash != IPCP_CALLID_SYNC) {
                 /* Not a response */
                 if ((display_mask & DM_IPC) != 0) {
@@ -331 +332 @@
         }
         
-        hash = hash & ~IPC_CALLID_ANSWERED;
-        
         item = hash_table_find(&pending_calls, &hash);
         if (item == NULL)

uspace/lib/c/generic/async.c

@@ -1341 +1341 @@
         
         /* Kernel notification */
-        if ((callid & IPC_CALLID_NOTIFICATION)) {
+        if (call->flags & IPC_CALLID_NOTIFICATION) {
                 fibril_t *fibril = (fibril_t *) __tcb_get()->fibril_data;
                 unsigned oldsw = fibril->switches;
@@ -1495 +1495 @@
                 }
                 
-                if (callid & IPC_CALLID_ANSWERED)
+                if (call.flags & IPC_CALLID_ANSWERED)
                         continue;
                 

uspace/lib/c/generic/ipc.c

@@ -259 +259 @@
  * @param flags Flags passed to SYS_IPC_WAIT (blocking, nonblocking).
  *
- * @return Hash of the call. Note that certain bits have special
- *         meaning: IPC_CALLID_ANSWERED is set in an answer
- *         and IPC_CALLID_NOTIFICATION is used for notifications.
- *
+ * @return Hash of the call.
  */
 ipc_callid_t ipc_wait_cycle(ipc_call_t *call, sysarg_t usec,
@@ -271 +268 @@
         
         /* Handle received answers */
-        if (callid & IPC_CALLID_ANSWERED)
+        if (callid && (call->flags & IPC_CALLID_ANSWERED))
                 handle_answer(callid, call);
         
@@ -301 +298 @@
         do {
                 callid = ipc_wait_cycle(call, usec, SYNCH_FLAGS_NONE);
-        } while (callid & IPC_CALLID_ANSWERED);
+        } while (callid && (call->flags & IPC_CALLID_ANSWERED));
         
         return callid;
@@ -322 +319 @@
                 callid = ipc_wait_cycle(call, SYNCH_NO_TIMEOUT,
                     SYNCH_FLAGS_NON_BLOCKING);
-        } while (callid & IPC_CALLID_ANSWERED);
+        } while (callid && (call->flags & IPC_CALLID_ANSWERED));
         
         return callid;

uspace/lib/c/include/ipc/common.h

@@ -49 +49 @@
         task_id_t in_task_id;
         sysarg_t in_phone_hash;
+        unsigned flags;
         struct async_call *label;
 } ipc_call_t;