Ignore:
File:
1 edited

Legend:

Unmodified
Added
Removed

  • uspace/srv/devman/main.c

    r3e6a98c5 r33fc3ae  
    4444#include <errno.h>
    4545#include <str_error.h>
    46 #include <stdbool.h>
     46#include <bool.h>
    4747#include <fibril_synch.h>
    4848#include <stdlib.h>
    … …  
    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);
    … …  
    432432       
    433433        fun_node_t *fun = create_fun_node();
    434         /* One reference for creation, one for us */
    435         fun_add_ref(fun);
    436434        fun_add_ref(fun);
    437435        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);
    444436       
    445437        if (!insert_fun_node(&device_tree, fun, fun_name, pdev)) {
    446438                fibril_rwlock_write_unlock(&tree->rwlock);
    447439                dev_del_ref(pdev);
    448                 fun_busy_unlock(fun);
    449                 fun_del_ref(fun);
    450440                delete_fun_node(fun);
    451441                async_answer_0(callid, ENOMEM);
    … …  
    460450        rc = online_function(fun);
    461451        if (rc != EOK) {
    462                 /* XXX Set some failed state? */
    463                 fun_busy_unlock(fun);
    464                 fun_del_ref(fun);
     452                /* XXX clean up */
    465453                async_answer_0(callid, rc);
    466454                return;
    467455        }
    468        
    469         fun_busy_unlock(fun);
    470         fun_del_ref(fun);
    471456       
    472457        /* Return device handle to parent's driver. */
    … …  
    507492        if (rc == EOK) {
    508493                loc_service_add_to_cat(fun->service_id, cat_id);
    509                 log_msg(LOG_DEFAULT, LVL_NOTE, "Function `%s' added to category `%s'.",
     494                log_msg(LVL_NOTE, "Function `%s' added to category `%s'.",
    510495                    fun->pathname, cat_name);
    511496        } else {
    512                 log_msg(LOG_DEFAULT, LVL_ERROR, "Failed adding function `%s' to category "
     497                log_msg(LVL_ERROR, "Failed adding function `%s' to category "
    513498                    "`%s'.", fun->pathname, cat_name);
    514499        }
    … …  
    529514        int rc;
    530515       
    531         log_msg(LOG_DEFAULT, LVL_DEBUG, "devman_drv_fun_online()");
     516        log_msg(LVL_DEBUG, "devman_drv_fun_online()");
    532517       
    533518        fun = find_fun_node(&device_tree, IPC_GET_ARG1(*icall));
    … …  
    536521                return;
    537522        }
    538        
    539         fun_busy_lock(fun);
    540523       
    541524        fibril_rwlock_read_lock(&device_tree.rwlock);
    542525        if (fun->dev == NULL || fun->dev->drv != drv) {
    543526                fibril_rwlock_read_unlock(&device_tree.rwlock);
    544                 fun_busy_unlock(fun);
    545527                fun_del_ref(fun);
    546528                async_answer_0(iid, ENOENT);
    … …  
    551533        rc = online_function(fun);
    552534        if (rc != EOK) {
    553                 fun_busy_unlock(fun);
    554535                fun_del_ref(fun);
    555536                async_answer_0(iid, (sysarg_t) rc);
    … …  
    557538        }
    558539       
    559         fun_busy_unlock(fun);
    560540        fun_del_ref(fun);
    561541       
    … …  
    579559        }
    580560       
    581         fun_busy_lock(fun);
    582        
    583561        fibril_rwlock_write_lock(&device_tree.rwlock);
    584562        if (fun->dev == NULL || fun->dev->drv != drv) {
    585                 fun_busy_unlock(fun);
    586563                fun_del_ref(fun);
    587564                async_answer_0(iid, ENOENT);
    … …  
    592569        rc = offline_function(fun);
    593570        if (rc != EOK) {
    594                 fun_busy_unlock(fun);
    595571                fun_del_ref(fun);
    596572                async_answer_0(iid, (sysarg_t) rc);
    … …  
    598574        }
    599575       
    600         fun_busy_unlock(fun);
    601576        fun_del_ref(fun);
    602577        async_answer_0(iid, (sysarg_t) EOK);
    … …  
    616591        }
    617592       
    618         fun_busy_lock(fun);
    619        
    620593        fibril_rwlock_write_lock(&tree->rwlock);
    621594       
    622         log_msg(LOG_DEFAULT, LVL_DEBUG, "devman_remove_function(fun='%s')", fun->pathname);
     595        log_msg(LVL_DEBUG, "devman_remove_function(fun='%s')", fun->pathname);
    623596       
    624597        /* Check function state */
    625598        if (fun->state == FUN_REMOVED) {
    626599                fibril_rwlock_write_unlock(&tree->rwlock);
    627                 fun_busy_unlock(fun);
    628                 fun_del_ref(fun);
    629600                async_answer_0(callid, ENOENT);
    630601                return;
    … …  
    653624                        /* Verify that driver succeeded and removed all functions */
    654625                        if (gone_rc != EOK || !list_empty(&dev->functions)) {
    655                                 log_msg(LOG_DEFAULT, LVL_ERROR, "Driver did not remove "
     626                                log_msg(LVL_ERROR, "Driver did not remove "
    656627                                    "functions for device that is gone. "
    657628                                    "Device node is now defunct.");
    … …  
    667638                                if (gone_rc == EOK)
    668639                                        gone_rc = ENOTSUP;
    669                                 fun_busy_unlock(fun);
    670                                 fun_del_ref(fun);
    671640                                async_answer_0(callid, gone_rc);
    672641                                return;
    … …  
    692661                        rc = loc_service_unregister(fun->service_id);
    693662                        if (rc != EOK) {
    694                                 log_msg(LOG_DEFAULT, LVL_ERROR, "Failed unregistering tree "
     663                                log_msg(LVL_ERROR, "Failed unregistering tree "
    695664                                    "service.");
    696665                                fibril_rwlock_write_unlock(&tree->rwlock);
    697                                 fun_busy_unlock(fun);
    698666                                fun_del_ref(fun);
    699667                                async_answer_0(callid, EIO);
    … …  
    705673        remove_fun_node(&device_tree, fun);
    706674        fibril_rwlock_write_unlock(&tree->rwlock);
    707         fun_busy_unlock(fun);
    708675       
    709676        /* Delete ref added when inserting function into tree */
    … …  
    712679        fun_del_ref(fun);
    713680       
    714         log_msg(LOG_DEFAULT, LVL_DEBUG, "devman_remove_function() succeeded.");
     681        log_msg(LVL_DEBUG, "devman_remove_function() succeeded.");
    715682        async_answer_0(callid, EOK);
    716683}
    … …  
    726693       
    727694        initialize_running_driver(driver, &device_tree);
    728         log_msg(LOG_DEFAULT, LVL_DEBUG, "The `%s` driver was successfully initialized.",
     695        log_msg(LVL_DEBUG, "The `%s` driver was successfully initialized.",
    729696            driver->name);
    730697        return 0;
    … …  
    742709        client = async_get_client_data();
    743710        if (client == NULL) {
    744                 log_msg(LOG_DEFAULT, LVL_ERROR, "Failed to allocate client data.");
     711                log_msg(LVL_ERROR, "Failed to allocate client data.");
    745712                return;
    746713        }
    … …  
    12291196         */
    12301197        if (dev == NULL) {
    1231                 log_msg(LOG_DEFAULT, LVL_ERROR, "IPC forwarding failed - no device or "
     1198                log_msg(LVL_ERROR, "IPC forwarding failed - no device or "
    12321199                    "function with handle %" PRIun " was found.", handle);
    12331200                async_answer_0(iid, ENOENT);
    … …  
    12361203
    12371204        if (fun == NULL && !drv_to_parent) {
    1238                 log_msg(LOG_DEFAULT, LVL_ERROR, NAME ": devman_forward error - cannot "
     1205                log_msg(LVL_ERROR, NAME ": devman_forward error - cannot "
    12391206                    "connect to handle %" PRIun ", refers to a device.",
    12401207                    handle);
    … …  
    12641231       
    12651232        if (driver == NULL) {
    1266                 log_msg(LOG_DEFAULT, LVL_ERROR, "IPC forwarding refused - " \
     1233                log_msg(LVL_ERROR, "IPC forwarding refused - " \
    12671234                    "the device %" PRIun " is not in usable state.", handle);
    12681235                async_answer_0(iid, ENOENT);
    … …  
    12771244       
    12781245        if (!driver->sess) {
    1279                 log_msg(LOG_DEFAULT, LVL_ERROR,
     1246                log_msg(LVL_ERROR,
    12801247                    "Could not forward to driver `%s'.", driver->name);
    12811248                async_answer_0(iid, EINVAL);
    … …  
    12841251
    12851252        if (fun != NULL) {
    1286                 log_msg(LOG_DEFAULT, LVL_DEBUG,
     1253                log_msg(LVL_DEBUG,
    12871254                    "Forwarding request for `%s' function to driver `%s'.",
    12881255                    fun->pathname, driver->name);
    12891256        } else {
    1290                 log_msg(LOG_DEFAULT, LVL_DEBUG,
     1257                log_msg(LVL_DEBUG,
    12911258                    "Forwarding request for `%s' device to driver `%s'.",
    12921259                    dev->pfun->pathname, driver->name);
    … …  
    13201287       
    13211288        if (fun == NULL || fun->dev == NULL || fun->dev->drv == NULL) {
    1322                 log_msg(LOG_DEFAULT, LVL_WARN, "devman_connection_loc(): function "
     1289                log_msg(LVL_WARN, "devman_connection_loc(): function "
    13231290                    "not found.\n");
    13241291                fibril_rwlock_read_unlock(&device_tree.rwlock);
    … …  
    13381305        async_exchange_end(exch);
    13391306       
    1340         log_msg(LOG_DEFAULT, LVL_DEBUG,
     1307        log_msg(LVL_DEBUG,
    13411308            "Forwarding loc service request for `%s' function to driver `%s'.",
    13421309            fun->pathname, driver->name);
    … …  
    13941361static bool devman_init(void)
    13951362{
    1396         log_msg(LOG_DEFAULT, LVL_DEBUG, "devman_init - looking for available drivers.");
     1363        log_msg(LVL_DEBUG, "devman_init - looking for available drivers.");
    13971364       
    13981365        /* Initialize list of available drivers. */
    … …  
    14001367        if (lookup_available_drivers(&drivers_list,
    14011368            DRIVER_DEFAULT_STORE) == 0) {
    1402                 log_msg(LOG_DEFAULT, LVL_FATAL, "No drivers found.");
     1369                log_msg(LVL_FATAL, "No drivers found.");
    14031370                return false;
    14041371        }
    14051372       
    1406         log_msg(LOG_DEFAULT, LVL_DEBUG, "devman_init - list of drivers has been initialized.");
     1373        log_msg(LVL_DEBUG, "devman_init - list of drivers has been initialized.");
    14071374       
    14081375        /* Create root device node. */
    14091376        if (!init_device_tree(&device_tree, &drivers_list)) {
    1410                 log_msg(LOG_DEFAULT, LVL_FATAL, "Failed to initialize device tree.");
     1377                log_msg(LVL_FATAL, "Failed to initialize device tree.");
    14111378                return false;
    14121379        }
    … …  
    14281395        printf("%s: HelenOS Device Manager\n", NAME);
    14291396       
    1430         int rc = log_init(NAME);
     1397        int rc = log_init(NAME, LVL_WARN);
    14311398        if (rc != EOK) {
    14321399                printf("%s: Error initializing logging subsystem.\n", NAME);
    … …  
    14401407       
    14411408        if (!devman_init()) {
    1442                 log_msg(LOG_DEFAULT, LVL_ERROR, "Error while initializing service.");
     1409                log_msg(LVL_ERROR, "Error while initializing service.");
    14431410                return -1;
    14441411        }
    … …  
    14471414        rc = service_register(SERVICE_DEVMAN);
    14481415        if (rc != EOK) {
    1449                 log_msg(LOG_DEFAULT, LVL_ERROR, "Failed registering as a service.");
     1416                log_msg(LVL_ERROR, "Failed registering as a service.");
    14501417                return rc;
    14511418        }
Note: See TracChangeset for help on using the changeset viewer.