ivi-compositor: Add support for multiple app_ids
[src/agl-compositor.git] / src / compositor.c
index dbc89e2..617f1b5 100644 (file)
@@ -73,6 +73,12 @@ to_ivi_compositor(struct weston_compositor *ec)
        return weston_compositor_get_user_data(ec);
 }
 
+static void
+sigint_helper(int sig)
+{
+       raise(SIGUSR2);
+}
+
 static void
 handle_output_destroy(struct wl_listener *listener, void *data)
 {
@@ -81,7 +87,8 @@ handle_output_destroy(struct wl_listener *listener, void *data)
        output = wl_container_of(listener, output, output_destroy);
        assert(output->output == data);
 
-       if (output->fullscreen_view.fs->view) {
+       if (output->fullscreen_view.fs &&
+           output->fullscreen_view.fs->view) {
                weston_surface_destroy(output->fullscreen_view.fs->view->surface);
                output->fullscreen_view.fs->view = NULL;
        }
@@ -106,19 +113,19 @@ static void
 ivi_output_configure_app_id(struct ivi_output *ivi_output)
 {
        if (ivi_output->config) {
-               if (ivi_output->app_id != NULL)
+               if (ivi_output->app_ids != NULL)
                        return;
 
                weston_config_section_get_string(ivi_output->config,
                                                 "agl-shell-app-id",
-                                                &ivi_output->app_id,
+                                                &ivi_output->app_ids,
                                                 NULL);
 
-               if (ivi_output->app_id == NULL)
+               if (ivi_output->app_ids == NULL)
                        return;
 
                weston_log("Will place app_id %s on output %s\n",
-                               ivi_output->app_id, ivi_output->name);
+                               ivi_output->app_ids, ivi_output->name);
        }
 }
 
@@ -321,6 +328,23 @@ parse_transform(const char *transform, uint32_t *out)
        return -1;
 }
 
+static int
+parse_activation_area(const char *geometry, struct ivi_output *output)
+{
+       int n;
+       unsigned width, height, x, y;
+
+       n = sscanf(geometry, "%ux%u+%u,%u", &width, &height, &x, &y);
+       if (n != 4) {
+               return -1;
+       }
+       output->area_activation.width = width;
+       output->area_activation.height = height;
+       output->area_activation.x = x;
+       output->area_activation.y = y;
+       return 0;
+}
+
 static int
 configure_output(struct ivi_output *output)
 {
@@ -344,6 +368,10 @@ configure_output(struct ivi_output *output)
                if (parse_transform(t, &transform) < 0)
                        weston_log("Invalid transform \"%s\" for output %s\n",
                                   t, output->name);
+               weston_config_section_get_string(section, "activation-area", &t, "");
+               if (parse_activation_area(t, output) < 0)
+                       weston_log("Invalid activation-area \"%s\" for output %s\n",
+                                  t, output->name);
                free(t);
        }
 
@@ -460,8 +488,21 @@ head_disable(struct ivi_compositor *ivi, struct weston_head *head)
 
        weston_head_detach(head);
        if (count_heads(ivi_output->output) == 0) {
-               weston_output_disable(ivi_output->output);
+               if (ivi_output->output) {
+                       /* ivi_output->output destruction may be deferred in
+                        * some cases (see drm_output_destroy()), so we need to
+                        * forcibly trigger the destruction callback now, or
+                        * otherwise would later access data that we are about
+                        * to free
+                        */
+                       struct weston_output *save = ivi_output->output;
+
+                       handle_output_destroy(&ivi_output->output_destroy, save);
+                       weston_output_destroy(save);
+               }
        }
+       wl_list_remove(&ivi_output->link);
+       free(ivi_output);
 }
 
 static struct weston_config_section *
@@ -1239,11 +1280,12 @@ choose_default_backend(void)
 }
 
 static int
-compositor_init_config(struct weston_compositor *compositor,
-                      struct weston_config *config)
+compositor_init_config(struct ivi_compositor *ivi)
 {
        struct xkb_rule_names xkb_names;
        struct weston_config_section *section;
+       struct weston_compositor *compositor = ivi->compositor;
+       struct weston_config *config = ivi->config;
        int repaint_msec;
        bool vt_switching;
        bool require_input;
@@ -1276,6 +1318,11 @@ compositor_init_config(struct weston_compositor *compositor,
        /* agl-compositor.ini [core] */
        section = weston_config_get_section(config, "core", NULL, NULL);
 
+       weston_config_section_get_bool(section, "disable-cursor",
+                                      &ivi->disable_cursor, false);
+       weston_config_section_get_bool(section, "activate-by-default",
+                                      &ivi->activate_by_default, true);
+
        weston_config_section_get_bool(section, "require-input", &require_input, true);
        compositor->require_input = require_input;
 
@@ -1608,7 +1655,7 @@ int wet_main(int argc, char *argv[], const struct weston_testsuite_data *test_da
        char *cmdline;
        struct wl_display *display = NULL;
        struct wl_event_loop *loop;
-       struct wl_event_source *signals[2] = { 0 };
+       struct wl_event_source *signals[3] = { 0 };
        struct weston_config_section *section;
        /* Command line options */
        char *backend = NULL;
@@ -1625,6 +1672,7 @@ int wet_main(int argc, char *argv[], const struct weston_testsuite_data *test_da
        struct weston_log_subscriber *logger;
        int ret = EXIT_FAILURE;
        bool xwayland = false;
+       struct sigaction action;
 
        const struct weston_option core_options[] = {
                { WESTON_OPTION_STRING, "backend", 'B', &backend },
@@ -1691,11 +1739,6 @@ int wet_main(int argc, char *argv[], const struct weston_testsuite_data *test_da
                if (!backend)
                        backend = choose_default_backend();
        }
-       /* from [core] */
-       weston_config_section_get_bool(section, "hide-cursor",
-                                      &ivi.hide_cursor, false);
-       weston_config_section_get_bool(section, "activate-by-default",
-                                      &ivi.activate_by_default, true);
 
        display = wl_display_create();
        loop = wl_display_get_event_loop(display);
@@ -1707,9 +1750,26 @@ int wet_main(int argc, char *argv[], const struct weston_testsuite_data *test_da
 
        signals[0] = wl_event_loop_add_signal(loop, SIGTERM, on_term_signal,
                                              display);
-       signals[1] = wl_event_loop_add_signal(loop, SIGINT, on_term_signal,
+       signals[1] = wl_event_loop_add_signal(loop, SIGUSR2, on_term_signal,
+                                             display);
+       signals[2] = wl_event_loop_add_signal(loop, SIGCHLD, sigchld_handler,
                                              display);
 
+       /* When debugging the compositor, if use wl_event_loop_add_signal() to
+        * catch SIGINT, the debugger can't catch it, and attempting to stop
+        * the compositor from within the debugger results in weston exiting
+        * cleanly.
+        *
+        * Instead, use the sigaction() function, which sets up the signal in a
+        * way that gdb can successfully catch, but have the handler for SIGINT
+        * send SIGUSR2 (xwayland uses SIGUSR1), which we catch via
+        * wl_event_loop_add_signal().
+        */
+       action.sa_handler = sigint_helper;
+       sigemptyset(&action.sa_mask);
+       action.sa_flags = 0;
+       sigaction(SIGINT, &action, NULL);
+
        for (size_t i = 0; i < ARRAY_LENGTH(signals); ++i)
                if (!signals[i])
                        goto error_signals;
@@ -1720,7 +1780,7 @@ int wet_main(int argc, char *argv[], const struct weston_testsuite_data *test_da
                goto error_signals;
        }
 
-       if (compositor_init_config(ivi.compositor, ivi.config) < 0)
+       if (compositor_init_config(&ivi) < 0)
                goto error_compositor;
 
        if (load_backend(&ivi, backend, &argc, argv) < 0) {
@@ -1771,7 +1831,12 @@ int wet_main(int argc, char *argv[], const struct weston_testsuite_data *test_da
        weston_compositor_wake(ivi.compositor);
 
        ivi_shell_create_global(&ivi);
-       ivi_launch_shell_client(&ivi);
+
+       ivi_launch_shell_client(&ivi, "shell-client",
+                               &ivi.shell_client.client);
+       ivi_launch_shell_client(&ivi, "shell-client-ext",
+                               &ivi.shell_client_ext.client);
+
        if (debug)
                ivi_screenshooter_create(&ivi);
        ivi_agl_systemd_notify(&ivi);