[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] spice: Disallow use of gl + TCP port
From: |
Christophe Fergeau |
Subject: |
Re: [Qemu-devel] [PATCH] spice: Disallow use of gl + TCP port |
Date: |
Tue, 15 Mar 2016 10:49:55 +0100 |
User-agent: |
Mutt/1.5.24 (2015-08-30) |
On Mon, Mar 14, 2016 at 09:41:34AM -0600, Eric Blake wrote:
> On 03/14/2016 05:41 AM, Christophe Fergeau wrote:
> > Currently, virgl support has to go through a local unix socket, trying
> > to connect to a VM using -spice gl through spice://localhost:5900 will
> > only result in a black screen.
> > This commit errors out when the user tries to start a VM with both GL
> > support and a port/tls-port set.
> > This would fit better in spice-server, but currently QEMU does not call
> > into spice-server when parsing 'gl' on its command line, so we have to
> > do this check in QEMU instead.
> >
> > Signed-off-by: Christophe Fergeau <address@hidden>
> > ---
> > ui/spice-core.c | 4 ++++
> > 1 file changed, 4 insertions(+)
> >
> > diff --git a/ui/spice-core.c b/ui/spice-core.c
> > index 7987a4e..94f3236 100644
> > --- a/ui/spice-core.c
> > +++ b/ui/spice-core.c
> > @@ -844,6 +844,10 @@ void qemu_spice_init(void)
> >
> > #ifdef HAVE_SPICE_GL
> > if (qemu_opt_get_bool(opts, "gl", 0)) {
> > + if ((port != 0) || (tls_port != 0)) {
>
> Overparenthesized; you could write:
>
> if (port || tls_port) {
>
> for the same effect with less typing.
Yeah I know I'm overly verbose with these tests, the parentheses make it
explicit that there are no operator priority issues, the != 0 emphasize
it's an integer type which is being handled.
I'll change it to your recommendation before sending a v2.
Christophe
signature.asc
Description: PGP signature
Re: [Qemu-devel] [PATCH] spice: Disallow use of gl + TCP port, Eric Blake, 2016/03/14
- Re: [Qemu-devel] [PATCH] spice: Disallow use of gl + TCP port,
Christophe Fergeau <=
Re: [Qemu-devel] [PATCH] spice: Disallow use of gl + TCP port, Gerd Hoffmann, 2016/03/18