Re: [PATCH v3 3/6] vbus: add a "vbus-proxy" bus model for vbus_driver objects

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Avi Kivity
Date: Tuesday, August 18, 2009 - 10:22 pm

On 08/19/2009 07:27 AM, Gregory Haskins wrote:

I guess he meant venet vs virtio-net.  Without venet vbus is currently 
userless.


Others and I have shown you its wrong.  There's no inherent performance 
problem in pci.  The vbus approach has inherent problems (the biggest of 
which is compatibility, the second managability).


s/kvm/kvm stack/.  virtio/pci is part of the kvm stack, even if it is 
not part of kvm itself.  If vbus/venet were to be merged, users and 
developers would have to choose one or the other.  That's the 
fragmentation I'm worried about.  And you can prefix that with "fact:" 
as well.


That does nothing to improve virtio.  Existing guests (Linux and 
Windows) which support virtio will cease to work if the host moves to 
vbus-virtio.  Existing hosts (running virtio-pci) won't be able to talk 
to newer guests running virtio-vbus.  The patch doesn't improve 
performance without the entire vbus stack in the host kernel and a 
vbus-virtio-net-host host kernel driver.

Perhaps if you posted everything needed to make vbus-virtio work and 
perform we could compare that to vhost-net and you'll see another reason 
why vhost-net is the better approach.


There's no way we can adapt vbus to our needs.  Don't you think we'd 
preferred it rather than writing our own?  the current virtio-net issues 
are hurting us.

Our needs are compatibility, performance, and managability.  vbus fails 
all three, your impressive venet numbers notwithstanding.


You could come up with uses where vbus truly is superior to 
virtio/pci/whatever (not words about etch constraints).  Showing some of 
those non-virt uses, for example.  The fact that your only user 
duplicates existing functionality doesn't help.



That's a step in the right direction.


Note whenever I mention migration, large guests, or Windows you say 
these are not your design requirements.  The best technical solution 
will have to consider those.


I've spent quite a lot of time arguing with you, no doubt influenced by 
the fact that you can write a lot faster than I can read.


That's one of the kvm strong points...

-- 
I have a truly marvellous patch that fixes the bug which this
signature is too narrow to contain.

--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[PATCH v3 0/6] AlacrityVM guest drivers, Gregory Haskins, (Fri Aug 14, 8:42 am)
[PATCH v3 1/6] shm-signal: shared-memory signals, Gregory Haskins, (Fri Aug 14, 8:42 am)
[PATCH v3 4/6] vbus-proxy: add a pci-to-vbus bridge, Gregory Haskins, (Fri Aug 14, 8:43 am)
[PATCH v3 5/6] ioq: add driver-side vbus helpers, Gregory Haskins, (Fri Aug 14, 8:43 am)
[PATCH v3 6/6] net: Add vbus_enet driver, Gregory Haskins, (Fri Aug 14, 8:43 am)
Re: [PATCH v3 3/6] vbus: add a "vbus-proxy" bus model for ..., Avi Kivity, (Tue Aug 18, 10:22 pm)
configfs/sysfs, Avi Kivity, (Wed Aug 19, 1:12 pm)
Re: configfs/sysfs, Ingo Molnar, (Wed Aug 19, 1:48 pm)
Re: configfs/sysfs, Avi Kivity, (Wed Aug 19, 1:53 pm)
Re: configfs/sysfs, Nicholas A. Bellinger, (Wed Aug 19, 2:19 pm)
Re: configfs/sysfs, Gregory Haskins, (Wed Aug 19, 3:15 pm)
Re: configfs/sysfs, Joel Becker, (Wed Aug 19, 3:16 pm)
Re: [Alacrityvm-devel] configfs/sysfs, Alex Tsariounov, (Wed Aug 19, 4:48 pm)
Re: configfs/sysfs, Nicholas A. Bellinger, (Wed Aug 19, 4:54 pm)
Re: configfs/sysfs, Avi Kivity, (Wed Aug 19, 11:09 pm)
Re: configfs/sysfs, Joel Becker, (Thu Aug 20, 3:48 pm)
Re: configfs/sysfs, Avi Kivity, (Thu Aug 20, 9:14 pm)