[dpdk-dev] [PATCH v2] net/i40e: fix incorrect checksum flag of L4 checksum

2020-12-02 Thread Murphy Yang
When send tunneled packet that inner L4 checksum value is correct, the test_pmd output log shows 'ol_flags' value is 'PKT_RX_L4_CKSUM_UNKNOWN', but expected value is 'PKT_RX_L4_CKSUM_GOOD'. If the inner l4 checksum is correctly, mark the 'PKT_RX_L4_CKSUM_GOOD' flag to 'l3_l4e_flags' for sse and 'l

Re: [dpdk-dev] [PATCH v4 3/4] test: add test case to validate VFIO DMA map/unmap

2020-12-02 Thread Nithin Dabilpuram
On Wed, Dec 02, 2020 at 11:23:09AM -0800, David Christensen wrote: > > > On 12/1/20 9:46 PM, Nithin Dabilpuram wrote: > > Test case mmap's system pages and tries to performs a user > > DMA map and unmap both partially and fully. > > > > Signed-off-by: Nithin Dabilpuram > > Acked-by: Anatoly Bur

[dpdk-dev] [PATCH] net/iavf: improve default RSS

2020-12-02 Thread Xuan Ding
This patch adds support to actively configure RSS. Any kernel PF enabled default RSS will be disabled during initialization. Currently supported default rss_type: ipv4[6], ipv4[6]_udp, ipv4[6]_tcp, ipv4[6]_sctp. Signed-off-by: Xuan Ding --- drivers/net/iavf/iavf.h| 12 - drivers/net/

[dpdk-dev] [PATCH v2 1/2] net/mlx5: optimize hash list synchronization

2020-12-02 Thread Suanming Mou
Since all the hash table operations are related with one dedicated bucket, the hash table lock and gen_cnt can be allocated per-bucket. Currently, the hash table uses one global lock to protect all the buckets, that global lock avoids the buckets to be operated at one time, it hurts the hash table

[dpdk-dev] [PATCH v2 2/2] net/mlx5: optimize the hash list entry memory

2020-12-02 Thread Suanming Mou
Currently, the hash list saves the hash key in the hash entry. And the key is mostly used to get the bucket index only. Save the entire 64 bits key to the entry will not be a good option if the key is only used to get the bucket index. Since 64 bits costs more memory for the entry, mostly the sign

[dpdk-dev] [PATCH v2 0/2] net/mlx5: hash list optimization

2020-12-02 Thread Suanming Mou
This patch set contains the hash list optimization for the lock and memory. v2: - Remove the __rte_pack in struct mlx5_hlist_entry. Suanming Mou (2): net/mlx5: optimize hash list synchronization net/mlx5: optimize the hash list entry memory drivers/net/mlx5/linux/mlx5_os.c | 5 +- drivers

Re: [dpdk-dev] [PATCH v2] windows: build warnings with clang

2020-12-02 Thread Dmitry Kozlyuk
On Sun, 29 Nov 2020 16:00:24 +, Nick Connolly wrote: > Microsoft CRT defines Windows-specific secure alternatives to > standard library functions and triggers warnings when "insecure" > functions are used [1]. However, calling code already has all > necessary checks around those functions, so t

[dpdk-dev] [PATCH 2/4] vdpa/mlx5: default polling mode delay time to zero

2020-12-02 Thread Xueming Li
To improve performance and latency, this patch set Rx polling mode default delay time to zero. Signed-off-by: Xueming Li Acked-by: Matan Azrad --- drivers/vdpa/mlx5/mlx5_vdpa.h | 2 +- drivers/vdpa/mlx5/mlx5_vdpa_event.c | 3 ++- 2 files changed, 3 insertions(+), 2 deletions(-) diff --gi

[dpdk-dev] [PATCH 4/4] vdpa/mlx5: set default event mode to polling

2020-12-02 Thread Xueming Li
For better performance and latency, this patch sets default event handling mode to polling mode which uses dedicate thread per device to poll and process event. Signed-off-by: Xueming Li Acked-by: Matan Azrad --- doc/guides/vdpadevs/mlx5.rst | 2 +- drivers/vdpa/mlx5/mlx5_vdpa.c | 2 +- 2 file

[dpdk-dev] [PATCH 3/4] vdpa/mlx5: add cpu core parameter to bind polling thread

2020-12-02 Thread Xueming Li
This patch adds new device argument to specify cpu core affinity to event polling thread for better latency and throughput. The thread could be also located by name "vDPA-mlx5-". Signed-off-by: Xueming Li Acked-by: Matan Azrad --- doc/guides/vdpadevs/mlx5.rst| 5 + drivers/vdpa/mlx

[dpdk-dev] [PATCH 1/4] vdpa/mlx5: set polling mode default delay to zero

2020-12-02 Thread Xueming Li
To improve throughput and latency, this patch allows Rx polling timer delay to 0us. Signed-off-by: Xueming Li Acked-by: Matan Azrad --- doc/guides/vdpadevs/mlx5.rst | 3 +-- drivers/vdpa/mlx5/mlx5_vdpa.c | 9 +++-- 2 files changed, 4 insertions(+), 8 deletions(-) diff --git a/doc/guides/v

[dpdk-dev] [PATCH v2] usertools: show hugepages on POWER systems

2020-12-02 Thread David Christensen
The IBM PowerNV systems include NUMA nodes that don't have associated CPUs or hugepage memory. Here is an example on an IBM AC922 system: $ lscpu ... NUMA node0 CPU(s): 0-63 NUMA node8 CPU(s): 64-127 NUMA node252 CPU(s): ... $ numastat -m ... Node 0 Node 8

Re: [dpdk-dev] Printfs in Signal Handler

2020-12-02 Thread Thomas Monjalon
01/12/2020 10:24, Thomas Monjalon: > 01/12/2020 04:39, prateekag: > > printf is not signal safe and may lead to deadlock if kept in signal > > handler > > and > > signal comes in two quick succession. It will not lead to incorrect > > behavior > > and it is a highly unlikely event. And an exper

Re: [dpdk-dev] [PATCH v4 3/4] test: add test case to validate VFIO DMA map/unmap

2020-12-02 Thread David Christensen
On 12/1/20 9:46 PM, Nithin Dabilpuram wrote: Test case mmap's system pages and tries to performs a user DMA map and unmap both partially and fully. Signed-off-by: Nithin Dabilpuram Acked-by: Anatoly Burakov --- app/test/meson.build | 1 + app/test/test_vfio.c | 103

[dpdk-dev] [PATCH v2] license: add licenses for exception cases

2020-12-02 Thread John McNamara
The license/exceptions.txt file lists a small number of files that have licenses that are exceptions to the three main licenses defined in the Intellectual Property Policy of the DPDK Charter. The three exception licenses are MIT, ISC and BSD-2-Clause. Each of these licenses states that the conten

Re: [dpdk-dev] [PATCH v4 2/4] vfio: fix DMA mapping granularity for type1 IOVA as VA

2020-12-02 Thread David Christensen
On 12/1/20 9:46 PM, Nithin Dabilpuram wrote: Partial unmapping is not supported for VFIO IOMMU type1 by kernel. Though kernel gives return as zero, the unmapped size returned will not be same as expected. So check for returned unmap size and return error. For IOVA as PA, DMA mapping is alread

Re: [dpdk-dev] [PATCH v4 1/4] vfio: revert changes for map contiguous areas in one go

2020-12-02 Thread David Christensen
On 12/1/20 9:46 PM, Nithin Dabilpuram wrote: In order to save DMA entries limited by kernel both for externel memory and hugepage memory, an attempt was made to map physically contiguous memory in one go. This cannot be done as VFIO IOMMU type1 does not support partially unmapping a previously

Re: [dpdk-dev] [PATCH 1/1] usertools: show hugepages on POWER systems

2020-12-02 Thread Stephen Hemminger
On Wed, 2 Dec 2020 10:19:59 -0800 David Christensen wrote: > The IBM PowerNV systems include NUMA nodes that don't have associated > CPUs or hugepage memory. Here is an example on an IBM AC922 system: > > $ lscpu > ... > NUMA node0 CPU(s): 0-63 > NUMA node8 CPU(s): 64-127 > NUMA node252 CP

[dpdk-dev] [PATCH 1/1] usertools: show hugepages on POWER systems

2020-12-02 Thread David Christensen
The IBM PowerNV systems include NUMA nodes that don't have associated CPUs or hugepage memory. Here is an example on an IBM AC922 system: $ lscpu ... NUMA node0 CPU(s): 0-63 NUMA node8 CPU(s): 64-127 NUMA node252 CPU(s): ... $ numastat -m ... Node 0 Node 8

[dpdk-dev] [PATCH] doc: add profile with vtune section to prog guide

2020-12-02 Thread eugeny . parshutin
From: Eugeny Parshutin Return back 'profiling with vtune' section to profiling programmers guide with updated instruction on how to enable vtune profiling with meson configuration option. Signed-off-by: Eugeny Parshutin --- doc/guides/prog_guide/profile_app.rst | 14 ++ 1 file chan

Re: [dpdk-dev] [PATCH v1 00/38] net/mvpp2: misc updates

2020-12-02 Thread Liron Himi
Acked-by: Liron Himi -Original Message- From: lir...@marvell.com Sent: Wednesday, 2 December 2020 12:12 To: Jerin Jacob Kollanukkaran Cc: dev@dpdk.org; Liron Himi Subject: [PATCH v1 00/38] net/mvpp2: misc updates From: Liron Himi This patch series align the mainline driver with al

[dpdk-dev] Printfs in Signal Handler

2020-12-02 Thread prateekag
printf is not signal safe and may lead to deadlock if kept in signal handler and signal comes in two quick succession. It will not lead to incorrect behavior and it is a highly unlikely event. And an expert may understand why this issue happened. Is it worthwhile to change these printfs to un

[dpdk-dev] [PATCH v1 2/2] doc: update armada docs

2020-12-02 Thread lironh
From: Liron Himi - update musdk requirement branch - add the use of PKG_CONFIG Signed-off-by: Liron Himi Reviewed-by: Liron Himi --- doc/guides/cryptodevs/mvsam.rst | 61 ++--- doc/guides/nics/mvneta.rst | 45 doc/guides/nics/mvpp2.rst

[dpdk-dev] [PATCH v1 1/2] meson: update meson build for armada drivers

2020-12-02 Thread lironh
From: Liron Himi With pkg-config support available within musdk library, meson option 'lib_musdk_dir' can be removed. PKG_CONFIG_PATH environment variable should be set appropriately to use the musdk library. Signed-off-by: Liron Himi Reviewed-by: Liron Himi --- drivers/common/mvep/meson.buil

[dpdk-dev] [PATCH v1 0/2] armada: introduce musdk pkg-config

2020-12-02 Thread lironh
From: Liron Himi - update meson files to use the musdk pkg-config - update armada docs Liron Himi (2): meson: update meson build for armada drivers doc: update armada docs doc/guides/cryptodevs/mvsam.rst| 61 +++--- doc/guides/nics/mvneta.rst | 45 ++

[dpdk-dev] [PATCH V1] app/testpmd: fix queue stats mapping configuration

2020-12-02 Thread Min Hu (Connor)
From: Huisong Li Currently, the queue stats mapping has the following problems: 1) Many PMD drivers don't support queue stats mapping. But there is no failure message after executing the command "set stat_qmap rx 0 2 2". 2) Once queue mapping is set, unrelated and unmapped queues are also displa

Re: [dpdk-dev] [PATCH 4/4] net/hns3: fix FEC state query

2020-12-02 Thread Min Hu (Connor)
在 2020/11/20 22:35, Ferruh Yigit 写道: On 11/20/2020 2:33 PM, Ferruh Yigit wrote: On 11/20/2020 11:27 AM, Lijun Ou wrote: From: "Min Hu (Conor)" As FEC is not supported below 10 Gbps, CMD(0x031A) offered from Firmware read will return fail in 10 Gbps device. This patch will prevent read thi

Re: [dpdk-dev] [PATCH v4 3/6] app/testpmd: remove restriction on txpkts set

2020-12-02 Thread Ferruh Yigit
On 11/27/2020 1:05 PM, Slava Ovsiienko wrote: -Original Message- From: Ferruh Yigit Sent: Thursday, November 26, 2020 14:38 To: Slava Ovsiienko ; NBU-Contact-Thomas Monjalon ; Wei Hu (Xavier) Cc: dev@dpdk.org; xavier.hu...@huawei.com Subject: Re: [dpdk-dev] [PATCH v4 3/6] app/testpmd: r

[dpdk-dev] [PATCH 2/2] net/mlx5: optimize the hash list entry memory

2020-12-02 Thread Suanming Mou
Currently, the hash list saves the hash key in the hash entry. And the key is mostly used to get the bucket index only. Save the entire 64 bits key to the entry will not be a good option if the key is only used to get the bucket index. Since 64 bits costs more memory for the entry, mostly the sign

[dpdk-dev] [PATCH 1/2] net/mlx5: optimize hash list synchronization

2020-12-02 Thread Suanming Mou
Since all the hash table operations are related with one dedicated bucket, the hash table lock and gen_cnt can be allocated per-bucket. Currently, the hash table uses one global lock to protect all the buckets, that global lock avoids the buckets to be operated at one time, it hurts the hash table

[dpdk-dev] [PATCH 0/2] net/mlx5: hash list optimization

2020-12-02 Thread Suanming Mou
This patch set contains the hash list optimization for the lock and memory. Suanming Mou (2): net/mlx5: optimize hash list synchronization net/mlx5: optimize the hash list entry memory drivers/net/mlx5/linux/mlx5_os.c | 5 ++- drivers/net/mlx5/mlx5.c | 3 +- drivers/net/mlx5/mlx5_

[dpdk-dev] [v2 PATCH] usertools: show valid hugepage sizes if user requests an invalid hugepage size

2020-12-02 Thread Sarosh Arif
If user requests a hugepage size which is not supported by the system, currently user gets an error message saying that the requested size is not a valid system huge page size. In addition to this if we display the valid hugepage sizes it will be convenient for the user to request the right size ne

Re: [dpdk-dev] [PATCH] net/ixgbe: check the result of setting VF MTU

2020-12-02 Thread Wang, Haiyue
> -Original Message- > From: Zhang,Alvin > Sent: Wednesday, December 2, 2020 17:48 > To: Guo, Jia ; Wang, Haiyue > Cc: dev@dpdk.org; Zhang, AlvinX ; sta...@dpdk.org > Subject: [PATCH] net/ixgbe: check the result of setting VF MTU > > From: Alvin Zhang > > If a VF request to set a inval

Re: [dpdk-dev] [RFC V4] app/testpmd: fix queue stats mapping configuration

2020-12-02 Thread Ferruh Yigit
On 11/30/2020 8:29 AM, Min Hu (Connor) wrote: Hi, Ferruh, and all, are there any comments about this set of patch? I am good with this one, would you mind sending an actual patch (not RFC) on top of the latest head? Thanks, ferruh 在 2020/11/23 17:51, Ferruh Yigit 写道: On 11/21/2020 4

[dpdk-dev] FOSDEM 2021 SDN Devroom CFP

2020-12-02 Thread Kinsella, Ray
We are pleased to announce the Call for Participation for the FOSDEM 2021 Software Defined Networking devroom! FOSDEM is a free event for software developers to meet, share ideas and collaborate. - FOSDEM is free to attend. There is no registration. - FOSDEM website: http://www.fosdem.org/ - FOSDE

[dpdk-dev] [PATCH v1 38/38] net/mvpp2: add fill_bpool_buffs to cfg file

2020-12-02 Thread lironh
From: Dana Vardi extend config file with 'fill_bpool_buffs' which control the amount of refill buffers Signed-off-by: Dana Vardi Reviewed-by: Liron Himi --- drivers/net/mvpp2/mrvl_ethdev.c | 7 --- drivers/net/mvpp2/mrvl_ethdev.h | 3 +++ drivers/net/mvpp2/mrvl_qos.c| 21 +++

[dpdk-dev] [PATCH v1 37/38] net/mvpp2: update qos defaults parameter name

2020-12-02 Thread lironh
From: Dana Vardi 'global_default' is only being used for 'qos' so adding 'qos' into its name Signed-off-by: Dana Vardi Reviewed-by: Liron Himi --- drivers/net/mvpp2/mrvl_ethdev.c | 2 +- drivers/net/mvpp2/mrvl_qos.c| 12 ++-- drivers/net/mvpp2/mrvl_qos.h| 2 +- 3 files chang

[dpdk-dev] [PATCH v1 36/38] net/mvpp2: forward bad packets support

2020-12-02 Thread lironh
From: Dana Vardi extend the config file with option to forward packets that were marked as "l2 bad pkts". by default the driver drop those packets Signed-off-by: Dana Vardi Reviewed-by: Liron Himi --- drivers/net/mvpp2/mrvl_ethdev.c | 10 -- drivers/net/mvpp2/mrvl_ethdev.h | 2 ++ dr

[dpdk-dev] [PATCH v1 35/38] net/mvpp2: support custom header before ethernet

2020-12-02 Thread lironh
From: Dana Vardi extend 'start_hdr' options with custom header. Signed-off-by: Dana Vardi Reviewed-by: Liron Himi --- drivers/net/mvpp2/mrvl_qos.c | 5 + 1 file changed, 5 insertions(+) diff --git a/drivers/net/mvpp2/mrvl_qos.c b/drivers/net/mvpp2/mrvl_qos.c index c7bd8825c..10e8636b4 10

[dpdk-dev] [PATCH v1 34/38] net/mvpp2: consider ptype in cksum info

2020-12-02 Thread lironh
From: Liron Himi provide checksum information based on the ptype. Signed-off-by: Liron Himi Reviewed-by: Liron Himi --- drivers/net/mvpp2/mrvl_ethdev.c | 34 +++-- 1 file changed, 20 insertions(+), 14 deletions(-) diff --git a/drivers/net/mvpp2/mrvl_ethdev.c b/dri

[dpdk-dev] [PATCH v1 32/38] net/mvpp2: apply flow-ctrl after port init

2020-12-02 Thread lironh
From: Liron Himi in case ppio was not initialized yet (only at 'start' function) the flow-ctrl setting should be saved for later stage. Signed-off-by: Liron Himi Reviewed-by: Liron Himi --- drivers/net/mvpp2/mrvl_ethdev.c | 26 +- drivers/net/mvpp2/mrvl_ethdev.h | 2 +

[dpdk-dev] [PATCH v1 33/38] net/mvpp2: change dsa_mode naming

2020-12-02 Thread lironh
From: Dana Vardi change 'dsa_mode' to 'start_hdr' in config file Signed-off-by: Dana Vardi Reviewed-by: Liron Himi --- drivers/net/mvpp2/mrvl_qos.c | 29 +++-- 1 file changed, 19 insertions(+), 10 deletions(-) diff --git a/drivers/net/mvpp2/mrvl_qos.c b/drivers/net/mv

[dpdk-dev] [PATCH v1 31/38] net/mvpp2: add support of LINK_SPEED_2_5G

2020-12-02 Thread lironh
From: Meir Levi update capability with 2.5G support Signed-off-by: Meir Levi Reviewed-by: Liron Himi --- drivers/net/mvpp2/mrvl_ethdev.c | 4 1 file changed, 4 insertions(+) diff --git a/drivers/net/mvpp2/mrvl_ethdev.c b/drivers/net/mvpp2/mrvl_ethdev.c index 9fd9269eb..d388fde96 100644

[dpdk-dev] [PATCH v1 30/38] net/mvpp2: expose max mtu size

2020-12-02 Thread lironh
From: Liron Himi expose max-mtu based on the max frame size that l4 checksum generation can be done by HW. Signed-off-by: Liron Himi Reviewed-by: Yuri Chipchev --- drivers/net/mvpp2/mrvl_ethdev.c | 24 ++-- drivers/net/mvpp2/mrvl_ethdev.h | 1 + 2 files changed, 23 insert

[dpdk-dev] [PATCH v1 29/38] net/mvpp2: autoneg disable handling

2020-12-02 Thread lironh
From: Yuri Chipchev flow control autoneg disable is not supported Signed-off-by: Yuri Chipchev Reviewed-by: Liron Himi --- drivers/net/mvpp2/mrvl_ethdev.c | 9 +++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/drivers/net/mvpp2/mrvl_ethdev.c b/drivers/net/mvpp2/mrvl_ethd

[dpdk-dev] [PATCH v1 28/38] net/mvpp2: propagate port-id in udata64

2020-12-02 Thread lironh
From: Liron Himi mbuf->port can be override and used for eventdev so saving the port-id information in another field tht can be query by application Signed-off-by: Liron Himi Reviewed-by: Liron Himi --- drivers/net/mvpp2/mrvl_ethdev.c | 1 + 1 file changed, 1 insertion(+) diff --git a/driver

[dpdk-dev] [PATCH v1 27/38] net/mvpp2: dummy pool creation

2020-12-02 Thread lironh
From: Liron Himi Currently the HW is configured with only one pool which its buffer size may be larger than the rx-fifo-size. In that situation, frame size larger than the fifo-size is gets dropped due to fifo overrun. this is cause because the HW works in cut-through mode which waits to have in

[dpdk-dev] [PATCH v1 26/38] net/mvpp2: rearrange functions order

2020-12-02 Thread lironh
From: Liron Himi rearrange functions order Signed-off-by: Liron Himi Reviewed-by: Liron Himi --- drivers/net/mvpp2/mrvl_ethdev.c | 98 - 1 file changed, 49 insertions(+), 49 deletions(-) diff --git a/drivers/net/mvpp2/mrvl_ethdev.c b/drivers/net/mvpp2/mrvl_eth

[dpdk-dev] [PATCH v1 25/38] net/mvpp2: support udf configuration

2020-12-02 Thread lironh
From: Liron Himi extend the config file with 'udf' (user-defined) settings Signed-off-by: Liron Himi Reviewed-by: Liron Himi --- drivers/net/mvpp2/mrvl_ethdev.c | 4 +- drivers/net/mvpp2/mrvl_qos.c| 212 drivers/net/mvpp2/mrvl_qos.h| 3 + 3 files c

[dpdk-dev] [PATCH v1 24/38] net/mvpp2: move common functions to common location

2020-12-02 Thread lironh
From: Liron Himi move common functions to common location Signed-off-by: Liron Himi Reviewed-by: Liron Himi --- drivers/net/mvpp2/mrvl_ethdev.h | 41 + drivers/net/mvpp2/mrvl_qos.c| 24 --- 2 files changed, 41 insertions(+), 24 deletions(-)

[dpdk-dev] [PATCH v1 23/38] net/mvpp2: skip qos init if not requested

2020-12-02 Thread lironh
From: Liron Himi skip qos init if not requested Signed-off-by: Liron Himi Reviewed-by: Liron Himi --- drivers/net/mvpp2/mrvl_qos.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/drivers/net/mvpp2/mrvl_qos.c b/drivers/net/mvpp2/mrvl_qos.c index 18cf470dd..d8f6dd5c6 100

[dpdk-dev] [PATCH v1 22/38] net/mvpp2: flow: add support for RAW type

2020-12-02 Thread lironh
From: Liron Himi add support for RAW type and connect it to MUSDK UDF Signed-off-by: Liron Himi Reviewed-by: Liron Himi --- drivers/net/mvpp2/mrvl_ethdev.h | 1 + drivers/net/mvpp2/mrvl_flow.c | 141 2 files changed, 142 insertions(+) diff --git a/drivers

[dpdk-dev] [PATCH v1 20/38] net/mvpp2: flow: support generic pattern combinations

2020-12-02 Thread lironh
From: Liron Himi Currently only specific pattern combinations are supported. this makes it hard to support additional pattern. in addition there is no a real limitation that prevent any combination. This patch iterate the input patterns and convert them to a mvpp2 API. Signed-off-by: Liron Himi

[dpdk-dev] [PATCH v1 21/38] net/mvpp2: flow: build table key along with rule

2020-12-02 Thread lironh
From: Liron Himi build table key along with rule Signed-off-by: Liron Himi Reviewed-by: Liron Himi --- drivers/net/mvpp2/mrvl_ethdev.h | 33 +--- drivers/net/mvpp2/mrvl_flow.c | 257 +--- 2 files changed, 106 insertions(+), 184 deletions(-) diff --git a/drivers

[dpdk-dev] [PATCH v1 19/38] net/mvpp2: replace 'qos_cfg' with 'cfg'

2020-12-02 Thread lironh
From: Liron Himi as the config file is not just for 'qos' it is more accurate to replace the name from 'qos_cfg' to 'cfg' Signed-off-by: Liron Himi Reviewed-by: Liron Himi --- drivers/net/mvpp2/mrvl_ethdev.c | 12 ++-- drivers/net/mvpp2/mrvl_qos.c| 31 +++--

[dpdk-dev] [PATCH v1 18/38] net/mvpp2: adjust the number of unicast address

2020-12-02 Thread lironh
From: Liron Himi HW support 25 mac address for filtering plus one for the primary mac address. Signed-off-by: Liron Himi Reviewed-by: Liron Himi --- drivers/net/mvpp2/mrvl_ethdev.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/drivers/net/mvpp2/mrvl_ethdev.c b/driv

[dpdk-dev] [PATCH v1 16/38] net/mvpp2: add dsa mode support

2020-12-02 Thread lironh
From: Liron Himi extend the config file with 'start-hdr' field. currently 'eth' (default) and 'dsa' headers are supported. Signed-off-by: Liron Himi Reviewed-by: Liron Himi --- drivers/net/mvpp2/mrvl_ethdev.c | 4 ++ drivers/net/mvpp2/mrvl_qos.c| 65 ++--- dri

[dpdk-dev] [PATCH v1 17/38] net/mvpp2: add TX flow control

2020-12-02 Thread lironh
From: Yuri Chipchev add tx flow control operations. Signed-off-by: Yuri Chipchev Reviewed-by: Liron Himi --- drivers/net/mvpp2/mrvl_ethdev.c | 59 - 1 file changed, 51 insertions(+), 8 deletions(-) diff --git a/drivers/net/mvpp2/mrvl_ethdev.c b/drivers/net/mvp

[dpdk-dev] [PATCH v1 15/38] net/mvpp2: only use ol_flags for checksum generation offload

2020-12-02 Thread lironh
From: Liron Himi according to the dpdk spec, only 'ol_flags' should be used for tx checksum generation Signed-off-by: Liron Himi Reviewed-by: Liron Himi --- drivers/net/mvpp2/mrvl_ethdev.c | 60 ++--- 1 file changed, 26 insertions(+), 34 deletions(-) diff --git a/

[dpdk-dev] [PATCH v1 14/38] net/mvpp2: add vlan offload support

2020-12-02 Thread lironh
From: Yuri Chipchev enable vlan filter configuration Signed-off-by: Yuri Chipchev Reviewed-by: Liron Himi --- drivers/net/mvpp2/mrvl_ethdev.c | 110 +--- 1 file changed, 86 insertions(+), 24 deletions(-) diff --git a/drivers/net/mvpp2/mrvl_ethdev.c b/drivers/net/m

[dpdk-dev] [PATCH v1 13/38] net/mvpp2: add loopback support

2020-12-02 Thread lironh
From: Yuri Chipchev add support for loopback mode Signed-off-by: Yuri Chipchev Reviewed-by: Liron Himi --- drivers/net/mvpp2/mrvl_ethdev.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/drivers/net/mvpp2/mrvl_ethdev.c b/drivers/net/mvpp2/mrvl_ethdev.c index c70a8fe93..5cd9ee38d 100

[dpdk-dev] [PATCH v1 12/38] net/mvpp2: save initial configuration

2020-12-02 Thread lironh
From: Yuri Chipchev save configuration that was done prior 'start' as only then the ppio is being configured. Signed-off-by: Yuri Chipchev Reviewed-by: Liron Himi --- drivers/net/mvpp2/mrvl_ethdev.c | 107 +++- 1 file changed, 92 insertions(+), 15 deletions(-) dif

[dpdk-dev] [PATCH v1 11/38] net/mvpp2: align checking order

2020-12-02 Thread lironh
From: Yuri Chipchev first check for 'isolated' and then for '!ppio' Signed-off-by: Yuri Chipchev Reviewed-by: Liron Himi --- drivers/net/mvpp2/mrvl_ethdev.c | 30 ++ 1 file changed, 18 insertions(+), 12 deletions(-) diff --git a/drivers/net/mvpp2/mrvl_ethdev.c b/d

[dpdk-dev] [PATCH v1 10/38] net/mvpp2: cosmetic changes to cookie usage

2020-12-02 Thread lironh
From: Yuri Chipchev No need to add high address to cookie on transmit side, as it has already 64bit value Signed-off-by: Yuri Chipchev Reviewed-by: Liron Himi --- drivers/net/mvpp2/mrvl_ethdev.c | 13 +++-- 1 file changed, 7 insertions(+), 6 deletions(-) diff --git a/drivers/net/mvpp

[dpdk-dev] [PATCH v1 09/38] net/mvpp2: extend xstats support

2020-12-02 Thread lironh
From: Yuri Chipchev add xstats_by_id callbacks Signed-off-by: Yuri Chipchev Reviewed-by: Liron Himi --- drivers/net/mvpp2/mrvl_ethdev.c | 90 + 1 file changed, 90 insertions(+) diff --git a/drivers/net/mvpp2/mrvl_ethdev.c b/drivers/net/mvpp2/mrvl_ethdev.c inde

[dpdk-dev] [PATCH v1 08/38] net/mvpp2: rss reservation

2020-12-02 Thread lironh
From: Yuri Chipchev reserve 4 rss tables for lk-4.14 support Signed-off-by: Yuri Chipchev Reviewed-by: Liron Himi --- drivers/net/mvpp2/mrvl_ethdev.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/mvpp2/mrvl_ethdev.c b/drivers/net/mvpp2/mrvl_ethdev.c index d81

[dpdk-dev] [PATCH v1 06/38] net/mvpp2: fix frame size checking

2020-12-02 Thread lironh
From: Liron Himi need to add CRC len to the frame-size to compare against max_rx_pkt_len which includes it. Fixes: 79ec62028 ("net/mvpp2: update MTU and MRU related calculations") Cc: sta...@dpdk.org Signed-off-by: Liron Himi Reviewed-by: Yuri Chipchev --- drivers/net/mvpp2/mrvl_ethdev.c | 3

[dpdk-dev] [PATCH v1 07/38] net/mvpp2: reduce prints on rx path

2020-12-02 Thread lironh
From: Liron Himi Fixes: acab7d58c ("net/mvpp2: convert to dynamic logging") Cc: sta...@dpdk.org Signed-off-by: Liron Himi Reviewed-by: Dana Tearosh --- drivers/net/mvpp2/mrvl_ethdev.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/mvpp2/mrvl_ethdev.c b/drivers

[dpdk-dev] [PATCH v1 05/38] net/mvpp2: remove CRC len from MRU validation

2020-12-02 Thread lironh
From: Liron Himi CRC is being removed by HW before packet get write to the buffer, so CRC len should not be included in MRU validation Fixes: 79ec62028 ("net/mvpp2: update MTU and MRU related calculations") Cc: sta...@dpdk.org Signed-off-by: Liron Himi Reviewed-by: Yuri Chipchev --- drivers/

[dpdk-dev] [PATCH v1 04/38] net/mvpp2: skip vlan flush

2020-12-02 Thread lironh
From: Liron Himi vlan-flush in MUSDK is not supported yet. until it does, the code should be skipped as currently an redundant error message is displayed. Fixes: a8f3d6783 ("net/mrvl: support VLAN filtering") Cc: sta...@dpdk.org Signed-off-by: Liron Himi Reviewed-by: Liron Himi --- drivers/

[dpdk-dev] [PATCH v1 03/38] net/mvpp2: fix rx/tx bytes statistics

2020-12-02 Thread lironh
From: Yuri Chipchev 4B of CRC was not included in the bytes statistics Fixes: bdffe0c70 ("net/mrvl: support basic stats") Cc: sta...@dpdk.org Signed-off-by: Yuri Chipchev Reviewed-by: Liron Himi --- drivers/net/mvpp2/mrvl_ethdev.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) di

[dpdk-dev] [PATCH v1 01/38] net/mvpp2: fix stack corruption

2020-12-02 Thread lironh
From: Yuri Chipchev Fixes stack corruption in mrvl_fill_bpool function in case num > MRVL_PP2_RXD_MAX Fixes: c3637258d894 ("net/mrvl: fix Rx descriptors number") Cc: sta...@dpdk.org Signed-off-by: Yuri Chipchev Reviewed-by: Liron Himi --- drivers/net/mvpp2/mrvl_ethdev.c | 4 ++-- 1 file chan

[dpdk-dev] [PATCH v1 02/38] net/mvpp2: remove debug log on fast-path

2020-12-02 Thread lironh
From: Liron Himi in case of non-ip frame the current code reached the 'default' case which result with function call to log a msg. those kind of calls should not be performed on fast-path. The performance for this kind of frames increased by 50% Fixes: acab7d58c ("net/mvpp2: convert to dynamic

[dpdk-dev] [PATCH v1 00/38] net/mvpp2: misc updates

2020-12-02 Thread lironh
From: Liron Himi This patch series align the mainline driver with all changes since 19.11 some of the patches are fixes which should be pushed to stable Dana Vardi (5): net/mvpp2: change dsa_mode naming net/mvpp2: support custom header before ethernet net/mvpp2: forward bad packets support

[dpdk-dev] [PATCH] net/ixgbe: check the result of setting VF MTU

2020-12-02 Thread Zhang,Alvin
From: Alvin Zhang If a VF request to set a invalid maximum packet length value, The PF kernel driver may disable its reception. This patch add codes to output information and return the error status. Signed-off-by: Alvin Zhang Fixes: 12cd03db ("ixgbevf: allow to set MTU") Cc: sta...@dpdk.

Re: [dpdk-dev] [PATCH 1/2] net/i40e: support GTP checksum offload

2020-12-02 Thread Guo, Jia
Acked-by: Jeff Guo > -Original Message- > From: Murphy Yang > Sent: Wednesday, November 25, 2020 5:32 PM > To: dev@dpdk.org > Cc: Yang, Qiming ; Yang, SteveX > ; Xing, Beilei ; Guo, Jia > ; Lu, Wenzhuo ; Yang, > MurphyX > Subject: [PATCH 1/2] net/i40e: support GTP checksum offload > >