[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH] libxl: Set VNC password through QMP



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 :(.

I see, I didn't realise the upstream qemu was missing a "just find me a
port" option.

> I hope this respond to your question.

Yes, thanks.



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.