[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] qxl: Default to PCI revision 4 when spice-serve
From: |
Søren Sandmann |
Subject: |
Re: [Qemu-devel] [PATCH] qxl: Default to PCI revision 4 when spice-server is new enough |
Date: |
Thu, 04 Oct 2012 19:31:34 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/23.1 (gnu/linux) |
Paolo Bonzini <address@hidden> writes:
> Il 03/10/2012 19:57, Søren Sandmann ha scritto:
>> From: Søren Sandmann Pedersen <address@hidden>
>>
>> When spice-server is newer than 0.12.0, the default revision should be
>> QXL_REVISION_STABLE_V12.
>>
>> Signed-off-by: Soren Sandmann <address@hidden>
>> ---
>> hw/qxl.h | 3 +--
>> 1 files changed, 1 insertions(+), 2 deletions(-)
>>
>> diff --git a/hw/qxl.h b/hw/qxl.h
>> index 5553824..4460bb2 100644
>> --- a/hw/qxl.h
>> +++ b/hw/qxl.h
>> @@ -129,8 +129,7 @@ typedef struct PCIQXLDevice {
>> } \
>> } while (0)
>>
>> -#if 0
>> -/* spice-server 0.12 is still in development */
>> +#if SPICE_SERVER_VERSION >= 0x000c00 /* 0x000c00 == 0.12.0 */
>> #define QXL_DEFAULT_REVISION QXL_REVISION_STABLE_V12
>> #else
>> #define QXL_DEFAULT_REVISION QXL_REVISION_STABLE_V10
>>
>
> No, that's not possible. The revision must be the same for all machines
> started with the same "-M" argument. So your only choice is to make
> 0.12 a requirement for QEMU 1.3. Then you can use compatibility
> properties (see the long list of strings in hw/pc_piix.c) to ensure that
> pc-1.2 and earlier machines use the previous revision.
Spice server 0.12 is already, so I'll send a patch to require that
unconditionally. I'm assuming that later strings override earlier ones
so that the revision=3 I added won't take effect for previous machines.
Søren