All of lore.kernel.org
 help / color / mirror / Atom feed
From: Gerd Hoffmann <kraxel@redhat.com>
To: Anthony Liguori <aliguori@us.ibm.com>
Cc: Peter Maydell <peter.maydell@linaro.org>,
	"Mian M. Hamayun" <m.hamayun@virtualopensystems.com>,
	patches@linaro.org, qemu-devel@nongnu.org,
	Markus Armbruster <armbru@redhat.com>,
	kvmarm@lists.cs.columbia.edu
Subject: Re: [Qemu-devel] Versioned machine types for ARM/non-x86 ? (Was Re: [PATCH v4 0/2] ARM: add 'virt' platform)
Date: Mon, 05 Aug 2013 17:06:46 +0200	[thread overview]
Message-ID: <51FFBF86.5030802@redhat.com> (raw)
In-Reply-To: <87y58gp6q5.fsf@codemonkey.ws>

  Hi,

>>> However, unlike PC, I'd like to do linear versioning and avoid bumping
>>> at every release.
>>>
>>> IOW, spapr-1, spapr-2, spapr-3, etc.
>>>
>>> I think virt ought to try to do the same.
>>
>> Any particular reason why ? I kind of like the clarity of having the
>> version match the release version. Avoids needing to lookup a magic
>> decoder ring to figure out which QEMU version maps to which machine
>> version.

+1, /me likes the version-based naming too.  It's also easier to handle
on source code level as it makes it easier to reuse the #defines we
already have for pc compat properties.

> (1) reduces testing matrix by having fewer versions

I doubt that is going to fly.  It's not like we do new pc-* machine
types just for the snake of creating them, there is no policy we have to
have a new one for each release.  Usually we create them in case there
is an actual need, i.e. a incompatible change which needs a compat
property.  Which so far was the case for (almost?) every release.

> (2) makes people
> think more carefully about whether it's really necessary to break
> compatibility.

Often it's not about incompatibilities but about new features which we
wanna have enabled by default.

cheers,
  Gerd

      reply	other threads:[~2013-08-05 15:07 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-08-05 11:18 [Qemu-devel] [PATCH v4 0/2] ARM: add 'virt' platform Peter Maydell
2013-08-05 11:18 ` [Qemu-devel] [PATCH v4 1/2] ARM: Allow boards to provide an fdt blob Peter Maydell
2013-08-05 11:18 ` [Qemu-devel] [PATCH v4 2/2] ARM: Add 'virt' platform Peter Maydell
2013-08-05 11:20   ` Peter Maydell
2013-08-05 11:48   ` Anup Patel
2013-08-05 12:01     ` Peter Maydell
2013-08-05 12:22       ` Anup Patel
2013-08-05 12:28         ` Peter Maydell
2013-08-05 12:37           ` Anup Patel
2013-08-05 12:43             ` Peter Maydell
2013-08-05 12:49 ` [Qemu-devel] Versioned machine types for ARM/non-x86 ? (Was Re: [PATCH v4 0/2] ARM: add 'virt' platform) Daniel P. Berrange
2013-08-05 13:02   ` Peter Maydell
2013-08-05 13:50     ` Daniel P. Berrange
2013-08-05 13:28   ` Anthony Liguori
2013-08-05 13:49     ` Daniel P. Berrange
2013-08-05 14:12       ` Anthony Liguori
2013-08-05 15:06         ` Gerd Hoffmann [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=51FFBF86.5030802@redhat.com \
    --to=kraxel@redhat.com \
    --cc=aliguori@us.ibm.com \
    --cc=armbru@redhat.com \
    --cc=kvmarm@lists.cs.columbia.edu \
    --cc=m.hamayun@virtualopensystems.com \
    --cc=patches@linaro.org \
    --cc=peter.maydell@linaro.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.