qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Jan Kiszka <jan.kiszka@web.de>
To: Gleb Natapov <gleb@redhat.com>
Cc: qemu-devel <qemu-devel@nongnu.org>, kvm <kvm@vger.kernel.org>
Subject: [Qemu-devel] Re: qemu-kvm vs. qemu: Terminate cpu loop on reset?
Date: Fri, 07 Jan 2011 20:33:20 +0100	[thread overview]
Message-ID: <4D276A80.6000007@web.de> (raw)
In-Reply-To: <20110107191020.GE10205@redhat.com>

[-- Attachment #1: Type: text/plain, Size: 2416 bytes --]

Am 07.01.2011 20:10, Gleb Natapov wrote:
>>>> We are on a good track now. I predict that we will be left with only one
>>>> or two major additional features in qemu-kvm in a few months from now,
>>>> no more duplications with subtle differences, and production-grade kvm
>>>> upstream stability.
>>>>
>>> You are optimistic. My prediction is that it will take at least one major RHEL
>>> release until such merged code base will become production-grade. That
>>> is when most bugs that were introduced by eliminating subtle differences
>>> between working and non-working version will be found :)
>>
>> The more upstream code qemu-kvm stresses, the faster this convergence
>> will become. And there is really not that much left. E.g, I've a
>> qemu-kvm-x86.c here that is <400 LOC.
>>
> That's what I don't get. Why working qemu-kvm should stress non working
> upstream code? Just remove upstream code and replace it with qemu-kvm
> version.

We are 3/4 (if not more) done with refactoring qemu-kvm into a clean
state, removing lots of cruft from libkvm days and early kvm modules. We
achieved this by creating a "fork of the fork": upstream kvm. We may
argue a lot about pros and cons of this approach, but it is a fact now.
And a lot of effort would be wasted as well by throwing this away.

Moreover, taking off the x86 glasses: ppc and s390 rely on upstream kvm.
So it is impossible to drop those bits without breaking all non-x86 kvm
archs.

> 
>>>
>>> BTW Do you have a plan how to move upstream to thread per vcpu?
>>
>> Upstream has this already, but it's - once again - a different
>> implementation. Understanding those differences is one of the next steps.
>>
> I see only two threads on upstream no matter how much vcpus I configure.

/me sees a lot of them. Did you enable io-thread support? Otherwise kvm
is run just like tcg in single-thread mode.

> 
>> In fact, as posted recently, unifying the execution model
>> implementations is the only big problem I see. In-kernel irqchips and
>> device assignment are things that can live in qemu-kvm without much
>> conflicts until they are finally mergable.
>>
> Upstream kvm is kinda useless without in-kernel irqchips.

Not if its code serves the rest of qemu-kvm without further patches (and
merge conflicts). And we only need to sort out the execution loop and
threading stuff to get there.

Jan


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 259 bytes --]

  reply	other threads:[~2011-01-07 19:33 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-01-07 15:57 [Qemu-devel] qemu-kvm vs. qemu: Terminate cpu loop on reset? Jan Kiszka
2011-01-07 16:53 ` [Qemu-devel] " Gleb Natapov
2011-01-07 16:59   ` Jan Kiszka
2011-01-07 17:16     ` Gleb Natapov
2011-01-07 17:30       ` Jan Kiszka
2011-01-07 17:53         ` Gleb Natapov
2011-01-07 18:24           ` Jan Kiszka
2011-01-07 18:32             ` Jan Kiszka
2011-01-07 19:10             ` Gleb Natapov
2011-01-07 19:33               ` Jan Kiszka [this message]
2011-01-07 21:19                 ` Gleb Natapov
2011-01-08  9:12                   ` Jan Kiszka

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=4D276A80.6000007@web.de \
    --to=jan.kiszka@web.de \
    --cc=gleb@redhat.com \
    --cc=kvm@vger.kernel.org \
    --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).