Re: [dpdk-dev] [PATCH 1/2] net: fix ESP header byte ordering definition

2018-01-19 Thread De Lara Guarch, Pablo
> -Original Message- > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of De Lara Guarch, > Pablo > Sent: Friday, January 19, 2018 9:34 AM > To: Nelio Laranjeiro ; dev@dpdk.org; Olivier > Matz > Cc: bor...@mellanox.com > Subject: Re: [dpdk-dev] [PATCH 1/2]

Re: [dpdk-dev] [PATCH 1/2] net: fix ESP header byte ordering definition

2018-01-19 Thread De Lara Guarch, Pablo
> -Original Message- > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Nelio Laranjeiro > Sent: Thursday, January 11, 2018 9:16 AM > To: dev@dpdk.org; Olivier Matz > Cc: bor...@mellanox.com > Subject: [dpdk-dev] [PATCH 1/2] net: fix ESP header byte ordering definition > > ESP heade

Re: [dpdk-dev] [PATCH 1/2] net: fix ESP header byte ordering definition

2018-01-19 Thread De Lara Guarch, Pablo
> -Original Message- > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Nélio Laranjeiro > Sent: Tuesday, January 16, 2018 3:52 PM > To: Olivier Matz > Cc: dev@dpdk.org; bor...@mellanox.com > Subject: Re: [dpdk-dev] [PATCH 1/2] net: fix ESP header byte ordering &g

Re: [dpdk-dev] [PATCH 1/2] net: fix ESP header byte ordering definition

2018-01-16 Thread Nélio Laranjeiro
On Tue, Jan 16, 2018 at 03:24:09PM +0100, Olivier Matz wrote: > On Thu, Jan 11, 2018 at 10:15:58AM +0100, Nelio Laranjeiro wrote: > > ESP header is defined in the RFC2406 [1] as Big Endian fields it should use > > the corresponding types in DPDK as well. > > > > Fixes: d4b684f7197a ("net: add ESP

Re: [dpdk-dev] [PATCH 1/2] net: fix ESP header byte ordering definition

2018-01-16 Thread Olivier Matz
On Thu, Jan 11, 2018 at 10:15:58AM +0100, Nelio Laranjeiro wrote: > ESP header is defined in the RFC2406 [1] as Big Endian fields it should use > the corresponding types in DPDK as well. > > Fixes: d4b684f7197a ("net: add ESP header to generic flow steering") I wonder if we should really mark thi