[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 19/24] console: add and use qemu_display_find_de
From: |
Gerd Hoffmann |
Subject: |
Re: [Qemu-devel] [PATCH 19/24] console: add and use qemu_display_find_default |
Date: |
Fri, 17 Nov 2017 14:24:54 +0100 |
User-agent: |
NeoMutt/20171027 |
Hi,
> > - dpy.type = DISPLAY_TYPE_NONE;
> > + if (!qemu_display_find_default(&dpy)) {
> > + dpy.type = DISPLAY_TYPE_NONE;
> > +#if defined(CONFIG_VNC)
> > + vnc_parse("localhost:0,to=99,id=default", &error_abort);
> > #endif
>
> Mostly some questions on this:
>
> - I'm curious, why is VNC not just one of the ones searched for in
> qemu_display_find_default()?
vnc is a bit different. It's a remote display protocol, not a (local)
user interface. There is no DISPLAY_TYPE_VNC for that reason (without
this series too). And IMO it was a bad idea to add -display vnc=$config
(same as -vnc $config) in the first place.
There can be only one user interface instance, i.e. you can't have gtk
and sdl at the same time. With vnc (and spice) that works though, i.e.
you can start qemu with some local user interface and enable vnc or
spice (even both) remote access at the same time.
> - What if there is another display type added? Would it be added to
> qemu_display_find_default() or in this if statement?
qemu_display_find_default() most likely,
but depends on what exactly it is.
> - Is there a reason that VNC doesn't follow the same registration
> mechanism you're proposing here (as in you've no changes to that
> module in this patch set).
See above ;)
> - In the spirit of encapsulation, would it not make sense for the
> vnc_parse() to be in the VNC code itself as a fall back if no
> specific options are given to the -vnc option?
It's not that simple. The default vnc server is started only in case
no (graphical) local user interface is available, so it's not something
the vnc code can figure purely on its own.
cheers,
Gerd
- [Qemu-devel] [PATCH 04/24] vl: rename DisplayType to LegacyDisplayType, (continued)
- [Qemu-devel] [PATCH 04/24] vl: rename DisplayType to LegacyDisplayType, Gerd Hoffmann, 2017/11/17
- [Qemu-devel] [PATCH 13/24] vl: drop request_opengl variable, Gerd Hoffmann, 2017/11/17
- [Qemu-devel] [PATCH 02/24] sdl: remove -alt-grab and -ctrl-grab support, Gerd Hoffmann, 2017/11/17
- [Qemu-devel] [PATCH 03/24] sdl: use ctrl-alt-g as grab hotkey, Gerd Hoffmann, 2017/11/17
- [Qemu-devel] [PATCH 09/24] curses: use DisplayOptions, Gerd Hoffmann, 2017/11/17
- [Qemu-devel] [PATCH 21/24] sdl: build as module, Gerd Hoffmann, 2017/11/17
- [Qemu-devel] [PATCH 19/24] console: add and use qemu_display_find_default, Gerd Hoffmann, 2017/11/17
[Qemu-devel] [PATCH 17/24] curses: hook up to display registry, Gerd Hoffmann, 2017/11/17
[Qemu-devel] [PATCH 18/24] egl-headless: hook up to display registry, Gerd Hoffmann, 2017/11/17
[Qemu-devel] [PATCH 01/24] sdl: remove -no-frame support, Gerd Hoffmann, 2017/11/17