Changeset 593e023 in mainline for uspace/app/init/init.c

Timestamp:

2014-08-12T17:14:32Z (11 years ago)

Author:

Martin Decky <martin@…>

Branches:

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

Children:

c3bdc92

Parents:

ce3efa0

Message:

allow compositor and console to coexist side-by-side, use the input server as a poor man's seat arbitrator

• kernel console notifies both about the release and grab events
• input server arbitrates the seat selection between kernel console and any number of user space UIs (currently the console server and the compositor server)
• input port yield and reclaim methods have been removed (they are used only on Ski and Niagara, both already need a more generic mechanism for the kernel/user space cooperation)
• console and compositor server keep track of the kernel console via the input arbitration
• move the waiting for a character device from init and terminal widget to getterm

File:

• uspace/app/init/init.c (modified) (2 diffs)

uspace/app/init/init.c

@@ -272 +272 @@
 }
 
-static void getterm(const char *svc, const char *app, bool wmsg)
-{
-        char term[LOC_NAME_MAXLEN];
-        snprintf(term, LOC_NAME_MAXLEN, "%s/%s", LOCFS_MOUNT_POINT, svc);
-        
-        printf("%s: Spawning %s %s %s\n", NAME, APP_GETTERM, term, app);
-        
-        /* Wait for the terminal service to be ready */
-        service_id_t service_id;
-        int rc = loc_service_get_id(svc, &service_id, IPC_FLAG_BLOCKING);
-        if (rc != EOK) {
-                printf("%s: Error waiting on %s (%s)\n", NAME, term,
-                    str_error(rc));
-                return;
-        }
-        
-        if (wmsg) {
-                rc = task_spawnl(NULL, APP_GETTERM, APP_GETTERM, "-w", term,
-                    app, NULL);
-                if (rc != EOK) {
-                        printf("%s: Error spawning %s -w %s %s (%s)\n", NAME,
-                            APP_GETTERM, term, app, str_error(rc));
-                }
+static void getterm(const char *svc, const char *app, bool msg)
+{
+        if (msg) {
+                printf("%s: Spawning %s %s %s --msg --wait -- %s\n", NAME,
+                    APP_GETTERM, svc, LOCFS_MOUNT_POINT, app);
+                
+                int rc = task_spawnl(NULL, APP_GETTERM, APP_GETTERM, svc,
+                    LOCFS_MOUNT_POINT, "--msg", "--wait", "--", app, NULL);
+                if (rc != EOK)
+                        printf("%s: Error spawning %s %s %s --msg --wait -- %s\n",
+                            NAME, APP_GETTERM, svc, LOCFS_MOUNT_POINT, app);
         } else {
-                rc = task_spawnl(NULL, APP_GETTERM, APP_GETTERM, term, app,
-                    NULL);
-                if (rc != EOK) {
-                        printf("%s: Error spawning %s %s %s (%s)\n", NAME,
-                            APP_GETTERM, term, app, str_error(rc));
-                }
+                printf("%s: Spawning %s %s %s --wait -- %s\n", NAME,
+                    APP_GETTERM, svc, LOCFS_MOUNT_POINT, app);
+                
+                int rc = task_spawnl(NULL, APP_GETTERM, APP_GETTERM, svc,
+                    LOCFS_MOUNT_POINT, "--wait", "--", app, NULL);
+                if (rc != EOK)
+                        printf("%s: Error spawning %s %s %s --wait -- %s\n",
+                            NAME, APP_GETTERM, svc, LOCFS_MOUNT_POINT, app);
         }
 }
@@ -364 +354 @@
                 gui_start("/app/vlaunch", HID_COMPOSITOR_SERVER);
                 gui_start("/app/vterm", HID_COMPOSITOR_SERVER);
-        } else {
-                rc = console(HID_INPUT, HID_OUTPUT);
-                if (rc == EOK) {
-                        getterm("term/vc0", "/app/bdsh", true);
-                        getterm("term/vc1", "/app/bdsh", false);
-                        getterm("term/vc2", "/app/bdsh", false);
-                        getterm("term/vc3", "/app/bdsh", false);
-                        getterm("term/vc4", "/app/bdsh", false);
-                        getterm("term/vc5", "/app/bdsh", false);
-                        getterm("term/vc6", "/app/klog", false);
-                }
+        }
+        
+        rc = console(HID_INPUT, HID_OUTPUT);
+        if (rc == EOK) {
+                getterm("term/vc0", "/app/bdsh", true);
+                getterm("term/vc1", "/app/bdsh", false);
+                getterm("term/vc2", "/app/bdsh", false);
+                getterm("term/vc3", "/app/bdsh", false);
+                getterm("term/vc4", "/app/bdsh", false);
+                getterm("term/vc5", "/app/bdsh", false);
         }