From: Paul Brook <paul@codesourcery.com>
To: qemu-devel@nongnu.org
Cc: Andrea Arcangeli <aarcange@redhat.com>
Subject: Re: [Qemu-devel] allow sysenter on 32bit guests running on vmx host
Date: Thu, 25 Jun 2009 23:12:08 +0100 [thread overview]
Message-ID: <200906252312.08945.paul@codesourcery.com> (raw)
In-Reply-To: <20090625210249.GB12992@random.random>
On Thursday 25 June 2009, Andrea Arcangeli wrote:
> On Thu, Jun 25, 2009 at 06:39:19PM +0100, Paul Brook wrote:
> > > What's missing is a comment saying "KVM changes this vendor id to
> > > Intel when run on a VMX-capable host".
> >
> > I don't see how this is anything other than a KVM bug. The user requested
> > an AMD vendor ID. Arbitrarily changing it to something else is just plain
> > wrong.
>
> Oh my, no -cpu is passed on the command line, and by default the only
> thing the user asks is always only one: performance.
I don't buy this. I'd expect a a good proportion of users to care more about
correct operation over absolute performance. A fast VM is no good if it
doesn't actually work.
On closer inspection I notice that we use an AMD vendor ID for the "qemu64"
cpu. IMO this is wrong, we should be using our own ID. However this does not
change the underlying problem - KVM absolutely should not be unilaterally
changing the reported vendor ID. Maybe select a different CPU by default, and
probably fail to run if the selected CPU ID is incompatible with the host
hardware features, but not arbitrarily mutate the provided CPUID.
Paul
next prev parent reply other threads:[~2009-06-25 22:12 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-06-23 16:21 [Qemu-devel] allow sysenter on 32bit guests running on vmx host Andrea Arcangeli
2009-06-24 17:29 ` Jamie Lokier
2009-06-24 17:48 ` Filip Navara
2009-06-24 21:13 ` Andrea Arcangeli
2009-06-24 21:12 ` Andrea Arcangeli
2009-06-24 21:39 ` Jamie Lokier
2009-06-24 22:32 ` Andrea Arcangeli
2009-06-25 8:11 ` [Qemu-devel] KVMs default CPU type (was: allow sysenter on 32bit guests running on vmx host) Andre Przywara
2009-06-25 8:29 ` [Qemu-devel] KVMs default CPU type Avi Kivity
2009-06-26 0:42 ` [Qemu-devel] KVMs default CPU type (was: allow sysenter on 32bit guests running on vmx host) Andrea Arcangeli
2009-06-26 1:06 ` Andrea Arcangeli
2009-06-25 17:39 ` [Qemu-devel] allow sysenter on 32bit guests running on vmx host Paul Brook
2009-06-25 21:02 ` Andrea Arcangeli
2009-06-25 22:12 ` Paul Brook [this message]
2009-06-25 23:27 ` Andrea Arcangeli
2009-06-25 23:49 ` Paul Brook
2009-06-26 0:06 ` Andrea Arcangeli
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=200906252312.08945.paul@codesourcery.com \
--to=paul@codesourcery.com \
--cc=aarcange@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;
as well as URLs for NNTP newsgroup(s).