All of lore.kernel.org
 help / color / mirror / Atom feed
From: Gregory Haskins <gregory.haskins@gmail.com>
To: Pantelis Koukousoulas <pktoss@gmail.com>
Cc: kvm@vger.kernel.org, alacrityvm-devel@lists.sourceforge.net
Subject: Re: [PATCH 0/7] AlacrityVM guest drivers Reply-To:
Date: Fri, 07 Aug 2009 08:14:20 -0400	[thread overview]
Message-ID: <4A7C1A9C.9000600@gmail.com> (raw)
In-Reply-To: <1295ed070908060928s7b02e8bbtaf2b9b5c35679c0d@mail.gmail.com>

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

[not sure if it was intentional, but you dropped the CC list.
Therefore, I didn't see this until I caught up on my kvm@vger reading]

Pantelis Koukousoulas wrote:
> How hard would it be to implement virtio over vbus and perhaps the
> virtio-net backend?

It should be relatively trivial.  I have already written the transport
(called virtio-vbus) that would allow the existing front-end
(virtio-net) to work without modification.

http://lkml.org/lkml/2009/4/21/427

All that is needed is to take venet-tap as an example and port it to
something virtio compatible (via that patch I posted) on the backend.  I
have proposed this as an alternative to venet, but so far I have not had
any takers to help with this effort.  Likewise, I am too busy with the
infrastructure to take this on myself.


> 
> This would leave only one variable in the comparison, clear misconceptions and
> make evaluation easier by judging each of vbus, venet etc separately on its own
> merits.
> 
> The way things are now, it is unclear exactly where those performance
> improvements are coming from (or how much each component contributes)
> because there are too many variables.
> 
> Replacing virtio-net by venet would be a hard proposition if only because
> virtio-net has (closed source) windows drivers available. There has to be
> shown that venet by itself does something significantly better that
> virtio-net can't be modified to do comparably well.

I am not proposing anyone replace virtio-net.  It will continue to work
fine despite the existence of an alternative, and KVM can continue to
standardize on it if that is what KVM wants to do.

> 
> Having venet in addition to virtio-net is also difficult, given that having only
> one set of paravirtual drivers in the kernel was the whole point behind virtio.

As it stands right now, virtio-net fails to meet my performance goals,
and venet meets them (or at least, gets much closer, but I will not
rest..).  So, at least for AlacrityVM, I will continue to use and
promote it when performance matters.  If at some time in the future I
can get virtio-net to work in my environment in a comparable and
satisfactory way, I will consider migrating to it and deprecating venet.

Until then, having two drivers is ok, and no-one has to use the one they
don't like.  I certainly do not think having more than one driver that
speaks 802.x ethernet in the kernel tree is without precedent. ;)

Kind Regards,
-Greg


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

  reply	other threads:[~2009-08-07 12:14 UTC|newest]

Thread overview: 62+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-08-03 17:17 [PATCH 0/7] AlacrityVM guest drivers Gregory Haskins
2009-08-03 17:17 ` [PATCH 1/7] shm-signal: shared-memory signals Gregory Haskins
2009-08-06 13:56   ` Arnd Bergmann
2009-08-06 15:11     ` Gregory Haskins
2009-08-06 20:51       ` Ira W. Snyder
2009-08-03 17:17 ` [PATCH 2/7] ioq: Add basic definitions for a shared-memory, lockless queue Gregory Haskins
2009-08-03 17:17 ` [PATCH 3/7] vbus: add a "vbus-proxy" bus model for vbus_driver objects Gregory Haskins
2009-08-03 17:17 ` [PATCH 4/7] vbus-proxy: add a pci-to-vbus bridge Gregory Haskins
2009-08-06 14:42   ` Arnd Bergmann
2009-08-06 15:59     ` Gregory Haskins
2009-08-06 17:03       ` Arnd Bergmann
2009-08-06 21:04         ` Gregory Haskins
2009-08-06 22:57           ` Arnd Bergmann
2009-08-07  4:42             ` Gregory Haskins
2009-08-07 14:57               ` Arnd Bergmann
2009-08-07 15:44                 ` Gregory Haskins
2009-08-07 15:44                   ` Gregory Haskins
2009-08-07 15:55               ` Ira W. Snyder
2009-08-07 18:25                 ` Gregory Haskins
2009-08-03 17:17 ` [PATCH 5/7] ioq: add driver-side vbus helpers Gregory Haskins
2009-08-03 17:18 ` [PATCH 6/7] net: Add vbus_enet driver Gregory Haskins
2009-08-03 18:30   ` Stephen Hemminger
2009-08-03 20:10     ` Gregory Haskins
2009-08-03 20:19       ` Stephen Hemminger
2009-08-03 20:24         ` Gregory Haskins
2009-08-03 20:29           ` Stephen Hemminger
2009-08-04  1:14   ` [PATCH v2] " Gregory Haskins
2009-08-04  2:38     ` David Miller
2009-08-04 13:57       ` [Alacrityvm-devel] " Gregory Haskins
2009-10-02 15:33     ` [PATCH v3] " Gregory Haskins
2009-08-03 17:18 ` [PATCH 7/7] venet: add scatter-gather/GSO support Gregory Haskins
2009-08-03 18:32   ` Stephen Hemminger
2009-08-03 19:30     ` Gregory Haskins
2009-08-03 18:33   ` Stephen Hemminger
2009-08-03 19:57     ` Gregory Haskins
2009-08-06  8:19 ` [PATCH 0/7] AlacrityVM guest drivers Reply-To: Michael S. Tsirkin
2009-08-06 10:17   ` Michael S. Tsirkin
2009-08-06 12:09     ` Gregory Haskins
2009-08-06 12:08   ` Gregory Haskins
2009-08-06 12:24     ` Michael S. Tsirkin
2009-08-06 13:00       ` Gregory Haskins
2009-08-06 12:54     ` Avi Kivity
2009-08-06 13:03       ` Gregory Haskins
2009-08-06 13:44         ` Avi Kivity
2009-08-06 13:45           ` Gregory Haskins
2009-08-06 13:57             ` Avi Kivity
2009-08-06 14:06               ` Gregory Haskins
2009-08-06 15:40                 ` Arnd Bergmann
2009-08-06 15:45                   ` Michael S. Tsirkin
2009-08-06 16:28                     ` Pantelis Koukousoulas
2009-08-07 12:14                       ` Gregory Haskins [this message]
2009-08-06 15:50                   ` Avi Kivity
2009-08-06 16:55                     ` Gregory Haskins
2009-08-09  7:48                       ` Avi Kivity
2009-08-06 16:29                   ` Gregory Haskins
2009-08-06 23:23                     ` Ira W. Snyder
2009-08-06 13:59             ` Michael S. Tsirkin
2009-08-06 14:07               ` Gregory Haskins
2009-08-07 14:19   ` Anthony Liguori
2009-08-07 15:05     ` [PATCH 0/7] AlacrityVM guest drivers Gregory Haskins
2009-08-07 15:46       ` Anthony Liguori
2009-08-07 18:04         ` Gregory Haskins

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=4A7C1A9C.9000600@gmail.com \
    --to=gregory.haskins@gmail.com \
    --cc=alacrityvm-devel@lists.sourceforge.net \
    --cc=kvm@vger.kernel.org \
    --cc=pktoss@gmail.com \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.