On Wed, Jun 27, 2018 at 02:53:28PM +0100, Daniel P. Berrangé wrote:
On Wed, Jun 27, 2018 at 03:34:37PM +0200, Erik Skultety wrote:
> See the RFC here:
>
https://www.redhat.com/archives/libvir-list/2018-May/msg02218.html
>
> Since RFC:
> - split graphics 'gl' to a standalone structure since it's now SPICE,
SDL,
> *and* VNC that will support it
>
> - egl-headless support for VNC, since VNC doesn't support OpenGL natively like
> SPICE does
>
> - added a new attribute 'native' for spice which will instruct libvirt to
use
> '-display egl-headless' instead of libvirt trying to figure this on out by
> itself, since egl-headless might have other uses besides mdev with VNC
Can you elaborate on this part, as it feels a bit odd to me.
Currently -display accepts vnc or spice or sdl or gtk or none
Each of these -display options corresponds to a different <graphics type=XXX/>
element in libvirt XML, except 'none' which corresponds to no <graphics>
at
all.
We already allow multiple <graphics> elements to turn on say both SPICE and
VNC at once.
So why would we want to treat egl-headless differently as an attribute under
the SPICE graphics, as opposed to just having <graphics type=egl-headless>
as a thing in its own right.
Egl-headless doesn't make sense on its own, neither does it make sense with
anything else but SPICE and VNC. And since we're talking about dri render
nodes, I figured it would be better to design this as an attribute to the gl
element rather than a standalone element.
Erik