Changeset f5837524 in mainline for kernel/generic/src/ipc/ops/concttome.c

Timestamp:

2018-10-29T17:15:02Z (5 years ago)

Author:

Jakub Jermar <jakub@…>

Children:

eec201d

Parents:

184f2f8a

git-author:

Jakub Jermar <jakub@…> (2018-10-28 12:42:35)

git-committer:

Jakub Jermar <jakub@…> (2018-10-29 17:15:02)

Message:

Use user-defined labels instead of phone hashes

This commit changes the way how the async framework maps incomming calls
to connections. Instead of abusing the kernel addresses of attached
phones as identifiers, the IPC_M_CONNECT_TO_ME and IPC_M_CONNECT_ME_TO
messages allow the server to specify an arbitrary label which is
remembered in the connected phone and consequently imprinted on each
call which is routed through this phone.

The async framework uses the address of the connection structure as the
label. This removes the need for a connection hash table because each
incoming call already remembers the connection in its label.

To disambiguate this new label and the other user-defined label used for
answers, the call structure now has the request_label member for the
former and answer_label member for the latter.

This commit also moves the kernel definition of ipc_data_t to abi/ and
removes the uspace redefinition thereof. Finally, when forwarding the
IPC_M_CONNECT_TO_ME call, the phone capability and the kernel object
allocated in request_process are now correctly disposed of.

File:

• kernel/generic/src/ipc/ops/concttome.c (modified) (4 diffs)

kernel/generic/src/ipc/ops/concttome.c

@@ -42 +42 @@
 static int request_process(call_t *call, answerbox_t *box)
 {
-        cap_phone_handle_t phone_handle;
-        kobject_t *phone_obj;
-        errno_t rc = phone_alloc(TASK, false, &phone_handle, &phone_obj);
-        call->priv = (sysarg_t) phone_obj;
-        IPC_SET_ARG5(call->data,
-            (rc == EOK) ? CAP_HANDLE_RAW(phone_handle) : CAP_NIL);
+        cap_phone_handle_t phandle = CAP_NIL;
+        kobject_t *pobj = NULL;
+        errno_t rc = phone_alloc(TASK, false, &phandle, &pobj);
+        if (rc == EOK) {
+                /*
+                 * Set the sender-assigned label to the new phone.
+                 */
+                pobj->phone->label = IPC_GET_ARG5(call->data);
+        }
+        call->priv = (sysarg_t) pobj;
+        IPC_SET_ARG5(call->data, CAP_HANDLE_RAW(phandle));
         return 0;
 }
@@ -53 +58 @@
 static errno_t answer_cleanup(call_t *answer, ipc_data_t *olddata)
 {
-        cap_phone_handle_t phone_handle = (cap_handle_t) IPC_GET_ARG5(*olddata);
-        kobject_t *phone_obj = (kobject_t *) answer->priv;
+        cap_phone_handle_t phandle = (cap_handle_t) IPC_GET_ARG5(*olddata);
+        kobject_t *pobj = (kobject_t *) answer->priv;
 
-        if (CAP_HANDLE_VALID(phone_handle)) {
-                kobject_put(phone_obj);
-                cap_free(TASK, phone_handle);
+        if (CAP_HANDLE_VALID(phandle)) {
+                kobject_put(pobj);
+                cap_free(TASK, phandle);
         }
 
@@ -66 +71 @@
 static errno_t answer_preprocess(call_t *answer, ipc_data_t *olddata)
 {
-        cap_phone_handle_t phone_handle = (cap_handle_t) IPC_GET_ARG5(*olddata);
-        kobject_t *phone_obj = (kobject_t *) answer->priv;
+        cap_phone_handle_t phandle = (cap_handle_t) IPC_GET_ARG5(*olddata);
+        kobject_t *pobj = (kobject_t *) answer->priv;
 
         if (IPC_GET_RETVAL(answer->data) != EOK) {
                 /* The connection was not accepted */
                 answer_cleanup(answer, olddata);
-        } else if (CAP_HANDLE_VALID(phone_handle)) {
+        } else if (CAP_HANDLE_VALID(phandle)) {
                 /*
                  * The connection was accepted
@@ -81 +86 @@
                  * will be consumed by ipc_phone_connect().
                  */
-                kobject_add_ref(phone_obj);
+                kobject_add_ref(pobj);
 
-                if (ipc_phone_connect(phone_obj->phone,
+                if (ipc_phone_connect(pobj->phone,
                     &answer->sender->answerbox)) {
                         /* Pass the reference to the capability */
-                        cap_publish(TASK, phone_handle, phone_obj);
-                        /* Set 'phone hash' as ARG5 of response */
-                        IPC_SET_ARG5(answer->data,
-                            (sysarg_t) phone_obj->phone);
+                        cap_publish(TASK, phandle, pobj);
                 } else {
                         /* The answerbox is shutting down. */