From: "Dr. David Alan Gilbert" <dgilbert@redhat.com>
To: Markus Armbruster <armbru@redhat.com>
Cc: Paolo Bonzini <pbonzini@redhat.com>,
quintela@redhat.com, Laszlo Ersek <lersek@redhat.com>,
qemu-devel@nongnu.org, lcapitulino@redhat.com
Subject: Re: [Qemu-devel] [PATCH 0/3] Migrate with destination version
Date: Mon, 2 Jun 2014 14:50:05 +0100 [thread overview]
Message-ID: <20140602135004.GF2634@work-vm> (raw)
In-Reply-To: <871tv7bhyn.fsf@blackfin.pond.sub.org>
* Markus Armbruster (armbru@redhat.com) wrote:
> "Dr. David Alan Gilbert" <dgilbert@redhat.com> writes:
>
> > * Paolo Bonzini (pbonzini@redhat.com) wrote:
> [...]
> >> Version numbers open a huge compatibility can of worms (what is a version
> >> number?)
> >
> > No, a version number here is very well defined; it's the output of
> > query-version'
> > (or 'info version' for the HMP world).
>
> Okay, make it "version numbers are a huge, well-defined compatibility
> can of worms" :)
>
> > How you do comparisons on that, and in particular how you parse the 'package'
> > text is a different matter, however I suggest that parsing the 'package'
> > text is downstream's problem and they provide that text.
>
> This means that a downstream has to examine and possibly adapt the
> version number conditionals when it backports anything.
I was thinking of adding a few routines for version number comparison, in the hope
that they would help cover these two problems; but that I really thought didn't
make sense until we had some feel about how they would be used.
> >> and don't solve the fundamental problem of migration receiving
> >> insufficient testing with upstream QEMU versions.
> >
> > Indeed; and it's not meant to - in no way is this an excuse for inadequate
> > testing - however, we're all human, and bugs happen, this is just providing
> > a way to get out of some of the mess afterwards.
>
> Understand. It's a last resort, to be used only when the alternatives
> are all even worse.
Exactly.
> If we put this in, we'll have to review all attempts to use it very
> critically: is there really, really no practical alternative?
Yep.
> Can you explain why we should put it in before we have a user?
If we have it in then we can get libvirt to use it.
If we have it all in place then if we do hit a compatibility
issue where we need to use it, then it's easy.
If we don't put it in, then if we hit this type of situation
we suddenly need to coordinate a modification of libvirt as well.
However, it gets trickier since the destination libvirt has to know to send
the version information to the source; so we can only use this trick between
versions where the libvirt on both ends knows about it; so if we suddenly hit
a compatibility issue where we need it then we'd have to update libvirt on both
sides, which is too late.
So putting this in now, and getting it into libvirt means everyone has the info.
Dave
--
Dr. David Alan Gilbert / dgilbert@redhat.com / Manchester, UK
next prev parent reply other threads:[~2014-06-02 13:50 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <1401276034-30486-1-git-send-email-dgilbert@redhat.com>
[not found] ` <5385CC10.6030909@redhat.com>
[not found] ` <20140528120444.GE2372@work-vm>
2014-06-02 12:42 ` [Qemu-devel] [PATCH 0/3] Migrate with destination version Markus Armbruster
2014-06-02 13:50 ` Dr. David Alan Gilbert [this message]
2014-06-02 15:51 ` Paolo Bonzini
2014-06-02 16:40 ` Dr. David Alan Gilbert
2014-06-02 17:39 ` Paolo Bonzini
2014-06-02 17:44 ` Dr. David Alan Gilbert
2014-06-02 20:05 ` Paolo Bonzini
2014-06-04 18:51 ` 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=20140602135004.GF2634@work-vm \
--to=dgilbert@redhat.com \
--cc=armbru@redhat.com \
--cc=lcapitulino@redhat.com \
--cc=lersek@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).