Skip to content

Commit

Permalink
ui: use "vc" chardev for dbus, gtk & spice-app
Browse files Browse the repository at this point in the history
Those display have their own implementation of "vc" chardev, which
doesn't use pixman. They also don't implement the width/height/cols/rows
options, so qemu_display_get_vc() should return a compatible argument.

This patch was meant to be with the pixman series, when the "vc" field
was introduced. It fixes a regression where VC are created on the
tty (or null) instead of the display own "vc" implementation.

Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>
Acked-by: Thomas Huth <thuth@redhat.com>
  • Loading branch information
elmarco committed Nov 21, 2023
1 parent 6956264 commit b7f1bb3
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 0 deletions.
1 change: 1 addition & 0 deletions ui/dbus.c
Original file line number Diff line number Diff line change
Expand Up @@ -518,6 +518,7 @@ static QemuDisplay qemu_display_dbus = {
.type = DISPLAY_TYPE_DBUS,
.early_init = early_dbus_init,
.init = dbus_init,
.vc = "vc",
};

static void register_dbus(void)
Expand Down
1 change: 1 addition & 0 deletions ui/gtk.c
Original file line number Diff line number Diff line change
Expand Up @@ -2534,6 +2534,7 @@ static QemuDisplay qemu_display_gtk = {
.type = DISPLAY_TYPE_GTK,
.early_init = early_gtk_display_init,
.init = gtk_display_init,
.vc = "vc",
};

static void register_gtk(void)
Expand Down
1 change: 1 addition & 0 deletions ui/spice-app.c
Original file line number Diff line number Diff line change
Expand Up @@ -220,6 +220,7 @@ static QemuDisplay qemu_display_spice_app = {
.type = DISPLAY_TYPE_SPICE_APP,
.early_init = spice_app_display_early_init,
.init = spice_app_display_init,
.vc = "vc",
};

static void register_spice_app(void)
Expand Down

0 comments on commit b7f1bb3

Please sign in to comment.