[dpdk-dev] DPDK (v17.11) ACL table field format definition enhancement: 'offset' field covering headroom space.

2019-08-17 Thread Victor Huertas
Hi all, I am developing an application with DPDK (v17.11) using a concatenation of pipelines. Now I find myself on the definition of an ACL table and more precisely the format of the ACL fields. One of the parameters of this field format is the 'offset', which indicates the number of bytes till t

[dpdk-dev] DPDK (v17.11) ACL table field format definition enhancement: 'offset' field covering headroom space.

2019-08-17 Thread Victor Huertas
Hi all, I am developing an application with DPDK (v17.11) using a concatenation of pipelines. Now I find myself on the definition of an ACL table and more precisely the format of the ACL fields. One of the parameters of this field format is the 'offset', which indicates the number of bytes till t

Re: [dpdk-dev] [PATCH 2/7] ethdev: add mbuf RSS update as a offload

2019-08-17 Thread Pavan Nikhilesh Bhagavatula
>-Original Message- >From: dev On Behalf Of Andrew Rybchenko >Sent: Friday, August 16, 2019 1:18 PM >To: Pavan Nikhilesh Bhagavatula ; Jerin >Jacob Kollanukkaran ; ferruh.yi...@intel.com; >John McNamara ; Marko Kovacevic >; Thomas Monjalon > >Cc: dev@dpdk.org >Subject: Re: [dpdk-dev] [PA

Re: [dpdk-dev] [PATCH 4/7] net: update Rx RSS hash offload capabilities

2019-08-17 Thread Pavan Nikhilesh Bhagavatula
>Subject: Re: [dpdk-dev] [PATCH 4/7] net: update Rx RSS hash offload >capabilities > >Prefix "net: " is typically used for lib/librte_net. It should be >"drivers/net: " here. Ack will fix in v2. > >On 8/16/19 8:55 AM, pbhagavat...@marvell.com wrote: >> From: Pavan Nikhilesh >> >> Add DEV_RX_OFFL

Re: [dpdk-dev] [PATCH 5/7] net: update Rx flow action offload capabilities

2019-08-17 Thread Pavan Nikhilesh Bhagavatula
>Prefix "net: " is typically used for lib/librte_net. It should be >"drivers/net: " here. > >"Rx flow action offload" sounds strange. May be "Rx flow mark offload". > >On 8/16/19 8:55 AM, pbhagavat...@marvell.com wrote: >> From: Pavan Nikhilesh >> >> Add DEV_RX_OFFLOAD_FLOW_MARK flag for all suppo

Re: [dpdk-dev] [EXT] Re: [PATCH 3/7] ethdev: add flow action type update as an offload

2019-08-17 Thread Pavan Nikhilesh Bhagavatula
>> enable/disable PMDs write to `rte_mbuf::hash::fdir::hi`. > >Notes similar to RSS hash. > >It requires better motivation why. It lets Rx queue know that >it will be used as flow action MARK target and the queue should >be configured to deliver the mark from NIC to PMD and processed >in the driver

Re: [dpdk-dev] [EXT] Re: [PATCH 1/7] ethdev: add set ptype function

2019-08-17 Thread Pavan Nikhilesh Bhagavatula
>-Original Message- >From: Andrew Rybchenko >Sent: Friday, August 16, 2019 1:53 PM >To: Pavan Nikhilesh Bhagavatula ; Jerin >Jacob Kollanukkaran ; ferruh.yi...@intel.com; >John McNamara ; Marko Kovacevic >; Thomas Monjalon > >Cc: dev@dpdk.org >Subject: [EXT] Re: [dpdk-dev] [PATCH 1/7] et

Re: [dpdk-dev] [PATCH 6/7] net: add ptype set default functionality

2019-08-17 Thread Pavan Nikhilesh Bhagavatula
>> >> Add ptype set default functionality i.e. enable ptype by default and >> report ptypes even if application disables ptypes. >> >> Signed-off-by: Pavan Nikhilesh > >May be it is better to have >rte_ethdev_supported_ptype_set_dummy() >in ethdev library and use it in PMDs instead of duplicating

Re: [dpdk-dev] [PATCH 2/7] ethdev: add mbuf RSS update as a offload

2019-08-17 Thread Shahaf Shuler
Friday, August 16, 2019 10:48 AM, Andrew Rybchenko: > Subject: Re: [dpdk-dev] [PATCH 2/7] ethdev: add mbuf RSS update as a > offload > > On 8/16/19 8:55 AM, pbhagavat...@marvell.com wrote: > > From: Pavan Nikhilesh > > > > Add new Rx offload flag `DEV_RX_OFFLOAD_RSS_HASH` which can be used > to >

Re: [dpdk-dev] [PATCH 3/7] ethdev: add flow action type update as an offload

2019-08-17 Thread Shahaf Shuler
Friday, August 16, 2019 11:05 AM, Andrew Rybchenko: > ; Thomas Monjalon > Cc: dev@dpdk.org > Subject: Re: [dpdk-dev] [PATCH 3/7] ethdev: add flow action type update as > an offload > > On 8/16/19 8:55 AM, pbhagavat...@marvell.com wrote: > > From: Pavan Nikhilesh > > > > Add new Rx offload flag `

Re: [dpdk-dev] [PATCH 2/7] ethdev: add mbuf RSS update as a offload

2019-08-17 Thread Andrew Rybchenko
On 8/18/19 7:52 AM, Shahaf Shuler wrote: Friday, August 16, 2019 10:48 AM, Andrew Rybchenko: Subject: Re: [dpdk-dev] [PATCH 2/7] ethdev: add mbuf RSS update as a offload On 8/16/19 8:55 AM, pbhagavat...@marvell.com wrote: From: Pavan Nikhilesh Add new Rx offload flag `DEV_RX_OFFLOAD_RSS_HASH

Re: [dpdk-dev] [PATCH 3/7] ethdev: add flow action type update as an offload

2019-08-17 Thread Andrew Rybchenko
On 8/18/19 7:59 AM, Shahaf Shuler wrote: Friday, August 16, 2019 11:05 AM, Andrew Rybchenko: ; Thomas Monjalon Cc: dev@dpdk.org Subject: Re: [dpdk-dev] [PATCH 3/7] ethdev: add flow action type update as an offload On 8/16/19 8:55 AM, pbhagavat...@marvell.com wrote: From: Pavan Nikhilesh Add

Re: [dpdk-dev] [PATCH 2/7] ethdev: add mbuf RSS update as a offload

2019-08-17 Thread Shahaf Shuler
Sunday, August 18, 2019 8:39 AM, Andrew Rybchenko: > ; Thomas Monjalon > Cc: dev@dpdk.org > Subject: Re: [dpdk-dev] [PATCH 2/7] ethdev: add mbuf RSS update as a > offload > > On 8/18/19 7:52 AM, Shahaf Shuler wrote: > > Friday, August 16, 2019 10:48 AM, Andrew Rybchenko: > >> Subject: Re: [dpdk-d

Re: [dpdk-dev] [PATCH 3/7] ethdev: add flow action type update as an offload

2019-08-17 Thread Shahaf Shuler
Sunday, August 18, 2019 8:57 AM, Andrew Rybchenko: > Subject: Re: [dpdk-dev] [PATCH 3/7] ethdev: add flow action type update as > an offload > > On 8/18/19 7:59 AM, Shahaf Shuler wrote: > > Friday, August 16, 2019 11:05 AM, Andrew Rybchenko: > >> ; Thomas Monjalon > > >> Cc: dev@dpdk.org > >> Sub

[dpdk-dev] [PATCH] eal: unmap undesired address upon config reattach failure

2019-08-17 Thread Arnon Warshavsky
Have rte_eal_config_reattach clean up the mapped address which is a valid address but not the one intended. Coverity issue: 343439 Signed-off-by: Arnon Warshavsky --- lib/librte_eal/linux/eal/eal.c | 1 + 1 file changed, 1 insertion(+) diff --git a/lib/librte_eal/linux/eal/eal.c b/lib/librte_ea