Ignore:
File:
1 edited

Legend:

Unmodified
Added
Removed

  • uspace/srv/devman/main.c

    ra1a101d r3f57fb7  
    7373        char *drv_name = NULL;
    7474
    75         log_msg(LOG_DEFAULT, LVL_DEBUG, "devman_driver_register");
     75        log_msg(LVL_DEBUG, "devman_driver_register");
    7676       
    7777        /* Get driver name. */
    … …  
    8282        }
    8383
    84         log_msg(LOG_DEFAULT, LVL_DEBUG, "The `%s' driver is trying to register.",
     84        log_msg(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(LOG_DEFAULT, LVL_ERROR, "No driver named `%s' was found.", drv_name);
     90                log_msg(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(LOG_DEFAULT, LVL_ERROR, "Driver '%s' already started.\n",
     104                log_msg(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(LOG_DEFAULT, LVL_NOTE, "Driver '%s' started manually.\n",
     114                log_msg(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(LOG_DEFAULT, LVL_DEBUG, "Creating connection to the `%s' driver.",
     127        log_msg(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(LOG_DEFAULT, LVL_NOTE,
     138        log_msg(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(LOG_DEFAULT, LVL_ERROR, "Failed to create initialization fibril " \
     149                log_msg(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(LOG_DEFAULT, LVL_ERROR,
     178                log_msg(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(LOG_DEFAULT, LVL_ERROR, "Failed to allocate match id.");
     186                log_msg(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(LOG_DEFAULT, LVL_ERROR, "Failed to receive match id string: %s.",
     200                log_msg(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(LOG_DEFAULT, LVL_DEBUG, "Received match id `%s', score %d.",
     207        log_msg(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(LOG_DEFAULT, LVL_WARN, "Function %s is already on line.",
     250                log_msg(LVL_WARN, "Function %s is already on line.",
    251251                    fun->pathname);
    252252                return EOK;
    … …  
    264264        }
    265265       
    266         log_msg(LOG_DEFAULT, LVL_DEBUG, "devman_add_function(fun=\"%s\")", fun->pathname);
     266        log_msg(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(LOG_DEFAULT, LVL_ERROR, "Failed to create fibril for "
     284                        log_msg(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(LOG_DEFAULT, LVL_WARN, "Function %s is already off line.",
     307                log_msg(LVL_WARN, "Function %s is already off line.",
    308308                    fun->pathname);
    309309                return EOK;
    … …  
    311311       
    312312        if (fun->ftype == fun_inner) {
    313                 log_msg(LOG_DEFAULT, LVL_DEBUG, "Offlining inner function %s.",
     313                log_msg(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(LOG_DEFAULT, LVL_ERROR, "Failed unregistering tree service.");
     361                        log_msg(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(LOG_DEFAULT, LVL_ERROR,
     393                log_msg(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         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 */
     421        if (find_fun_node_in_device(tree, pdev, fun_name) != NULL) {
    424422                fibril_rwlock_write_unlock(&tree->rwlock);
    425423                dev_del_ref(pdev);
    … …  
    432430       
    433431        fun_node_t *fun = create_fun_node();
    434         /* One reference for creation, one for us */
    435         fun_add_ref(fun);
    436432        fun_add_ref(fun);
    437433        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);
    444434       
    445435        if (!insert_fun_node(&device_tree, fun, fun_name, pdev)) {
    446436                fibril_rwlock_write_unlock(&tree->rwlock);
    447437                dev_del_ref(pdev);
    448                 fun_busy_unlock(fun);
    449                 fun_del_ref(fun);
    450438                delete_fun_node(fun);
    451439                async_answer_0(callid, ENOMEM);
    … …  
    460448        rc = online_function(fun);
    461449        if (rc != EOK) {
    462                 /* XXX Set some failed state? */
    463                 fun_busy_unlock(fun);
    464                 fun_del_ref(fun);
     450                /* XXX clean up */
    465451                async_answer_0(callid, rc);
    466452                return;
    467453        }
    468        
    469         fun_busy_unlock(fun);
    470         fun_del_ref(fun);
    471454       
    472455        /* Return device handle to parent's driver. */
    … …  
    507490        if (rc == EOK) {
    508491                loc_service_add_to_cat(fun->service_id, cat_id);
    509                 log_msg(LOG_DEFAULT, LVL_NOTE, "Function `%s' added to category `%s'.",
     492                log_msg(LVL_NOTE, "Function `%s' added to category `%s'.",
    510493                    fun->pathname, cat_name);
    511494        } else {
    512                 log_msg(LOG_DEFAULT, LVL_ERROR, "Failed adding function `%s' to category "
     495                log_msg(LVL_ERROR, "Failed adding function `%s' to category "
    513496                    "`%s'.", fun->pathname, cat_name);
    514497        }
    … …  
    529512        int rc;
    530513       
    531         log_msg(LOG_DEFAULT, LVL_DEBUG, "devman_drv_fun_online()");
     514        log_msg(LVL_DEBUG, "devman_drv_fun_online()");
    532515       
    533516        fun = find_fun_node(&device_tree, IPC_GET_ARG1(*icall));
    … …  
    536519                return;
    537520        }
    538        
    539         fun_busy_lock(fun);
    540521       
    541522        fibril_rwlock_read_lock(&device_tree.rwlock);
    542523        if (fun->dev == NULL || fun->dev->drv != drv) {
    543524                fibril_rwlock_read_unlock(&device_tree.rwlock);
    544                 fun_busy_unlock(fun);
    545525                fun_del_ref(fun);
    546526                async_answer_0(iid, ENOENT);
    … …  
    551531        rc = online_function(fun);
    552532        if (rc != EOK) {
    553                 fun_busy_unlock(fun);
    554533                fun_del_ref(fun);
    555534                async_answer_0(iid, (sysarg_t) rc);
    … …  
    557536        }
    558537       
    559         fun_busy_unlock(fun);
    560538        fun_del_ref(fun);
    561539       
    … …  
    579557        }
    580558       
    581         fun_busy_lock(fun);
    582        
    583559        fibril_rwlock_write_lock(&device_tree.rwlock);
    584560        if (fun->dev == NULL || fun->dev->drv != drv) {
    585                 fun_busy_unlock(fun);
    586561                fun_del_ref(fun);
    587562                async_answer_0(iid, ENOENT);
    … …  
    592567        rc = offline_function(fun);
    593568        if (rc != EOK) {
    594                 fun_busy_unlock(fun);
    595569                fun_del_ref(fun);
    596570                async_answer_0(iid, (sysarg_t) rc);
    … …  
    598572        }
    599573       
    600         fun_busy_unlock(fun);
    601574        fun_del_ref(fun);
    602575        async_answer_0(iid, (sysarg_t) EOK);
    … …  
    616589        }
    617590       
    618         fun_busy_lock(fun);
    619        
    620591        fibril_rwlock_write_lock(&tree->rwlock);
    621592       
    622         log_msg(LOG_DEFAULT, LVL_DEBUG, "devman_remove_function(fun='%s')", fun->pathname);
     593        log_msg(LVL_DEBUG, "devman_remove_function(fun='%s')", fun->pathname);
    623594       
    624595        /* Check function state */
    625596        if (fun->state == FUN_REMOVED) {
    626597                fibril_rwlock_write_unlock(&tree->rwlock);
    627                 fun_busy_unlock(fun);
    628                 fun_del_ref(fun);
    629598                async_answer_0(callid, ENOENT);
    630599                return;
    … …  
    653622                        /* Verify that driver succeeded and removed all functions */
    654623                        if (gone_rc != EOK || !list_empty(&dev->functions)) {
    655                                 log_msg(LOG_DEFAULT, LVL_ERROR, "Driver did not remove "
     624                                log_msg(LVL_ERROR, "Driver did not remove "
    656625                                    "functions for device that is gone. "
    657626                                    "Device node is now defunct.");
    … …  
    667636                                if (gone_rc == EOK)
    668637                                        gone_rc = ENOTSUP;
    669                                 fun_busy_unlock(fun);
    670                                 fun_del_ref(fun);
    671638                                async_answer_0(callid, gone_rc);
    672639                                return;
    … …  
    692659                        rc = loc_service_unregister(fun->service_id);
    693660                        if (rc != EOK) {
    694                                 log_msg(LOG_DEFAULT, LVL_ERROR, "Failed unregistering tree "
     661                                log_msg(LVL_ERROR, "Failed unregistering tree "
    695662                                    "service.");
    696663                                fibril_rwlock_write_unlock(&tree->rwlock);
    697                                 fun_busy_unlock(fun);
    698664                                fun_del_ref(fun);
    699665                                async_answer_0(callid, EIO);
    … …  
    705671        remove_fun_node(&device_tree, fun);
    706672        fibril_rwlock_write_unlock(&tree->rwlock);
    707         fun_busy_unlock(fun);
    708673       
    709674        /* Delete ref added when inserting function into tree */
    … …  
    712677        fun_del_ref(fun);
    713678       
    714         log_msg(LOG_DEFAULT, LVL_DEBUG, "devman_remove_function() succeeded.");
     679        log_msg(LVL_DEBUG, "devman_remove_function() succeeded.");
    715680        async_answer_0(callid, EOK);
    716681}
    … …  
    726691       
    727692        initialize_running_driver(driver, &device_tree);
    728         log_msg(LOG_DEFAULT, LVL_DEBUG, "The `%s` driver was successfully initialized.",
     693        log_msg(LVL_DEBUG, "The `%s` driver was successfully initialized.",
    729694            driver->name);
    730695        return 0;
    … …  
    742707        client = async_get_client_data();
    743708        if (client == NULL) {
    744                 log_msg(LOG_DEFAULT, LVL_ERROR, "Failed to allocate client data.");
     709                log_msg(LVL_ERROR, "Failed to allocate client data.");
    745710                return;
    746711        }
    … …  
    12291194         */
    12301195        if (dev == NULL) {
    1231                 log_msg(LOG_DEFAULT, LVL_ERROR, "IPC forwarding failed - no device or "
     1196                log_msg(LVL_ERROR, "IPC forwarding failed - no device or "
    12321197                    "function with handle %" PRIun " was found.", handle);
    12331198                async_answer_0(iid, ENOENT);
    … …  
    12361201
    12371202        if (fun == NULL && !drv_to_parent) {
    1238                 log_msg(LOG_DEFAULT, LVL_ERROR, NAME ": devman_forward error - cannot "
     1203                log_msg(LVL_ERROR, NAME ": devman_forward error - cannot "
    12391204                    "connect to handle %" PRIun ", refers to a device.",
    12401205                    handle);
    … …  
    12641229       
    12651230        if (driver == NULL) {
    1266                 log_msg(LOG_DEFAULT, LVL_ERROR, "IPC forwarding refused - " \
     1231                log_msg(LVL_ERROR, "IPC forwarding refused - " \
    12671232                    "the device %" PRIun " is not in usable state.", handle);
    12681233                async_answer_0(iid, ENOENT);
    … …  
    12771242       
    12781243        if (!driver->sess) {
    1279                 log_msg(LOG_DEFAULT, LVL_ERROR,
     1244                log_msg(LVL_ERROR,
    12801245                    "Could not forward to driver `%s'.", driver->name);
    12811246                async_answer_0(iid, EINVAL);
    … …  
    12841249
    12851250        if (fun != NULL) {
    1286                 log_msg(LOG_DEFAULT, LVL_DEBUG,
     1251                log_msg(LVL_DEBUG,
    12871252                    "Forwarding request for `%s' function to driver `%s'.",
    12881253                    fun->pathname, driver->name);
    12891254        } else {
    1290                 log_msg(LOG_DEFAULT, LVL_DEBUG,
     1255                log_msg(LVL_DEBUG,
    12911256                    "Forwarding request for `%s' device to driver `%s'.",
    12921257                    dev->pfun->pathname, driver->name);
    … …  
    13201285       
    13211286        if (fun == NULL || fun->dev == NULL || fun->dev->drv == NULL) {
    1322                 log_msg(LOG_DEFAULT, LVL_WARN, "devman_connection_loc(): function "
     1287                log_msg(LVL_WARN, "devman_connection_loc(): function "
    13231288                    "not found.\n");
    13241289                fibril_rwlock_read_unlock(&device_tree.rwlock);
    … …  
    13381303        async_exchange_end(exch);
    13391304       
    1340         log_msg(LOG_DEFAULT, LVL_DEBUG,
     1305        log_msg(LVL_DEBUG,
    13411306            "Forwarding loc service request for `%s' function to driver `%s'.",
    13421307            fun->pathname, driver->name);
    … …  
    13941359static bool devman_init(void)
    13951360{
    1396         log_msg(LOG_DEFAULT, LVL_DEBUG, "devman_init - looking for available drivers.");
     1361        log_msg(LVL_DEBUG, "devman_init - looking for available drivers.");
    13971362       
    13981363        /* Initialize list of available drivers. */
    … …  
    14001365        if (lookup_available_drivers(&drivers_list,
    14011366            DRIVER_DEFAULT_STORE) == 0) {
    1402                 log_msg(LOG_DEFAULT, LVL_FATAL, "No drivers found.");
     1367                log_msg(LVL_FATAL, "No drivers found.");
    14031368                return false;
    14041369        }
    14051370       
    1406         log_msg(LOG_DEFAULT, LVL_DEBUG, "devman_init - list of drivers has been initialized.");
     1371        log_msg(LVL_DEBUG, "devman_init - list of drivers has been initialized.");
    14071372       
    14081373        /* Create root device node. */
    14091374        if (!init_device_tree(&device_tree, &drivers_list)) {
    1410                 log_msg(LOG_DEFAULT, LVL_FATAL, "Failed to initialize device tree.");
     1375                log_msg(LVL_FATAL, "Failed to initialize device tree.");
    14111376                return false;
    14121377        }
    … …  
    14281393        printf("%s: HelenOS Device Manager\n", NAME);
    14291394       
    1430         int rc = log_init(NAME);
     1395        int rc = log_init(NAME, LVL_WARN);
    14311396        if (rc != EOK) {
    14321397                printf("%s: Error initializing logging subsystem.\n", NAME);
    … …  
    14401405       
    14411406        if (!devman_init()) {
    1442                 log_msg(LOG_DEFAULT, LVL_ERROR, "Error while initializing service.");
     1407                log_msg(LVL_ERROR, "Error while initializing service.");
    14431408                return -1;
    14441409        }
    … …  
    14471412        rc = service_register(SERVICE_DEVMAN);
    14481413        if (rc != EOK) {
    1449                 log_msg(LOG_DEFAULT, LVL_ERROR, "Failed registering as a service.");
     1414                log_msg(LVL_ERROR, "Failed registering as a service.");
    14501415                return rc;
    14511416        }
Note: See TracChangeset for help on using the changeset viewer.