From: Avi Kivity <avi@redhat.com>
To: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com>
Cc: Ingo Molnar <mingo@elte.hu>,
Anthony Liguori <anthony@codemonkey.ws>,
Andi Kleen <andi@firstfloor.org>,
Gregory Haskins <gregory.haskins@gmail.com>,
kvm@vger.kernel.org, Andrew Morton <akpm@linux-foundation.org>,
torvalds@linux-foundation.org,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
netdev@vger.kernel.org,
"alacrityvm-devel@lists.sourceforge.net"
<alacrityvm-devel@lists.sourceforge.net>
Subject: Re: [GIT PULL] AlacrityVM guest drivers for 2.6.33
Date: Wed, 23 Dec 2009 15:31:11 +0200 [thread overview]
Message-ID: <4B321B9F.6030707@redhat.com> (raw)
In-Reply-To: <200912231407.20130.bzolnier@gmail.com>
On 12/23/2009 03:07 PM, Bartlomiej Zolnierkiewicz wrote:
>
>> That is a very different situation from the AlacrityVM patches, which:
>>
>> - Are a pure software concept and any compatibility mismatch is
>> self-inflicted. The patches are in fact breaking the ABI to KVM
>> intentionally (for better or worse).
>>
> Care to explain the 'breakage' and why KVM is more special in this regard
> than other parts of the kernel (where we don't keep any such requirements)?
>
The device model is exposed to the guest. If you change it, the guest
breaks.
So we have two options:
- phase out virtio, users don't see new improvements, ask them to
change to vbus/venet
- maintain the two in parallel
Neither appeals to me.
> Truth to be told KVM is just another driver/subsystem and Gregory's changes
> are only 4KLOC of clean and easily maintainable code..
>
This 4K is only the beginning. There are five more virtio drivers, plus
features in virtio-net not ported to venet, plus the host support, plus
qemu support, plus Windows drivers, plus adapters for non-pci (lguest
and s390), plus live migration support. vbus itself still has scaling
issues.
Virtio was under development for years. Sure you can focus on one
dimension only (performance) and get good results but real life is more
complicated.
> I certainly missed the time when KVM became officially part of core ABI..
>
It's more akin to the hardware interface. We don't change the hardware
underneath the guest.
>> Overlap and forking can still be done in special circumstances, when a project
>> splits and a hostile fork is inevitable due to prolongued and irreconcilable
>> differences between the parties and if there's no strong technical advantage
>> on either side. I havent seen evidence of this yet though: Gregory claims that
>> he wants to 'work with the community' and the KVM guys seem to agree violently
>> that performance can be improved - and are doing so (and are asking Gregory to
>> take part in that effort).
>>
> How it is different from any past forks?
>
> The odium of proving that the existing framework is sufficient was always on
> original authors or current maintainers.
>
> KVM guys were offered assistance from Gregory and had few months to prove that
> they can get the same kind of performance using existing architecture and they
> DID NOT do it.
>
Look at the results from Chris Wright's presentation. Hopefully in a
few days some results from vhost-net.
> Then please try harder. Gregory posted his initial patches in August,
> it is December now and we only see artificial road-blocks instead of code
> from KVM folks.
>
What artificial road blocks?
--
error compiling committee.c: too many arguments to function
next prev parent reply other threads:[~2009-12-23 13:31 UTC|newest]
Thread overview: 105+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-12-07 18:53 [GIT PULL] AlacrityVM guest drivers for 2.6.33 Gregory Haskins
2009-12-18 21:51 ` Ingo Molnar
2009-12-21 15:34 ` Gregory Haskins
2009-12-21 15:43 ` Avi Kivity
2009-12-21 16:04 ` Gregory Haskins
2009-12-21 16:37 ` Anthony Liguori
2009-12-21 16:40 ` Avi Kivity
2009-12-21 16:56 ` Gregory Haskins
2009-12-21 17:05 ` Avi Kivity
2009-12-21 17:24 ` Gregory Haskins
2009-12-21 16:46 ` Gregory Haskins
2009-12-21 17:20 ` Anthony Liguori
2009-12-21 17:44 ` Gregory Haskins
2009-12-22 0:12 ` Anthony Liguori
2009-12-22 7:59 ` Ingo Molnar
2009-12-22 11:49 ` Andi Kleen
2009-12-22 15:31 ` Anthony Liguori
2009-12-22 16:01 ` Bartlomiej Zolnierkiewicz
2009-12-22 16:21 ` Anthony Liguori
2009-12-23 6:51 ` Ingo Molnar
2009-12-23 10:13 ` Andi Kleen
2009-12-23 10:22 ` Avi Kivity
2009-12-23 12:14 ` Andi Kleen
2009-12-23 12:48 ` Avi Kivity
2009-12-23 17:00 ` Chris Wright
2009-12-23 17:20 ` Andi Kleen
2009-12-23 15:03 ` Anthony Liguori
2009-12-23 17:52 ` [Alacrityvm-devel] " Peter W. Morreale
2009-12-23 18:17 ` Gregory Haskins
2009-12-23 18:22 ` Chris Wright
2009-12-23 18:15 ` Gregory Haskins
2009-12-23 20:20 ` Avi Kivity
2009-12-23 21:21 ` Gregory Haskins
2009-12-27 9:15 ` Avi Kivity
2009-12-27 13:18 ` Gregory Haskins
2009-12-27 13:27 ` Avi Kivity
2009-12-27 13:39 ` Gregory Haskins
2009-12-27 13:49 ` Avi Kivity
2009-12-28 1:01 ` Gregory Haskins
2009-12-23 18:51 ` Ingo Molnar
2009-12-23 19:27 ` Andi Kleen
2009-12-23 20:26 ` Avi Kivity
2009-12-24 6:58 ` Gleb Natapov
2009-12-24 20:41 ` Roland Dreier
2009-12-23 13:07 ` Bartlomiej Zolnierkiewicz
2009-12-23 13:31 ` Avi Kivity [this message]
2009-12-23 14:08 ` Bartlomiej Zolnierkiewicz
2009-12-23 14:28 ` Avi Kivity
2009-12-23 14:57 ` Anthony Liguori
2009-12-23 18:31 ` Chris Wright
2009-12-23 16:44 ` Gregory Haskins
2009-12-23 17:10 ` Andi Kleen
2009-12-23 17:17 ` Gregory Haskins
2009-12-23 17:29 ` Linus Torvalds
2009-12-23 23:27 ` Anthony Liguori
2009-12-23 17:33 ` Andi Kleen
2009-12-23 20:36 ` Avi Kivity
2009-12-23 21:01 ` Avi Kivity
2009-12-24 9:36 ` Gregory Haskins
2009-12-27 9:33 ` Avi Kivity
2009-12-27 13:34 ` Gregory Haskins
2009-12-27 13:49 ` Avi Kivity
2009-12-27 14:29 ` Gregory Haskins
2009-12-24 9:31 ` Gregory Haskins
2009-12-27 9:29 ` Avi Kivity
2009-12-22 16:21 ` Andi Kleen
2009-12-22 16:27 ` Anthony Liguori
2009-12-22 17:33 ` Andi Kleen
2009-12-22 17:35 ` Avi Kivity
2009-12-22 21:14 ` Anthony Liguori
2009-12-23 0:02 ` Chris Wright
2009-12-23 15:12 ` Anthony Liguori
2009-12-23 15:17 ` Chris Wright
2009-12-22 17:05 ` Avi Kivity
2009-12-22 20:41 ` Gregory Haskins
2009-12-22 7:22 ` Gleb Natapov
2009-12-22 7:57 ` Ingo Molnar
2009-12-22 17:36 ` Gregory Haskins
2009-12-22 18:53 ` Avi Kivity
2009-12-22 18:56 ` Gregory Haskins
2009-12-22 19:15 ` Gregory Haskins
2009-12-22 19:25 ` Avi Kivity
2009-12-22 19:32 ` Gregory Haskins
2009-12-22 19:37 ` Gregory Haskins
2009-12-22 19:38 ` Avi Kivity
2009-12-22 19:41 ` Gregory Haskins
2009-12-22 19:43 ` Avi Kivity
2009-12-22 19:47 ` Gregory Haskins
2009-12-22 19:40 ` Avi Kivity
2009-12-22 19:39 ` Davide Libenzi
2009-12-22 19:53 ` Gregory Haskins
2009-12-23 1:05 ` Davide Libenzi
2009-12-23 6:15 ` Kyle Moffett
2009-12-23 15:09 ` Anthony Liguori
2009-12-23 19:54 ` [Alacrityvm-devel] " Ira W. Snyder
2009-12-23 22:58 ` Anthony Liguori
2009-12-23 23:42 ` Ira W. Snyder
2009-12-24 17:09 ` Anthony Liguori
2009-12-25 0:38 ` Ira W. Snyder
2009-12-24 4:52 ` Kyle Moffett
2009-12-24 16:57 ` Anthony Liguori
2009-12-23 17:34 ` Gregory Haskins
2009-12-23 19:28 ` Ira W. Snyder
2009-12-23 19:50 ` Andi Kleen
2009-12-24 10:01 ` Michael S. Tsirkin
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=4B321B9F.6030707@redhat.com \
--to=avi@redhat.com \
--cc=akpm@linux-foundation.org \
--cc=alacrityvm-devel@lists.sourceforge.net \
--cc=andi@firstfloor.org \
--cc=anthony@codemonkey.ws \
--cc=bzolnier@gmail.com \
--cc=gregory.haskins@gmail.com \
--cc=kvm@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mingo@elte.hu \
--cc=netdev@vger.kernel.org \
--cc=torvalds@linux-foundation.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).