All of lore.kernel.org
 help / color / mirror / Atom feed
From: Paolo Bonzini <pbonzini@redhat.com>
To: Alexey Kardashevskiy <aik@ozlabs.ru>,
	"qemu-devel@nongnu.org" <qemu-devel@nongnu.org>
Cc: "Paul Mackerras" <paulus@samba.org>,
	"Andreas Färber" <afaerber@suse.de>,
	"Alexander Graf" <agraf@suse.com>
Subject: Re: [Qemu-devel] machines and versions - why so many?
Date: Mon, 23 Jun 2014 16:59:20 +0200	[thread overview]
Message-ID: <53A840C8.8060607@redhat.com> (raw)
In-Reply-To: <53A8366B.1020301@ozlabs.ru>

Il 23/06/2014 16:15, Alexey Kardashevskiy ha scritto:
> Hi!
>
> I have been hearing recently that we (server PPC) should have more that
> just one pseries machine in QEMU because this is what everybody else does :)

If you want to support live upgrades from QEMU 2.1 to QEMU 2.2, you 
should indeed prepare by adding a stable name such as pseries-2.1 
already in QEMU 2.1.

> My current understanding is that multiple machines (like
> pc-i440fx-1.4..2.1, and many others) are needed:
>
> 3) for devices which are created explicitly and for which we want some
> capabilities be disabled and we do not want to bother about this every time
> we run QEMU;

This is not the exact reason.  What you wrote _could_ be a reason to use 
compat_properties, but it is not why x86 has multiple machine types.

The reason is because when you migrate from QEMU 1.7 to QEMU 2.1, the 
machine should behave exactly the same as it did in QEMU 1.7.  Any 
guest-visible change introduced between QEMU 1.7 and QEMU 2.1 (*) must 
be removed from QEMU 2.1.  This is the reason why x86 has one machine 
type per version.

     (*) X got faster is not a guest-visible change.  A guest-visible
         change is something like:

         - a register gets a new default value

         - a register that was read-only is now read-write

         - the size of a memory region changed (e.g. a ROM BAR grew
           from 128k to 256k)

         Sometimes, if no known guest is using the register (e.g. only
         firmware uses it) it is okay to skip the creation of the
         compat property.  However, if in doubt, better add it.

> ok, this one makes some sense for "pseries" to have (and upcoming
> endianness register on VGA seems to be the case) but it seems that adding a
> "compat" or "feature" property to the VGA device (and other devices which
> deal with this kind of compatibility) is still more architecturally correct
> thing to do, and let libvirt deal with the rest.

Yes, this sounds sensible.

Paolo

> Since I (almost) always miss the bigger picture, what do I miss now? :) Thanks!
>
>

  reply	other threads:[~2014-06-23 14:59 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-06-23 14:15 [Qemu-devel] machines and versions - why so many? Alexey Kardashevskiy
2014-06-23 14:59 ` Paolo Bonzini [this message]
2014-06-23 21:35   ` Alexey Kardashevskiy
2014-06-23 21:41     ` Andreas Färber
2014-06-23 22:33       ` Alexey Kardashevskiy
2014-06-24  5:21         ` Paolo Bonzini
2014-06-24 11:15           ` Andreas Färber
2014-06-24 13:05             ` Alexey Kardashevskiy
2014-06-24 12:38       ` Marcel Apfelbaum
2014-06-23 21:41     ` Peter Maydell
2014-06-24  5:15     ` Paolo Bonzini
2014-06-24  5:37       ` Alexey Kardashevskiy
2014-06-24  8:17         ` Markus Armbruster
2014-06-24  9:10           ` Alexey Kardashevskiy
2014-06-24 11:22           ` Andreas Färber
2014-06-24 12:56             ` Alexey Kardashevskiy
2014-06-23 15:16 ` Markus Armbruster
2014-06-24  1:06   ` Alexey Kardashevskiy

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=53A840C8.8060607@redhat.com \
    --to=pbonzini@redhat.com \
    --cc=afaerber@suse.de \
    --cc=agraf@suse.com \
    --cc=aik@ozlabs.ru \
    --cc=paulus@samba.org \
    --cc=qemu-devel@nongnu.org \
    /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.