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
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
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
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
> -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,
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
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