Re: [Qemu-trivial] [PATCH] wdt_i6300esb: fix vmstate versioning

[Peter Maydell]

On 12 June 2013 21:11, mdroth <address@hidden> wrote:
> On Tue, May 21, 2013 at 05:32:57PM -0500, Michael Roth wrote:
>> When this VMSD was introduced it's version fields were set to
>> sizeof(I6300State), making them essentially random from build to build,
>> version to version.
>>
>> To fix this, we lock in a high version id and low minimum version id to
>> support old->new migration from all prior versions of this device's
>> state. This should work since the device state has not changed since
>> its introduction.
>>
>> The potentially breaks migration from 1.5+ to 1.5, but since the
>> versioning was essentially random prior to this patch, new->old
>> migration was not consistently functional to begin with.
>>
>> Reported-by: Nicholas Thomas <address@hidden>
>> Suggested-by: Peter Maydell <address@hidden>
>> Cc: address@hidden
>> Signed-off-by: Michael Roth <address@hidden>
>
> CC'ing qemu-trivial. Looking to get this in for 1.5.1

This is a good patch but it definitely doesn't seem like
-trivial material to me. -trivial isn't "way to get patches
in that would otherwise fall through the cracks" :-)

thanks
-- PMM