On Mon, Sep 22, 2014 at 01:28:30PM +0100, Alex Bligh wrote:
Michael,
(Reordering)
On 22 Sep 2014, at 12:50, Michael S. Tsirkin <mst(a)redhat.com> wrote:
>
>> @@ -257,6 +259,11 @@ static int pit_dispatch_post_load(void *opaque, int
version_id)
>> return 0;
>> }
>>
>> +static bool has_irq_disabled(void *opaque, int version_id)
>> +{
>> + return (version_id >= 3) || ((version_id == 2) &&
qemu_kvm_1_0_compat);
>> +}
>> +
> Can you add a device property like we do for
> other compat behaviours?
> That's better than a global variable.
> Same for other devices.
I can see 'opaque' points to PIIX4PMState or PITCommonState. Do you mean
to put them in there? Could you point to an example to copy.
Look for PC_COMPAT_ macros, we have a ton of flags like this.
--
Alex Bligh