Re: [dpdk-dev] [PATCH v2 1/4] ethdev: add Rx offload outer UDP checksum definition

2018-10-08 Thread Jerin Jacob
> , "Mcnamara, John" , > "Kovacevic, Marko" , Olivier Matz > , "dev@dpdk.org" , > "shah...@mellanox.com" , "didier.pall...@6wind.com" > > Subject: Re: [dpdk-dev] [PATCH v2 1/4] ethdev: add Rx offload outer UDP > checksum

Re: [dpdk-dev] [PATCH v2 1/4] ethdev: add Rx offload outer UDP checksum definition

2018-10-08 Thread Thomas Monjalon
08/10/2018 14:25, Jerin Jacob: > From: Ferruh Yigit > > On 10/8/2018 12:55 PM, Jerin Jacob wrote: > > > From: Ferruh Yigit > > >> On 10/8/2018 10:37 AM, Jerin Jacob wrote: > > >>> From: Thomas Monjalon > > 08/10/2018 10:24, Jerin Jacob: > > > From: Ferruh Yigit > > >> On 10/6/2018

Re: [dpdk-dev] [PATCH v2 1/4] ethdev: add Rx offload outer UDP checksum definition

2018-10-08 Thread Jerin Jacob
> , "Mcnamara, John" , > "Kovacevic, Marko" , Olivier Matz > , "dev@dpdk.org" , > "shah...@mellanox.com" , "didier.pall...@6wind.com" > > Subject: Re: [dpdk-dev] [PATCH v2 1/4] ethdev: add Rx offload outer UDP > checksum

Re: [dpdk-dev] [PATCH v2 1/4] ethdev: add Rx offload outer UDP checksum definition

2018-10-08 Thread Ferruh Yigit
u, Wenzhuo" , "Wu, >> Jingjing" , "Iremonger, Bernard" >> , "Mcnamara, John" , >> "Kovacevic, Marko" , Olivier Matz >> , "dev@dpdk.org" , >> "shah...@mellanox.com" , "didier.pall...@6wind.com" >

Re: [dpdk-dev] [PATCH v2 1/4] ethdev: add Rx offload outer UDP checksum definition

2018-10-08 Thread Jerin Jacob
> , "Mcnamara, John" , > "Kovacevic, Marko" , Olivier Matz > , "dev@dpdk.org" , > "shah...@mellanox.com" , "didier.pall...@6wind.com" > > Subject: Re: [dpdk-dev] [PATCH v2 1/4] ethdev: add Rx offload outer UDP > checksum

Re: [dpdk-dev] [PATCH v2 1/4] ethdev: add Rx offload outer UDP checksum definition

2018-10-08 Thread Ferruh Yigit
enzhuo" >> , "Wu, Jingjing" , >> "Iremonger, Bernard" , "Mcnamara, John" >> , "Kovacevic, Marko" , >> Olivier Matz , "dev@dpdk.org" , >> "shah...@mellanox.com" , "didier.pall...@6wind.com" >

Re: [dpdk-dev] [PATCH v2 1/4] ethdev: add Rx offload outer UDP checksum definition

2018-10-08 Thread Jerin Jacob
rd" , "Mcnamara, John" > , "Kovacevic, Marko" , > Olivier Matz , "dev@dpdk.org" , > "shah...@mellanox.com" , "didier.pall...@6wind.com" > > Subject: Re: [dpdk-dev] [PATCH v2 1/4] ethdev: add Rx offload outer UDP > c

Re: [dpdk-dev] [PATCH v2 1/4] ethdev: add Rx offload outer UDP checksum definition

2018-10-08 Thread Thomas Monjalon
08/10/2018 10:24, Jerin Jacob: > From: Ferruh Yigit > > On 10/6/2018 1:18 PM, Ananyev, Konstantin wrote: > > > From: Jerin Jacob [mailto:jerin.ja...@caviumnetworks.com] > > >> From: Thomas Monjalon > > >>> However, we should re-visit the flag PKT_RX_EIP_CKSUM_BAD. > > >> > > >> Do we need to bloc

Re: [dpdk-dev] [PATCH v2 1/4] ethdev: add Rx offload outer UDP checksum definition

2018-10-08 Thread Jerin Jacob
, "Mcnamara, John" > , "Kovacevic, Marko" , > Olivier Matz , "dev@dpdk.org" , > "shah...@mellanox.com" , "didier.pall...@6wind.com" > > Subject: Re: [dpdk-dev] [PATCH v2 1/4] ethdev: add Rx offload outer UDP > checksum definit

Re: [dpdk-dev] [PATCH v2 1/4] ethdev: add Rx offload outer UDP checksum definition

2018-10-08 Thread Ferruh Yigit
>> ; Lu, Wenzhuo ; Wu, >> Jingjing ; Iremonger, Bernard >> ; Mcnamara, John ; >> Kovacevic, Marko ; Olivier Matz >> ; dev@dpdk.org; shah...@mellanox.com; >> Ananyev, Konstantin ; didier.pall...@6wind.com >> Subject: Re: [dpdk-dev] [PATCH v2 1/4] ethdev

Re: [dpdk-dev] [PATCH v2 1/4] ethdev: add Rx offload outer UDP checksum definition

2018-10-06 Thread Ananyev, Konstantin
ohn ; > Kovacevic, Marko ; Olivier Matz > ; dev@dpdk.org; shah...@mellanox.com; > Ananyev, Konstantin ; didier.pall...@6wind.com > Subject: Re: [dpdk-dev] [PATCH v2 1/4] ethdev: add Rx offload outer UDP > checksum definition > > -Original Message- > > Date: Sat, 0

Re: [dpdk-dev] [PATCH v2 1/4] ethdev: add Rx offload outer UDP checksum definition

2018-10-06 Thread Jerin Jacob
k.org, shah...@mellanox.com, > "Ananyev, Konstantin" , > didier.pall...@6wind.com > Subject: Re: [dpdk-dev] [PATCH v2 1/4] ethdev: add Rx offload outer UDP > checksum definition > > > 05/10/2018 22:04, Ferruh Yigit: > > On 10/4/2018 6:59 AM, Jerin Jacob wrote:

Re: [dpdk-dev] [PATCH v2 1/4] ethdev: add Rx offload outer UDP checksum definition

2018-10-05 Thread Thomas Monjalon
05/10/2018 22:04, Ferruh Yigit: > On 10/4/2018 6:59 AM, Jerin Jacob wrote: > > From: Andrew Rybchenko > >> On 03.10.2018 21:14, Jerin Jacob wrote: > >>> From: Andrew Rybchenko > On 03.10.2018 20:12, Jerin Jacob wrote: > > From: Jerin Jacob > >> From: Andrew Rybchenko > >>> 3. P

Re: [dpdk-dev] [PATCH v2 1/4] ethdev: add Rx offload outer UDP checksum definition

2018-10-05 Thread Ferruh Yigit
ic , >> Thomas Monjalon , Ferruh Yigit >> , Olivier Matz , >> dev@dpdk.org, shah...@mellanox.com, "Ananyev, Konstantin" >> >> Subject: Re: [dpdk-dev] [PATCH v2 1/4] ethdev: add Rx offload outer UDP >> checksum definition >> User-Agent: Mozilla/5.0 (X

Re: [dpdk-dev] [PATCH v2 1/4] ethdev: add Rx offload outer UDP checksum definition

2018-10-05 Thread Ferruh Yigit
ic , >> Thomas Monjalon , Ferruh Yigit >> , Olivier Matz , >> dev@dpdk.org, shah...@mellanox.com, "Ananyev, Konstantin" >> >> Subject: Re: [dpdk-dev] [PATCH v2 1/4] ethdev: add Rx offload outer UDP >> checksum definition >> User-Agent: Mozilla/5.0 (X

Re: [dpdk-dev] [PATCH v2 1/4] ethdev: add Rx offload outer UDP checksum definition

2018-10-03 Thread Jerin Jacob
k.org, shah...@mellanox.com, "Ananyev, Konstantin" > > Subject: Re: [dpdk-dev] [PATCH v2 1/4] ethdev: add Rx offload outer UDP > checksum definition > User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 > Thunderbird/52.9.1 > > > Hi Je

Re: [dpdk-dev] [PATCH v2 1/4] ethdev: add Rx offload outer UDP checksum definition

2018-10-03 Thread Thomas Monjalon
03/10/2018 21:47, Andrew Rybchenko: > On 03.10.2018 21:14, Jerin Jacob wrote: > > From: Andrew Rybchenko > >>> From: Jerin Jacob > From: Andrew Rybchenko > > On 10/2/18 10:24 PM, Jerin Jacob wrote: > > > > Introduced DEV_RX_OFFLOAD_OUTER_UDP_CKSUM Rx offload flag and > > PKT

Re: [dpdk-dev] [PATCH v2 1/4] ethdev: add Rx offload outer UDP checksum definition

2018-10-03 Thread Andrew Rybchenko
, dev@dpdk.org, shah...@mellanox.com, "Ananyev, Konstantin" Subject: Re: [dpdk-dev] [PATCH v2 1/4] ethdev: add Rx offload outer UDP checksum definition User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.9.1 On 03.10.2018 20:12, Jerin J

Re: [dpdk-dev] [PATCH v2 1/4] ethdev: add Rx offload outer UDP checksum definition

2018-10-03 Thread Jerin Jacob
k.org, shah...@mellanox.com, "Ananyev, Konstantin" > > Subject: Re: [dpdk-dev] [PATCH v2 1/4] ethdev: add Rx offload outer UDP > checksum definition > User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 > Thunderbird/52.9.1 > > On 03.10.2018 20:12, Je

Re: [dpdk-dev] [PATCH v2 1/4] ethdev: add Rx offload outer UDP checksum definition

2018-10-03 Thread Andrew Rybchenko
@dpdk.org, shah...@mellanox.com, "Ananyev, Konstantin" Subject: Re: [dpdk-dev] [PATCH v2 1/4] ethdev: add Rx offload outer UDP checksum definition User-Agent: Mutt/1.10.1 (2018-07-13) External Email -Original Message- Date: Wed, 3 Oct 2018 10:34:52 +0300 From: Andrew Ryb

Re: [dpdk-dev] [PATCH v2 1/4] ethdev: add Rx offload outer UDP checksum definition

2018-10-03 Thread Jerin Jacob
k.org, shah...@mellanox.com, "Ananyev, Konstantin" > > Subject: Re: [dpdk-dev] [PATCH v2 1/4] ethdev: add Rx offload outer UDP > checksum definition > User-Agent: Mutt/1.10.1 (2018-07-13) > > External Email > > -Original Message- > > Date: Wed,

Re: [dpdk-dev] [PATCH v2 1/4] ethdev: add Rx offload outer UDP checksum definition

2018-10-03 Thread Iremonger, Bernard
Hi Jerin, > > > Subject: [dpdk-dev] [PATCH v2 1/4] ethdev: add Rx offload outer UDP > > > checksum definition > > > > > > Introduced DEV_RX_OFFLOAD_OUTER_UDP_CKSUM Rx offload flag and > > > PKT_RX_EL4_CKSUM_BAD mbuf ol_flags to detect outer

Re: [dpdk-dev] [PATCH v2 1/4] ethdev: add Rx offload outer UDP checksum definition

2018-10-03 Thread Jerin Jacob
uot;Yigit, Ferruh" , Andrew Rybchenko > , Olivier Matz > CC: "dev@dpdk.org" , "shah...@mellanox.com" > > Subject: RE: [dpdk-dev] [PATCH v2 1/4] ethdev: add Rx offload outer UDP > checksum definition > > > Hi Jerin, Hi Iremonger, > > &g

Re: [dpdk-dev] [PATCH v2 1/4] ethdev: add Rx offload outer UDP checksum definition

2018-10-03 Thread Jerin Jacob
arko" , Thomas Monjalon > , "Yigit, Ferruh" , Andrew > Rybchenko , Olivier Matz > , "dev@dpdk.org" , > "shah...@mellanox.com" > Subject: RE: [dpdk-dev] [PATCH v2 1/4] ethdev: add Rx offload outer UDP > checksum definition > > &

Re: [dpdk-dev] [PATCH v2 1/4] ethdev: add Rx offload outer UDP checksum definition

2018-10-03 Thread Iremonger, Bernard
erruh ; Andrew Rybchenko > ; Olivier Matz > Cc: dev@dpdk.org; shah...@mellanox.com; Jerin Jacob > > Subject: [dpdk-dev] [PATCH v2 1/4] ethdev: add Rx offload outer UDP checksum > definition > > Introduced DEV_RX_OFFLOAD_OUTER_UDP_CKSUM Rx offload flag and > PKT_RX_EL4_CKSU

Re: [dpdk-dev] [PATCH v2 1/4] ethdev: add Rx offload outer UDP checksum definition

2018-10-03 Thread Ananyev, Konstantin
njalon > ; Yigit, Ferruh ; Andrew > Rybchenko ; Olivier Matz > ; dev@dpdk.org; shah...@mellanox.com > Subject: Re: [dpdk-dev] [PATCH v2 1/4] ethdev: add Rx offload outer UDP > checksum definition > > -Original Message- > > Date: Wed, 3 Oct 2018 09:17:18 + &

Re: [dpdk-dev] [PATCH v2 1/4] ethdev: add Rx offload outer UDP checksum definition

2018-10-03 Thread Jerin Jacob
arko" , Thomas Monjalon > , "Yigit, Ferruh" , Andrew > Rybchenko , Olivier Matz > , "dev@dpdk.org" , > "shah...@mellanox.com" > Subject: RE: [dpdk-dev] [PATCH v2 1/4] ethdev: add Rx offload outer UDP > checksum definition > > &g

Re: [dpdk-dev] [PATCH v2 1/4] ethdev: add Rx offload outer UDP checksum definition

2018-10-03 Thread Ananyev, Konstantin
njalon > ; Yigit, Ferruh ; Andrew > Rybchenko ; Olivier Matz > ; dev@dpdk.org; shah...@mellanox.com > Subject: Re: [dpdk-dev] [PATCH v2 1/4] ethdev: add Rx offload outer UDP > checksum definition > > -Original Message- > > Date: Wed, 3 Oct 2018 08:53:22 + &

Re: [dpdk-dev] [PATCH v2 1/4] ethdev: add Rx offload outer UDP checksum definition

2018-10-03 Thread Jerin Jacob
, > Thomas Monjalon , "Yigit, Ferruh" > , Andrew Rybchenko , > Olivier Matz > CC: "dev@dpdk.org" , "shah...@mellanox.com" > > Subject: RE: [dpdk-dev] [PATCH v2 1/4] ethdev: add Rx offload outer UDP > checksum definition > > >

Re: [dpdk-dev] [PATCH v2 1/4] ethdev: add Rx offload outer UDP checksum definition

2018-10-03 Thread Ananyev, Konstantin
Hi Jerin, > > Introduced DEV_RX_OFFLOAD_OUTER_UDP_CKSUM Rx offload flag and > PKT_RX_EL4_CKSUM_BAD mbuf ol_flags to detect outer UDP checksum > failure. > > - To use hardware Rx outer UDP checksum offload, the user needs to > configure DEV_RX_OFFLOAD_OUTER_UDP_CKSUM offload flags in slowpath. >

Re: [dpdk-dev] [PATCH v2 1/4] ethdev: add Rx offload outer UDP checksum definition

2018-10-03 Thread Andrew Rybchenko
On 10/3/18 11:35 AM, Thomas Monjalon wrote: 03/10/2018 09:57, Jerin Jacob: From: Andrew Rybchenko 1. I'm not sure that it is OK that mbuf and ethdev changes go in one patch. It seems typically mbuf changes go separately and mbuf changes should be applied to main dpdk repo. I don't hav

Re: [dpdk-dev] [PATCH v2 1/4] ethdev: add Rx offload outer UDP checksum definition

2018-10-03 Thread Thomas Monjalon
03/10/2018 09:57, Jerin Jacob: > From: Andrew Rybchenko > > 1. I'm not sure that it is OK that mbuf and ethdev changes go in one patch. > >It seems typically mbuf changes go separately and mbuf changes should > >be applied to main dpdk repo. > > I don't have strong opinion on this. If the

Re: [dpdk-dev] [PATCH v2 1/4] ethdev: add Rx offload outer UDP checksum definition

2018-10-03 Thread Jerin Jacob
k.org, shah...@mellanox.com, "Ananyev, Konstantin" > > Subject: Re: [dpdk-dev] [PATCH v2 1/4] ethdev: add Rx offload outer UDP > checksum definition > User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 > Thunderbird/60.0 > >

Re: [dpdk-dev] [PATCH v2 1/4] ethdev: add Rx offload outer UDP checksum definition

2018-10-03 Thread Andrew Rybchenko
On 10/2/18 10:24 PM, Jerin Jacob wrote: Introduced DEV_RX_OFFLOAD_OUTER_UDP_CKSUM Rx offload flag and PKT_RX_EL4_CKSUM_BAD mbuf ol_flags to detect outer UDP checksum failure. - To use hardware Rx outer UDP checksum offload, the user needs to configure DEV_RX_OFFLOAD_OUTER_UDP_CKSUM offload flags

[dpdk-dev] [PATCH v2 1/4] ethdev: add Rx offload outer UDP checksum definition

2018-10-02 Thread Jerin Jacob
Introduced DEV_RX_OFFLOAD_OUTER_UDP_CKSUM Rx offload flag and PKT_RX_EL4_CKSUM_BAD mbuf ol_flags to detect outer UDP checksum failure. - To use hardware Rx outer UDP checksum offload, the user needs to configure DEV_RX_OFFLOAD_OUTER_UDP_CKSUM offload flags in slowpath. - Driver updates the PKT_RX