Re: [Qemu-devel] [PATCH] add VNC reverse connections

2008-01-17 Thread Eddie Kohler
Well, attached is a patch with ",reverse" syntax, anyway; hopefully one of these syntaxes is OK! Eddie Anthony Liguori wrote: Daniel P. Berrange wrote: On Wed, Jan 16, 2008 at 07:09:03AM -0800, Eddie Kohler wrote: Daniel P. Berrange wrote: We already have the ability to pass multiple

Re: [Qemu-devel] [PATCH] add VNC reverse connections

2008-01-16 Thread Eddie Kohler
Anthony Liguori wrote: This doesn't feel like an option to me, though; rather a different means of connecting. Among other things, in "-vnc :0", the QEMU VNC server opens port 5900. But the client's listening port for reverse connections defaults to 5500. "-vnc :-400,rev" is clearly insane,

Re: [Qemu-devel] [PATCH] add VNC reverse connections

2008-01-16 Thread Anthony Liguori
Daniel P. Berrange wrote: On Wed, Jan 16, 2008 at 07:09:03AM -0800, Eddie Kohler wrote: Daniel P. Berrange wrote: We already have the ability to pass multiple flags / options to the VNC driver as a post-fix to the host:port pair, so I'm not a fan of introducing a new option as a prefix.

Re: [Qemu-devel] [PATCH] add VNC reverse connections

2008-01-16 Thread Daniel P. Berrange
On Wed, Jan 16, 2008 at 07:09:03AM -0800, Eddie Kohler wrote: > Daniel P. Berrange wrote: > >We already have the ability to pass multiple flags / options to the VNC > >driver as a post-fix to the host:port pair, so I'm not a fan of introducing > >a new option as a prefix. If using existing options

Re: [Qemu-devel] [PATCH] add VNC reverse connections

2008-01-16 Thread Anthony Liguori
Eddie Kohler wrote: Daniel P. Berrange wrote: We already have the ability to pass multiple flags / options to the VNC driver as a post-fix to the host:port pair, so I'm not a fan of introducing a new option as a prefix. If using existing options syntax, it could look like: -vnc :5500,rev

Re: [Qemu-devel] [PATCH] add VNC reverse connections

2008-01-16 Thread Eddie Kohler
Daniel P. Berrange wrote: We already have the ability to pass multiple flags / options to the VNC driver as a post-fix to the host:port pair, so I'm not a fan of introducing a new option as a prefix. If using existing options syntax, it could look like: -vnc :5500,rev -vnc read.cs.ucla.edu:5

Re: [Qemu-devel] [PATCH] add VNC reverse connections

2008-01-16 Thread Anthony Liguori
Daniel P. Berrange wrote: On Tue, Jan 15, 2008 at 11:27:15PM -0800, Eddie Kohler wrote: Hi all, This patch against current CVS adds VNC reverse connections, where the server connects actively to a waiting client, as in "-vnc rev:5500" or "-vnc rev:read.cs.ucla.edu:5500". This is quite use

Re: [Qemu-devel] [PATCH] add VNC reverse connections

2008-01-16 Thread Daniel P. Berrange
On Tue, Jan 15, 2008 at 11:27:15PM -0800, Eddie Kohler wrote: > Hi all, > > This patch against current CVS adds VNC reverse connections, where the > server connects actively to a waiting client, as in "-vnc rev:5500" or > "-vnc rev:read.cs.ucla.edu:5500". This is quite useful if the user expect

Re: [Qemu-devel] [PATCH] add VNC reverse connections

2008-01-16 Thread Avi Kivity
Eddie Kohler wrote: Hi all, This patch against current CVS adds VNC reverse connections, where the server connects actively to a waiting client, as in "-vnc rev:5500" or "-vnc rev:read.cs.ucla.edu:5500". This is quite useful if the user expects to run QEMU many times in succession (for examp

[Qemu-devel] [PATCH] add VNC reverse connections

2008-01-15 Thread Eddie Kohler
Hi all, This patch against current CVS adds VNC reverse connections, where the server connects actively to a waiting client, as in "-vnc rev:5500" or "-vnc rev:read.cs.ucla.edu:5500". This is quite useful if the user expects to run QEMU many times in succession (for example, is debugging a to