"Michael S. Tsirkin" <m...@redhat.com> wrote: > From: Peter Maydell <peter.mayd...@linaro.org> > > At the moment we require vmstate definitions to set minimum_version_id_old > to the same value as minimum_version_id if they do not provide a > load_state_old handler. Since the load_state_old functionality is > required only for a handful of devices that need to retain migration > compatibility with a pre-vmstate implementation, this means the bulk > of devices have pointless boilerplate. Relax the definition so that > minimum_version_id_old is ignored if there is no load_state_old handler. > > Note that under the old scheme we would segfault if the vmstate > specified a minimum_version_id_old that was less than minimum_version_id > but did not provide a load_state_old function, and the incoming state > specified a version number between minimum_version_id_old and > minimum_version_id. Under the new scheme this will just result in > our failing the migration. > > Signed-off-by: Peter Maydell <peter.mayd...@linaro.org> > Reviewed-by: Michael S. Tsirkin <m...@redhat.com> > Signed-off-by: Michael S. Tsirkin <m...@redhat.com>
Reviewed-by: Juan Quintela <quint...@redhat.com> Applied