From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:59582) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZH7fW-0007Gc-NF for qemu-devel@nongnu.org; Mon, 20 Jul 2015 05:47:31 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ZH7fR-0000iE-NU for qemu-devel@nongnu.org; Mon, 20 Jul 2015 05:47:30 -0400 Received: from mx1.redhat.com ([209.132.183.28]:54072) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZH7fR-0000i5-HD for qemu-devel@nongnu.org; Mon, 20 Jul 2015 05:47:25 -0400 Date: Mon, 20 Jul 2015 11:46:49 +0200 From: Martin Kletzander Message-ID: <20150720094645.GG3887@wheatley> References: <1437381803-2936-1-git-send-email-fziglio@redhat.com> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="oOB74oR0WcNeq9Zb" Content-Disposition: inline In-Reply-To: <1437381803-2936-1-git-send-email-fziglio@redhat.com> Subject: Re: [Qemu-devel] [libvirt] [PATCH] qxl: Fix new function name for spice-server library List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Frediano Ziglio Cc: peter.maydell@linaro.org, pkrempa@redhat.com, libvir-list@redhat.com, qemu-devel@nongnu.org, cfergeau@redhat.com, pgrunt@redhat.com, kraxel@redhat.com, spice-devel@lists.freedesktop.org --oOB74oR0WcNeq9Zb Content-Type: text/plain; charset=us-ascii; format=flowed Content-Disposition: inline On Mon, Jul 20, 2015 at 09:43:23AM +0100, Frediano Ziglio wrote: >The new spice-server function to limit the number of monitors (0.12.6) >changed while development from spice_qxl_set_monitors_config_limit to >spice_qxl_max_monitors (accepted upstream). >By mistake I post patch with former name. >This patch fix the function name. > >Signed-off-by: Frediano Ziglio > >--- > hw/display/qxl.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > >I tested again doing a clean build, unfortunately I did some mistake >and my tests worked. > >diff --git a/hw/display/qxl.c b/hw/display/qxl.c >index 4e5ff69..2288238 100644 >--- a/hw/display/qxl.c >+++ b/hw/display/qxl.c >@@ -273,8 +273,7 @@ static void qxl_spice_monitors_config_async(PCIQXLDevice *qxl, int replay) > } else { > #if SPICE_SERVER_VERSION >= 0x000c06 /* release 0.12.6 */ > if (qxl->max_outputs) { >- spice_qxl_set_monitors_config_limit(&qxl->ssd.qxl, >- qxl->max_outputs); >+ spice_qxl_set_max_monitors(&qxl->ssd.qxl, qxl->max_outputs); > } > #endif > qxl->guest_monitors_config = qxl->ram->monitors_config; >-- >2.1.0 > Same as the fix I did in order for this to work with upstream spice. ACK. Weak, though, as I'm not a privileged one. Martin --oOB74oR0WcNeq9Zb Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iQIcBAEBCAAGBQJVrMOFAAoJEAgfwp8kF4bd6FYP/3kPCLkIwe8pE6joxKVfXMUT PHcO6l+zAMkT31XNcfkjV33xmosLI8ST7zYSseLE5pbWBn7rcTXc7oqfeLJMnfoL 5VHwsOI+HxHPRe5RwbwvVmLnFJOvD/MnwgZDcGMcqtfst+sIUwqOijKY4ITgzEz3 LS334ofxOVLDtCW2bQPUlzxCgFtdsCrcqx31NmxPKzLn1GTDlSjrkrC8iSgQC9lo FBjhIbk4e5Zupsy2g9NWabRcvU6FARNE3DAg91Bdy6XEwutGTikDMq9tXHr7bUzc iz7JxHri/OfTAXOo5gulshXpSAcwPaL2FDNQB+jQ8zIWYQqOZawGlwXwOsqpSOH5 mOFQU41z6wmrHe8/3wnPwCH4/KbCB7H1zZOKQJHzFkNPz3MrgCdBSOwNDGifTOCK ddwgq/YkFPf1Rb5yorUvT3hzV4PjoJtT2bziTdocRz1cobisHELV+CVbjKL0rlcb ZXmlee7Hixj1ZMWE2DCJFw9G5mtkE/If/0ekxqXD0epNNdZUEq+VhaEnpVQNFe7m 7/Q+9rGvuIlytSOtLttXU7DO2w7WQPOR+qHC64kJGRY8Ai6vkS2JeTHEfYAkgnLg ynC1OTTLrqL7w/bb94i6WQike4gLp/u0g3ykT1bD9bJxsZaACNrX7LEjiAalPWq4 b5du3fDZvIZUspwab/Ul =L/V4 -----END PGP SIGNATURE----- --oOB74oR0WcNeq9Zb--