[libvirt] [PATCH 2/4] Add a <graphics> type for SPICE protocol

[Dan Kenigsberg]

On Wed, Oct 28, 2009 at 07:57:05PM +0000, Daniel P. Berrange wrote:
> On Sun, Oct 25, 2009 at 10:54:19PM +0000, Daniel P. Berrange wrote:
> > On Sun, Oct 25, 2009 at 05:15:26PM +0200, Dan Kenigsberg wrote:
> > > Just stumpbled on another issue:
> > > 
> > > On Tue, Sep 29, 2009 at 04:43:50PM +0100, Daniel P. Berrange wrote:
> > > > @@ -366,6 +367,13 @@ struct _virDomainGraphicsDef {
> > > >              char *display;
> > > >              int fullscreen : 1;
> > > >          } desktop;
> > > > +        struct {
> > > > +            int port;
> > > > +            int tlsPort;
> > > > +            char *listenAddr;
> > > > +            char *keymap;
> > > > +            char *passwd;
> > > > +        } spice;
> > > >      } data;
> > > >  };
> > > 
> > > just like vnc, spice needs autoport here (as well as in the schema).
> > > 
> > > I wanted to ignore that, however, migration is impossibly unreliable
> > > without autoport=yes - the destination may have the port binded to
> > > something else.
> > 
> > Ok, i'll add that.
> 
> BTW, is there any recommended port range to use for SPICE    TCP and
> TLS ports ?  VNC for example starts at 5900 and searches upwards

currently I use 5900 upward for spice ports too, and 5900 downward for
tlsPort. (better keep it this way to save changes in iptables)