Re: [Qemu-devel] [PATCH 0/6] vnc: add support for multiple vnc server instances.

[Daniel P. Berrange]

On Wed, Oct 15, 2014 at 04:29:47PM +0200, Gerd Hoffmann wrote:
>   Hi,
> 
> > > To be done: monitor support.  I think the best way to handle this is to
> > > introduce a new command to query vnc server state, which returns a list
> > > of vnc servers but otherwise works like "query-vnc".  Alternative
> > > approach would be to add a optional 'id=' parameter to query-vnc, but
> > > then you'll need a new list-vnc command.
> > > 
> > > Opinions on this?
> > 
> > I don't think it makes a whole lot of difference really. Either way
> > mgmt apps like libvirt will need updating to deal with the new monitor
> > commands.
> 
> One thing that comes do mind is that if we do a completely new command
> we can easily fix any shortcomings of the existing query-vnc command.
> 
> The listening address could be changed to be a *list* of addresses,
> allowing to notify libvirt about multiple listening sockets.  One ipv4,
> one ipv6 being a use case.

That sounds like a good reason. I can even imagine in the future there
could be multiple IPv4 and multiple IPv6 addresses per VNC server.
eg if the host has many NICs present and we only want to listen on a
subset of them we might want to pass multiple IPv4 addresses to QEMU
on the CLI.

Regards,
Daniel
-- 
|: http://berrange.com      -o-    http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org              -o-             http://virt-manager.org :|
|: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org       -o-       http://live.gnome.org/gtk-vnc :|