[Spice-devel] [libvirt] [PATCH] qxl: Fix new function name for spice-server library

Marc-André Lureau marcandre.lureau at gmail.com
Wed Jul 22 07:37:05 PDT 2015


Hi

On Mon, Jul 20, 2015 at 11:46 AM, Martin Kletzander <mkletzan at redhat.com> wrote:
> 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 <fziglio at redhat.com>
>>

ACK, I'll try sending a proper pull request for 2.4 since Gerd is on holidays

>> ---
>> 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
H>>
>
> 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
>
> _______________________________________________
> Spice-devel mailing list
> Spice-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/spice-devel
>



-- 
Marc-André Lureau


More information about the Spice-devel mailing list