From: Avi Kivity <avi@redhat.com>
To: Gregory Haskins <gregory.haskins@gmail.com>
Cc: Andi Kleen <andi@firstfloor.org>,
kvm@vger.kernel.org,
Bartlomiej Zolnierkiewicz <bzolnier@gmail.com>,
netdev@vger.kernel.org,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
"alacrityvm-devel@lists.sourceforge.net"
<alacrityvm-devel@lists.sourceforge.net>,
Anthony Liguori <anthony@codemonkey.ws>,
Ingo Molnar <mingo@elte.hu>,
torvalds@linux-foundation.org,
Andrew Morton <akpm@linux-foundation.org>
Subject: Re: [Alacrityvm-devel] [GIT PULL] AlacrityVM guest drivers for 2.6.33
Date: Wed, 23 Dec 2009 22:20:32 +0200 [thread overview]
Message-ID: <4B327B90.2050403@redhat.com> (raw)
In-Reply-To: <4B325E33.8060407@gmail.com>
On 12/23/2009 08:15 PM, Gregory Haskins wrote:
> On 12/23/09 5:22 AM, Avi Kivity wrote:
>
>
>> There was no attempt by Gregory to improve virtio-net.
>>
> If you truly do not understand why your statement is utterly wrong at
> this point in the discussion, I feel sorry for you. If you are trying
> to be purposely disingenuous, you should be ashamed of yourself. In any
> case, your statement is demonstrably bogus, but you should already know
> this given that we talked about at least several times.
>
There's no need to feel sorry for me, thanks. There's no reason for me
to be ashamed, either. And there's no need to take the discussion to
personal levels. Please keep it technical.
> To refresh your memory: http://patchwork.kernel.org/patch/17428/
>
This is not an attempt to improve virtio-net, it's an attempt to push
vbus. With this, virtio-net doesn't become any faster, since the
greatest bottleneck is not removed, it remains in userspace.
If you wanted to improve virtio-net, you would port venet-host to the
virtio-net guest/host interface, and port any secret sauce in
venet(-guest) to virtio-net. After that we could judge what vbus'
contribution to the equation is.
> In case its not blatantly clear, which I would hope it would be to
> anyone that understands the problem space: What that patch would do is
> allow an unmodified virtio-net to bridge to a vbus based virtio-net
> backend. (Also note that this predates vhost-net by months (the date in
> that thread is 4/9/2009) in case you are next going to try to argue that
> it does nothing over vhost-net).
>
Without the backend, it is useless. It demonstrates vbus' flexibility
quite well, but does nothing for virtio-net or its users, at least
without a lot more work.
> This would mean that virtio-net would gain most of the benefits I have
> been advocating (fewer exits, cheaper exits, concurrent execution, etc).
> So this would very much improve virtio-net indeed, given how poorly the
> current backend was performing. I tried to convince the team to help me
> build it out to completion on multiple occasions, but that request was
> answered with "sorry, we are doing our own thing instead". You can say
> that you didn't like my approach, since that is a subjective opinion.
> But to say that I didn't attempt to improve it is a flat out wrong, and
> I do not appreciate it.
>
Cutting down on the rhetoric is more important than cutting down exits
at this point in time.
--
I have a truly marvellous patch that fixes the bug which this
signature is too narrow to contain.
next prev parent reply other threads:[~2009-12-23 20:21 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 [this message]
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
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=4B327B90.2050403@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).