Changes in kernel/generic/src/ipc/sysipc.c [33adc6ce:057d21a] in mainline

File:

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

kernel/generic/src/ipc/sysipc.c

@@ -531 +531 @@
     unative_t arg1, unative_t arg2, unative_t arg3, ipc_data_t *data)
 {
-        call_t *call;
+        call_t call;
         phone_t *phone;
         int res;
@@ -538 +538 @@
         GET_CHECK_PHONE(phone, phoneid, return ENOENT);
 
-        call = ipc_call_alloc(0);
-        IPC_SET_METHOD(call->data, method);
-        IPC_SET_ARG1(call->data, arg1);
-        IPC_SET_ARG2(call->data, arg2);
-        IPC_SET_ARG3(call->data, arg3);
+        ipc_call_static_init(&call);
+        IPC_SET_METHOD(call.data, method);
+        IPC_SET_ARG1(call.data, arg1);
+        IPC_SET_ARG2(call.data, arg2);
+        IPC_SET_ARG3(call.data, arg3);
         /*
          * To achieve deterministic behavior, zero out arguments that are beyond
          * the limits of the fast version.
          */
-        IPC_SET_ARG4(call->data, 0);
-        IPC_SET_ARG5(call->data, 0);
-
-        if (!(res = request_preprocess(call, phone))) {
+        IPC_SET_ARG4(call.data, 0);
+        IPC_SET_ARG5(call.data, 0);
+
+        if (!(res = request_preprocess(&call, phone))) {
 #ifdef CONFIG_UDEBUG
                 udebug_stoppable_begin();
 #endif
-                rc = ipc_call_sync(phone, call);
+                rc = ipc_call_sync(phone, &call);
 #ifdef CONFIG_UDEBUG
                 udebug_stoppable_end();
 #endif
-                if (rc != EOK) {
-                        /* The call will be freed by ipc_cleanup(). */
+                if (rc != EOK)
                         return rc;
-                }
-                process_answer(call);
+                process_answer(&call);
 
         } else {
-                IPC_SET_RETVAL(call->data, res);
-        }
-        rc = STRUCT_TO_USPACE(&data->args, &call->data.args);
-        ipc_call_free(call);
+                IPC_SET_RETVAL(call.data, res);
+        }
+        rc = STRUCT_TO_USPACE(&data->args, &call.data.args);
         if (rc != 0)
                 return rc;
@@ -587 +584 @@
     ipc_data_t *reply)
 {
-        call_t *call;
+        call_t call;
         phone_t *phone;
         int res;
         int rc;
 
+        ipc_call_static_init(&call);
+        rc = copy_from_uspace(&call.data.args, &question->args,
+            sizeof(call.data.args));
+        if (rc != 0)
+                return (unative_t) rc;
+
         GET_CHECK_PHONE(phone, phoneid, return ENOENT);
 
-        call = ipc_call_alloc(0);
-        rc = copy_from_uspace(&call->data.args, &question->args,
-            sizeof(call->data.args));
-        if (rc != 0) {
-                ipc_call_free(call);
-                return (unative_t) rc;
-        }
-
-
-        if (!(res = request_preprocess(call, phone))) {
+        if (!(res = request_preprocess(&call, phone))) {
 #ifdef CONFIG_UDEBUG
                 udebug_stoppable_begin();
 #endif
-                rc = ipc_call_sync(phone, call);
+                rc = ipc_call_sync(phone, &call);
 #ifdef CONFIG_UDEBUG
                 udebug_stoppable_end();
 #endif
-                if (rc != EOK) {
-                        /* The call will be freed by ipc_cleanup(). */
+                if (rc != EOK)
                         return rc;
-                }
-                process_answer(call);
+                process_answer(&call);
         } else 
-                IPC_SET_RETVAL(call->data, res);
-
-        rc = STRUCT_TO_USPACE(&reply->args, &call->data.args);
-        ipc_call_free(call);
+                IPC_SET_RETVAL(call.data, res);
+
+        rc = STRUCT_TO_USPACE(&reply->args, &call.data.args);
         if (rc != 0)
                 return rc;
@@ -1000 +991 @@
 
         if (call->flags & IPC_CALL_NOTIF) {
+                ASSERT(! (call->flags & IPC_CALL_STATIC_ALLOC));
+
                 /* Set in_phone_hash to the interrupt counter */
                 call->data.phone = (void *) call->priv;
@@ -1012 +1005 @@
         if (call->flags & IPC_CALL_ANSWERED) {
                 process_answer(call);
+
+                ASSERT(! (call->flags & IPC_CALL_STATIC_ALLOC));
 
                 if (call->flags & IPC_CALL_DISCARD_ANSWER) {