On Tue, 14 Apr 2015 16:37:34 +0000 "Rogers, Gerald" <gerald.rog...@intel.com> wrote:
> Pravin, > > There are some deployments that are using or require older OS > distributions using qemu versions prior to version 2.1, thus would > require the support for vHost-cuse. Most general reason is > qualification cycles for new OS versions. The patch provides compile > time support for choosing which vHost version to use. The support > for vhost-cuse should be kept for at least several releases with > documentation indicating it will be eventually deprecated. My concern here is to have to fix or improve vhost-cuse forever because of new users. So, I suggest to put a paragraph in the documentation highlighting that vhost-cuse is available only to be used with older qemu up to version <2.1.0/?> and that it will be deprecated in favor of vhost-user in the future. Regarding to the version, the RFC patch says: +DPDK vhost-user VM configuration: +--------------------------------- +DPDK vhost-user works with QEMU v2.2.0. I think that is a typo and the version is actually v2.1.0. Also, dpdk-2.0 is released so now the patch can be merged. Thanks, fbl > > Gerald > > > On 4/2/15, 10:31 PM, "Pravin Shelar" <pshe...@nicira.com> wrote: > > >On Thu, Apr 2, 2015 at 8:37 AM, Traynor, Kevin > ><kevin.tray...@intel.com> wrote: > >> > >>> -----Original Message----- > >>> From: dev [mailto:dev-boun...@openvswitch.org] On Behalf Of Pravin > >>>Shelar > >>> Sent: Friday, March 27, 2015 6:08 PM > >>> To: Flavio Leitner > >>> Cc: dev@openvswitch.org > >>> Subject: Re: [ovs-dev] [PATCH RFC 1/1] netdev-dpdk: add dpdk > >>>vhost-user ports > >>> > >>> On Fri, Mar 27, 2015 at 6:57 AM, Flavio Leitner <f...@sysclose.org> > >>>wrote: > >>> > On Fri, 20 Mar 2015 10:53:14 -0700 > >>> > Pravin Shelar <pshe...@nicira.com> wrote: > >>> > > >>> >> On Thu, Mar 19, 2015 at 11:48 AM, Ciara Loftus > >>> >> <ciara.lof...@intel.com> wrote: > >>> >> > This patch adds support for a new port type to the userspace > >>> >> > datapath called dpdkvhostuser. It adds to the existing > >>> >> > infrastructure of vhost-cuse, however disables vhost-cuse > >>> >> > ports in favour of vhost-user ports. > >>> >> > > >>> >> > A new dpdkvhostuser port will create a unix domain socket > >>> >> > which > >>>when > >>> >> > provided to QEMU is used to facilitate communication between > >>> >> > the virtio-net device on the VM and the OVS port. > >>> >> > > >>> >> Thanks for the patch. I have pushed OVS DPDK vHost cuse patch. > >>> >> Once > >>>we > >>> >> add support for vhost user, vhost-cuse support should be > >>> >> dropped. > >>> > > >>> > Do you mean to literally take vhost-cuse out of the code? If > >>> > so, yes, I think that makes sense. > >>> > > >>> > >>> Yes. thats what I meant. > >> > >> One issue with removing is that vhost-user is only supported from > >> QEMU > >>2.1.0 > >> onwards. If anyone is using an older version of QEMU they would > >> not be > >>able > >> to use vhost-user, whereas vhost-cuse will support older versions > >> of > >>QEMU. > >> > > > >Are there distribution still using older QEMU? > _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
