[ovs-dev] [PATCH v8] netdev-dpdk: add dpdk vhost-cuse ports

[Pravin Shelar]

On Mon, Mar 16, 2015 at 10:27 AM, Traynor, Kevin
<kevin.traynor at intel.com> wrote:
>
>> -----Original Message-----
>> From: Pravin Shelar [mailto:pshelar at nicira.com]
>> Sent: Monday, March 16, 2015 2:46 AM
>> To: Traynor, Kevin
>> Cc: dev at openvswitch.org
>> Subject: Re: [ovs-dev] [PATCH v8] netdev-dpdk: add dpdk vhost-cuse ports
>>
>> On Thu, Mar 12, 2015 at 7:18 AM, Traynor, Kevin <kevin.traynor at intel.com>
>> wrote:
>> >
>> >> -----Original Message-----
>> >> From: Pravin Shelar [mailto:pshelar at nicira.com]
>> >> Sent: Thursday, March 12, 2015 7:20 AM
>> >> To: Traynor, Kevin
>> >> Cc: dev at openvswitch.org
>> >> Subject: Re: [ovs-dev] [PATCH v8] netdev-dpdk: add dpdk vhost-cuse ports
>> >>
>> >> On Thu, Mar 5, 2015 at 1:42 PM, Kevin Traynor <kevin.traynor at intel.com>
>> >> wrote:
>> >> > This patch adds support for a new port type to userspace datapath
>> >> > called dpdkvhost. This allows KVM (QEMU) to offload the servicing
>> >> > of virtio-net devices to its associated dpdkvhost port. Instructions
>> >> > for use are in INSTALL.DPDK.
>> >> >
>> >> > This has been tested on Intel multi-core platforms and with clients
>> >> > that have virtio-net interfaces.
>> >> >
>> >>
>> >> What is required QEMU version for this patch ?
>> >
>> > It's been tested with 2.1.0 and 2.2.0. It has also previously been tested
>> > with 1.6.2 but only the '-mem-path /dev/hugepages -mem-prealloc' flags are
>> > needed to share the hugepage for that version. It won't work with 2.0.0 as
>> > there isn't a way to share the hugepage.
>> >
>>
>> Thanks for the info. I will try it this week.
>> I had another question about netdev-dpdk mutex, why is it converted to
>> spin_lock?
>
> We changed it based on this review comment:
> ' - ovs-mutex is bit heavy weight, can you use rte-spin-lock '
>
But the mutex is not used in packet rx or tx path. So it should not matter.

> There was a few different mutexes, so we thought it was a general comment and
> converted them to spinlocks.