qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Anthony Liguori <anthony@codemonkey.ws>
To: Alexander Graf <agraf@suse.de>
Cc: Anthony PERARD <anthony.perard@citrix.com>,
	QEMU-devel <qemu-devel@nongnu.org>
Subject: Re: [Qemu-devel] [PATCH] Introduce -accel command option.
Date: Tue, 16 Nov 2010 11:20:50 -0600	[thread overview]
Message-ID: <4CE2BD72.5030502@codemonkey.ws> (raw)
In-Reply-To: <E5DEFA0B-2A77-4F56-B8AA-76DBCBC11C3B@suse.de>

On 11/16/2010 10:59 AM, Alexander Graf wrote:
>
>> Surely, only if it is running on a Xen Dom0. If you use -M xenpv on a KVM
>> host, then -M xenpv should imply -machine accel=kvm (ie it would be using
>> xenner)
>>      
> Actually, it should imply -machine accel=kvm,tcg :). Accelerators really are not a machine property. In an ideal world, -M pc would just work with xen hvm if -accel xen is given.
>    

No, an accelerator is both a CPU selection and a machine 
characteristic.  For KVM, we overload -cpu to modify both the KVM CPU 
and the TCG CPU both this won't work with accel=xen.  We probably 
shouldn't do this with KVM either because there's a significant 
different between trying to do cpuid masking with KVM and modifying the 
TCG cpu emulation support.

Both KVM and Xen have other impacts on the platform devices though.  KVM 
does not support SMM so it disables that in the i440fx.  KVM prefers to 
use it's own in-kernel local APIC (and IOAPIC).  That makes it a 
property of the machine.

Regards,

Anthony Liguori

> Alex
>
>    

  reply	other threads:[~2010-11-16 17:20 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-11-15 15:45 [Qemu-devel] [PATCH] Introduce -accel command option anthony.perard
2010-11-15 16:54 ` Anthony Liguori
2010-11-16 16:10   ` Anthony PERARD
2010-11-16 16:41     ` Anthony Liguori
2010-11-16 16:55       ` Daniel P. Berrange
2010-11-16 16:59         ` Alexander Graf
2010-11-16 17:20           ` Anthony Liguori [this message]
2010-11-16 17:24             ` Alexander Graf
2010-11-16 17:27               ` Anthony Liguori
2010-11-16 17:32                 ` Alexander Graf
2010-11-16 18:22                 ` Anthony PERARD
2010-11-16 18:49                   ` Alexander Graf
2010-11-16 18:57                     ` Anthony Liguori
2010-11-16 17:07         ` Anthony Liguori

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=4CE2BD72.5030502@codemonkey.ws \
    --to=anthony@codemonkey.ws \
    --cc=agraf@suse.de \
    --cc=anthony.perard@citrix.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).