Re: [ovs-dev] [PATCH RFC v3 1/1] netdev-dpdk: Add support for DPDK 16.07

2016-08-03 Thread Loftus, Ciara
> > Given that using vhost PMD doesn't seem viable in the very short term, I > think we should stick with the vhost lib. > I sent a patch for ovsrcu to add a new RCU protected array index. > > http://openvswitch.org/pipermail/dev/2016-August/077097.html > Thanks, > Daniele Thanks Daniele, I

Re: [ovs-dev] [PATCH RFC v3 1/1] netdev-dpdk: Add support for DPDK 16.07

2016-08-02 Thread Daniele Di Proietto
Given that using vhost PMD doesn't seem viable in the very short term, I think we should stick with the vhost lib. I sent a patch for ovsrcu to add a new RCU protected array index. http://openvswitch.org/pipermail/dev/2016-August/077097.html Thanks, Daniele 2016-07-28 6:26 GMT-07:00 Loftus,

Re: [ovs-dev] [PATCH RFC v3 1/1] netdev-dpdk: Add support for DPDK 16.07

2016-08-02 Thread Daniele Di Proietto
2016-07-24 11:06 GMT-07:00 Ben Pfaff : > On Sun, Jul 24, 2016 at 10:17:13AM -0400, Aaron Conole wrote: > > Daniele Di Proietto writes: > > > > > Thanks for the patch. > > > > > > I have another concern with this. If we're still going to rely on RCU > to > > >

Re: [ovs-dev] [PATCH RFC v3 1/1] netdev-dpdk: Add support for DPDK 16.07

2016-07-28 Thread Loftus, Ciara
> > Thanks for the patch. > I have another concern with this.  If we're still going to rely on RCU to > protect > the vhost device (and as pointed out by Ilya, I think we should) we need to > use RCU-like semantics on the vid array index. I'm not sure a boolean flag is > going to be enough. >

Re: [ovs-dev] [PATCH RFC v3 1/1] netdev-dpdk: Add support for DPDK 16.07

2016-07-24 Thread Ben Pfaff
On Sun, Jul 24, 2016 at 10:17:13AM -0400, Aaron Conole wrote: > Daniele Di Proietto writes: > > > Thanks for the patch. > > > > I have another concern with this. If we're still going to rely on RCU to > > protect the vhost device (and as pointed out by Ilya, I think we

Re: [ovs-dev] [PATCH RFC v3 1/1] netdev-dpdk: Add support for DPDK 16.07

2016-07-24 Thread Aaron Conole
Hi Daniele, Daniele Di Proietto writes: > Thanks for the patch. > > I have another concern with this. If we're still going to rely on RCU to > protect the vhost device (and as pointed out by Ilya, I think we should) we > need to use RCU-like semantics on the vid array

Re: [ovs-dev] [PATCH RFC v3 1/1] netdev-dpdk: Add support for DPDK 16.07

2016-07-22 Thread Daniele Di Proietto
Thanks for the patch. I have another concern with this. If we're still going to rely on RCU to protect the vhost device (and as pointed out by Ilya, I think we should) we need to use RCU-like semantics on the vid array index. I'm not sure a boolean flag is going to be enough. CCing Jarno: We

[ovs-dev] [PATCH RFC v3 1/1] netdev-dpdk: Add support for DPDK 16.07

2016-07-22 Thread Ciara Loftus
This commit introduces support for DPDK 16.07 and consequently breaks compatibility with DPDK 16.04. DPDK 16.07 introduces some changes to various APIs. These have been updated in OVS, including: * xstats API: changes to structure of xstats * vhost API: replace virtio-net references with 'vid'