From: Anthony Liguori <anthony@codemonkey.ws>
To: Paolo Bonzini <pbonzini@redhat.com>
Cc: quintela@redhat.com, qemu-devel@nongnu.org, mst@redhat.com
Subject: Re: [Qemu-devel] [RFC PATCH 0/4] Fix subsection ambiguity in the migration format
Date: Fri, 29 Jul 2011 08:14:46 -0500 [thread overview]
Message-ID: <4E32B246.6060001@codemonkey.ws> (raw)
In-Reply-To: <CAHFMJ7uAf4OmqQtUALyWa4Nmii=oZsHTYHPpBbuaN9mg2-Hrvw@mail.gmail.com>
On 07/25/2011 04:10 PM, Paolo Bonzini wrote:
> On Thu, Jun 30, 2011 at 17:46, Paolo Bonzini<pbonzini@redhat.com> wrote:
> I have now tested this series (exactly as sent) both by examining
> manually the differences between the two formats on the same guest
> state, and by a mix of saves/restores (new on new, 0.14 on new
> pc-0.14, new pc-0.14 on 0.14; also the same combinations on RHEL). It
> always does what is expected.
>
> Michael Tsirkin objected that the format should be passed as a
> parameter in the migrate command. I kind of agree, however since this
> is a real bug you would need to bump the default for new machine
> types, and this default would still go in the QEMUMachine struct like
> I am doing. So I consider the two settings to be orthogonal. Also,
> the alternative requires changes to the whole management stack and if
> the default is not changed it imposes a broken format unless you
> update the management tools. Clearly much less bang for the buck.
>
> I think this is ready to go into 0.15. The bug happens when migrating
> to 0.14 a pc-0.14 machine created with QEMU 0.15 and which has a
> floppy. The media changed subsection is almost always included, and
> this causes problems when migrating to 0.14 which didn't have any
> subsection for the floppy device. While QEMU support for migration to
> old version admittedly depends on luck, this isn't true of certain
> downstreams :) which would like to have an unambiguous migration
> format.
I really hate the idea of changing the migration format moments before
the release.
Since subsections are optional, can't we take the offending subsections,
remove them, bump the section version numbers and make the fields required?
That "fixes" this issue temporarily without changing the format and we
can change the format for 1.0.
Regards,
Anthony Liguori
>
> Paolo
>
>
next prev parent reply other threads:[~2011-07-29 13:14 UTC|newest]
Thread overview: 47+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-06-30 15:46 [Qemu-devel] [RFC PATCH 0/4] Fix subsection ambiguity in the migration format Paolo Bonzini
2011-06-30 15:46 ` [Qemu-devel] [RFC PATCH 1/4] add support for machine models to specify their " Paolo Bonzini
2011-06-30 18:11 ` Michael S. Tsirkin
2011-07-01 6:10 ` Paolo Bonzini
2011-07-29 13:08 ` Anthony Liguori
2011-07-29 14:35 ` Paolo Bonzini
2011-06-30 15:46 ` [Qemu-devel] [RFC PATCH 2/4] add pc-0.14 machine Paolo Bonzini
2011-08-05 19:26 ` Bruce Rogers
2011-08-05 19:41 ` Anthony Liguori
2011-06-30 15:46 ` [Qemu-devel] [RFC PATCH 3/4] savevm: define new unambiguous migration format Paolo Bonzini
2011-07-29 13:12 ` Anthony Liguori
2011-07-29 14:35 ` Paolo Bonzini
2011-06-30 15:46 ` [Qemu-devel] [RFC PATCH 4/4] Partially revert "savevm: fix corruption in vmstate_subsection_load()." Paolo Bonzini
2011-07-25 21:10 ` [Qemu-devel] [RFC PATCH 0/4] Fix subsection ambiguity in the migration format Paolo Bonzini
2011-07-25 23:23 ` Anthony Liguori
2011-07-26 9:42 ` Daniel P. Berrange
2011-07-26 9:48 ` Stefan Hajnoczi
2011-07-26 12:51 ` Stefan Hajnoczi
2011-07-26 13:00 ` Anthony Liguori
2011-07-26 12:07 ` Juan Quintela
2011-07-26 12:37 ` Anthony Liguori
2011-07-26 20:13 ` Juan Quintela
2011-07-26 21:46 ` Anthony Liguori
2011-07-26 22:22 ` Peter Maydell
2011-07-26 23:08 ` Anthony Liguori
2011-07-29 14:03 ` Kevin Wolf
2011-07-29 14:28 ` Anthony Liguori
2011-07-29 15:18 ` Kevin Wolf
2011-07-29 22:28 ` Anthony Liguori
2011-07-31 10:48 ` Dor Laor
2011-07-31 11:37 ` Peter Maydell
2011-07-31 11:45 ` Dor Laor
2011-07-31 18:46 ` Christoph Hellwig
2011-07-31 20:43 ` Dor Laor
2011-07-31 20:55 ` Anthony Liguori
2011-07-31 23:10 ` Christoph Hellwig
2011-08-01 0:15 ` Anthony Liguori
2011-08-01 7:54 ` Christoph Hellwig
2011-08-01 13:53 ` Anthony Liguori
2011-08-04 14:59 ` Luiz Capitulino
2011-07-31 20:43 ` Anthony Liguori
2011-07-31 20:57 ` Dor Laor
2011-07-31 21:03 ` Anthony Liguori
2011-07-31 21:25 ` Dor Laor
2011-07-31 21:49 ` Anthony Liguori
2011-07-29 13:14 ` Anthony Liguori [this message]
2011-07-29 14:49 ` Paolo Bonzini
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4E32B246.6060001@codemonkey.ws \
--to=anthony@codemonkey.ws \
--cc=mst@redhat.com \
--cc=pbonzini@redhat.com \
--cc=qemu-devel@nongnu.org \
--cc=quintela@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).