From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:42610) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1T8am4-0008D2-0L for qemu-devel@nongnu.org; Mon, 03 Sep 2012 13:49:24 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1T8am3-0002mC-3T for qemu-devel@nongnu.org; Mon, 03 Sep 2012 13:49:23 -0400 Received: from smtp.nfit.au.dk ([130.225.17.180]:47425 helo=nysmtp.nfit.au.dk) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1T8am2-0002m5-SN for qemu-devel@nongnu.org; Mon, 03 Sep 2012 13:49:23 -0400 From: sandmann@cs.au.dk (=?utf-8?Q?S=C3=B8ren?= Sandmann) References: <704493541.25479840.1346657644362.JavaMail.root@redhat.com> Date: Mon, 03 Sep 2012 19:49:20 +0200 In-Reply-To: (=?utf-8?Q?=22S=C3=B8ren?= Sandmann"'s message of "Mon, 3 Sep 2012 19:33:27 +0200") Message-ID: MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [PATCH-v2 spice 1/2] Set a8 capability in the QXL device if supported by the client List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Alon Levy Cc: qemu-devel@nongnu.org, spice-devel@lists.freedesktop.org, =?utf-8?Q?S=C3=B8ren?= Sandmann Pedersen , kraxel@redhat.com S=C3=B8ren Sandmann writes: > Alon Levy writes: > >>> --- >>> server/red_worker.c | 2 ++ >>> spice-common | 2 +- >>> 2 files changed, 3 insertions(+), 1 deletion(-) >>>=20 >>> diff --git a/server/red_worker.c b/server/red_worker.c >>> index 843f559..23f3464 100644 >>> --- a/server/red_worker.c >>> +++ b/server/red_worker.c >>> @@ -10377,6 +10377,8 @@ static void >>> handle_new_display_channel(RedWorker *worker, RedClient *client, Red >>> SET_CAP(caps, SPICE_DISPLAY_CAP_MONITORS_CONFIG); >>> if (red_channel_client_test_remote_cap(rcc, >>> SPICE_DISPLAY_CAP_COMPOSITE)) >>> SET_CAP(caps, SPICE_DISPLAY_CAP_COMPOSITE); >>> + if (red_channel_client_test_remote_cap(rcc, >>> SPICE_DISPLAY_CAP_COMPOSITE)) >>> + SET_CAP(caps, SPICE_DISPLAY_CAP_A8_SURFACE); >> >> Didn't you mean to test remote SPICE_DISPLAY_CAP_A8_SURFACE? > > Yes, good catch. I'll fix before pushing. I realized that I didn't actually send the whole patch series for spice-server. There are five patches rather than two. Soren