qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Anthony Liguori <anthony@codemonkey.ws>
To: qemu-devel@nongnu.org
Cc: Paul Brook <paul@codesourcery.com>
Subject: Re: [Qemu-devel] Re: KQEMU code organization
Date: Thu, 29 May 2008 12:43:57 -0500	[thread overview]
Message-ID: <483EEB5D.1030004@codemonkey.ws> (raw)
In-Reply-To: <20080529163536.GD21610@shareable.org>

Jamie Lokier wrote:
> Paul Brook wrote:
>   
>>> I.e. kvm has two sub-modules for Intel VT and AMD SVM extensions (I
>>> think that's their names).  It would be great if it hard a third KQEMU
>>> sub-module (which would of course be the most complicated ;-)
>>>       
>> I believe this is also a prerequisite for getting kqemu merged into
>> maintream kernels, which IMHO is the only sane goal to have. Out of
>> tree kernel modules simply aren't worth the effort.
>>     
>
> I think there's utility in crossover between both of them too.
>   

There are some architectural incompatibilities.  For instance, KVM 
support guest SMP but the code TCG generates does not ensure atomic 
operations are truly atomic.  In general, it may not be possible to do 
this across architectures without employing the use of a big lock.

Also, when you mix dynamic translation in userspace with direct 
execution, it implies you have to completely flush the shadow page table 
cache.  This is going to severely impact performance so I don't know 
that there are a lot of circumstances where using TCG would improve 
performance.

KVM already does some instruction patching FWIW.  For instance, TPR 
accesses are modified in Windows guests to prevent a vmexit from 
occurring since Windows accesses the TPR so frequently.

Regards,

Anthony Liguori

> Sometimes it would be nice to have the speed and directness of kvm,
> with the code scanning and replacement abilities of kqemu to block
> particular instructions, pretend to be a specific CPU model, or
> replace some hardware-accessing instruction sequences instead of
> trapping and emulating them - without the guest seeing the replacement.
>
> -- Jamie
>
>
>   

  reply	other threads:[~2008-05-29 17:52 UTC|newest]

Thread overview: 31+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-05-27 16:56 [Qemu-devel] KQEMU code organization Jan Kiszka
2008-05-27 17:20 ` Ben Taylor
2008-05-27 18:25   ` [Qemu-devel] " Jan Kiszka
2008-05-27 20:58 ` [Qemu-devel] " Fabrice Bellard
2008-05-27 21:40   ` [Qemu-devel] " Jan Kiszka
2008-05-27 22:11 ` [Qemu-devel] " Fabrice Bellard
2008-05-28 16:02   ` [Qemu-devel] " Jan Kiszka
2008-05-28 16:37     ` Fabrice Bellard
2008-05-28 16:55       ` Jan Kiszka
2008-05-28 18:34         ` Jan Kiszka
2008-05-29 12:29         ` Fabrice Bellard
2008-05-29 13:16           ` Jan Kiszka
2008-05-29 16:13           ` Jamie Lokier
2008-05-29 16:26             ` Paul Brook
2008-05-29 16:35               ` Jamie Lokier
2008-05-29 17:43                 ` Anthony Liguori [this message]
2008-05-29 21:46                   ` Fabrice Bellard
2008-05-30  3:32                     ` Mulyadi Santosa
2008-05-30  8:14                       ` Andreas Färber
2008-05-29 16:26             ` Anthony Liguori
2008-05-29 16:53               ` Jan Kiszka
2008-05-29 17:48                 ` Anthony Liguori
2008-05-31 10:18                 ` Avi Kivity
2008-06-02 16:34                   ` Jamie Lokier
2008-05-29 21:52               ` Fabrice Bellard
2008-05-31 10:06                 ` Avi Kivity
2008-06-01 22:58                 ` Anthony Liguori
2008-06-02  9:02                   ` Fabrice Bellard
2008-06-02 13:25                     ` Anthony Liguori
2008-05-29 16:48             ` Jan Kiszka
2008-05-29 17:47               ` 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=483EEB5D.1030004@codemonkey.ws \
    --to=anthony@codemonkey.ws \
    --cc=paul@codesourcery.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).