On Thu, 2012-02-09 at 10:58 +0000, Anthony PERARD wrote:
> On Thu, 9 Feb 2012, Ian Campbell wrote:
>
> > On Wed, 2012-02-08 at 17:24 +0000, Anthony PERARD wrote:
> > > On Wed, 8 Feb 2012, Ian Campbell wrote:
> > > > > @@ -287,13 +285,16 @@ static char ** libxl__build_device_model_args_new(libxl__gc *gc,
> > > > > }
> > > > >
> > > > > if (strchr(listen, ':') != NULL)
> > > > > - flexarray_append(dm_args,
> > > > > - libxl__sprintf(gc, "%s%s", listen,
> > > > > - info->vncunused ? ",to=99" : ""));
> > > > > + vncarg = libxl__sprintf(gc, "%s", listen);
> > > > > else
> > > > > - flexarray_append(dm_args,
> > > > > - libxl__sprintf(gc, "%s:%d%s", listen, display,
> > > > > - info->vncunused ? ",to=99" : ""));
> > > > > + vncarg = libxl__sprintf(gc, "%s:%d", listen, display);
> > > > > + if (info->vncpasswd && info->vncpasswd[0]) {
> > > > > + vncarg = libxl__sprintf(gc, "%s,password", vncarg);
> > > > > + }
> > > > > + if (info->vncunused) {
> > > > > + vncarg = libxl__sprintf(gc, "%s,to=99", vncarg);
> > > >
> > > > Not new, but I've been meaning to ask: what is to=99 for here? It seems
> > > > a bit arbitrary and the default behaviour without it appears to be to
> > > > keep looking for an available port which sounds like what we want.
> > >
> > > QEMU will search for an open port until it reach the vnc display port 99
> > > (5999 TCP port).
> > >
> > > For the 99, I probably read this number somewhere. But after checking in
> > > QEMU code, I do not see any limitation for this number. So we could
> > > probably change this number for the port number max (-5900, default vnc
> > > port).
> >
> > Can we not just omit it altogether and let qemu do its default thing?
>
> I did not find any default configuration for a VNC display in QEMU. We
> have to provide every option we want. The minimum is hostname:display,
> and QEMU will only tries to bind this address, or fails and quit. So to
> "simulated" -vncunused, we have to provide a ",to=i" options to try
> every single port between 5900 and 5900+i, and this option needs to be
> last :(.
