Changeset 91b60499 in mainline for uspace/lib/c/generic/async.c

Timestamp:

2017-09-30T06:29:42Z (7 years ago)

Author:

Jakub Jermar <jakub@…>

Branches:

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

Children:

300f4c4

Parents:

d076f16 (diff), 6636fb19 (diff)
Note: this is a merge changeset, the changes displayed below correspond to the merge itself.
Use the (diff) links above to see all the changes relative to each parent.

Message:

Merge support for capabilities from lp:~jakub/helenos/caps

This commit introduces capabilities as task-local names for references to kernel
objects. Kernel objects are reference-counted wrappers for a select group of
objects allocated in and by the kernel that can be made accessible to userspace
in a controlled way via integer handles.

So far, a kernel object encapsulates either an irq_t or a phone_t.

Support for the former lead to the removal of kernel-assigned devnos and
unsecure deregistration of IRQs in which a random task was able to unregister
some other task's IRQ.

File:

• uspace/lib/c/generic/async.c (modified) (5 diffs)

uspace/lib/c/generic/async.c

@@ -1048 +1048 @@
  *
  * @param inr     IRQ number.
- * @param devno   Device number of the device generating inr.
  * @param handler Notification handler.
  * @param data    Notification handler client data.
  * @param ucode   Top-half pseudocode handler.
  *
- * @return Zero on success or a negative error code.
- *
- */
-int async_irq_subscribe(int inr, int devno,
-    async_notification_handler_t handler, void *data, const irq_code_t *ucode)
+ * @return IRQ capability handle on success.
+ * @return Negative error code.
+ *
+ */
+int async_irq_subscribe(int inr, async_notification_handler_t handler,
+    void *data, const irq_code_t *ucode)
 {
         notification_t *notification =
@@ -1077 +1077 @@
         futex_up(&async_futex);
         
-        return ipc_irq_subscribe(inr, devno, imethod, ucode);
+        return ipc_irq_subscribe(inr, imethod, ucode);
 }
 
 /** Unsubscribe from IRQ notification.
  *
- * @param inr     IRQ number.
- * @param devno   Device number of the device generating inr.
+ * @param cap     IRQ capability handle. 
  *
  * @return Zero on success or a negative error code.
  *
  */
-int async_irq_unsubscribe(int inr, int devno)
+int async_irq_unsubscribe(int cap)
 {
         // TODO: Remove entry from hash table
         //       to avoid memory leak
         
-        return ipc_irq_unsubscribe(inr, devno);
+        return ipc_irq_unsubscribe(cap);
 }
 
@@ -1384 +1383 @@
                 async_new_connection(call->in_task_id, in_phone_hash, callid,
                     call, handler, data);
-                return;
-        }
-        
-        /* Cloned connection */
-        if (IPC_GET_IMETHOD(*call) == IPC_M_CLONE_ESTABLISH) {
-                // TODO: Currently ignores ports altogether
-                
-                /* Open new connection with fibril, etc. */
-                async_new_connection(call->in_task_id, IPC_GET_ARG5(*call),
-                    callid, call, fallback_port_handler, fallback_port_data);
                 return;
         }
@@ -2111 +2100 @@
         
         return EOK;
-}
-
-/** Wrapper for making IPC_M_CLONE_ESTABLISH calls using the async framework.
- *
- * Ask for a cloned connection to some service.
- *
- * @param mgmt Exchange management style.
- * @param exch Exchange for sending the message.
- *
- * @return New session on success or NULL on error.
- *
- */
-async_sess_t *async_clone_establish(exch_mgmt_t mgmt, async_exch_t *exch)
-{
-        if (exch == NULL) {
-                errno = ENOENT;
-                return NULL;
-        }
-        
-        async_sess_t *sess = (async_sess_t *) malloc(sizeof(async_sess_t));
-        if (sess == NULL) {
-                errno = ENOMEM;
-                return NULL;
-        }
-        
-        ipc_call_t result;
-        
-        amsg_t *msg = amsg_create();
-        if (!msg) {
-                free(sess);
-                errno = ENOMEM;
-                return NULL;
-        }
-        
-        msg->dataptr = &result;
-        msg->wdata.active = true;
-        
-        ipc_call_async_0(exch->phone, IPC_M_CLONE_ESTABLISH, msg,
-            reply_received);
-        
-        sysarg_t rc;
-        async_wait_for((aid_t) msg, &rc);
-        
-        if (rc != EOK) {
-                errno = rc;
-                free(sess);
-                return NULL;
-        }
-        
-        int phone = (int) IPC_GET_ARG5(result);
-        
-        if (phone < 0) {
-                errno = phone;
-                free(sess);
-                return NULL;
-        }
-        
-        sess->iface = 0;
-        sess->mgmt = mgmt;
-        sess->phone = phone;
-        sess->arg1 = 0;
-        sess->arg2 = 0;
-        sess->arg3 = 0;
-        
-        fibril_mutex_initialize(&sess->remote_state_mtx);
-        sess->remote_state_data = NULL;
-        
-        list_initialize(&sess->exch_list);
-        fibril_mutex_initialize(&sess->mutex);
-        atomic_set(&sess->refcnt, 0);
-        
-        return sess;
 }
 
@@ -3153 +3070 @@
 }
 
-/** Wrapper for sending an exchange over different exchange for cloning
- *
- * @param exch       Exchange to be used for sending.
- * @param clone_exch Exchange to be cloned.
- *
- */
-int async_exchange_clone(async_exch_t *exch, async_exch_t *clone_exch)
-{
-        return async_req_1_0(exch, IPC_M_CONNECTION_CLONE, clone_exch->phone);
-}
-
-/** Wrapper for receiving the IPC_M_CONNECTION_CLONE calls.
- *
- * If the current call is IPC_M_CONNECTION_CLONE then a new
- * async session is created for the accepted phone.
- *
- * @param mgmt Exchange management style.
- *
- * @return New async session or NULL on failure.
- *
- */
-async_sess_t *async_clone_receive(exch_mgmt_t mgmt)
-{
-        /* Accept the phone */
-        ipc_call_t call;
-        ipc_callid_t callid = async_get_call(&call);
-        int phone = (int) IPC_GET_ARG1(call);
-        
-        if ((IPC_GET_IMETHOD(call) != IPC_M_CONNECTION_CLONE) ||
-            (phone < 0)) {
-                async_answer_0(callid, EINVAL);
-                return NULL;
-        }
-        
-        async_sess_t *sess = (async_sess_t *) malloc(sizeof(async_sess_t));
-        if (sess == NULL) {
-                async_answer_0(callid, ENOMEM);
-                return NULL;
-        }
-        
-        sess->iface = 0;
-        sess->mgmt = mgmt;
-        sess->phone = phone;
-        sess->arg1 = 0;
-        sess->arg2 = 0;
-        sess->arg3 = 0;
-        
-        fibril_mutex_initialize(&sess->remote_state_mtx);
-        sess->remote_state_data = NULL;
-        
-        list_initialize(&sess->exch_list);
-        fibril_mutex_initialize(&sess->mutex);
-        atomic_set(&sess->refcnt, 0);
-        
-        /* Acknowledge the cloned phone */
-        async_answer_0(callid, EOK);
-        
-        return sess;
-}
-
 /** Wrapper for receiving the IPC_M_CONNECT_TO_ME calls.
  *