Re: [ovs-dev] [PATCH] netdev-dpdk: Modify rings creation attributes

2015-11-16 Thread Mauricio Vásquez
I have sent a new patch modifying the commit title and commit message: http://openvswitch.org/pipermail/dev/2015-November/062257.html Mauricio V, On 16 November 2015 at 19:39, Flavio Leitner wrote: > On Mon, Nov 16, 2015 at 11:25:11AM +0100, Mauricio Vásquez wrote: > > On 16 November 2015 at 06

Re: [ovs-dev] [PATCH] netdev-dpdk: Modify rings creation attributes

2015-11-16 Thread Flavio Leitner
On Mon, Nov 16, 2015 at 11:25:11AM +0100, Mauricio Vásquez wrote: > On 16 November 2015 at 06:48, Flavio Leitner wrote: > > On Wed, Nov 11, 2015 at 09:22:39AM +0100, Mauricio Vásquez wrote: > > > Although netdev does explicit locking, it is only valid from the ovs > > > perspective, then only the

Re: [ovs-dev] [PATCH] netdev-dpdk: Modify rings creation attributes

2015-11-16 Thread Mauricio Vásquez
On 16 November 2015 at 06:48, Flavio Leitner wrote: > On Wed, Nov 11, 2015 at 09:22:39AM +0100, Mauricio Vásquez wrote: > > The commit title can be more clear like > "netdev-dpdk: assume dpdr peer can be multi-producer/consumer" > I agree, you just missed a letter, "netdev-dpdk: assume dpdkr peer

Re: [ovs-dev] [PATCH] netdev-dpdk: Modify rings creation attributes

2015-11-15 Thread Flavio Leitner
On Wed, Nov 11, 2015 at 09:22:39AM +0100, Mauricio Vásquez wrote: The commit title can be more clear like "netdev-dpdk: assume dpdr peer can be multi-producer/consumer" > Although netdev does explicit locking, it is only valid from the ovs > perspective, then only the ring ends used by ovs shoul

Re: [ovs-dev] [PATCH] netdev-dpdk: Modify rings creation attributes

2015-11-11 Thread Traynor, Kevin
> -Original Message- > From: dev [mailto:dev-boun...@openvswitch.org] On Behalf Of Mauricio Vásquez > Sent: Wednesday, November 11, 2015 8:23 AM > To: dev@openvswitch.org > Subject: [ovs-dev] [PATCH] netdev-dpdk: Modify rings creation attributes > > Although netdev does explicit locking,

Re: [ovs-dev] [PATCH] netdev-dpdk: Modify rings creation attributes

2015-11-10 Thread Ben Pfaff
On Wed, Nov 11, 2015 at 12:10:06AM +0100, Mauricio Vásquez wrote: > Right now I realized that I did not respect the 79 characters width limit > on > the description. > > Should I resend the path? > > Thank you very much. It looks to me like your patch is word-wrapped. That is a more serious iss

Re: [ovs-dev] [PATCH] netdev-dpdk: Modify rings creation attributes

2015-11-10 Thread Mauricio Vásquez
Right now I realized that I did not respect the 79 characters width limit on the description. Should I resend the path? Thank you very much. On 10 November 2015 at 23:42, Mauricio Vásquez < mauricio.vasquezber...@studenti.polito.it> wrote: > Although netdev does explicit locking, it is only val