compositor: Use stdint for specifing integer storage
[src/agl-compositor.git] / src / compositor.c
index 2bbdce8..078157a 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * Copyright © 2012-2021 Collabora, Ltd.
+ * Copyright © 2012-2024 Collabora, Ltd.
  *
  * Permission is hereby granted, free of charge, to any person obtaining
  * a copy of this software and associated documentation files (the
@@ -39,8 +39,8 @@
 
 #include <libweston/backend-drm.h>
 #include <libweston/backend-wayland.h>
-#ifdef HAVE_BACKEND_HEADLESS
-#include <libweston/backend-headless.h>
+#ifdef HAVE_BACKEND_RDP
+#include <libweston/backend-rdp.h>
 #endif
 #ifdef HAVE_BACKEND_X11
 #include <libweston/backend-x11.h>
 #include <libweston/windowed-output-api.h>
 #include <libweston/config-parser.h>
 #include <libweston/weston-log.h>
-#include <weston/weston.h>
+#include <weston.h>
 
 #include "shared/os-compatibility.h"
 #include "shared/helpers.h"
 
+#include "config.h"
 #include "agl-shell-server-protocol.h"
 
 #ifdef HAVE_REMOTING
 #include "remote.h"
 #endif
 
-#ifdef HAVE_WALTHAM
-#include <waltham-transmitter/transmitter_api.h>
-#endif
+#define WINDOWED_DEFAULT_WIDTH 1024
+#define WINDOWED_DEFAULT_HEIGHT 768
 
 static int cached_tm_mday = -1;
 static struct weston_log_scope *log_scope;
@@ -73,6 +73,104 @@ to_ivi_compositor(struct weston_compositor *ec)
        return weston_compositor_get_user_data(ec);
 }
 
+void
+ivi_process_destroy(struct wet_process *process, int status, bool call_cleanup)
+{
+        wl_list_remove(&process->link);
+        if (call_cleanup && process->cleanup)
+                process->cleanup(process, status, process->cleanup_data);
+        free(process->path);
+        free(process);
+}
+
+struct ivi_output_config *
+ivi_init_parsed_options(struct weston_compositor *compositor)
+{
+       struct ivi_compositor *ivi = to_ivi_compositor(compositor);
+       struct ivi_output_config *config;
+
+       config = zalloc(sizeof *config);
+       if (!config)
+               return NULL;
+
+       config->width = 0;
+       config->height = 0;
+       config->scale = 0;
+       config->transform = UINT32_MAX;
+
+       ivi->parsed_options = config;
+
+       return config;
+}
+
+static void
+screenshot_allow_all(struct wl_listener *l, struct weston_output_capture_attempt *att)
+{
+       att->authorized = true;
+}
+
+
+static void
+sigint_helper(int sig)
+{
+       raise(SIGUSR2);
+}
+
+struct {
+        char *name;
+        enum weston_renderer_type renderer;
+} renderer_name_map[] = {
+        { "auto", WESTON_RENDERER_AUTO },
+        { "gl", WESTON_RENDERER_GL },
+        { "noop", WESTON_RENDERER_NOOP },
+        { "pixman", WESTON_RENDERER_PIXMAN },
+};
+
+struct {
+       char *short_name;
+       char *long_name;
+       enum weston_compositor_backend backend;
+} backend_name_map[] = {
+       { "drm", "drm-backend.so", WESTON_BACKEND_DRM },
+       { "rdp", "rdp-backend.so", WESTON_BACKEND_RDP },
+       { "wayland", "wayland-backend.so", WESTON_BACKEND_WAYLAND },
+       { "x11", "x11-backend.so", WESTON_BACKEND_X11 },
+};
+
+bool
+get_backend_from_string(const char *name, enum weston_compositor_backend *backend)
+{
+       size_t i;
+
+       for (i = 0; i < ARRAY_LENGTH(backend_name_map); i++) {
+               if (strcmp(name, backend_name_map[i].short_name) == 0 ||
+                   strcmp(name, backend_name_map[i].long_name) == 0) {
+                       *backend = backend_name_map[i].backend;
+                       return true;
+               }
+       }
+
+       return false;
+}
+
+bool
+get_renderer_from_string(const char *name, enum weston_renderer_type *renderer)
+{
+       size_t i;
+
+       if (!name)
+               name = "auto";
+
+       for (i = 0; i < ARRAY_LENGTH(renderer_name_map); i++) {
+               if (strcmp(name, renderer_name_map[i].name) == 0) {
+                       *renderer = renderer_name_map[i].renderer;
+                       return true;
+               }
+       }
+
+       return false;
+}
+
 static void
 handle_output_destroy(struct wl_listener *listener, void *data)
 {
@@ -81,11 +179,15 @@ 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) {
-               weston_surface_destroy(output->fullscreen_view.fs->view->surface);
+       if (output->fullscreen_view.fs &&
+           output->fullscreen_view.fs->view) {
+               weston_surface_unref(output->fullscreen_view.fs->view->surface);
+               weston_buffer_destroy_solid(output->fullscreen_view.buffer_ref);
                output->fullscreen_view.fs->view = NULL;
        }
 
+       ivi_layout_save(output->ivi, output);
+
        output->output = NULL;
        wl_list_remove(&output->output_destroy.link);
 }
@@ -97,6 +199,9 @@ to_ivi_output(struct weston_output *o)
        struct ivi_output *output;
 
        listener = weston_output_get_destroy_listener(o, handle_output_destroy);
+       if (!listener)
+               return NULL;
+
        output = wl_container_of(listener, output, output_destroy);
 
        return output;
@@ -106,25 +211,26 @@ 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);
        }
 }
 
 static struct ivi_output *
 ivi_ensure_output(struct ivi_compositor *ivi, char *name,
-                 struct weston_config_section *config)
+                 struct weston_config_section *config,
+                 struct weston_head *head)
 {
        struct ivi_output *output = NULL;
        wl_list_for_each(output, &ivi->outputs, link) {
@@ -144,19 +250,45 @@ ivi_ensure_output(struct ivi_compositor *ivi, char *name,
        output->name = name;
        output->config = config;
 
-       output->output = weston_compositor_create_output(ivi->compositor, name);
+       output->output =
+               weston_compositor_create_output(ivi->compositor, head, head->name);
        if (!output->output) {
                free(output->name);
                free(output);
                return NULL;
        }
 
+       /* simple_output_configure might assume we have an ivi_output created
+        * by this point, which we do but we can only link it to a
+        * weston_output through the destroy listener, so install it earlier
+        * before actually running the callback handler */
        output->output_destroy.notify = handle_output_destroy;
        weston_output_add_destroy_listener(output->output,
                                           &output->output_destroy);
 
+       if (ivi->simple_output_configure) {
+               int ret = ivi->simple_output_configure(output->output);
+               if (ret < 0) {
+                       weston_log("Configuring output \"%s\" failed.\n",
+                                       weston_head_get_name(head));
+                       weston_output_destroy(output->output);
+                       ivi->init_failed = true;
+                       return NULL;
+               }
+
+               if (weston_output_enable(output->output) < 0) {
+                       weston_log("Enabling output \"%s\" failed.\n",
+                                       weston_head_get_name(head));
+                       weston_output_destroy(output->output);
+                       ivi->init_failed = true;
+                       return NULL;
+               }
+       }
+
+
        wl_list_insert(&ivi->outputs, &output->link);
        ivi_output_configure_app_id(output);
+
        return output;
 }
 
@@ -208,63 +340,14 @@ add_head_destroyed_listener(struct weston_head *head)
 }
 
 static int
-drm_configure_output(struct ivi_output *output)
+ivi_configure_windowed_output_from_config(struct ivi_output *output,
+                                         struct ivi_output_config *defaults)
 {
        struct ivi_compositor *ivi = output->ivi;
        struct weston_config_section *section = output->config;
-       enum weston_drm_backend_output_mode mode =
-               WESTON_DRM_BACKEND_OUTPUT_PREFERRED;
-       char *modeline = NULL;
-       char *gbm_format = NULL;
-       char *seat = NULL;
-
-       if (section) {
-               char *m;
-               weston_config_section_get_string(section, "mode", &m, "preferred");
-
-               /* This should have been handled earlier */
-               assert(strcmp(m, "off") != 0);
-
-               if (ivi->cmdline.use_current_mode || strcmp(m, "current") == 0) {
-                       mode = WESTON_DRM_BACKEND_OUTPUT_CURRENT;
-               } else if (strcmp(m, "preferred") != 0) {
-                       modeline = m;
-                       m = NULL;
-               }
-               free(m);
-
-               weston_config_section_get_string(section, "gbm-format",
-                                                &gbm_format, NULL);
-
-               weston_config_section_get_string(section, "seat", &seat, "");
-       }
-
-       if (ivi->drm_api->set_mode(output->output, mode, modeline) < 0) {
-               weston_log("Cannot configure output using weston_drm_output_api.\n");
-               free(modeline);
-               return -1;
-       }
-       free(modeline);
-
-       ivi->drm_api->set_gbm_format(output->output, gbm_format);
-       free(gbm_format);
-
-       ivi->drm_api->set_seat(output->output, seat);
-       free(seat);
-
-       return 0;
-}
-
-#define WINDOWED_DEFAULT_WIDTH 1024
-#define WINDOWED_DEFAULT_HEIGHT 768
-
-static int
-windowed_configure_output(struct ivi_output *output)
-{
-       struct ivi_compositor *ivi = output->ivi;
-       struct weston_config_section *section = output->config;
-       int width = WINDOWED_DEFAULT_WIDTH;
-       int height = WINDOWED_DEFAULT_HEIGHT;
+       int width;
+       int height;
+       int scale;
 
        if (section) {
                char *mode;
@@ -273,18 +356,26 @@ windowed_configure_output(struct ivi_output *output)
                if (!mode || sscanf(mode, "%dx%d", &width, &height) != 2) {
                        weston_log("Invalid mode for output %s. Using defaults.\n",
                                   output->name);
-                       width = WINDOWED_DEFAULT_WIDTH;
-                       height = WINDOWED_DEFAULT_HEIGHT;
+                       width = defaults->width;
+                       height = defaults->height;
                }
                free(mode);
+       } else {
+               width = defaults->width;
+               height = defaults->height;
        }
 
+       scale = defaults->scale;
+
        if (ivi->cmdline.width)
                width = ivi->cmdline.width;
        if (ivi->cmdline.height)
                height = ivi->cmdline.height;
        if (ivi->cmdline.scale)
-               weston_output_set_scale(output->output, ivi->cmdline.scale);
+               scale = ivi->cmdline.scale;
+
+       weston_output_set_scale(output->output, scale);
+       weston_output_set_transform(output->output, defaults->transform);
 
        if (ivi->window_api->output_set_size(output->output, width, height) < 0) {
                weston_log("Cannot configure output '%s' using weston_windowed_output_api.\n",
@@ -292,7 +383,9 @@ windowed_configure_output(struct ivi_output *output)
                return -1;
        }
 
-       weston_log("Configured windowed_output_api to %dx%d\n", width, height);
+
+       weston_log("Configured windowed_output_api to %dx%d, scale %d\n",
+                  width, height, scale);
 
        return 0;
 }
@@ -301,14 +394,14 @@ static int
 parse_transform(const char *transform, uint32_t *out)
 {
        static const struct { const char *name; uint32_t token; } transforms[] = {
-               { "normal",     WL_OUTPUT_TRANSFORM_NORMAL },
-               { "90",         WL_OUTPUT_TRANSFORM_90 },
-               { "180",        WL_OUTPUT_TRANSFORM_180 },
-               { "270",        WL_OUTPUT_TRANSFORM_270 },
-               { "flipped",    WL_OUTPUT_TRANSFORM_FLIPPED },
-               { "flipped-90", WL_OUTPUT_TRANSFORM_FLIPPED_90 },
-               { "flipped-180", WL_OUTPUT_TRANSFORM_FLIPPED_180 },
-               { "flipped-270", WL_OUTPUT_TRANSFORM_FLIPPED_270 },
+               { "normal",             WL_OUTPUT_TRANSFORM_NORMAL },
+               { "rotate-90",          WL_OUTPUT_TRANSFORM_90 },
+               { "rotate-180",         WL_OUTPUT_TRANSFORM_180 },
+               { "rotate-270",         WL_OUTPUT_TRANSFORM_270 },
+               { "flipped",            WL_OUTPUT_TRANSFORM_FLIPPED },
+               { "flipped-rotate-90",  WL_OUTPUT_TRANSFORM_FLIPPED_90 },
+               { "flipped-rotate-180", WL_OUTPUT_TRANSFORM_FLIPPED_180 },
+               { "flipped-rotate-270", WL_OUTPUT_TRANSFORM_FLIPPED_270 },
        };
 
        for (size_t i = 0; i < ARRAY_LENGTH(transforms); i++)
@@ -321,20 +414,35 @@ parse_transform(const char *transform, uint32_t *out)
        return -1;
 }
 
+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)
+drm_configure_output(struct ivi_output *output)
 {
        struct ivi_compositor *ivi = output->ivi;
        struct weston_config_section *section = output->config;
        int32_t scale = 1;
        uint32_t transform = WL_OUTPUT_TRANSFORM_NORMAL;
-
-       /*
-        * This can happen with the wayland backend with 'sprawl'. The config
-        * is hard-coded, so we don't need to do anything.
-        */
-       if (!ivi->drm_api && !ivi->window_api)
-               return 0;
+       enum weston_drm_backend_output_mode mode =
+               WESTON_DRM_BACKEND_OUTPUT_PREFERRED;
+       char *modeline = NULL;
+       char *gbm_format = NULL;
+       char *seat = NULL;
 
        if (section) {
                char *t;
@@ -344,16 +452,51 @@ 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);
        }
 
        weston_output_set_scale(output->output, scale);
        weston_output_set_transform(output->output, transform);
 
-       if (ivi->drm_api)
-               return drm_configure_output(output);
-       else
-               return windowed_configure_output(output);
+       if (section) {
+               char *m;
+               weston_config_section_get_string(section, "mode", &m, "preferred");
+
+               /* This should have been handled earlier */
+               assert(strcmp(m, "off") != 0);
+
+               if (ivi->cmdline.use_current_mode || strcmp(m, "current") == 0) {
+                       mode = WESTON_DRM_BACKEND_OUTPUT_CURRENT;
+               } else if (strcmp(m, "preferred") != 0) {
+                       modeline = m;
+                       m = NULL;
+               }
+               free(m);
+
+               weston_config_section_get_string(section, "gbm-format",
+                                                &gbm_format, NULL);
+
+               weston_config_section_get_string(section, "seat", &seat, "");
+       }
+
+       if (ivi->drm_api->set_mode(output->output, mode, modeline) < 0) {
+               weston_log("Cannot configure output using weston_drm_output_api.\n");
+               free(modeline);
+               return -1;
+       }
+       free(modeline);
+
+       ivi->drm_api->set_gbm_format(output->output, gbm_format);
+       free(gbm_format);
+
+       ivi->drm_api->set_seat(output->output, seat);
+       free(seat);
+
+       return 0;
 }
 
 /*
@@ -368,7 +511,7 @@ try_attach_heads(struct ivi_output *output)
 {
        size_t fail_len = 0;
 
-       for (size_t i = 0; i < output->add_len; ++i) {
+       for (size_t i = 1; i < output->add_len; i++) {
                if (weston_output_attach_head(output->output, output->add[i]) < 0) {
                        struct weston_head *tmp = output->add[i];
                        memmove(&output->add[fail_len + 1], output->add[fail_len],
@@ -376,10 +519,42 @@ try_attach_heads(struct ivi_output *output)
                        output->add[fail_len++] = tmp;
                }
        }
-
        return fail_len;
 }
 
+/* Place output exactly to the right of the most recently enabled output.
+ *
+ * Historically, we haven't given much thought to output placement,
+ * simply adding outputs in a horizontal line as they're enabled. This
+ * function simply sets an output's x coordinate to the right of the
+ * most recently enabled output, and its y to zero.
+ *
+ * If you're adding new calls to this function, you're also not giving
+ * much thought to output placement, so please consider carefully if
+ * it's really doing what you want.
+ *
+ * You especially don't want to use this for any code that won't
+ * immediately enable the passed output.
+ */
+static void
+weston_output_lazy_align(struct weston_output *output)
+{
+       struct weston_compositor *c;
+       struct weston_output *peer;
+       int next_x = 0;
+
+       /* Put this output to the right of the most recently enabled output */
+       c = output->compositor;
+       if (!wl_list_empty(&c->output_list)) {
+               peer = container_of(c->output_list.prev,
+                               struct weston_output, link);
+               next_x = peer->pos.c.x + peer->width;
+       }
+       output->pos.c.x = next_x;
+       output->pos.c.y = 0;
+}
+
+
 /*
  * Like try_attach_heads, this reorganizes the output's add array into a failed
  * and successful section.
@@ -391,6 +566,8 @@ try_enable_output(struct ivi_output *output, size_t i)
        for (; i < output->add_len; ++i) {
                struct weston_head *head;
 
+               weston_output_lazy_align(output->output);
+
                if (weston_output_enable(output->output) == 0)
                        break;
 
@@ -413,7 +590,7 @@ try_attach_enable_heads(struct ivi_output *output)
 
        fail_len = try_attach_heads(output);
 
-       if (configure_output(output) < 0)
+       if (drm_configure_output(output) < 0)
                return -1;
 
        fail_len = try_enable_output(output, fail_len);
@@ -426,6 +603,8 @@ try_attach_enable_heads(struct ivi_output *output)
        for (size_t i = fail_len; i < output->add_len; ++i)
                add_head_destroyed_listener(output->add[i]);
 
+       ivi_layout_restore(output->ivi, output);
+
        output->add_len = fail_len;
        return 0;
 }
@@ -442,7 +621,7 @@ process_output(struct ivi_output *output)
 }
 
 static void
-head_disable(struct ivi_compositor *ivi, struct weston_head *head)
+drm_head_disable(struct ivi_compositor *ivi, struct weston_head *head)
 {
        struct weston_output *output;
        struct ivi_output *ivi_output;
@@ -460,8 +639,22 @@ 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->app_ids);
+       free(ivi_output);
 }
 
 static struct weston_config_section *
@@ -498,13 +691,14 @@ find_controlling_output_config(struct weston_config *config,
 }
 
 static void
-head_prepare_enable(struct ivi_compositor *ivi, struct weston_head *head)
+drm_head_prepare_enable(struct ivi_compositor *ivi, struct weston_head *head)
 {
        const char *name = weston_head_get_name(head);
        struct weston_config_section *section;
        struct ivi_output *output;
        char *output_name = NULL;
 
+
        section = find_controlling_output_config(ivi->config, name);
        if (section) {
                char *mode;
@@ -525,7 +719,7 @@ head_prepare_enable(struct ivi_compositor *ivi, struct weston_head *head)
        if (!output_name)
                return;
 
-       output = ivi_ensure_output(ivi, output_name, section);
+       output = ivi_ensure_output(ivi, output_name, section, head);
        if (!output)
                return;
 
@@ -536,7 +730,7 @@ head_prepare_enable(struct ivi_compositor *ivi, struct weston_head *head)
 }
 
 static void
-heads_changed(struct wl_listener *listener, void *arg)
+drm_heads_changed(struct wl_listener *listener, void *arg)
 {
        struct weston_compositor *compositor = arg;
        struct weston_head *head = NULL;
@@ -550,9 +744,9 @@ heads_changed(struct wl_listener *listener, void *arg)
                bool non_desktop = weston_head_is_non_desktop(head);
 
                if (connected && !enabled && !non_desktop)
-                       head_prepare_enable(ivi, head);
+                       drm_head_prepare_enable(ivi, head);
                else if (!connected && enabled)
-                       head_disable(ivi, head);
+                       drm_head_disable(ivi, head);
                else if (enabled && changed)
                        weston_log("Detected a monitor change on head '%s', "
                                   "not bothering to do anything about it.\n",
@@ -572,37 +766,89 @@ heads_changed(struct wl_listener *listener, void *arg)
        }
 }
 
-#ifdef HAVE_WALTHAM
-static int
-load_waltham_plugin(struct ivi_compositor *ivi, struct weston_config *config)
+static void
+simple_head_enable(struct ivi_compositor *ivi, struct weston_head *head)
 {
-       struct weston_compositor *compositor = ivi->compositor;
-       int (*module_init)(struct weston_compositor *wc);
+       struct ivi_output *output;
+       struct weston_config_section *section;
+       char *output_name = NULL;
+       const char *name = weston_head_get_name(head);
 
-       module_init = weston_load_module("waltham-transmitter.so",
-                                        "wet_module_init");
-       if (!module_init)
-               return -1;
+       section = find_controlling_output_config(ivi->config, name);
+       if (section) {
+               char *mode;
 
-       if (module_init(compositor) < 0)
-               return -1;
+               weston_config_section_get_string(section, "mode", &mode, NULL);
+               if (mode && strcmp(mode, "off") == 0) {
+                       free(mode);
+                       return;
+               }
+               free(mode);
 
-       ivi->waltham_transmitter_api = weston_get_transmitter_api(compositor);
-       if (!ivi->waltham_transmitter_api) {
-               weston_log("Failed to load waltham-transmitter plugin.\n");
-               return -1;
+               weston_config_section_get_string(section, "name",
+                                                &output_name, NULL);
+       } else {
+               output_name = strdup(name);
        }
 
-       weston_log("waltham-transmitter plug-in loaded\n");
-       return 0;
+       if (!output_name)
+               return;
+
+       output = ivi_ensure_output(ivi, output_name, section, head);
+       if (!output) {
+               weston_log("Failed to create output %s\n", output_name);
+               return;
+       }
+
+       add_head_destroyed_listener(head);
 }
-#else
-static int
-load_waltham_plugin(struct ivi_compositor *ivi, struct weston_config *config)
+
+
+static void
+simple_head_disable(struct weston_head *head)
 {
-       return -1;
+       struct weston_output *output;
+       struct wl_listener *listener;
+
+       listener = weston_head_get_destroy_listener(head, handle_head_destroy);
+       wl_list_empty(&listener->link);
+
+       output = weston_head_get_output(head);
+       assert(output);
+       weston_output_destroy(output);
+}
+
+
+static void
+simple_heads_changed(struct wl_listener *listener, void *arg)
+{
+       struct weston_compositor *compositor = arg;
+       struct ivi_compositor *ivi = to_ivi_compositor(compositor);
+       struct weston_head *head = NULL;
+       bool connected;
+       bool enabled;
+       bool changed;
+       bool non_desktop;
+
+       while ((head = weston_compositor_iterate_heads(ivi->compositor, head))) {
+               connected = weston_head_is_connected(head);
+               enabled = weston_head_is_enabled(head);
+               changed = weston_head_is_device_changed(head);
+               non_desktop = weston_head_is_non_desktop(head);
+
+               if (connected && !enabled && !non_desktop) {
+                       simple_head_enable(ivi, head);
+               } else if (!connected && enabled) {
+                       simple_head_disable(head);
+               } else if (enabled && changed) {
+                       weston_log("Detected a monitor change on head '%s', "
+                                       "not bothering to do anything about it.\n",
+                                       weston_head_get_name(head));
+               }
+               weston_head_reset_device_changed(head);
+       }
 }
-#endif
+
 
 #ifdef HAVE_REMOTING
 static int
@@ -703,7 +949,7 @@ remote_output_init(struct ivi_output *ivi_output,
                goto err;
        }
 
-       if (weston_output_enable(ivi_output->output) < 0) {
+       if ((ret = weston_output_enable(ivi_output->output)) < 0) {
                weston_log("Enabling remoted output \"%s\" failed.\n",
                                output_name);
                goto err;
@@ -780,62 +1026,6 @@ ivi_enable_remote_outputs(struct ivi_compositor *ivi)
        }
 }
 
-static void
-ivi_enable_waltham_outputs(struct ivi_compositor *ivi)
-{
-       struct weston_config_section *transmitter_section = NULL;
-       const char *sect_name;
-       struct weston_config *config = ivi->config;
-
-       while (weston_config_next_section(config, &transmitter_section, &sect_name)) {
-               if (strcmp(sect_name, "transmitter-output"))
-                       continue;
-
-               struct ivi_output *ivi_output = NULL;
-               bool output_found = false;
-               char *_name = NULL;
-
-               weston_config_section_get_string(transmitter_section,
-                               "name", &_name, NULL);
-               wl_list_for_each(ivi_output, &ivi->outputs, link) {
-                       if (!strcmp(ivi_output->name, _name)) {
-                               output_found = true;
-                               break;
-                       }
-               }
-
-               if (output_found) {
-                       free(_name);
-                       continue;
-               }
-
-               ivi_output = zalloc(sizeof(*ivi_output));
-               if (!ivi_output) {
-                       free(_name);
-                       continue;
-               }
-
-               ivi_output->ivi = ivi;
-               ivi_output->name = _name;
-               ivi_output->config = transmitter_section;
-
-               if (remote_output_init(ivi_output, ivi->compositor,
-                                       transmitter_section, ivi->remoting_api)) {
-                       free(ivi_output->name);
-                       free(ivi_output);
-                       continue;
-               }
-
-               ivi_output->type = OUTPUT_WALTHAM;
-               ivi_output->output_destroy.notify = handle_output_destroy;
-               weston_output_add_destroy_listener(ivi_output->output,
-                               &ivi_output->output_destroy);
-
-               wl_list_insert(&ivi->outputs, &ivi_output->link);
-               ivi_output_configure_app_id(ivi_output);
-       }
-}
-
 static int
 load_remoting_plugin(struct ivi_compositor *ivi, struct weston_config *config)
 {
@@ -843,7 +1033,8 @@ load_remoting_plugin(struct ivi_compositor *ivi, struct weston_config *config)
        int (*module_init)(struct weston_compositor *wc);
 
        module_init = weston_load_module("remoting-plugin.so",
-                                        "weston_module_init");
+                                        "weston_module_init",
+                                        LIBWESTON_MODULEDIR);
        if (!module_init)
                return -1;
 
@@ -864,7 +1055,8 @@ load_remoting_plugin(struct weston_compositor *compositor, struct weston_config
 #endif
 
 static int
-load_drm_backend(struct ivi_compositor *ivi, int *argc, char *argv[])
+load_drm_backend(struct ivi_compositor *ivi, int *argc, char *argv[],
+                enum weston_renderer_type renderer)
 {
        struct weston_drm_backend_config config = {
                .base = {
@@ -874,20 +1066,25 @@ load_drm_backend(struct ivi_compositor *ivi, int *argc, char *argv[])
        };
        struct weston_config_section *section;
        int use_current_mode = 0;
-       int use_pixman = 0;
+       bool force_pixman = false;
        bool use_shadow;
-       int ret;
+       bool without_input = false;
 
        const struct weston_option options[] = {
                { WESTON_OPTION_STRING, "seat", 0, &config.seat_id },
-               { WESTON_OPTION_INTEGER, "tty", 0, &config.tty },
                { WESTON_OPTION_STRING, "drm-device", 0, &config.specific_device },
                { WESTON_OPTION_BOOLEAN, "current-mode", 0, &use_current_mode },
-               { WESTON_OPTION_BOOLEAN, "use-pixman", 0, &use_pixman },
+               { WESTON_OPTION_BOOLEAN, "use-pixman", 0, &force_pixman },
+               { WESTON_OPTION_BOOLEAN, "continue-without-input", false, &without_input }
        };
 
        parse_options(options, ARRAY_LENGTH(options), argc, argv);
-       config.use_pixman = use_pixman;
+
+       if (force_pixman)
+               config.renderer = WESTON_RENDERER_PIXMAN;
+       else
+               config.renderer = WESTON_RENDERER_AUTO;
+
        ivi->cmdline.use_current_mode = use_current_mode;
 
        section = weston_config_get_section(ivi->config, "core", NULL, NULL);
@@ -898,30 +1095,38 @@ load_drm_backend(struct ivi_compositor *ivi, int *argc, char *argv[])
        weston_config_section_get_bool(section, "pixman-shadow", &use_shadow, 1);
        config.use_pixman_shadow = use_shadow;
 
-       ret = weston_compositor_load_backend(ivi->compositor, WESTON_BACKEND_DRM,
-                                            &config.base);
-       if (ret < 0)
-               return ret;
+       if (without_input)
+               ivi->compositor->require_input = !without_input;
+
+       ivi->heads_changed.notify = drm_heads_changed;
+       weston_compositor_add_heads_changed_listener(ivi->compositor,
+                                                    &ivi->heads_changed);
+
+       if (!weston_compositor_load_backend(ivi->compositor, WESTON_BACKEND_DRM,
+                                           &config.base)) {
+               weston_log("Failed to load DRM backend\n");
+               return -1;
+       }
 
        ivi->drm_api = weston_drm_output_get_api(ivi->compositor);
        if (!ivi->drm_api) {
                weston_log("Cannot use drm output api.\n");
-               ret = -1;
                goto error;
        }
 
        load_remoting_plugin(ivi, ivi->config);
-       load_waltham_plugin(ivi, ivi->config);
+
+       return 0;
 
 error:
        free(config.gbm_format);
        free(config.seat_id);
-       return ret;
+       return -1;
 }
 
 static void
 windowed_parse_common_options(struct ivi_compositor *ivi, int *argc, char *argv[],
-                             bool *use_pixman, bool *fullscreen, int *output_count)
+                             bool *force_pixman, bool *fullscreen, int *output_count)
 {
        struct weston_config_section *section;
        bool pixman;
@@ -931,17 +1136,17 @@ windowed_parse_common_options(struct ivi_compositor *ivi, int *argc, char *argv[
                { WESTON_OPTION_INTEGER, "width", 0, &ivi->cmdline.width },
                { WESTON_OPTION_INTEGER, "height", 0, &ivi->cmdline.height },
                { WESTON_OPTION_INTEGER, "scale", 0, &ivi->cmdline.scale },
-               { WESTON_OPTION_BOOLEAN, "use-pixman", 0, &pixman },
+               { WESTON_OPTION_BOOLEAN, "use-pixman", 0, &force_pixman },
                { WESTON_OPTION_BOOLEAN, "fullscreen", 0, &fs },
                { WESTON_OPTION_INTEGER, "output-count", 0, output_count },
        };
 
        section = weston_config_get_section(ivi->config, "core", NULL, NULL);
-       weston_config_section_get_bool(section, "use-pixman", &pixman, 0);
+       weston_config_section_get_bool(section, "use-pixman", &pixman, false);
 
        *output_count = 1;
        parse_options(options, ARRAY_LENGTH(options), argc, argv);
-       *use_pixman = pixman;
+       *force_pixman = pixman;
        *fullscreen = fs;
 }
 
@@ -972,7 +1177,7 @@ windowed_create_outputs(struct ivi_compositor *ivi, int output_count,
                        continue;
                }
 
-               if (ivi->window_api->create_head(ivi->compositor, output_name) < 0) {
+               if (ivi->window_api->create_head(ivi->backend, output_name) < 0) {
                        free(output_name);
                        return -1;
                }
@@ -985,7 +1190,7 @@ windowed_create_outputs(struct ivi_compositor *ivi, int output_count,
                if (asprintf(&default_output, "%s%d", name_prefix, i) < 0)
                        return -1;
 
-               if (ivi->window_api->create_head(ivi->compositor, default_output) < 0) {
+               if (ivi->window_api->create_head(ivi->backend, default_output) < 0) {
                        free(default_output);
                        return -1;
                }
@@ -996,8 +1201,30 @@ windowed_create_outputs(struct ivi_compositor *ivi, int output_count,
        return 0;
 }
 
+
 static int
-load_wayland_backend(struct ivi_compositor *ivi, int *argc, char *argv[])
+wayland_backend_output_configure(struct weston_output *output)
+{
+       struct ivi_output *ivi_output = to_ivi_output(output);
+
+       struct ivi_output_config defaults = {
+               .width = WINDOWED_DEFAULT_WIDTH,
+               .height = WINDOWED_DEFAULT_HEIGHT,
+               .scale = 1,
+               .transform = WL_OUTPUT_TRANSFORM_NORMAL
+       };
+
+       if (!ivi_output) {
+               weston_log("Failed to configure and enable Wayland output. No ivi-output available!\n");
+               return -1;
+       }
+
+       return ivi_configure_windowed_output_from_config(to_ivi_output(output), &defaults);
+}
+
+static int
+load_wayland_backend(struct ivi_compositor *ivi, int *argc, char *argv[],
+                    enum weston_renderer_type renderer)
 {
        struct weston_wayland_backend_config config = {
                .base = {
@@ -1008,14 +1235,14 @@ load_wayland_backend(struct ivi_compositor *ivi, int *argc, char *argv[])
        struct weston_config_section *section;
        int sprawl = 0;
        int output_count;
-       int ret;
+       bool force_pixman = false;
 
        const struct weston_option options[] = {
                { WESTON_OPTION_STRING, "display", 0, &config.display_name },
                { WESTON_OPTION_STRING, "sprawl", 0, &sprawl },
        };
 
-       windowed_parse_common_options(ivi, argc, argv, &config.use_pixman,
+       windowed_parse_common_options(ivi, argc, argv, &force_pixman,
                                      &config.fullscreen, &output_count);
 
        parse_options(options, ARRAY_LENGTH(options), argc, argv);
@@ -1027,14 +1254,23 @@ load_wayland_backend(struct ivi_compositor *ivi, int *argc, char *argv[])
        weston_config_section_get_int(section, "cursor-size",
                                      &config.cursor_size, 32);
 
-       ret = weston_compositor_load_backend(ivi->compositor, WESTON_BACKEND_WAYLAND,
-                                            &config.base);
+       ivi->simple_output_configure = wayland_backend_output_configure;
+       ivi->heads_changed.notify = simple_heads_changed;
+       weston_compositor_add_heads_changed_listener(ivi->compositor,
+                                                    &ivi->heads_changed);
+
+       ivi->backend = weston_compositor_load_backend(ivi->compositor,
+                                                     WESTON_BACKEND_WAYLAND,
+                                                     &config.base);
+       if (!ivi->backend) {
+               weston_log("Failed to create Wayland backend!\n");
+       }
 
        free(config.cursor_theme);
        free(config.display_name);
 
-       if (ret < 0)
-               return ret;
+       if (!ivi->backend)
+               return -1;
 
        ivi->window_api = weston_windowed_output_get_api(ivi->compositor);
 
@@ -1053,7 +1289,29 @@ load_wayland_backend(struct ivi_compositor *ivi, int *argc, char *argv[])
 
 #ifdef HAVE_BACKEND_X11
 static int
-load_x11_backend(struct ivi_compositor *ivi, int *argc, char *argv[])
+x11_backend_output_configure(struct weston_output *output)
+{
+       struct ivi_output *ivi_output = to_ivi_output(output);
+
+       struct ivi_output_config defaults = {
+               .width = WINDOWED_DEFAULT_WIDTH,
+               .height = WINDOWED_DEFAULT_HEIGHT,
+               .scale = 1,
+               .transform = WL_OUTPUT_TRANSFORM_NORMAL
+       };
+
+       if (!ivi_output) {
+               weston_log("Failed to configure and enable X11 output. No ivi-output available!\n");
+               return -1;
+       }
+
+
+       return ivi_configure_windowed_output_from_config(ivi_output, &defaults);
+}
+
+static int
+load_x11_backend(struct ivi_compositor *ivi, int *argc, char *argv[],
+                enum weston_renderer_type renderer)
 {
        struct weston_x11_backend_config config = {
                .base = {
@@ -1063,23 +1321,35 @@ load_x11_backend(struct ivi_compositor *ivi, int *argc, char *argv[])
        };
        int no_input = 0;
        int output_count;
-       int ret;
+       bool force_pixman = false;
 
        const struct weston_option options[] = {
               { WESTON_OPTION_BOOLEAN, "no-input", 0, &no_input },
        };
 
-       windowed_parse_common_options(ivi, argc, argv, &config.use_pixman,
+       windowed_parse_common_options(ivi, argc, argv, &force_pixman,
                                      &config.fullscreen, &output_count);
 
        parse_options(options, ARRAY_LENGTH(options), argc, argv);
+       if (force_pixman)
+               config.renderer = WESTON_RENDERER_PIXMAN;
+       else
+               config.renderer = WESTON_RENDERER_AUTO;
        config.no_input = no_input;
 
-       ret = weston_compositor_load_backend(ivi->compositor, WESTON_BACKEND_X11,
-                                            &config.base);
+       ivi->simple_output_configure = x11_backend_output_configure;
 
-       if (ret < 0)
-               return ret;
+       ivi->heads_changed.notify = simple_heads_changed;
+       weston_compositor_add_heads_changed_listener(ivi->compositor,
+                                                    &ivi->heads_changed);
+
+       ivi->backend = weston_compositor_load_backend(ivi->compositor,
+                                                     WESTON_BACKEND_X11,
+                                                     &config.base);
+       if (!ivi->backend) {
+               weston_log("Failed to create X11 backend!\n");
+               return -1;
+       }
 
        ivi->window_api = weston_windowed_output_get_api(ivi->compositor);
        if (!ivi->window_api) {
@@ -1091,84 +1361,204 @@ load_x11_backend(struct ivi_compositor *ivi, int *argc, char *argv[])
 }
 #else
 static int
-load_x11_backend(struct ivi_compositor *ivi, int *argc, char *argv[])
+load_x11_backend(struct ivi_compositor *ivi, int *argc, char **argv,
+                enum weston_renderer_type renderer)
 {
        return -1;
 }
 #endif
 
-#ifdef HAVE_BACKEND_HEADLESS
+#ifdef HAVE_BACKEND_RDP
+static void
+weston_rdp_backend_config_init(struct weston_rdp_backend_config *config)
+{
+        config->base.struct_version = WESTON_RDP_BACKEND_CONFIG_VERSION;
+        config->base.struct_size = sizeof(struct weston_rdp_backend_config);
+
+        config->renderer = WESTON_RENDERER_AUTO;
+        config->bind_address = NULL;
+        config->port = 3389;
+        config->rdp_key = NULL;
+        config->server_cert = NULL;
+        config->server_key = NULL;
+        config->env_socket = 0;
+        config->external_listener_fd = -1;
+        config->no_clients_resize = 0;
+        config->force_no_compression = 0;
+        config->remotefx_codec = true;
+        config->refresh_rate = RDP_DEFAULT_FREQ;
+
+}
+
 static int
-load_headless_backend(struct ivi_compositor *ivi, int *argc, char **argv)
+rdp_backend_output_configure(struct weston_output *output)
 {
-       struct weston_headless_backend_config config = {};
-       int ret = 0;
+       struct ivi_compositor *ivi = to_ivi_compositor(output->compositor);
+       struct ivi_output_config *parsed_options = ivi->parsed_options;
+       const struct weston_rdp_output_api *api =
+               weston_rdp_output_get_api(output->compositor);
+       int width = 640;
+       int height = 480;
+       struct weston_config_section *section;
+       uint32_t transform = WL_OUTPUT_TRANSFORM_NORMAL;
+       char *transform_string;
+       struct weston_mode new_mode = {};
 
-       bool use_pixman;
-       bool fullscreen;
-       bool use_gl;
-       int output_count;
+       assert(parsed_options);
 
-       struct weston_compositor *c = ivi->compositor;
+       if (!api) {
+               weston_log("Cannot use weston_rdp_output_api.\n");
+               return -1;
+       }
 
-       const struct weston_option options[] = {
-               { WESTON_OPTION_BOOLEAN, "use-pixman", 0, &use_pixman },
-               { WESTON_OPTION_BOOLEAN, "use-gl", 0, &use_gl },
-       };
+       section = weston_config_get_section(ivi->config, "rdp", NULL, NULL);
 
-       windowed_parse_common_options(ivi, argc, argv, &use_pixman,
-                       &fullscreen, &output_count);
+       if (parsed_options->width)
+               width = parsed_options->width;
 
-       parse_options(options, ARRAY_LENGTH(options), argc, argv);
-       config.use_pixman = use_pixman;
-       config.use_gl = use_gl;
+       if (parsed_options->height)
+               height = parsed_options->height;
 
-       config.base.struct_version = WESTON_HEADLESS_BACKEND_CONFIG_VERSION;
-       config.base.struct_size = sizeof(struct weston_headless_backend_config);
+       weston_output_set_scale(output, 1);
 
-       /* load the actual headless-backend and configure it */
-       ret = weston_compositor_load_backend(c, WESTON_BACKEND_HEADLESS,
-                                            &config.base);
-       if (ret < 0)
-               return ret;
+       weston_config_section_get_int(section, "width",
+                       &width, width);
 
-       ivi->window_api = weston_windowed_output_get_api(c);
-       if (!ivi->window_api) {
-               weston_log("Cannot use weston_windowed_output_api.\n");
+       weston_config_section_get_int(section, "height",
+                       &height, height);
+
+       if (parsed_options->transform)
+               transform = parsed_options->transform;
+
+       weston_config_section_get_string(section, "transform",
+                       &transform_string, "normal");
+
+       if (parse_transform(transform_string, &transform) < 0) {
+               weston_log("Invalid transform \"%s\" for output %s\n",
+                          transform_string, output->name);
                return -1;
        }
 
-       if (ivi->window_api->create_head(c, "headless") < 0) {
-               weston_log("Cannot create headless back-end\n");
+
+       new_mode.width = width;
+       new_mode.height = height;
+
+       weston_log("Setting modeline to %dx%d\n", width, height);
+
+       api->output_set_mode(output, &new_mode);
+       weston_output_set_transform(output, transform);
+
+       return 0;
+}
+
+static int
+load_rdp_backend(struct ivi_compositor *ivi, int *argc, char **argv,
+                enum weston_renderer_type renderer)
+{
+       struct weston_rdp_backend_config config = {};
+       struct weston_config_section *section;
+       bool no_remotefx_codec = false;
+
+       struct ivi_output_config *parsed_options = ivi_init_parsed_options(ivi->compositor);
+       if (!parsed_options)
+               return -1;
+
+       weston_rdp_backend_config_init(&config);
+
+       const struct weston_option rdp_options[] = {
+               { WESTON_OPTION_BOOLEAN, "env-socket", 0, &config.env_socket },
+               { WESTON_OPTION_INTEGER, "external-listener-fd", 0, &config.external_listener_fd },
+               { WESTON_OPTION_INTEGER, "width", 0, &parsed_options->width },
+               { WESTON_OPTION_INTEGER, "height", 0, &parsed_options->height },
+               { WESTON_OPTION_STRING,  "address", 0, &config.bind_address },
+               { WESTON_OPTION_INTEGER, "port", 0, &config.port },
+               { WESTON_OPTION_BOOLEAN, "no-clients-resize", 0, &config.no_clients_resize },
+               { WESTON_OPTION_STRING,  "rdp4-key", 0, &config.rdp_key },
+               { WESTON_OPTION_STRING,  "rdp-tls-cert", 0, &config.server_cert },
+               { WESTON_OPTION_STRING,  "rdp-tls-key", 0, &config.server_key },
+               { WESTON_OPTION_INTEGER, "scale", 0, &parsed_options->scale },
+               { WESTON_OPTION_BOOLEAN, "force-no-compression", 0, &config.force_no_compression },
+               { WESTON_OPTION_BOOLEAN, "no-remotefx-codec", 0, &no_remotefx_codec },
+       };
+
+       config.remotefx_codec = !no_remotefx_codec;
+       config.renderer = renderer;
+
+       section = weston_config_get_section(ivi->config, "rdp", NULL, NULL);
+
+       weston_config_section_get_int(section, "refresh-rate",
+                       &config.refresh_rate, RDP_DEFAULT_FREQ);
+
+       weston_config_section_get_string(section, "tls-cert",
+                       &config.server_cert, config.server_cert);
+
+       weston_config_section_get_string(section, "tls-key",
+                       &config.server_key, config.server_key);
+
+
+       parse_options(rdp_options, ARRAY_LENGTH(rdp_options), argc, argv);
+       weston_log("No clients resize: %d\n", config.no_clients_resize);
+
+       ivi->simple_output_configure = rdp_backend_output_configure;
+
+       ivi->heads_changed.notify = simple_heads_changed;
+       weston_compositor_add_heads_changed_listener(ivi->compositor,
+                                                    &ivi->heads_changed);
+
+       if (!weston_compositor_load_backend(ivi->compositor,
+                                           WESTON_BACKEND_RDP, &config.base)) {
+               weston_log("Failed to create RDP backend\n");
                return -1;
        }
 
+       free(config.bind_address);
+       free(config.rdp_key);
+       free(config.server_cert);
+       free(config.server_key);
+
        return 0;
 }
 #else
 static int
-load_headless_backend(struct ivi_compositor *ivi, int *argc, char **argv)
+load_rdp_backend(struct ivi_compositor *ivi, int *argc, char **argv,
+                enum weston_renderer_type renderer)
 {
        return -1;
 }
 #endif
 
+
 static int
-load_backend(struct ivi_compositor *ivi, const char *backend,
-            int *argc, char *argv[])
+load_backend(struct ivi_compositor *ivi, int *argc, char **argv,
+            const char *backend_name, const char *renderer_name)
 {
-       if (strcmp(backend, "drm-backend.so") == 0) {
-               return load_drm_backend(ivi, argc, argv);
-       } else if (strcmp(backend, "wayland-backend.so") == 0) {
-               return load_wayland_backend(ivi, argc, argv);
-       } else if (strcmp(backend, "x11-backend.so") == 0) {
-               return load_x11_backend(ivi, argc, argv);
-       } else if (strcmp(backend, "headless-backend.so") == 0) {
-               return load_headless_backend(ivi, argc, argv);
+       enum weston_compositor_backend backend;
+       enum weston_renderer_type renderer;
+
+       if (!get_backend_from_string(backend_name, &backend)) {
+               weston_log("Error: unknown backend \"%s\"\n", backend_name);
+               return -1;
        }
 
-       weston_log("fatal: unknown backend '%s'.\n", backend);
-       return -1;
+       if (!get_renderer_from_string(renderer_name, &renderer)) {
+               weston_log("Error: unknown renderer \"%s\"\n", renderer_name);
+               return -1;
+       }
+
+       switch (backend) {
+       case WESTON_BACKEND_DRM:
+               return load_drm_backend(ivi, argc, argv, renderer);
+       case WESTON_BACKEND_RDP:
+               return load_rdp_backend(ivi, argc, argv, renderer);
+       case WESTON_BACKEND_WAYLAND:
+               return load_wayland_backend(ivi, argc, argv, renderer);
+       case WESTON_BACKEND_X11:
+               return load_x11_backend(ivi, argc, argv, renderer);
+       default:
+               assert(!"unknown backend type in load_backend()");
+       }
+
+       return 0;
 }
 
 static int
@@ -1188,27 +1578,18 @@ load_modules(struct ivi_compositor *ivi, const char *modules,
                snprintf(buffer, sizeof buffer, "%.*s", (int) (end - p), p);
 
                if (strstr(buffer, "xwayland.so")) {
-                       weston_log("Xwayland plug-in not supported!\n");
-                       p = end;
-                       while (*p == ',')
-                               p++;
-                       continue;
-               }
-
-               if (strstr(buffer, "systemd-notify.so")) {
+                       *xwayland = true;
+               } else if (strstr(buffer, "systemd-notify.so")) {
                        weston_log("systemd-notify plug-in already loaded!\n");
-                       p = end;
-                       while (*p == ',')
-                               p++;
-                       continue;
-               }
+               } else {
+                       module_init = weston_load_module(buffer, "wet_module_init", WESTON_MODULEDIR);
+                       if (!module_init)
+                               return -1;
 
-               module_init = weston_load_module(buffer, "wet_module_init");
-               if (!module_init)
-                       return -1;
+                       if (module_init(ivi->compositor, *argc, argv) < 0)
+                               return -1;
 
-               if (module_init(ivi->compositor, *argc, argv) < 0)
-                       return -1;
+               }
 
                p = end;
                while (*p == ',')
@@ -1235,11 +1616,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;
@@ -1272,6 +1654,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;
 
@@ -1303,18 +1690,25 @@ to_ivi_surface(struct weston_surface *surface)
 
 static void
 activate_binding(struct weston_seat *seat,
-                struct weston_view *focus_view)
+                struct weston_view *focus_view, uint32_t flags)
 {
-       struct weston_surface *focus = focus_view->surface;
-       struct weston_surface *main_surface =
-               weston_surface_get_main_surface(focus);
-       struct ivi_surface *surface;
+       struct weston_surface *focus_surface;
+       struct weston_surface *main_surface;
+       struct ivi_surface *ivi_surface;
+       struct ivi_shell_seat *ivi_seat = get_ivi_shell_seat(seat);
+
+       if (!focus_view)
+               return;
+
+       focus_surface = focus_view->surface;
+       main_surface = weston_surface_get_main_surface(focus_surface);
 
-       surface = to_ivi_surface(main_surface);
-       if (!surface)
+       ivi_surface = to_ivi_surface(main_surface);
+       if (!ivi_surface)
                return;
 
-       weston_seat_set_keyboard_focus(seat, focus);
+       if (ivi_seat)
+               ivi_shell_activate_surface(ivi_surface, ivi_seat, flags);
 }
 
 static void
@@ -1327,7 +1721,8 @@ click_to_activate_binding(struct weston_pointer *pointer,
        if (pointer->focus == NULL)
                return;
 
-       activate_binding(pointer->seat, pointer->focus);
+       activate_binding(pointer->seat, pointer->focus,
+                        WESTON_ACTIVATE_FLAG_CLICKED);
 }
 
 static void
@@ -1340,7 +1735,8 @@ touch_to_activate_binding(struct weston_touch *touch,
        if (touch->focus == NULL)
                return;
 
-       activate_binding(touch->seat, touch->focus);
+       activate_binding(touch->seat, touch->focus,
+                        WESTON_ACTIVATE_FLAG_NONE);
 }
 
 static void
@@ -1449,7 +1845,7 @@ log_timestamp(char *buf, size_t len)
 
        strftime(timestr, sizeof(timestr), "%H:%M:%S", brokendown_time);
        /* if datestr is empty it prints only timestr*/
-       snprintf(buf, len, "%s[%s.%03li]", datestr,
+       snprintf(buf, len, "%s[%s.%03"PRIi64"]", datestr,
                        timestr, (tv.tv_usec / 1000));
 
        return buf;
@@ -1558,6 +1954,7 @@ usage(int error_code)
                        "\t\t\t\twayland-backend.so\n"
                        "\t\t\t\tx11-backend.so\n"
                        "\t\t\t\theadless-backend.so\n"
+               "  -r, --renderer=NAME\tName of renderer to use: auto, gl, noop, pixman\n"
                "  -S, --socket=NAME\tName of socket to listen on\n"
                "  --log=FILE\t\tLog to the given file\n"
                "  -c, --config=FILE\tConfig file to load, defaults to agl-compositor.ini\n"
@@ -1588,8 +1985,46 @@ copy_command_line(int argc, char * const argv[])
        return str;
 }
 
+#if !defined(BUILD_XWAYLAND)
+void *
+wet_load_xwayland(struct weston_compositor *comp)
+{
+       weston_log("Attempted to load xwayland library but compositor "
+                  "was *not* built with xwayland support!\n");
+       return NULL;
+}
+#endif
+
+static void
+weston_log_setup_scopes(struct weston_log_context *log_ctx,
+                       struct weston_log_subscriber *subscriber,
+                       const char *names)
+{
+       assert(log_ctx);
+       assert(subscriber);
+
+       char *tokenize = strdup(names);
+       char *token = strtok(tokenize, ",");
+       while (token) {
+               weston_log_subscribe(log_ctx, subscriber, token);
+               token = strtok(NULL, ",");
+       }
+       free(tokenize);
+}
+
+static void
+weston_log_subscribe_to_scopes(struct weston_log_context *log_ctx,
+                               struct weston_log_subscriber *logger,
+                               const char *debug_scopes)
+{
+        if (logger && debug_scopes)
+                weston_log_setup_scopes(log_ctx, logger, debug_scopes);
+        else
+                weston_log_subscribe(log_ctx, logger, "log");
+}
+
 WL_EXPORT
-int wet_main(int argc, char *argv[])
+int wet_main(int argc, char *argv[], const struct weston_testsuite_data *test_data)
 {
        struct ivi_compositor ivi = { 0 };
        char *cmdline;
@@ -1603,17 +2038,23 @@ int wet_main(int argc, char *argv[])
        char *log = NULL;
        char *modules = NULL;
        char *option_modules = NULL;
+       char *debug_scopes = NULL;
        int help = 0;
        int version = 0;
        int no_config = 0;
        int debug = 0;
+       bool list_debug_scopes = false;
        char *config_file = NULL;
        struct weston_log_context *log_ctx = NULL;
        struct weston_log_subscriber *logger;
        int ret = EXIT_FAILURE;
        bool xwayland = false;
+       struct sigaction action;
+       char *renderer = NULL;
+       struct wet_process *process, *process_tmp;
 
        const struct weston_option core_options[] = {
+               { WESTON_OPTION_STRING, "renderer", 'r', &renderer },
                { WESTON_OPTION_STRING, "backend", 'B', &backend },
                { WESTON_OPTION_STRING, "socket", 'S', &socket_name },
                { WESTON_OPTION_STRING, "log", 0, &log },
@@ -1623,9 +2064,12 @@ int wet_main(int argc, char *argv[])
                { WESTON_OPTION_BOOLEAN, "debug", 0, &debug },
                { WESTON_OPTION_STRING, "config", 'c', &config_file },
                { WESTON_OPTION_STRING, "modules", 0, &option_modules },
+               { WESTON_OPTION_STRING, "debug-scopes", 'l', &debug_scopes },
+               { WESTON_OPTION_STRING, "list-debug-scopes", 'L', &list_debug_scopes },
        };
 
        wl_list_init(&ivi.outputs);
+       wl_list_init(&ivi.saved_outputs);
        wl_list_init(&ivi.surfaces);
        wl_list_init(&ivi.pending_surfaces);
        wl_list_init(&ivi.popup_pending_apps);
@@ -1633,6 +2077,8 @@ int wet_main(int argc, char *argv[])
        wl_list_init(&ivi.split_pending_apps);
        wl_list_init(&ivi.remote_pending_apps);
        wl_list_init(&ivi.desktop_clients);
+       wl_list_init(&ivi.child_process_list);
+       wl_list_init(&ivi.pending_apps);
 
        /* Prevent any clients we spawn getting our stdin */
        os_fd_set_cloexec(STDIN_FILENO);
@@ -1645,24 +2091,25 @@ int wet_main(int argc, char *argv[])
 
        if (version) {
                printf(PACKAGE_STRING "\n");
-               return EXIT_SUCCESS;
+               ret = EXIT_SUCCESS;
+               goto exit_signals;
        }
 
-       log_ctx = weston_log_ctx_compositor_create();
+       log_ctx = weston_log_ctx_create();
        if (!log_ctx) {
                fprintf(stderr, "Failed to initialize weston debug framework.\n");
-               return ret;
+               goto exit_signals;
        }
 
-        log_scope = weston_compositor_add_log_scope(log_ctx, "log",
-                                                   "agl-compositor log\n",
-                                                   NULL, NULL, NULL);
+        log_scope = weston_log_ctx_add_log_scope(log_ctx, "log",
+                                                "agl-compositor log\n",
+                                                NULL, NULL, NULL);
 
        log_file_open(log);
        weston_log_set_handler(vlog, vlog_continue);
 
        logger = weston_log_subscriber_create_log(logfile);
-       weston_log_subscribe(log_ctx, logger, "log");
+       weston_log_subscribe_to_scopes(log_ctx, logger, debug_scopes);
 
        weston_log("Command line: %s\n", cmdline);
        free(cmdline);
@@ -1676,9 +2123,6 @@ int wet_main(int argc, char *argv[])
                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);
@@ -1690,32 +2134,48 @@ int wet_main(int argc, char *argv[])
 
        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, SIGQUIT, on_term_signal,
+       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;
 
-       ivi.compositor = weston_compositor_create(display, log_ctx, &ivi);
+       ivi.compositor = weston_compositor_create(display, log_ctx, &ivi, test_data);
        if (!ivi.compositor) {
                weston_log("fatal: failed to create compositor.\n");
                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) {
+       if (load_backend(&ivi, &argc, argv, backend, renderer) < 0) {
                weston_log("fatal: failed to create compositor backend.\n");
                goto error_compositor;
        }
 
-       ivi.heads_changed.notify = heads_changed;
-       weston_compositor_add_heads_changed_listener(ivi.compositor,
-                                                    &ivi.heads_changed);
+       if (weston_compositor_backends_loaded(ivi.compositor) < 0)
+                goto error_compositor;
+
+       weston_compositor_flush_heads_changed(ivi.compositor);
 
        if (ivi_desktop_init(&ivi) < 0)
                goto error_compositor;
@@ -1730,22 +2190,44 @@ int wet_main(int argc, char *argv[])
        if (load_modules(&ivi, option_modules, &argc, argv, &xwayland) < 0)
                goto error_compositor;
 
+       if (!xwayland) {
+               weston_config_section_get_bool(section, "xwayland", &xwayland,
+                                              false);
+       }
+
+       if (ivi_shell_init(&ivi) < 0)
+               goto error_compositor;
+
+       if (xwayland) {
+               if (!wet_load_xwayland(ivi.compositor))
+                       goto error_compositor;
+       }
+
        if (ivi_policy_init(&ivi) < 0)
                goto error_compositor;
 
-       if (ivi_shell_init(&ivi) < 0)
+
+       if (list_debug_scopes) {
+               struct weston_log_scope *nscope = NULL;
+
+               weston_log("Printing available debug scopes:\n");
+
+               while ((nscope = weston_log_scopes_iterate(log_ctx, nscope))) {
+                       weston_log("\tscope name: %s, desc: %s",
+                                       weston_log_scope_get_name(nscope),
+                                       weston_log_scope_get_description(nscope));
+               }
+
+               weston_log("\n");
+
                goto error_compositor;
+       }
 
        add_bindings(ivi.compositor);
 
-       weston_compositor_flush_heads_changed(ivi.compositor);
-
        if (ivi.remoting_api)
                ivi_enable_remote_outputs(&ivi);
 
-       if (ivi.waltham_transmitter_api)
-               ivi_enable_waltham_outputs(&ivi);
-
        if (create_listening_socket(display, socket_name) < 0)
                goto error_compositor;
 
@@ -1756,9 +2238,17 @@ int wet_main(int argc, char *argv[])
        weston_compositor_wake(ivi.compositor);
 
        ivi_shell_create_global(&ivi);
-       ivi_launch_shell_client(&ivi);
-       if (debug)
-               ivi_screenshooter_create(&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) {
+               weston_compositor_add_screenshot_authority(ivi.compositor,
+                                          &ivi.screenshot_auth,
+                                          screenshot_allow_all);
+       }
        ivi_agl_systemd_notify(&ivi);
 
        wl_display_run(display);
@@ -1768,18 +2258,24 @@ int wet_main(int argc, char *argv[])
        wl_display_destroy_clients(display);
 
 error_compositor:
-       weston_compositor_tear_down(ivi.compositor);
-
-       weston_compositor_log_scope_destroy(log_scope);
-       log_scope = NULL;
+       free(backend);
+       backend = NULL;
+       free(modules);
+       modules = NULL;
 
-       weston_log_ctx_compositor_destroy(ivi.compositor);
        weston_compositor_destroy(ivi.compositor);
 
-       weston_log_subscriber_destroy_log(logger);
+       weston_log_scope_destroy(log_scope);
+       log_scope = NULL;
+
+       weston_log_subscriber_destroy(logger);
+       weston_log_ctx_destroy(log_ctx);
 
        ivi_policy_destroy(ivi.policy);
 
+        wl_list_for_each_safe(process, process_tmp, &ivi.child_process_list, link)
+                ivi_process_destroy(process, 0, false);
+
 error_signals:
        for (size_t i = 0; i < ARRAY_LENGTH(signals); ++i)
                if (signals[i])
@@ -1791,5 +2287,10 @@ error_signals:
        if (ivi.config)
                weston_config_destroy(ivi.config);
 
+exit_signals:
+       free(log);
+       free(config_file);
+       free(socket_name);
+       free(option_modules);
        return ret;
 }