All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Michael S. Tsirkin" <mst@redhat.com>
To: Alex Bligh <alex@alex.org.uk>
Cc: "Ryan Harper" <ryan.harper@canonical.com>,
	"Serge Hallyn" <serge.hallyn@canonical.com>,
	"quintela@redhat.com" <quintela@redhat.com>,
	Libvirt <libvir-list@redhat.com>,
	"Serge Hallyn" <serge.hallyn@ubuntu.com>,
	"qemu-devel@nongnu.org" <qemu-devel@nongnu.org>,
	"Alexander Graf" <agraf@suse.de>,
	"Cole Robinson" <crobinso@redhat.com>,
	"Amit Shah" <amit.shah@redhat.com>,
	"Paolo Bonzini" <pbonzini@redhat.com>,
	"Bruce Rogers" <brogers@suse.com>,
	"Andreas Färber" <afaerber@suse.de>
Subject: Re: [Qemu-devel] [PATCH] [RFC] Add machine type pc-1.0-qemu-kvm for live migrate compatibility with qemu-kvm
Date: Mon, 4 Aug 2014 18:13:24 +0200	[thread overview]
Message-ID: <20140804161324.GA22228@redhat.com> (raw)
In-Reply-To: <387254CE-58BB-41B5-90F6-2D54112B4695@alex.org.uk>

On Mon, Aug 04, 2014 at 04:47:11PM +0100, Alex Bligh wrote:
> 
> On 4 Aug 2014, at 16:38, Serge Hallyn <serge.hallyn@ubuntu.com> wrote:
> 
> >> 
> >> If you really want it to be called pc-1.0, you
> >> can make it a machine property instead.
> >> E.g. qemu-kvm-compatibility.
> >> Teach management to set it if remote is qemu-kvm:
> >> 	-machine pc-1.0,qemu-kvm-compatibility=on
> > 
> > That sounds nice - Alex, what do you think?
> 
> Not having used the machine property stuff before,
> or played with libvirt much, I'm not sure how this
> helps libvirt.
> 
> I thought the issue here was that migrating from
> 1.0-qemu-kvm to 2.x OR 1.0-qemu-git to 2.x, libvirt
> is going to to supply the same command line.
> As
> libvirt doesn't know what the sender is (and
> it's not possible to detect this automatically -
> at least not without a far more intrusive patch),

Yes, this is up to higher level user.
At libvirt xml level, you would just specify
something like "legacy qemu-kvm compatibility" in the xml.

> one has to make a choice at build time as to what
> 'pc-1.0' represents.

There's no choice really. Downstreams must make sure
their machine types are distinct from upstream ones.
qemu-kvm as a downstream violated this rule but
I don't think this means upstream should violate it.

> This is what patch #2 does.
> I fully agree it is not pretty.

The problem is not prettyness.
The problem is, it creates a situation where two instances
of qemu have different ideas about what a specific
machine type is.


> So I am not sure why
> 	-machine pc-1.0,qemu-kvm-compatibility=on
> is any easier for libvirt than
> 	-machine pc-1.0-qemu-kvm
> 
> IE what does using a machine property rather than
> a machine type buy us?

Seems to be easier to understand that it maps to pc-1.0
on the other side.

> -- 
> Alex Bligh
> 
> 
> 

      reply	other threads:[~2014-08-04 16:13 UTC|newest]

Thread overview: 28+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-07-22 18:43 [Qemu-devel] [PATCH] [RFC] Add machine type pc-1.0-qemu-kvm for live migrate compatibility with qemu-kvm Alex Bligh
2014-07-25 16:01 ` Alex Bligh
2014-07-26  7:38   ` Paolo Bonzini
2014-07-27 13:09 ` Alex Bligh
2014-07-27 14:10 ` Andreas Färber
2014-07-27 17:04   ` Alex Bligh
2014-07-27 21:03 ` Alex Bligh
2014-07-29  4:16   ` Serge Hallyn
2014-07-29  7:31     ` Alex Bligh
2014-07-29 13:03       ` Serge E. Hallyn
2014-07-29 13:15         ` Alex Bligh
2014-07-29 13:21         ` Paolo Bonzini
2014-07-29 13:27           ` Serge Hallyn
2014-07-29 13:35             ` Paolo Bonzini
2014-07-29 13:39               ` Alex Bligh
2014-07-29 13:42                 ` Paolo Bonzini
2014-07-29 13:56                   ` Alex Bligh
2014-07-29 14:00                     ` Paolo Bonzini
2014-07-29 15:05                       ` Alex Bligh
2014-07-29 13:41               ` Serge Hallyn
2014-07-29 13:38           ` Alex Bligh
2014-07-29 13:41             ` Serge Hallyn
2014-08-04 13:34       ` Michael S. Tsirkin
2014-08-04 15:08         ` Serge Hallyn
2014-08-04 15:26           ` Michael S. Tsirkin
2014-08-04 15:38             ` Serge Hallyn
2014-08-04 15:47               ` Alex Bligh
2014-08-04 16:13                 ` Michael S. Tsirkin [this message]

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=20140804161324.GA22228@redhat.com \
    --to=mst@redhat.com \
    --cc=afaerber@suse.de \
    --cc=agraf@suse.de \
    --cc=alex@alex.org.uk \
    --cc=amit.shah@redhat.com \
    --cc=brogers@suse.com \
    --cc=crobinso@redhat.com \
    --cc=libvir-list@redhat.com \
    --cc=pbonzini@redhat.com \
    --cc=qemu-devel@nongnu.org \
    --cc=quintela@redhat.com \
    --cc=ryan.harper@canonical.com \
    --cc=serge.hallyn@canonical.com \
    --cc=serge.hallyn@ubuntu.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.