Ignore:
File:
1 edited

Legend:

Unmodified
Added
Removed

  • uspace/srv/devman/main.c

    r3f57fb7 ra1a101d  
    7373        char *drv_name = NULL;
    7474
    75         log_msg(LVL_DEBUG, "devman_driver_register");
     75        log_msg(LOG_DEFAULT, LVL_DEBUG, "devman_driver_register");
    7676       
    7777        /* Get driver name. */
    … …  
    8282        }
    8383
    84         log_msg(LVL_DEBUG, "The `%s' driver is trying to register.",
     84        log_msg(LOG_DEFAULT, LVL_DEBUG, "The `%s' driver is trying to register.",
    8585            drv_name);
    8686       
    … …  
    8888        driver = find_driver(&drivers_list, drv_name);
    8989        if (driver == NULL) {
    90                 log_msg(LVL_ERROR, "No driver named `%s' was found.", drv_name);
     90                log_msg(LOG_DEFAULT, LVL_ERROR, "No driver named `%s' was found.", drv_name);
    9191                free(drv_name);
    9292                drv_name = NULL;
    … …  
    102102        if (driver->sess) {
    103103                /* We already have a connection to the driver. */
    104                 log_msg(LVL_ERROR, "Driver '%s' already started.\n",
     104                log_msg(LOG_DEFAULT, LVL_ERROR, "Driver '%s' already started.\n",
    105105                    driver->name);
    106106                fibril_mutex_unlock(&driver->driver_mutex);
    … …  
    112112        case DRIVER_NOT_STARTED:
    113113                /* Somebody started the driver manually. */
    114                 log_msg(LVL_NOTE, "Driver '%s' started manually.\n",
     114                log_msg(LOG_DEFAULT, LVL_NOTE, "Driver '%s' started manually.\n",
    115115                    driver->name);
    116116                driver->state = DRIVER_STARTING;
    … …  
    125125       
    126126        /* Create connection to the driver. */
    127         log_msg(LVL_DEBUG, "Creating connection to the `%s' driver.",
     127        log_msg(LOG_DEFAULT, LVL_DEBUG, "Creating connection to the `%s' driver.",
    128128            driver->name);
    129129        driver->sess = async_callback_receive(EXCHANGE_PARALLEL);
    … …  
    136136        async_sess_args_set(driver->sess, DRIVER_DEVMAN, 0, 0);
    137137       
    138         log_msg(LVL_NOTE,
     138        log_msg(LOG_DEFAULT, LVL_NOTE,
    139139            "The `%s' driver was successfully registered as running.",
    140140            driver->name);
    … …  
    147147        fid_t fid = fibril_create(init_running_drv, driver);
    148148        if (fid == 0) {
    149                 log_msg(LVL_ERROR, "Failed to create initialization fibril " \
     149                log_msg(LOG_DEFAULT, LVL_ERROR, "Failed to create initialization fibril " \
    150150                    "for driver `%s'.", driver->name);
    151151                fibril_mutex_unlock(&driver->driver_mutex);
    … …  
    176176        callid = async_get_call(&call);
    177177        if (DEVMAN_ADD_MATCH_ID != IPC_GET_IMETHOD(call)) {
    178                 log_msg(LVL_ERROR,
     178                log_msg(LOG_DEFAULT, LVL_ERROR,
    179179                    "Invalid protocol when trying to receive match id.");
    180180                async_answer_0(callid, EINVAL);
    … …  
    184184       
    185185        if (match_id == NULL) {
    186                 log_msg(LVL_ERROR, "Failed to allocate match id.");
     186                log_msg(LOG_DEFAULT, LVL_ERROR, "Failed to allocate match id.");
    187187                async_answer_0(callid, ENOMEM);
    188188                return ENOMEM;
    … …  
    198198        if (rc != EOK) {
    199199                delete_match_id(match_id);
    200                 log_msg(LVL_ERROR, "Failed to receive match id string: %s.",
     200                log_msg(LOG_DEFAULT, LVL_ERROR, "Failed to receive match id string: %s.",
    201201                    str_error(rc));
    202202                return rc;
    … …  
    205205        list_append(&match_id->link, &match_ids->ids);
    206206       
    207         log_msg(LVL_DEBUG, "Received match id `%s', score %d.",
     207        log_msg(LOG_DEFAULT, LVL_DEBUG, "Received match id `%s', score %d.",
    208208            match_id->id, match_id->score);
    209209        return rc;
    … …  
    248248        if (fun->state == FUN_ON_LINE) {
    249249                fibril_rwlock_write_unlock(&device_tree.rwlock);
    250                 log_msg(LVL_WARN, "Function %s is already on line.",
     250                log_msg(LOG_DEFAULT, LVL_WARN, "Function %s is already on line.",
    251251                    fun->pathname);
    252252                return EOK;
    … …  
    264264        }
    265265       
    266         log_msg(LVL_DEBUG, "devman_add_function(fun=\"%s\")", fun->pathname);
     266        log_msg(LOG_DEFAULT, LVL_DEBUG, "devman_add_function(fun=\"%s\")", fun->pathname);
    267267       
    268268        if (fun->ftype == fun_inner) {
    … …  
    282282                fid_t assign_fibril = fibril_create(assign_driver_fibril, dev);
    283283                if (assign_fibril == 0) {
    284                         log_msg(LVL_ERROR, "Failed to create fibril for "
     284                        log_msg(LOG_DEFAULT, LVL_ERROR, "Failed to create fibril for "
    285285                            "assigning driver.");
    286286                        /* XXX Cleanup */
    … …  
    305305        if (fun->state == FUN_OFF_LINE) {
    306306                fibril_rwlock_write_unlock(&device_tree.rwlock);
    307                 log_msg(LVL_WARN, "Function %s is already off line.",
     307                log_msg(LOG_DEFAULT, LVL_WARN, "Function %s is already off line.",
    308308                    fun->pathname);
    309309                return EOK;
    … …  
    311311       
    312312        if (fun->ftype == fun_inner) {
    313                 log_msg(LVL_DEBUG, "Offlining inner function %s.",
     313                log_msg(LOG_DEFAULT, LVL_DEBUG, "Offlining inner function %s.",
    314314                    fun->pathname);
    315315               
    … …  
    359359                if (rc != EOK) {
    360360                        fibril_rwlock_write_unlock(&device_tree.rwlock);
    361                         log_msg(LVL_ERROR, "Failed unregistering tree service.");
     361                        log_msg(LOG_DEFAULT, LVL_ERROR, "Failed unregistering tree service.");
    362362                        return EIO;
    363363                }
    … …  
    391391        if (ftype != fun_inner && ftype != fun_exposed) {
    392392                /* Unknown function type */
    393                 log_msg(LVL_ERROR,
     393                log_msg(LOG_DEFAULT, LVL_ERROR,
    394394                    "Unknown function type %d provided by driver.",
    395395                    (int) ftype);
    … …  
    419419       
    420420        /* Check that function with same name is not there already. */
    421         if (find_fun_node_in_device(tree, pdev, fun_name) != NULL) {
     421        fun_node_t *tfun = find_fun_node_in_device(tree, pdev, fun_name);
     422        if (tfun) {
     423                fun_del_ref(tfun);      /* drop the new unwanted reference */
    422424                fibril_rwlock_write_unlock(&tree->rwlock);
    423425                dev_del_ref(pdev);
    … …  
    430432       
    431433        fun_node_t *fun = create_fun_node();
     434        /* One reference for creation, one for us */
     435        fun_add_ref(fun);
    432436        fun_add_ref(fun);
    433437        fun->ftype = ftype;
     438       
     439        /*
     440         * We can lock the function here even when holding the tree because
     441         * we know it cannot be held by anyone else yet.
     442         */
     443        fun_busy_lock(fun);
    434444       
    435445        if (!insert_fun_node(&device_tree, fun, fun_name, pdev)) {
    436446                fibril_rwlock_write_unlock(&tree->rwlock);
    437447                dev_del_ref(pdev);
     448                fun_busy_unlock(fun);
     449                fun_del_ref(fun);
    438450                delete_fun_node(fun);
    439451                async_answer_0(callid, ENOMEM);
    … …  
    448460        rc = online_function(fun);
    449461        if (rc != EOK) {
    450                 /* XXX clean up */
     462                /* XXX Set some failed state? */
     463                fun_busy_unlock(fun);
     464                fun_del_ref(fun);
    451465                async_answer_0(callid, rc);
    452466                return;
    453467        }
     468       
     469        fun_busy_unlock(fun);
     470        fun_del_ref(fun);
    454471       
    455472        /* Return device handle to parent's driver. */
    … …  
    490507        if (rc == EOK) {
    491508                loc_service_add_to_cat(fun->service_id, cat_id);
    492                 log_msg(LVL_NOTE, "Function `%s' added to category `%s'.",
     509                log_msg(LOG_DEFAULT, LVL_NOTE, "Function `%s' added to category `%s'.",
    493510                    fun->pathname, cat_name);
    494511        } else {
    495                 log_msg(LVL_ERROR, "Failed adding function `%s' to category "
     512                log_msg(LOG_DEFAULT, LVL_ERROR, "Failed adding function `%s' to category "
    496513                    "`%s'.", fun->pathname, cat_name);
    497514        }
    … …  
    512529        int rc;
    513530       
    514         log_msg(LVL_DEBUG, "devman_drv_fun_online()");
     531        log_msg(LOG_DEFAULT, LVL_DEBUG, "devman_drv_fun_online()");
    515532       
    516533        fun = find_fun_node(&device_tree, IPC_GET_ARG1(*icall));
    … …  
    519536                return;
    520537        }
     538       
     539        fun_busy_lock(fun);
    521540       
    522541        fibril_rwlock_read_lock(&device_tree.rwlock);
    523542        if (fun->dev == NULL || fun->dev->drv != drv) {
    524543                fibril_rwlock_read_unlock(&device_tree.rwlock);
     544                fun_busy_unlock(fun);
    525545                fun_del_ref(fun);
    526546                async_answer_0(iid, ENOENT);
    … …  
    531551        rc = online_function(fun);
    532552        if (rc != EOK) {
     553                fun_busy_unlock(fun);
    533554                fun_del_ref(fun);
    534555                async_answer_0(iid, (sysarg_t) rc);
    … …  
    536557        }
    537558       
     559        fun_busy_unlock(fun);
    538560        fun_del_ref(fun);
    539561       
    … …  
    557579        }
    558580       
     581        fun_busy_lock(fun);
     582       
    559583        fibril_rwlock_write_lock(&device_tree.rwlock);
    560584        if (fun->dev == NULL || fun->dev->drv != drv) {
     585                fun_busy_unlock(fun);
    561586                fun_del_ref(fun);
    562587                async_answer_0(iid, ENOENT);
    … …  
    567592        rc = offline_function(fun);
    568593        if (rc != EOK) {
     594                fun_busy_unlock(fun);
    569595                fun_del_ref(fun);
    570596                async_answer_0(iid, (sysarg_t) rc);
    … …  
    572598        }
    573599       
     600        fun_busy_unlock(fun);
    574601        fun_del_ref(fun);
    575602        async_answer_0(iid, (sysarg_t) EOK);
    … …  
    589616        }
    590617       
     618        fun_busy_lock(fun);
     619       
    591620        fibril_rwlock_write_lock(&tree->rwlock);
    592621       
    593         log_msg(LVL_DEBUG, "devman_remove_function(fun='%s')", fun->pathname);
     622        log_msg(LOG_DEFAULT, LVL_DEBUG, "devman_remove_function(fun='%s')", fun->pathname);
    594623       
    595624        /* Check function state */
    596625        if (fun->state == FUN_REMOVED) {
    597626                fibril_rwlock_write_unlock(&tree->rwlock);
     627                fun_busy_unlock(fun);
     628                fun_del_ref(fun);
    598629                async_answer_0(callid, ENOENT);
    599630                return;
    … …  
    622653                        /* Verify that driver succeeded and removed all functions */
    623654                        if (gone_rc != EOK || !list_empty(&dev->functions)) {
    624                                 log_msg(LVL_ERROR, "Driver did not remove "
     655                                log_msg(LOG_DEFAULT, LVL_ERROR, "Driver did not remove "
    625656                                    "functions for device that is gone. "
    626657                                    "Device node is now defunct.");
    … …  
    636667                                if (gone_rc == EOK)
    637668                                        gone_rc = ENOTSUP;
     669                                fun_busy_unlock(fun);
     670                                fun_del_ref(fun);
    638671                                async_answer_0(callid, gone_rc);
    639672                                return;
    … …  
    659692                        rc = loc_service_unregister(fun->service_id);
    660693                        if (rc != EOK) {
    661                                 log_msg(LVL_ERROR, "Failed unregistering tree "
     694                                log_msg(LOG_DEFAULT, LVL_ERROR, "Failed unregistering tree "
    662695                                    "service.");
    663696                                fibril_rwlock_write_unlock(&tree->rwlock);
     697                                fun_busy_unlock(fun);
    664698                                fun_del_ref(fun);
    665699                                async_answer_0(callid, EIO);
    … …  
    671705        remove_fun_node(&device_tree, fun);
    672706        fibril_rwlock_write_unlock(&tree->rwlock);
     707        fun_busy_unlock(fun);
    673708       
    674709        /* Delete ref added when inserting function into tree */
    … …  
    677712        fun_del_ref(fun);
    678713       
    679         log_msg(LVL_DEBUG, "devman_remove_function() succeeded.");
     714        log_msg(LOG_DEFAULT, LVL_DEBUG, "devman_remove_function() succeeded.");
    680715        async_answer_0(callid, EOK);
    681716}
    … …  
    691726       
    692727        initialize_running_driver(driver, &device_tree);
    693         log_msg(LVL_DEBUG, "The `%s` driver was successfully initialized.",
     728        log_msg(LOG_DEFAULT, LVL_DEBUG, "The `%s` driver was successfully initialized.",
    694729            driver->name);
    695730        return 0;
    … …  
    707742        client = async_get_client_data();
    708743        if (client == NULL) {
    709                 log_msg(LVL_ERROR, "Failed to allocate client data.");
     744                log_msg(LOG_DEFAULT, LVL_ERROR, "Failed to allocate client data.");
    710745                return;
    711746        }
    … …  
    11941229         */
    11951230        if (dev == NULL) {
    1196                 log_msg(LVL_ERROR, "IPC forwarding failed - no device or "
     1231                log_msg(LOG_DEFAULT, LVL_ERROR, "IPC forwarding failed - no device or "
    11971232                    "function with handle %" PRIun " was found.", handle);
    11981233                async_answer_0(iid, ENOENT);
    … …  
    12011236
    12021237        if (fun == NULL && !drv_to_parent) {
    1203                 log_msg(LVL_ERROR, NAME ": devman_forward error - cannot "
     1238                log_msg(LOG_DEFAULT, LVL_ERROR, NAME ": devman_forward error - cannot "
    12041239                    "connect to handle %" PRIun ", refers to a device.",
    12051240                    handle);
    … …  
    12291264       
    12301265        if (driver == NULL) {
    1231                 log_msg(LVL_ERROR, "IPC forwarding refused - " \
     1266                log_msg(LOG_DEFAULT, LVL_ERROR, "IPC forwarding refused - " \
    12321267                    "the device %" PRIun " is not in usable state.", handle);
    12331268                async_answer_0(iid, ENOENT);
    … …  
    12421277       
    12431278        if (!driver->sess) {
    1244                 log_msg(LVL_ERROR,
     1279                log_msg(LOG_DEFAULT, LVL_ERROR,
    12451280                    "Could not forward to driver `%s'.", driver->name);
    12461281                async_answer_0(iid, EINVAL);
    … …  
    12491284
    12501285        if (fun != NULL) {
    1251                 log_msg(LVL_DEBUG,
     1286                log_msg(LOG_DEFAULT, LVL_DEBUG,
    12521287                    "Forwarding request for `%s' function to driver `%s'.",
    12531288                    fun->pathname, driver->name);
    12541289        } else {
    1255                 log_msg(LVL_DEBUG,
     1290                log_msg(LOG_DEFAULT, LVL_DEBUG,
    12561291                    "Forwarding request for `%s' device to driver `%s'.",
    12571292                    dev->pfun->pathname, driver->name);
    … …  
    12851320       
    12861321        if (fun == NULL || fun->dev == NULL || fun->dev->drv == NULL) {
    1287                 log_msg(LVL_WARN, "devman_connection_loc(): function "
     1322                log_msg(LOG_DEFAULT, LVL_WARN, "devman_connection_loc(): function "
    12881323                    "not found.\n");
    12891324                fibril_rwlock_read_unlock(&device_tree.rwlock);
    … …  
    13031338        async_exchange_end(exch);
    13041339       
    1305         log_msg(LVL_DEBUG,
     1340        log_msg(LOG_DEFAULT, LVL_DEBUG,
    13061341            "Forwarding loc service request for `%s' function to driver `%s'.",
    13071342            fun->pathname, driver->name);
    … …  
    13591394static bool devman_init(void)
    13601395{
    1361         log_msg(LVL_DEBUG, "devman_init - looking for available drivers.");
     1396        log_msg(LOG_DEFAULT, LVL_DEBUG, "devman_init - looking for available drivers.");
    13621397       
    13631398        /* Initialize list of available drivers. */
    … …  
    13651400        if (lookup_available_drivers(&drivers_list,
    13661401            DRIVER_DEFAULT_STORE) == 0) {
    1367                 log_msg(LVL_FATAL, "No drivers found.");
     1402                log_msg(LOG_DEFAULT, LVL_FATAL, "No drivers found.");
    13681403                return false;
    13691404        }
    13701405       
    1371         log_msg(LVL_DEBUG, "devman_init - list of drivers has been initialized.");
     1406        log_msg(LOG_DEFAULT, LVL_DEBUG, "devman_init - list of drivers has been initialized.");
    13721407       
    13731408        /* Create root device node. */
    13741409        if (!init_device_tree(&device_tree, &drivers_list)) {
    1375                 log_msg(LVL_FATAL, "Failed to initialize device tree.");
     1410                log_msg(LOG_DEFAULT, LVL_FATAL, "Failed to initialize device tree.");
    13761411                return false;
    13771412        }
    … …  
    13931428        printf("%s: HelenOS Device Manager\n", NAME);
    13941429       
    1395         int rc = log_init(NAME, LVL_WARN);
     1430        int rc = log_init(NAME);
    13961431        if (rc != EOK) {
    13971432                printf("%s: Error initializing logging subsystem.\n", NAME);
    … …  
    14051440       
    14061441        if (!devman_init()) {
    1407                 log_msg(LVL_ERROR, "Error while initializing service.");
     1442                log_msg(LOG_DEFAULT, LVL_ERROR, "Error while initializing service.");
    14081443                return -1;
    14091444        }
    … …  
    14121447        rc = service_register(SERVICE_DEVMAN);
    14131448        if (rc != EOK) {
    1414                 log_msg(LVL_ERROR, "Failed registering as a service.");
     1449                log_msg(LOG_DEFAULT, LVL_ERROR, "Failed registering as a service.");
    14151450                return rc;
    14161451        }
Note: See TracChangeset for help on using the changeset viewer.