Changeset 9b1baac in mainline for uspace/srv/locsrv/locsrv.c

Timestamp:

2018-07-18T08:35:42Z (6 years ago)

Author:

Martin Decky <martin@…>

Branches:

lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export

Children:

0b05082

Parents:

edc64c0

Message:

ns: register service interfaces individually

Each service interface is now registered individually with the naming
service. This adds a degree of type safety, potentially allows the
individual interfaces to be implemented by independent tasks and moves
the code slightly closer to the full-fledged ports design.

Broker services (e.g. the location service) can still register a
fallback port for receiving connections to all interface types
explicitly using service_register_broker().

File:

• uspace/srv/locsrv/locsrv.c (modified) (1 diff)

uspace/srv/locsrv/locsrv.c

@@ -1533 +1533 @@
         }
 
-        port_id_t port;
-        errno_t rc = async_create_port(INTERFACE_LOC_SUPPLIER,
-            loc_connection_supplier, NULL, &port);
+        /* Register location service at naming service */
+        errno_t rc = service_register(SERVICE_LOC, INTERFACE_LOC_SUPPLIER,
+            loc_connection_supplier, NULL);
         if (rc != EOK) {
-                printf("%s: Error while creating supplier port: %s\n", NAME, str_error(rc));
+                printf("%s: Error while registering supplier service: %s\n", NAME, str_error(rc));
                 return rc;
         }
 
-        rc = async_create_port(INTERFACE_LOC_CONSUMER,
-            loc_connection_consumer, NULL, &port);
+        rc = service_register(SERVICE_LOC, INTERFACE_LOC_CONSUMER,
+            loc_connection_consumer, NULL);
         if (rc != EOK) {
-                printf("%s: Error while creating consumer port: %s\n", NAME, str_error(rc));
+                printf("%s: Error while registering consumer service: %s\n", NAME, str_error(rc));
                 return rc;
         }
 
-        /* Set a handler of incomming connections */
-        async_set_fallback_port_handler(loc_forward, NULL);
-
-        /* Register location service at naming service */
-        rc = service_register(SERVICE_LOC);
+        rc = service_register_broker(SERVICE_LOC, loc_forward, NULL);
         if (rc != EOK) {
-                printf("%s: Error while registering service: %s\n", NAME, str_error(rc));
+                printf("%s: Error while registering broker service: %s\n", NAME, str_error(rc));
                 return rc;
         }