Fork us on GitHub Follow us on Facebook Follow us on Twitter

Changeset 50ad3f3 in mainline


Ignore:
Timestamp:
2012-06-01T19:17:07Z (8 years ago)
Author:
Martin Decky <martin@…>
Branches:
master
Children:
1b054a6f
Parents:
b16e77d
Message:

trivial changes (error code reporting, messages)

Location:
uspace/srv
Files:
4 edited

Legend:

Unmodified
Added
Removed
  • uspace/srv/devman/devman.h

    rb16e77d r50ad3f3  
    210210};
    211211
    212 
    213212/** Represents device tree. */
    214213typedef struct dev_tree {
  • uspace/srv/devman/main.c

    rb16e77d r50ad3f3  
    245245       
    246246        fibril_rwlock_write_lock(&device_tree.rwlock);
    247 
     247       
    248248        if (fun->state == FUN_ON_LINE) {
    249249                fibril_rwlock_write_unlock(&device_tree.rwlock);
     
    259259                        return ENOMEM;
    260260                }
    261 
     261               
    262262                insert_dev_node(&device_tree, dev, fun);
    263263                dev_add_ref(dev);
     
    272272                /* Give one reference over to assign_driver_fibril(). */
    273273                dev_add_ref(dev);
     274               
    274275                /*
    275276                 * Try to find a suitable driver and assign it to the device.  We do
     
    288289                }
    289290                fibril_add_ready(assign_fibril);
    290         } else {
     291        } else
    291292                loc_register_tree_function(fun, &device_tree);
    292         }
    293293       
    294294        fibril_rwlock_write_unlock(&device_tree.rwlock);
     
    11491149                if (dev->pfun->dev != NULL)
    11501150                        driver = dev->pfun->dev->drv;
     1151               
    11511152                fwd_h = dev->pfun->handle;
    11521153        } else if (dev->state == DEVICE_USABLE) {
     
    11541155                driver = dev->drv;
    11551156                assert(driver != NULL);
    1156 
     1157               
    11571158                fwd_h = handle;
    11581159        }
     
    11811182
    11821183        if (fun != NULL) {
    1183                 log_msg(LVL_DEBUG, 
     1184                log_msg(LVL_DEBUG,
    11841185                    "Forwarding request for `%s' function to driver `%s'.",
    11851186                    fun->pathname, driver->name);
    11861187        } else {
    1187                 log_msg(LVL_DEBUG, 
     1188                log_msg(LVL_DEBUG,
    11881189                    "Forwarding request for `%s' device to driver `%s'.",
    11891190                    dev->pfun->pathname, driver->name);
     
    11931194        async_forward_fast(iid, exch, method, fwd_h, 0, IPC_FF_NONE);
    11941195        async_exchange_end(exch);
    1195 
     1196       
    11961197cleanup:
    11971198        if (dev != NULL)
    11981199                dev_del_ref(dev);
     1200       
    11991201        if (fun != NULL)
    12001202                fun_del_ref(fun);
     
    12991301                return false;
    13001302        }
    1301 
     1303       
    13021304        log_msg(LVL_DEBUG, "devman_init - list of drivers has been initialized.");
    1303 
     1305       
    13041306        /* Create root device node. */
    13051307        if (!init_device_tree(&device_tree, &drivers_list)) {
     
    13071309                return false;
    13081310        }
    1309 
     1311       
    13101312        /*
    13111313         * Caution: As the device manager is not a real loc
     
    13221324int main(int argc, char *argv[])
    13231325{
    1324         printf(NAME ": HelenOS Device Manager\n");
    1325 
    1326         if (log_init(NAME, LVL_WARN) != EOK) {
    1327                 printf(NAME ": Error initializing logging subsystem.\n");
    1328                 return -1;
     1326        printf("%s: HelenOS Device Manager\n", NAME);
     1327       
     1328        int rc = log_init(NAME, LVL_WARN);
     1329        if (rc != EOK) {
     1330                printf("%s: Error initializing logging subsystem.\n", NAME);
     1331                return rc;
    13291332        }
    13301333       
     
    13331336        async_set_client_data_destructor(devman_client_data_destroy);
    13341337        async_set_client_connection(devman_connection);
    1335 
     1338       
    13361339        if (!devman_init()) {
    13371340                log_msg(LVL_ERROR, "Error while initializing service.");
    13381341                return -1;
    13391342        }
    1340 
     1343       
    13411344        /* Register device manager at naming service. */
    1342         if (service_register(SERVICE_DEVMAN) != EOK) {
     1345        rc = service_register(SERVICE_DEVMAN);
     1346        if (rc != EOK) {
    13431347                log_msg(LVL_ERROR, "Failed registering as a service.");
    1344                 return -1;
    1345         }
    1346 
    1347         printf(NAME ": Accepting connections.\n");
     1348                return rc;
     1349        }
     1350       
     1351        printf("%s: Accepting connections.\n", NAME);
    13481352        task_retval(0);
    13491353        async_manager();
    1350 
     1354       
    13511355        /* Never reached. */
    13521356        return 0;
  • uspace/srv/locsrv/locsrv.c

    rb16e77d r50ad3f3  
    390390         */
    391391        list_initialize(&server->services);
    392 
    393392        link_initialize(&server->servers);
    394393       
     
    14901489       
    14911490        /* Register location service at naming service */
    1492         if (service_register(SERVICE_LOC) != EOK)
    1493                 return -1;
     1491        int rc = service_register(SERVICE_LOC);
     1492        if (rc != EOK)
     1493                return rc;
    14941494       
    14951495        printf("%s: Accepting connections\n", NAME);
  • uspace/srv/locsrv/locsrv.h

    rb16e77d r50ad3f3  
    8686        /** Link to global list of services (services_list) */
    8787        link_t services;
     88       
    8889        /** Link to server list of services (loc_server_t.services) */
    8990        link_t server_services;
     91       
    9092        /** Link to list of services in category (category_t.services) */
    9193        link_t cat_services;
     94       
    9295        /** List of category memberships (svc_categ_t) */
    9396        list_t cat_memb;
     97       
    9498        /** Unique service identifier */
    9599        service_id_t id;
     100       
    96101        /** Service namespace */
    97102        loc_namespace_t *namespace;
     103       
    98104        /** Service name */
    99105        char *name;
     106       
    100107        /** Supplier of this service */
    101108        loc_server_t *server;
     109       
    102110        /** Use this interface when forwarding to server. */
    103111        sysarg_t forward_interface;
Note: See TracChangeset for help on using the changeset viewer.