On Fri, 15 Feb 2019 at 09:38, Paolo Bonzini <pbonz...@redhat.com> wrote:
>
> On 14/02/19 23:04, Sven Schnelle wrote:
> > @@ -2202,6 +2224,7 @@ static const VMStateDescription vmstate_lsi_scsi = {
> > VMSTATE_UINT8(stime0, LSIState),
> > VMSTATE_UINT8(respid0, LSIState),
> > VMSTATE_UINT8(respid1, LSIState),
> > + VMSTATE_UINT8(sbcl, LSIState),
>
> This breaks live migration. You need to bump the version number higher
> in vmstate_lsi_scsi and use "VMSTATE_UINT8_V(sbcl, LSIState, 1)" so that
> the field is only marshalled/unmarshalled for version 1 of the migration
> state.
I thought we preferred to do this with migration subsections
rather than versioning fields these days ? In this case I
think a subsection that says "needed if sbcl is non-zero" would
do the right thing.
thanks
-- PMM
