Changes in uspace/srv/devman/main.c [228e490:0b5a4131] in mainline

File:

• uspace/srv/devman/main.c (modified) (19 diffs)

uspace/srv/devman/main.c

@@ -36 +36 @@
  */
 
-#include <inttypes.h>
 #include <assert.h>
 #include <ipc/services.h>
@@ -74 +73 @@
         
         iid = async_get_call(&icall);
-        if (IPC_GET_IMETHOD(icall) != DEVMAN_DRIVER_REGISTER) {
+        if (IPC_GET_METHOD(icall) != DEVMAN_DRIVER_REGISTER) {
                 ipc_answer_0(iid, EREFUSED);
                 return NULL;
@@ -109 +108 @@
         ipc_call_t call;
         ipc_callid_t callid = async_get_call(&call);
-        if (IPC_GET_IMETHOD(call) != IPC_M_CONNECT_TO_ME) {
+        if (IPC_GET_METHOD(call) != IPC_M_CONNECT_TO_ME) {
                 ipc_answer_0(callid, ENOTSUP);
                 ipc_answer_0(iid, ENOTSUP);
@@ -141 +140 @@
         
         callid = async_get_call(&call);
-        if (DEVMAN_ADD_MATCH_ID != IPC_GET_IMETHOD(call)) {
+        if (DEVMAN_ADD_MATCH_ID != IPC_GET_METHOD(call)) {
                 printf(NAME ": ERROR: devman_receive_match_id - invalid "
                     "protocol.\n");
@@ -184 +183 @@
  * @return              Zero on success, negative error code otherwise.
  */
-static int devman_receive_match_ids(sysarg_t match_count,
+static int devman_receive_match_ids(ipcarg_t match_count,
     match_id_list_t *match_ids)
 {
@@ -197 +196 @@
 }
 
-static int assign_driver_fibril(void *arg)
-{
-        node_t *node = (node_t *) arg;
-        assign_driver(node, &drivers_list, &device_tree);
-        return EOK;
-}
-
 /** Handle child device registration.
  *
@@ -211 +203 @@
 {
         devman_handle_t parent_handle = IPC_GET_ARG1(*call);
-        sysarg_t match_count = IPC_GET_ARG2(*call);
+        ipcarg_t match_count = IPC_GET_ARG2(*call);
         dev_tree_t *tree = &device_tree;
         
@@ -244 +236 @@
         
         devman_receive_match_ids(match_count, &node->match_ids);
-
-        /*
-         * Try to find a suitable driver and assign it to the device.  We do
-         * not want to block the current fibril that is used for processing
-         * incoming calls: we will launch a separate fibril to handle the
-         * driver assigning. That is because assign_driver can actually include
-         * task spawning which could take some time.
-         */
-        fid_t assign_fibril = fibril_create(assign_driver_fibril, node);
-        if (assign_fibril == 0) {
-                /*
-                 * Fallback in case we are out of memory.
-                 * Probably not needed as we will die soon anyway ;-).
-                 */
-                (void) assign_driver_fibril(node);
-        } else {
-                fibril_add_ready(assign_fibril);
-        }
-
+        
         /* Return device handle to parent's driver. */
         ipc_answer_1(callid, EOK, node->handle);
+        
+        /* Try to find suitable driver and assign it to the device. */
+        assign_driver(node, &drivers_list, &device_tree);
 }
 
@@ -319 +296 @@
         printf(NAME ": device '%s' added to class '%s', class name '%s' was "
             "asigned to it\n", dev->pathname, class_name, class_info->dev_name);
-
+        
         ipc_answer_0(callid, EOK);
 }
@@ -367 +344 @@
                 callid = async_get_call(&call);
                 
-                switch (IPC_GET_IMETHOD(call)) {
+                switch (IPC_GET_METHOD(call)) {
                 case IPC_M_PHONE_HUNGUP:
                         cont = false;
@@ -420 +397 @@
                 ipc_callid_t callid = async_get_call(&call);
                 
-                switch (IPC_GET_IMETHOD(call)) {
+                switch (IPC_GET_METHOD(call)) {
                 case IPC_M_PHONE_HUNGUP:
                         cont = false;
@@ -428 +405 @@
                         break;
                 default:
-                        ipc_answer_0(callid, ENOENT);
+                        if (!(callid & IPC_CALLID_NOTIFICATION))
+                                ipc_answer_0(callid, ENOENT);
                 }
         }
@@ -440 +418 @@
         node_t *dev = find_dev_node(&device_tree, handle);
         if (dev == NULL) {
-                printf(NAME ": devman_forward error - no device with handle %" PRIun
-                    " was found.\n", handle);
+                printf(NAME ": devman_forward error - no device with handle %x "
+                    "was found.\n", handle);
                 ipc_answer_0(iid, ENOENT);
                 return;
@@ -457 +435 @@
         
         if (driver == NULL) {
-                printf(NAME ": devman_forward error - the device is not in %" PRIun
-                    " usable state.\n", handle);
+                printf(NAME ": devman_forward error - the device is not in "
+                    "usable state.\n", handle);
                 ipc_answer_0(iid, ENOENT);
                 return;
@@ -472 +450 @@
                 printf(NAME ": devman_forward: cound not forward to driver %s ",
                     driver->name);
-                printf("the driver's phone is %" PRIun ").\n", driver->phone);
+                printf("the driver's phone is %x).\n", driver->phone);
                 ipc_answer_0(iid, EINVAL);
                 return;
@@ -486 +464 @@
 static void devman_connection_devmapper(ipc_callid_t iid, ipc_call_t *icall)
 {
-        devmap_handle_t devmap_handle = IPC_GET_IMETHOD(*icall);
+        devmap_handle_t devmap_handle = IPC_GET_METHOD(*icall);
         node_t *dev;
 
@@ -521 +499 @@
          * passes device handle to the driver as an ipc method.)
          */
-        if (IPC_GET_IMETHOD(*icall) != IPC_M_CONNECT_ME_TO)
+        if (IPC_GET_METHOD(*icall) != IPC_M_CONNECT_ME_TO)
                 devman_connection_devmapper(iid, icall);
 
@@ -531 +509 @@
         
         /* Select interface. */
-        switch ((sysarg_t) (IPC_GET_ARG1(*icall))) {
+        switch ((ipcarg_t) (IPC_GET_ARG1(*icall))) {
         case DEVMAN_DRIVER:
                 devman_connection_driver(iid, icall);
@@ -599 +577 @@
 
         /* Register device manager at naming service. */
-        sysarg_t phonead;
+        ipcarg_t phonead;
         if (ipc_connect_to_me(PHONE_NS, SERVICE_DEVMAN, 0, 0, &phonead) != 0)
                 return -1;