Changeset 5271e4c in mainline for uspace/srv/hid/display/display.c

Timestamp:

2020-06-22T12:20:42Z (4 years ago)

Author:

Jiri Svoboda <jiri@…>

Branches:

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

Children:

195b7b3

Parents:

66a408f7

git-author:

Jiri Svoboda <jiri@…> (2020-06-21 15:20:18)

git-committer:

Jiri Svoboda <jiri@…> (2020-06-22 12:20:42)

Message:

Duplicate rendering to additional output devices using a cloning GC

This gives the display server a pretty good illusion of rendering to just
one output device, while supporting multiple. This makes RFB work properly.

File:

• uspace/srv/hid/display/display.c (modified) (3 diffs)

uspace/srv/hid/display/display.c

@@ -42 +42 @@
 #include <stdlib.h>
 #include "client.h"
+#include "clonegc.h"
 #include "cursimg.h"
 #include "cursor.h"
@@ -505 +506 @@
                 disp->rect = ddev->info.rect;
 
+                /* Create cloning GC */
+                rc = ds_clonegc_create(ddev->gc, &disp->fbgc);
+                if (rc != EOK) {
+                        // XXX Remove output
+                        return ENOMEM;
+                }
+
                 /* Allocate backbuffer */
                 rc = ds_display_alloc_backbuf(disp);
+                if (rc != EOK) {
+                        // XXX Remove output
+                        // XXX Delete clone GC
+                        goto error;
+                }
+        } else {
+                /* Add new output device to cloning GC */
+                rc = ds_clonegc_add_output(disp->fbgc, ddev->gc);
                 if (rc != EOK)
                         goto error;
@@ -596 +612 @@
 static gfx_context_t *ds_display_get_unbuf_gc(ds_display_t *display)
 {
-        ds_ddev_t *ddev;
-
-        /*
-         * XXX To properly support multiple display devices, create
-         * a cloning GC that copies rendering operation to each output.
-         */
-        ddev = ds_display_first_ddev(display);
-        if (ddev == NULL)
-                return NULL;
-
-        return ddev->gc;
+        /* In case of unit tests */
+        if (display->fbgc == NULL)
+                return NULL;
+
+        return ds_clonegc_get_ctx(display->fbgc);
 }