On Thu, 2016-09-22 at 20:34 +1000, Alexey Kardashevskiy wrote: > > diff --git a/target-ppc/machine.c b/target-ppc/machine.c > > index 4820f22..1cf3779 100644 > > --- a/target-ppc/machine.c > > +++ b/target-ppc/machine.c > > @@ -563,8 +563,8 @@ const VMStateDescription vmstate_ppc_cpu = { > > > > /* Sanity checking */ > > VMSTATE_UINTTL_EQUAL(env.msr_mask, PowerPCCPU), > > - VMSTATE_UINT64_EQUAL(env.insns_flags, PowerPCCPU), > > - VMSTATE_UINT64_EQUAL(env.insns_flags2, PowerPCCPU), > > + VMSTATE_UNUSED(sizeof(target_ulong)), /* was > _EQUAL(env.insns_flags) */ > > + VMSTATE_UNUSED(sizeof(target_ulong)), /* was > _EQUAL(env.insns_flags2) */ > > VMSTATE_UINT32_EQUAL(env.nb_BATs, PowerPCCPU), > > VMSTATE_END_OF_LIST() > > }, > > > > TCG migration still remains broken with this.
TCG migration doesn't matter much ... yet, I think. KVM is what actual customers use, we can probably live with some TCG migration breakage. Hopefully we'll be done with P8 soon and it will be stable enough, and we'll be more careful with P9. Cheers, Ben.