qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
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

  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).