From: Eduardo Habkost <ehabkost@redhat.com>
To: Paolo Bonzini <pbonzini@redhat.com>
Cc: qemu-devel@nongnu.org, "Andreas Färber" <afaerber@suse.de>,
kvm@vger.kernel.org, "Aurelien Jarno" <aurelien@aurel32.net>
Subject: Re: [PATCH v2 0/6] target-i386: Make most CPU models work with "enforce" out of the box
Date: Tue, 26 Aug 2014 15:01:07 -0300 [thread overview]
Message-ID: <20140826180107.GD32084@thinpad.lan.raisama.net> (raw)
In-Reply-To: <53FC83F5.2090905@redhat.com>
On Tue, Aug 26, 2014 at 02:56:21PM +0200, Paolo Bonzini wrote:
> Il 25/08/2014 22:45, Eduardo Habkost ha scritto:
> >
> > TCG users expect the default CPU model to contain most TCG-supported features
> > (and it makes sense). See, for example, commit
> > f1e00a9cf326acc1f2386a72525af8859852e1df.
>
> It doesn't though (SMAP is the most egregious omission, and probably the
> main reason why people use QEMU TCG these days), and it raises the
> question of backwards-compatibility of qemu64---should we disable TCG
> features in old machine types? Probably yes, but we've never done that.
Had we changed qemu64, any changes to the feature set of qemu64 would
probably require compatibility code on old machine-types for KVM,
anyway. But the last time qemu64 was changed was in 2009 (commit
f1e00a9cf326acc1f2386a72525af8859852e1df), it looks like everybody was
afraid of touching "qemu64" because its purpose was not very clear.
So maybe that's good news, as things can be simpler if we make both TCG
and KVM have similar behavior:
* qemu64: a conservative default that should work out of the box on
most systems, for both TCG and KVM. That's already the current status,
we just need to document it.
* -cpu host: for people who want every possible feature to be enabled
(but without cross-version live-migration support). We can easily add
support for "-cpu host" to TCG, too.
--
Eduardo
next prev parent reply other threads:[~2014-08-26 18:01 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-08-25 20:45 [PATCH v2 0/6] target-i386: Make most CPU models work with "enforce" out of the box Eduardo Habkost
2014-08-25 20:45 ` [PATCH v2 1/6] pc: Create pc_compat_2_1() functions Eduardo Habkost
2014-08-25 20:45 ` [PATCH v2 2/6] target-i386: Rename KVM auto-feature-enable compat function Eduardo Habkost
2014-08-25 20:45 ` [PATCH v2 3/6] target-i386: Disable CPUID_ACPI by default on KVM mode Eduardo Habkost
2014-08-25 20:45 ` [PATCH v2 4/6] target-i386: Remove unsupported bits from all CPU models Eduardo Habkost
2014-08-25 20:45 ` [PATCH v2 5/6] target-i386: Don't enable nested VMX by default Eduardo Habkost
2014-08-25 20:45 ` [PATCH v2 6/6] target-i386: Disable SVM by default in KVM mode Eduardo Habkost
2014-08-26 12:56 ` [PATCH v2 0/6] target-i386: Make most CPU models work with "enforce" out of the box Paolo Bonzini
2014-08-26 18:01 ` Eduardo Habkost [this message]
2014-08-27 13:36 ` Paolo Bonzini
2014-08-27 14:05 ` Eduardo Habkost
2014-08-27 14:33 ` Paolo Bonzini
2014-08-27 15:42 ` Eduardo Habkost
2014-08-27 15:58 ` Andreas Färber
2014-08-27 16:08 ` Eduardo Habkost
2014-08-27 16:14 ` Andreas Färber
2014-08-27 16:18 ` Paolo Bonzini
2014-08-27 16:34 ` Eduardo Habkost
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=20140826180107.GD32084@thinpad.lan.raisama.net \
--to=ehabkost@redhat.com \
--cc=afaerber@suse.de \
--cc=aurelien@aurel32.net \
--cc=kvm@vger.kernel.org \
--cc=pbonzini@redhat.com \
--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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox