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@xxxxxxxxxx> 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 > > > -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list