Changes in uspace/srv/hid/compositor/compositor.c [8e4a408:071fefec] in mainline

File:

• uspace/srv/hid/compositor/compositor.c (modified) (7 diffs)

uspace/srv/hid/compositor/compositor.c

@@ -55 +55 @@
 #include <async.h>
 #include <loc.h>
+#include <devman.h>
 
 #include <event.h>
+#include <graph_iface.h>
 #include <io/keycode.h>
 #include <io/mode.h>
@@ -1147 +1149 @@
 }
 
-static async_sess_t *vsl_connect(service_id_t sid, const char *svc)
+static async_sess_t *vsl_connect(const char *svc)
 {
         int rc;
         async_sess_t *sess;
-
-        sess = loc_service_connect(EXCHANGE_SERIALIZE, sid, 0);
-        if (sess == NULL) {
-                printf("%s: Unable to connect to visualizer %s\n", NAME, svc);
+        service_id_t dsid;
+        devman_handle_t handle;
+
+        rc = loc_service_get_id(svc, &dsid, 0);
+        if (rc != EOK) {
                 return NULL;
         }
 
+        rc = devman_fun_sid_to_handle(dsid, &handle);
+        if (rc == EOK) {
+                sess = devman_device_connect(EXCHANGE_SERIALIZE, handle, 0);
+                if (sess == NULL) {
+                        printf("%s: Unable to connect to visualizer %s\n", NAME, svc);
+                        return NULL;
+                }
+                rc = graph_dev_connect(sess);
+                if (rc != EOK) {
+                        return NULL;
+                }
+        } else if (rc == ENOENT) {
+                sess = loc_service_connect(EXCHANGE_SERIALIZE, dsid, 0);
+                if (sess == NULL) {
+                        printf("%s: Unable to connect to visualizer %s\n", NAME, svc);
+                        return NULL;
+                }
+        } else {
+                return NULL;
+        }
+
         async_exch_t *exch = async_exchange_begin(sess);
-        rc = async_connect_to_me(exch, sid, 0, 0, vsl_notifications, NULL);
+        rc = async_connect_to_me(exch, dsid, 0, 0, vsl_notifications, NULL);
         async_exchange_end(exch);
 
@@ -1172 +1196 @@
 }
 
-static viewport_t *viewport_create(service_id_t sid)
+static viewport_t *viewport_create(const char *vsl_name)
 {
         int rc;
-        char *vsl_name = NULL;
-        viewport_t *vp = NULL;
-        bool claimed = false;
-
-        rc = loc_service_get_name(sid, &vsl_name);
-        if (rc != EOK)
-                goto error;
-
-        vp = (viewport_t *) calloc(1, sizeof(viewport_t));
-        if (!vp)
-                goto error;
+
+        viewport_t *vp = (viewport_t *) malloc(sizeof(viewport_t));
+        if (!vp) {
+                return NULL;
+        }
 
         link_initialize(&vp->link);
@@ -1192 +1210 @@
 
         /* Establish output bidirectional connection. */
-        vp->dsid = sid;
-        vp->sess = vsl_connect(sid, vsl_name);
-        if (vp->sess == NULL)
-                goto error;
+        vp->sess = vsl_connect(vsl_name);
+        rc = loc_service_get_id(vsl_name, &vp->dsid, 0);
+        if (vp->sess == NULL || rc != EOK) {
+                free(vp);
+                return NULL;
+        }
 
         /* Claim the given visualizer. */
         rc = visualizer_claim(vp->sess, 0);
         if (rc != EOK) {
+                async_hangup(vp->sess);
+                free(vp);
                 printf("%s: Unable to claim visualizer (%s)\n", NAME, str_error(rc));
-                goto error;
-        }
-
-        claimed = true;
+                return NULL;
+        }
 
         /* Retrieve the default mode. */
         rc = visualizer_get_default_mode(vp->sess, &vp->mode);
         if (rc != EOK) {
+                visualizer_yield(vp->sess);
+                async_hangup(vp->sess);
+                free(vp);
                 printf("%s: Unable to retrieve mode (%s)\n", NAME, str_error(rc));
-                goto error;
+                return NULL;
         }
 
@@ -1217 +1240 @@
             NULL, SURFACE_FLAG_SHARED);
         if (vp->surface == NULL) {
+                visualizer_yield(vp->sess);
+                async_hangup(vp->sess);
+                free(vp);
                 printf("%s: Unable to create surface (%s)\n", NAME, str_error(rc));
-                goto error;
+                return NULL;
         }
 
@@ -1225 +1251 @@
                 vp->mode.index, vp->mode.version, surface_direct_access(vp->surface));
         if (rc != EOK) {
+                visualizer_yield(vp->sess);
+                surface_destroy(vp->surface);
+                async_hangup(vp->sess);
+                free(vp);
                 printf("%s: Unable to set mode (%s)\n", NAME, str_error(rc));
-                goto error;
+                return NULL;
         }
 
         return vp;
-error:
-        if (claimed)
-                visualizer_yield(vp->sess);
-        if (vp->sess != NULL)
-                async_hangup(vp->sess);
-        free(vp);
-        free(vsl_name);
-        return NULL;
 }
 
@@ -2168 +2190 @@
                         continue;
                 
-                viewport_t *vp = viewport_create(svcs[i]);
-                if (vp != NULL)
-                        list_append(&vp->link, &viewport_list);
+                char *svc_name;
+                rc = loc_service_get_name(svcs[i], &svc_name);
+                if (rc == EOK) {
+                        viewport_t *vp = viewport_create(svc_name);
+                        if (vp != NULL) {
+                                list_append(&vp->link, &viewport_list);
+                        }
+                }
         }
         fibril_mutex_unlock(&viewport_list_mtx);