[dpdk-dev] [PATCH] update Red Hat roadmap for 21.02

2020-12-16 Thread Maxime Coquelin
Signed-off-by: Maxime Coquelin --- content/roadmap/_index.md | 2 ++ 1 file changed, 2 insertions(+) diff --git a/content/roadmap/_index.md b/content/roadmap/_index.md index 4566902..9d432df 100644 --- a/content/roadmap/_index.md +++ b/content/roadmap/_index.md @@ -20,6 +20,8 @@ This list is obv

[dpdk-dev] [PATCH] update Marvell roadmap for 21.02

2020-12-16 Thread jerinj
From: Jerin Jacob Signed-off-by: Jerin Jacob --- content/roadmap/_index.md | 5 + 1 file changed, 5 insertions(+) diff --git a/content/roadmap/_index.md b/content/roadmap/_index.md index 4566902..ad9fcd9 100644 --- a/content/roadmap/_index.md +++ b/content/roadmap/_index.md @@ -19,6 +19,9

Re: [dpdk-dev] 19.11.6 patches review and test

2020-12-16 Thread Pei Zhang
Hello Luca, The testing with dpdk 19.11.6-rc1 from Red Hat looks good. We tested below 17 scenarios and and all got PASS on RHEL8: (1)Guest with device assignment(PF) throughput testing(1G hugepage size): PASS (2)Guest with device assignment(PF) throughput testing(2M hugepage size) : PASS (3)Gue

[dpdk-dev] [dpdk-dev 21.02 0/5] enable UDP ecpri configure in dcf

2020-12-16 Thread Jeff Guo
Enabling ecpri UDP tunnel port configure in dcf. Jeff Guo (5): ethdev: add new tunnel type for ecpri net/ice/base: add new UDP tunnel type for ecpri net/ice: add ecpri package type net/ice: enable ecpri tunnel port configure in dcf app/testpmd: add new UDP tunnel port for ecpri app/tes

[dpdk-dev] [dpdk-dev 21.02 1/5] ethdev: add new tunnel type for ecpri

2020-12-16 Thread Jeff Guo
Add type of RTE_TUNNEL_TYPE_ECPRI into the enum of ethdev tunnel type. Signed-off-by: Jeff Guo --- lib/librte_ethdev/rte_ethdev.h | 1 + 1 file changed, 1 insertion(+) diff --git a/lib/librte_ethdev/rte_ethdev.h b/lib/librte_ethdev/rte_ethdev.h index f5f8919186..2cbce958cf 100644 --- a/lib/libr

[dpdk-dev] [dpdk-dev 21.02 2/5] net/ice/base: add new UDP tunnel type for ecpri

2020-12-16 Thread Jeff Guo
Add new UDP tunnel type for ecpri. Signed-off-by: Jeff Guo --- drivers/net/ice/base/ice_flex_pipe.c | 1 + drivers/net/ice/base/ice_flex_type.h | 1 + 2 files changed, 2 insertions(+) diff --git a/drivers/net/ice/base/ice_flex_pipe.c b/drivers/net/ice/base/ice_flex_pipe.c index 7594df1696..fa8

[dpdk-dev] [dpdk-dev 21.02 3/5] net/ice: add ecpri package type

2020-12-16 Thread Jeff Guo
Add new ecpri package type and process the new package type when handle each flow engines. Signed-off-by: Jeff Guo --- drivers/net/ice/ice_ethdev.c| 7 +- drivers/net/ice/ice_ethdev.h| 1 + drivers/net/ice/ice_fdir_filter.c | 66 +++ drivers/net/ice/ice_hash.c

[dpdk-dev] [dpdk-dev 21.02 4/5] net/ice: enable ecpri tunnel port configure in dcf

2020-12-16 Thread Jeff Guo
Add ecpri tunnel port add and rm ops to configure ecpri udp tunnel port in dcf. Signed-off-by: Jeff Guo --- drivers/net/ice/ice_dcf_ethdev.c | 67 1 file changed, 67 insertions(+) diff --git a/drivers/net/ice/ice_dcf_ethdev.c b/drivers/net/ice/ice_dcf_ethdev.c i

[dpdk-dev] [dpdk-dev 21.02 5/5] app/testpmd: add new UDP tunnel port for ecpri

2020-12-16 Thread Jeff Guo
Add new UDP tunnel port params for ecpri configuration, the command as below: testpmd> port config 0 udp_tunnel_port add ecpri 6789 testpmd> port config 0 udp_tunnel_port rm ecpri 6789 Signed-off-by: Jeff Guo --- app/test-pmd/cmdline.c | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-

Re: [dpdk-dev] 19.11.6 patches review and test

2020-12-16 Thread Luca Boccassi
Hi, Release date is fast approaching - could the questions below be looked, please? Thanks! On Thu, 2020-12-10 at 10:15 +, Luca Boccassi wrote: > On Thu, 2020-12-10 at 10:07 +, Chen, BoX C wrote: > > Hi Luca, > > Update LTS 19.11.6 test result for Intel part. All passed except the known

Re: [dpdk-dev] 19.11.6 patches review and test

2020-12-16 Thread Luca Boccassi
Thank you! On Wed, 2020-12-16 at 03:47 -0500, Pei Zhang wrote: > Hello Luca, > > The testing with dpdk 19.11.6-rc1 from Red Hat looks good. We tested below 17 > scenarios and and all got PASS on RHEL8: > > (1)Guest with device assignment(PF) throughput testing(1G hugepage size): PASS > (2)Guest

Re: [dpdk-dev] [PATCH] net/mlx5: fix buffer split offload advertising

2020-12-16 Thread Raslan Darawsheh
Hi, > -Original Message- > From: Viacheslav Ovsiienko > Sent: Friday, December 11, 2020 1:45 PM > To: dev@dpdk.org > Cc: Raslan Darawsheh ; Matan Azrad > ; sta...@dpdk.org > Subject: [PATCH] net/mlx5: fix buffer split offload advertising > > The buffer split Rx offload is not compatible w

Re: [dpdk-dev] [PATCH v5] ip_frag: remove padding length of fragment

2020-12-16 Thread Ananyev, Konstantin
Hi Yicai, > > In some situations, we would get several ip fragments, which total > > data length is less than min_ip_len(64) and padding with zeros. > > We simulated intermediate fragments by modifying the MTU. > > To illustrate the problem, we simplify the packet format and ignore > > the impa

Re: [dpdk-dev] [PATCH v5] ip_frag: remove padding length of fragment

2020-12-16 Thread luyicai
-Original Message- From: Ananyev, Konstantin [mailto:konstantin.anan...@intel.com] Sent: Wednesday, December 16, 2020 6:45 PM To: luyicai ; dev@dpdk.org Cc: Zhoujingbin (Robin, Russell Lab) ; chenchanghu ; Lilijun (Jerry) ; Linhaifeng ; Guohongzhi (Russell Lab) ; wangyunjian ; sta...@

Re: [dpdk-dev] [PATCH] crypto/qat: fix possible access to an uninitialized variable

2020-12-16 Thread Doherty, Declan
On 15/12/2020 6:23 PM, Vladimir Medvedkin wrote: QAT_LOG() has access to the uninitialized variable "name" Fixes: 21792c443205 ("crypto/qat: add multi-process handling of driver ID") Cc: sta...@dpdk.org Signed-off-by: Vladimir Medvedkin --- Acked-by: Declan Doherty

[dpdk-dev] [PATCH] net/ice: add GTP over GRE package name

2020-12-16 Thread Simei Su
Add new GTPoGRE package name and handle the new package when initializing each flow engine. Signed-off-by: Simei Su --- drivers/net/ice/ice_ethdev.c| 4 drivers/net/ice/ice_ethdev.h| 1 + drivers/net/ice/ice_fdir_filter.c | 4 drivers/net/ice/ice_hash.c | 4 +

Re: [dpdk-dev] 19.11.6 patches review and test

2020-12-16 Thread Govindharajan, Hariprasad
> -Original Message- > From: luca.bocca...@gmail.com > Sent: Thursday, December 3, 2020 9:39 AM > To: sta...@dpdk.org > Cc: dev@dpdk.org; Abhishek Marathe ; > Akhil Goyal ; Ali Alnubani ; > Walker, Benjamin ; David Christensen > ; Hemant Agrawal ; > Stokes, Ian ; Jerin Jacob ; > Mcnamara

Re: [dpdk-dev] [PATCH] app/testpmd: fix segment number check

2020-12-16 Thread Ferruh Yigit
On 12/11/2020 4:14 PM, Slava Ovsiienko wrote: Hi, Andrew Thank you for the review, please, see below. -Original Message- From: Andrew Boyer Sent: Friday, December 11, 2020 18:00 To: Slava Ovsiienko Cc: dev@dpdk.org; NBU-Contact-Thomas Monjalon ; ferruh.yi...@intel.com; sta...@dpdk.or

Re: [dpdk-dev] [v1, 2/2] bonding: fix PCI address comparison on non-pci ports

2020-12-16 Thread Min Hu (Connor)
Hi, sorry for late reply. I know what you mean. But "find_port_id_by_pci_addr" is one static type funtion. it is only used in the function "parse_port_id". what you modified in "find_port_id_by_pci_addr" is totally done before "find_port_id_by_pci_addr" in the function "parse_port_id". Tha

Re: [dpdk-dev] [PATCH] app/testpmd: fix segment number check

2020-12-16 Thread Slava Ovsiienko
Hi, Ferruh > -Original Message- > From: Ferruh Yigit > Sent: Wednesday, December 16, 2020 14:12 > To: Slava Ovsiienko ; Andrew Boyer > > Cc: dev@dpdk.org; NBU-Contact-Thomas Monjalon ; > sta...@dpdk.org > Subject: Re: [dpdk-dev] [PATCH] app/testpmd: fix segment number check > > On 12/11

Re: [dpdk-dev] [PATCH] app/testpmd: fix segment number check

2020-12-16 Thread Ferruh Yigit
On 12/11/2020 3:07 PM, Viacheslav Ovsiienko wrote: The --txpkts command line parameter was silently ignored due to application was unable to check the Tx queue ring sizes for non configured ports [1]. The "set txpkts " was also rejected if there was some stopped or /unconfigured port. May not

[dpdk-dev] [RFC v3 0/8] ethdev: introduce GENEVE header TLV option item

2020-12-16 Thread Shiri Kuzin
The Geneve tunneling protocol is designed to allow the user to specify some data context on the packet. The GENEVE TLV (Type-Length-Variable) Option is the mean intended to present the user data. In order to support GENEVE TLV Option the new rte_flow item "rte_flow_item_geneve_opt" is introduced.

[dpdk-dev] [RFC v3 2/8] app/testpmd: add GENEVE option item support

2020-12-16 Thread Shiri Kuzin
From: Viacheslav Ovsiienko The patch adds the GENEVE option rte flow item support to command line interpreter. The flow command with GENEVE option items looks like: flow create 0 ingress pattern eth / ipv4 / udp / geneve vni is 100 / geneve-opt class is 99 length is 1 type is 0 data is

[dpdk-dev] [RFC v3 3/8] common/mlx5: check GENEVE TLV support in HCA attributes

2020-12-16 Thread Shiri Kuzin
This is preparation step to support match on GENEVE TLV option. In this Patch we add the HCA attributes that will allow supporting GENEVE TLV option matching. Signed-off-by: Shiri Kuzin --- drivers/common/mlx5/mlx5_devx_cmds.c | 7 +++ drivers/common/mlx5/mlx5_devx_cmds.h | 4 driver

[dpdk-dev] [RFC v3 1/8] lib/librte_ethdev: introduce GENEVE header TLV option item

2020-12-16 Thread Shiri Kuzin
The Geneve tunneling protocol is designed to allow the user to specify some data context on the packet. The GENEVE TLV (Type-Length-Variable) Option is the mean intended to present the user data. In order to support GENEVE TLV Option the new rte_flow item "rte_flow_item_geneve_opt" is added. The n

[dpdk-dev] [RFC v3 4/8] common/mlx5: create GENEVE TLV option object with DevX

2020-12-16 Thread Shiri Kuzin
TLV object is a special firmware maintained entity used to support match on GENEVE header extension option. The TLV object is created with DevX API and accepts the option class, type and lehgth fields. The class type and length fields are set using MLX5_SET and the Devx object is created using ml

[dpdk-dev] [RFC v3 5/8] net/mlx5: create GENEVE TLV option management

2020-12-16 Thread Shiri Kuzin
Currently firmware supports the only TLV object per device to match on the GENEVE header option. This patch adds the simple TLV object management to the mlx5 PMD. Signed-off-by: Shiri Kuzin --- drivers/net/mlx5/mlx5.c | 2 + drivers/net/mlx5/mlx5.h | 13 + drivers/net/mlx

[dpdk-dev] [RFC v3 8/8] doc: update GENEVE TLV option support

2020-12-16 Thread Shiri Kuzin
GENEVE TLV option support added to mlx5 PMD. The limitations and support were updated in documentation. Signed-off-by: Shiri Kuzin --- doc/guides/nics/mlx5.rst | 18 +- 1 file changed, 17 insertions(+), 1 deletion(-) diff --git a/doc/guides/nics/mlx5.rst b/doc/guides/nics/mlx5.

[dpdk-dev] [RFC v3 6/8] net/mlx5: add GENEVE TLV option flow validation

2020-12-16 Thread Shiri Kuzin
This patch adds validation routine for the GENEVE header TLV option. The GENEVE TLV option match must include all fields with full masks due to NIC does not support masking on option class, type and length. The option data length must be non zero and provided data pattern should be zero neither d

[dpdk-dev] [RFC v3 7/8] net/mlx5: add GENEVE TLV option flow translation

2020-12-16 Thread Shiri Kuzin
The GENEVE TLV option matching flows must be created using a translation function. This function checks whether we already created a Devx object for the matching and either creates the objects or updates the reference counter. Signed-off-by: Shiri Kuzin --- drivers/net/mlx5/mlx5_flow_dv.c | 70

Re: [dpdk-dev] 19.11.6 patches review and test

2020-12-16 Thread Luca Boccassi
On Wed, 2020-12-16 at 12:11 +, Govindharajan, Hariprasad wrote: > > -Original Message- > > From: luca.bocca...@gmail.com > > Sent: Thursday, December 3, 2020 9:39 AM > > To: sta...@dpdk.org > > Cc: dev@dpdk.org; Abhishek Marathe ; > > Akhil Goyal ; Ali Alnubani ; > > Walker, Benjamin ;

[dpdk-dev] [PATCH v6] ip_frag: remove padding length of fragment

2020-12-16 Thread Yicai Lu
In some situations, we would get several ip fragments, which total data length is less than min_ip_len(64) and padding with zeros. We simulated intermediate fragments by modifying the MTU. To illustrate the problem, we simplify the packet format and ignore the impact of the packet header.In namespa

Re: [dpdk-dev] [PATCH v2] meson: update meson build for armada drivers

2020-12-16 Thread Ferruh Yigit
On 12/15/2020 10:23 PM, lir...@marvell.com wrote: From: Liron Himi With pkg-config support available within musdk library (from musdk-release-SDK-10.3.5.0-PR2 version), meson option 'lib_musdk_dir' can be removed. PKG_CONFIG_PATH environment variable should be set appropriately to use the musdk

Re: [dpdk-dev] [EXT] Re: [PATCH v2] meson: update meson build for armada drivers

2020-12-16 Thread Liron Himi
-Original Message- From: Ferruh Yigit Sent: Wednesday, 16 December 2020 15:37 To: Liron Himi Cc: dev@dpdk.org; Luca Boccassi ; Kevin Traynor ; Akhil Goyal Subject: [EXT] Re: [PATCH v2] meson: update meson build for armada drivers External Email --

Re: [dpdk-dev] [EXT] Re: [PATCH v2] meson: update meson build for armada drivers

2020-12-16 Thread Ferruh Yigit
On 12/16/2020 1:41 PM, Liron Himi wrote: -Original Message- From: Ferruh Yigit Sent: Wednesday, 16 December 2020 15:37 To: Liron Himi Cc: dev@dpdk.org; Luca Boccassi ; Kevin Traynor ; Akhil Goyal Subject: [EXT] Re: [PATCH v2] meson: update meson build for armada drivers External Ema

Re: [dpdk-dev] [EXT] Re: [PATCH v2] meson: update meson build for armada drivers

2020-12-16 Thread Ferruh Yigit
On 12/16/2020 1:41 PM, Liron Himi wrote: -Original Message- From: Ferruh Yigit Sent: Wednesday, 16 December 2020 15:37 To: Liron Himi Cc: dev@dpdk.org; Luca Boccassi ; Kevin Traynor ; Akhil Goyal Subject: [EXT] Re: [PATCH v2] meson: update meson build for armada drivers External Ema

[dpdk-dev] [RFC 0/6] add GTP PSC extension header support

2020-12-16 Thread Shiri Kuzin
In this RFC we add the support for GTP PSC in net/mlx5. This will enable matching on "rte_flow_item_gtp_psc" item. The support in testpmd is extended to support raw encap\decap of GTP PSC. Shiri Kuzin (4): common/mlx5: add matcher fields for GTP extensions net/mlx5: add GTP PSC flow validati

[dpdk-dev] [RFC 1/6] ethdev: update GTP headers

2020-12-16 Thread Shiri Kuzin
From: Viacheslav Ovsiienko This patch introduces the GTP header individual flag bit fields and the header optional word with N-PDU number, Sequence Number and Next Extension Header type. Signed-off-by: Viacheslav Ovsiienko --- lib/librte_net/rte_gtp.h | 33 ++--- 1

[dpdk-dev] [RFC 2/6] app/testpmd: add GTP PSC option support in raw sets

2020-12-16 Thread Shiri Kuzin
From: Viacheslav Ovsiienko This patch add support for generating GTP PDU container session option for the raw encap and raw decap sets. The generated options is single 32-bit word with minimal length specified as 4, no extra fields in the option are supported. The option item must be preceded wit

[dpdk-dev] [RFC 3/6] common/mlx5: add matcher fields for GTP extensions

2020-12-16 Thread Shiri Kuzin
This is a preparation step to support GTP extension header. In this patch we add the matcher fields that will be used to match on the GTP extension header. Signed-off-by: Shiri Kuzin --- drivers/common/mlx5/mlx5_prm.h | 7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/dri

[dpdk-dev] [RFC 5/6] net/mlx5: add GTP PSC item translation

2020-12-16 Thread Shiri Kuzin
This patch adds the translation function which sets the qfi, PDU type. The next extension header which indicates the following extension header type is set to 0x85 - a PDU session container. Signed-off-by: Shiri Kuzin --- drivers/net/mlx5/mlx5_flow_dv.c | 85 + 1

[dpdk-dev] [RFC 6/6] doc: update GTP extension header support

2020-12-16 Thread Shiri Kuzin
added GTP extension header support to mlx5 PMD. The limitations and support were updated in documentation. Signed-off-by: Shiri Kuzin --- doc/guides/nics/mlx5.rst | 5 + 1 file changed, 5 insertions(+) diff --git a/doc/guides/nics/mlx5.rst b/doc/guides/nics/mlx5.rst index 3bda0f8417..3b1a0

[dpdk-dev] [RFC 4/6] net/mlx5: add GTP PSC flow validation

2020-12-16 Thread Shiri Kuzin
In this patch we add validation routine for GTP PSC extension header. The GTP PSC extension header must follow the GTP item. Signed-off-by: Shiri Kuzin --- drivers/net/mlx5/mlx5_flow.h| 5 +++ drivers/net/mlx5/mlx5_flow_dv.c | 70 + 2 files changed, 75 inser

Re: [dpdk-dev] [EXTERNAL] Re: Windows DPDK real-time priority threads causing thread starvation

2020-12-16 Thread Tal Shnaiderman
It was suggested by DmitryM in the last DPDK Windows community call to have the priority parameter configurable (at least for Windows) to allow user to reduce the thread priority and avoid the issues mentioned below. > Subject: RE: [EXTERNAL] Re: [dpdk-dev] Windows DPDK real-time priority > thre

[dpdk-dev] [PATCH] net/bnxt: fix fallback mbuf allocation logic

2020-12-16 Thread Lance Richardson
From: Lance Richardson Fixes for fallback mbuf allocation logic. - Preserve raw (unmasked) producer index. - Iterate over all processed descriptors (representor and non-representor) when checking allocation status. - Invoke fallback allocation logic when an allocation failure h

[dpdk-dev] L4 rx checksum verification in tun/tap pmd

2020-12-16 Thread Jacques Fourie
Hi, I noticed that TCP rx checksum verification fails when using the tap pmd. In this particular case packets with correct checksums will always have PKT_RX_L4_CKSUM_BAD set in ol_flags. The rte_ipv4_udptcp_cksum() and rte_ipv6_udptcp_cksum() functions already return the ones complement of the che

[dpdk-dev] [PATCH v1 3/6] app/regex: read data file once at startup

2020-12-16 Thread Ophir Munk
Up to this commit the input data file was read from scratch for each QP, which is redundant. Starting from this commit the data file is read only once at startup. Each QP will clone the data. Signed-off-by: Ophir Munk --- app/test-regex/main.c | 63 +++

[dpdk-dev] [PATCH v1 2/6] app/regex: support multi QPs

2020-12-16 Thread Ophir Munk
Up to this commit the regex application used one QP which was assigned a number of jobs, each with a different segment of a file to parse. This commit adds support for multiple QPs assignments. All QPs will be assigned the same number of jobs, with the same segments of file to parse. It will enabl

[dpdk-dev] [PATCH v1 1/6] app/regex: move mem pool creation to worker routine

2020-12-16 Thread Ophir Munk
Function rte_pktmbuf_pool_create() is moved from init_port() routine to run_regex() routine. Looking forward on multi core support - init_port() will be called only once as part of application startup while mem pool creation should be called multiple times (per core). Signed-off-by: Ophir Munk --

[dpdk-dev] [PATCH v1 4/6] app/regex: support multi cores

2020-12-16 Thread Ophir Munk
Up to this commit the regex application was running with multiple QPs on a single core. This commit adds the option to specify a number of cores on which multiple QPs will run. A new parameter 'nb_lcores' was added to configure the number of cores: --nb_lcores . If not configured the number of cor

[dpdk-dev] [PATCH v1 6/6] app/regex: replace Linux clock() API with rdtsc

2020-12-16 Thread Ophir Munk
Performance measurement (elapsed time and Gbps) are based on Linux clock() API. The resolution is improved by replacing the clock() API with rte_rdtsc_precise() API. Signed-off-by: Ophir Munk --- app/test-regex/main.c | 32 ++-- 1 file changed, 14 insertions(+), 18 de

[dpdk-dev] [PATCH v1 5/6] app/regex: support performance measurements per QP

2020-12-16 Thread Ophir Munk
Up to this commit measuring the parsing elapsed time and Giga bits per second performance was done on the aggregation of all QPs (per core). This commit seperates the time measurements per individual QP. Signed-off-by: Ophir Munk --- app/test-regex/main.c | 33 ++---

[dpdk-dev] [PATCH v1 0/6] regex multi Q with multi cores support

2020-12-16 Thread Ophir Munk
This patchset enhances the regex application to support multi Q with multi cores. v1: initial release Ophir Munk (6): app/regex: move mem pool creation to worker routine app/regex: support multi QPs app/regex: read data file once at startup app/regex: support multi cores app/regex: sup

Re: [dpdk-dev] [RFC PATCH] build/pkg-config: remove machine arch flag

2020-12-16 Thread David Christensen
On 12/11/20 7:51 AM, Bruce Richardson wrote: Traditionally any apps built using DPDK had to support the same instruction sets as supported when the DPDK SDK itself was built, since that was "leaked" through to the end-app and DPDK headers via RTE_MACHINE_CPUFLAG_* values. Therefore, when conve

Re: [dpdk-dev] [PATCH 3/6] net/ionic: fully implement remove-on-close

2020-12-16 Thread Andrew Boyer
Hello Ferruh, > On Dec 15, 2020, at 8:42 AM, Ferruh Yigit wrote: > > On 12/10/2020 2:22 PM, Andrew Boyer wrote: >> This is now required behavior for a PMD. >> Remove the UNMAINTAINED flag. >> Signed-off-by: Andrew Boyer >> --- >> MAINTAINERS | 2 +- >> drivers/net/ionic/i

Re: [dpdk-dev] 19.11.6 patches review and test

2020-12-16 Thread Ali Alnubani
Hi, > -Original Message- > From: luca.bocca...@gmail.com > Sent: Thursday, December 3, 2020 11:39 AM > To: sta...@dpdk.org > Cc: dev@dpdk.org; Abhishek Marathe ; > Akhil Goyal ; Ali Alnubani ; > benjamin.wal...@intel.com; David Christensen ; > Hemant Agrawal ; Ian Stokes > ; Jerin Jacob ;

[dpdk-dev] [PATCH v2 0/7] net/ionic: fixes for stop and start

2020-12-16 Thread Andrew Boyer
These patches address issues found when testing port stop and start, link up and down, and queue stop and start. The UNMAINTAINED flag is removed in patch 3. Signed-off-by: Andrew Boyer --- v2: * Address Ferruh's comments * Insert a new patch "net/ionic: remove multi-LIF support" so that the

[dpdk-dev] [PATCH v2 1/7] net/ionic: preserve RSS state unless RETA size changes

2020-12-16 Thread Andrew Boyer
This preserves settings across a LIF stop/start. This will become important when link toggling is tied to LIF stop/start. Signed-off-by: Andrew Boyer --- drivers/net/ionic/ionic_lif.c | 44 --- drivers/net/ionic/ionic_lif.h | 1 + 2 files changed, 21 insertions(+

[dpdk-dev] [PATCH v2 2/7] net/ionic: preserve Rx mode across LIF stop/start

2020-12-16 Thread Andrew Boyer
Otherwise, non-default settings (like PROMISC) get reset. This will become important when link toggling is tied to LIF stop/start. Signed-off-by: Andrew Boyer --- drivers/net/ionic/ionic_lif.c | 18 +- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/drivers/net/ioni

[dpdk-dev] [PATCH v2 3/7] net/ionic: remove multi-LIF support

2020-12-16 Thread Andrew Boyer
This feature is unused, so remove it. There is exactly one adapter / lif / ethdev per port. Signed-off-by: Andrew Boyer --- drivers/net/ionic/ionic.h| 3 +- drivers/net/ionic/ionic_dev.c| 16 ++ drivers/net/ionic/ionic_dev.h| 8 ++--- drivers/net/ionic/ionic_ethdev.c |

[dpdk-dev] [PATCH v2 4/7] net/ionic: fully implement remove-on-close

2020-12-16 Thread Andrew Boyer
ionic_dev_close() is responsible for destroying the ethdev, lif, and adapter. eth_ionic_dev_remove() calls ionic_dev_close(). Remove-on-close is now required behavior for a PMD. Remove the UNMAINTAINED flag. Signed-off-by: Andrew Boyer --- MAINTAINERS | 2 +- drivers/net/i

[dpdk-dev] [PATCH v2 5/7] net/ionic: improve link state handling

2020-12-16 Thread Andrew Boyer
Add UP and FW_RESET state flags. Update the stack info when the link state changes. Convert set_link_up/set_link_down to lif_start/lif_stop. Condition reported link state on UP flag. Signed-off-by: Andrew Boyer --- drivers/net/ionic/ionic_ethdev.c | 49 ++- drivers/ne

[dpdk-dev] [PATCH v2 6/7] net/ionic: improve queue state handling

2020-12-16 Thread Andrew Boyer
Skip ionic_lif_[rxq|txq]_init() in queue start if it's already done. Move ionic_lif_[rxq|txq]_deinit() from queue stop to queue release. This allows the queues to be restarted. Signed-off-by: Andrew Boyer --- drivers/net/ionic/ionic_rxtx.c | 62 +++--- 1 file changed

[dpdk-dev] [PATCH v2 7/7] net/ionic: stop queues when LIF is stopped

2020-12-16 Thread Andrew Boyer
Otherwise they cannot be restarted, because the FW will reject INIT or ENA commands on queues which are already running. Signed-off-by: Andrew Boyer --- drivers/net/ionic/ionic_lif.c | 14 ++ 1 file changed, 14 insertions(+) diff --git a/drivers/net/ionic/ionic_lif.c b/drivers/net/i

Re: [dpdk-dev] [EXT] Re: [PATCH v2] meson: update meson build for armada drivers

2020-12-16 Thread Liron Himi
-Original Message- From: Ferruh Yigit Sent: Wednesday, 16 December 2020 15:49 To: Liron Himi Cc: dev@dpdk.org; Luca Boccassi ; Kevin Traynor ; Akhil Goyal Subject: Re: [EXT] Re: [PATCH v2] meson: update meson build for armada drivers On 12/16/2020 1:41 PM, Liron Himi wrote: > >

[dpdk-dev] [PATCH v3] meson: update meson build for armada drivers

2020-12-16 Thread lironh
From: Liron Himi With pkg-config support available within musdk library (from musdk-release-SDK-10.3.5.0-PR2 version), meson option 'lib_musdk_dir' can be removed. PKG_CONFIG_PATH environment variable should be set appropriately to use the musdk library. docs are updated with new musdk version a

Re: [dpdk-dev] [PATCH v13 00/12] Arm build options rework

2020-12-16 Thread Vimal Chungath
On 11/20/20 4:08 AM, Juraj Linkeš wrote: > The current way of specifying Arm configuration options is insufficient > since we can't identify the SoC we're building for from the MIDR > information. For example, we can't distinguish between N1SDP, Graviton2 > or Ampere Altra. > > Add a way to specify

[dpdk-dev] DPDK ENA PMD spurious ierrors

2020-12-16 Thread Roger Melton (rmelton)
We are seeing issues with the DPDK 18.11 ENA PMD incrementing rx_errors stat on good packets (checksum later validated by software). We’ve tried several versions from DPDK 18.11 stable, including 18.11.9 and 18.11.10. Looking through ENA PMD commits, I see there have been a number of rx stats

[dpdk-dev] [PATCH] net/enic: remove deprecated FDIR code

2020-12-16 Thread John Daley
The Flow Director (FDIR) API was removed in release 20.11. This patch removes the remainder of the FDIR code in the PMD. Signed-off-by: John Daley Reviewed-by: Hyong Youb Kim --- drivers/net/enic/enic.h | 23 -- drivers/net/enic/enic_clsf.c | 334

Re: [dpdk-dev] [PATCH v2] doc: fix some statements for ice vector PMD

2020-12-16 Thread Rong, Leyi
> -Original Message- > From: Zhang, Qi Z > Sent: Wednesday, December 16, 2020 8:22 AM > To: Rong, Leyi > Cc: Yigit, Ferruh ; dev@dpdk.org; Zhang, Qi Z > ; sta...@dpdk.org > Subject: [PATCH v2] doc: fix some statements for ice vector PMD > > 1. Add descriptions for how to select avx512

Re: [dpdk-dev] [PATCH v2] net/i40e: fix argument in RSS action

2020-12-16 Thread Xing, Beilei
Hi, According to Bo's test, this fix patch will cause other cases fail, so NACK. @Bo, could you please detail which cases will FAIL, thanks. BR, Beilei > -Original Message- > From: dev On Behalf Of Kumar Amber > Sent: Wednesday, November 11, 2020 6:11 PM > To: dev@dpdk.org > Cc: Zhao1,

[dpdk-dev] [PATCH] update Broadcom roadmap for 21.02

2020-12-16 Thread Ajit Khaparde
Signed-off-by: Ajit Khaparde --- content/roadmap/_index.md | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/content/roadmap/_index.md b/content/roadmap/_index.md index 4566902..80d328d 100644 --- a/content/roadmap/_index.md +++ b/content/roadmap/_index.md @@ -16,7 +16,8 @@ Th