From: "Michael S. Tsirkin" <mst@redhat.com>
To: Paolo Bonzini <pbonzini@redhat.com>
Cc: amit.shah@redhat.com, qemu-devel@nongnu.org,
"Dr. David Alan Gilbert" <dgilbert@redhat.com>,
quintela@redhat.com
Subject: Re: [Qemu-devel] [PATCH] Migration compatibility for serial
Date: Wed, 17 Jun 2015 16:44:28 +0200 [thread overview]
Message-ID: <20150617163731-mutt-send-email-mst@redhat.com> (raw)
In-Reply-To: <5581661A.1060101@redhat.com>
On Wed, Jun 17, 2015 at 02:20:42PM +0200, Paolo Bonzini wrote:
>
>
> On 17/06/2015 14:07, Dr. David Alan Gilbert wrote:
> > Of course for us on RHEL our minor releases don't correspond to
> > QEMU minor releases, so we already support migrating from our
> > downstream 7.1 (QEMU 2.1) derivative to our 7.0 (1.5.3) version.
> > And the reason for this patch series is to support something >2.2
> > migrating back to that 2.1 (or maybe even to that 1.5.3).
> >
> > I don't believe we're alone in wanting to be able to do that type
> > of thing;
>
> Others may prefer to have migration only work when it is absolutely sure
> that it works. It is much easier to add hacks on top of what upstream
> QEMU does (e.g. using the static checker), than to remove the hacks.
>
> If we really didn't care about others' support for bidirectional
> migration, we would have kept the static checker internal to Red Hat.
> Or we wouldn't have bothered to refine the .needed functions, and so on.
>
> Paolo
What we need to decide is how major is the breakage.
If it's minor - like some lost characters - then it's not
worth breaking migration for most users.
And I think this should be a property so people can
force strict mode if they really want to.
If it's a major breakage, it's harder to decide:
some people might be able to retry migration later.
Maybe a flag to enable this mode would make sense?
Also, maybe it would be better to fail migration on source
rather than send something destination can't handle?
But let's see what the symptoms are before we argue
about this option.
> > so you can either worry about not burdening upstream
> > with compatibility patches like this, or think it's not fair
> > to leave them out if others upstream might want them. How many
> > others? Well I'd say it's got to be more than some of the other
> > obscure features in QEMU!
next prev parent reply other threads:[~2015-06-17 14:44 UTC|newest]
Thread overview: 42+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-06-16 18:54 [Qemu-devel] [PATCH] Migration compatibility for serial Dr. David Alan Gilbert (git)
2015-06-16 20:49 ` Michael S. Tsirkin
2015-06-17 9:09 ` Dr. David Alan Gilbert
2015-06-17 6:52 ` Markus Armbruster
2015-06-17 6:58 ` Michael S. Tsirkin
2015-06-17 7:11 ` Markus Armbruster
2015-06-17 7:25 ` Michael S. Tsirkin
2015-06-17 7:41 ` Paolo Bonzini
2015-06-17 7:52 ` Michael S. Tsirkin
2015-06-17 8:11 ` Paolo Bonzini
2015-06-17 10:13 ` Michael S. Tsirkin
2015-06-17 10:48 ` Juan Quintela
2015-06-17 10:51 ` Dr. David Alan Gilbert
2015-06-17 10:56 ` Michael S. Tsirkin
2015-06-17 10:59 ` Dr. David Alan Gilbert
2015-06-17 11:36 ` Michael S. Tsirkin
2015-06-17 10:55 ` Michael S. Tsirkin
2015-06-17 11:33 ` Paolo Bonzini
2015-06-17 11:40 ` Dr. David Alan Gilbert
2015-06-17 11:44 ` Paolo Bonzini
2015-06-17 11:48 ` Michael S. Tsirkin
2015-06-17 12:07 ` Dr. David Alan Gilbert
2015-06-17 12:20 ` Paolo Bonzini
2015-06-17 14:44 ` Michael S. Tsirkin [this message]
2015-06-17 16:34 ` Juan Quintela
2015-06-17 16:39 ` Michael S. Tsirkin
2015-06-17 16:40 ` Paolo Bonzini
2015-06-17 11:45 ` Michael S. Tsirkin
2015-06-17 11:53 ` Paolo Bonzini
2015-06-17 11:54 ` Michael S. Tsirkin
2015-06-17 11:55 ` Paolo Bonzini
2015-06-17 11:58 ` Michael S. Tsirkin
2015-06-17 12:20 ` Paolo Bonzini
2015-06-17 14:50 ` Michael S. Tsirkin
2015-06-17 16:16 ` Paolo Bonzini
2015-06-17 16:34 ` Michael S. Tsirkin
2015-06-17 16:35 ` Paolo Bonzini
2015-06-17 9:26 ` Juan Quintela
2015-06-17 10:37 ` Michael S. Tsirkin
2015-06-17 8:37 ` Dr. David Alan Gilbert
2015-06-17 8:49 ` Paolo Bonzini
2015-06-17 9:38 ` Dr. David Alan Gilbert
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=20150617163731-mutt-send-email-mst@redhat.com \
--to=mst@redhat.com \
--cc=amit.shah@redhat.com \
--cc=dgilbert@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).