On 12 June 2013 21:11, mdroth <[email protected]> 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 <[email protected]> >> Suggested-by: Peter Maydell <[email protected]> >> Cc: [email protected] >> Signed-off-by: Michael Roth <[email protected]> > > 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
