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
>
>
next prev parent 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).