Re: [dpdk-dev] [PATCH v2 1/2] app/testpmd: add link speed check before port start

2021-05-05 Thread Huisong Li
在 2021/5/6 10:36, Li, Xiaoyun 写道: -Original Message- From: Huisong Li Sent: Tuesday, May 4, 2021 09:46 To: Li, Xiaoyun Cc: Yigit, Ferruh ; dev@dpdk.org Subject: Re: [dpdk-dev] [PATCH v2 1/2] app/testpmd: add link speed check before port start 在 2021/4/30 12:46, Li, Xiaoyun 写道: --

[dpdk-dev] [PATCH v3] app/crypto-perf: fix dereference of null return

2021-05-05 Thread Min Hu (Connor)
Return value of a function 'rte_zmalloc' is dereferenced without checking, and it may call segmentation fault. This patch fixed it. Fixes: f8be1786b1b8 ("app/crypto-perf: introduce performance test application") Cc: sta...@dpdk.org Signed-off-by: Min Hu (Connor) --- v3: * fix compiling warning.

Re: [dpdk-dev] [dpdk-stable] [PATCH v3] net/mlx5: fix tunnel offload private items location

2021-05-05 Thread Gregory Etelson
Hello Ferruh, > >> The were some questions around testpmd part of this patch in previous > >> version, they are not answered and this version is dropping the > >> testpmd part. > >> > > > > The tunnel offload API allows application to place tunnel offload elements > at any valid location in a flow

Re: [dpdk-dev] [PATCH] net/i40e: fix offload flag checking in simple Tx datapath

2021-05-05 Thread Zhou, JunX W
> -Original Message- > From: dev On Behalf Of Alvin Zhang > Sent: Tuesday, April 27, 2021 4:59 PM > To: Wu, Jingjing ; Xing, Beilei > Cc: dev@dpdk.org; Zhang, AlvinX ; sta...@dpdk.org > Subject: [dpdk-dev] [PATCH] net/i40e: fix offload flag checking in simple Tx > datapath Tested-by: Ju

[dpdk-dev] [PATCH v5 3/3] net/i40e: enable PCI bus master after reset

2021-05-05 Thread Haiyue Wang
The VF reset can be triggerred by the PF reset event, in this case, the PCI bus master will be cleared, then the VF is not allowed to issue any Memory or I/O Requests. So after the reset event is detected, always enable the PCI bus master. And if failed, the device or system may be in an invalid s

[dpdk-dev] [PATCH v5 2/3] net/iavf: enable PCI bus master after reset

2021-05-05 Thread Haiyue Wang
The VF reset can be triggerred by the PF reset event, in this case, the PCI bus master will be cleared, then the VF is not allowed to issue any Memory or I/O Requests. So after the reset event is detected, always enable the PCI bus master. And if failed, the device or system may be in an invalid s

[dpdk-dev] [PATCH v5 1/3] bus/pci: set PCI master in command register

2021-05-05 Thread Haiyue Wang
Add the API to set 'Bus Master Enable' bit to be enabled or disabled in the PCI command register. Signed-off-by: Haiyue Wang Acked-by: Ray Kinsella --- drivers/bus/pci/pci_common.c | 28 drivers/bus/pci/rte_bus_pci.h | 14 ++ drivers/bus/pci/version.map

[dpdk-dev] [PATCH v5 0/3] fix PF reset causes VF memory request failure

2021-05-05 Thread Haiyue Wang
By triggerring the VF reset from PF reset, echo 1 > /sys/bus/pci/devices/PF-BDF/reset the PCI bus master bit will cleared on VF, so the VF needs to enable this bit before restart. This patch set adds the API to enable PCI bus master. v5: error handling if bus master enable failed v4: ch

[dpdk-dev] [PATCH v2 2/2] examples/ethtool: add closing port operation

2021-05-05 Thread Min Hu (Connor)
From: Huisong Li Currently, ethtool directly ends the process after 'quit' cmd. In this case, software resources are not released and hardware resources of the device are not uninstalled. This patch adds closing port operation to release resources. Fixes: bda68ab9d1e7 ("examples/ethtool: add us

[dpdk-dev] [PATCH v2 1/2] examples/ethtool: fix Rx/Tx queue setup with rte socket id

2021-05-05 Thread Min Hu (Connor)
From: Chengwen Feng In DPDK, 'rte_socket_id' means the running socket while 'rte_eth_dev_socket_id' is the device socket. For better performance, memory which queue setup used and device should be in the same socket. This patch make sure it calls rte_eth_dev_socket_id API to get device socket_id

[dpdk-dev] [PATCH v2 0/2] fix bugs for ethtool APP

2021-05-05 Thread Min Hu (Connor)
This patch fixed fix bugs for ethtool APP. Chengwen Feng (1): examples/ethtool: fix Rx/Tx queue setup with rte socket id Huisong Li (1): examples/ethtool: add closing port operation examples/ethtool/ethtool-app/main.c | 18 ++ examples/ethtool/lib/rte_ethtool.c | 4 ++-- 2

Re: [dpdk-dev] [dpdk-stable] [PATCH v2 1/5] net/txgbe: fix RSS in QINQ

2021-05-05 Thread Jiawen Wu
On April 29, 2021 10:31 PM, Ferruh Yigit wrote: > On 4/29/2021 11:33 AM, Jiawen Wu wrote: > > Support to enable and disable QINQ hardware strip, when configure vlan > > offload with QINQ strip mask, to avoid RSS does not work for QINQ > > packets. > > > > Hi Jiawen, > > What was not working and f

Re: [dpdk-dev] [PATCH v4 2/3] net/iavf: enable PCI bus master after reset

2021-05-05 Thread Wang, Haiyue
> -Original Message- > From: David Marchand > Sent: Wednesday, May 5, 2021 16:39 > To: Wang, Haiyue > Cc: dev ; Zhang, Qi Z ; Wang, Liang-min > ; > Wu, Jingjing ; Xing, Beilei > Subject: Re: [PATCH v4 2/3] net/iavf: enable PCI bus master after reset > > On Wed, May 5, 2021 at 4:56 AM W

Re: [dpdk-dev] [PATCH v3] examples/rxtx_callbacks: fix port ID format specifier

2021-05-05 Thread Stephen Hemminger
On Thu, 6 May 2021 00:54:35 +0300 Dmitry Kozlyuk wrote: > This fixes -Wformat warning with clang 10.0.0 on Windows. > > Fixes: f8244c6399d9 ("ethdev: increase port id range") > Cc: sta...@dpdk.org > > Signed-off-by: Dmitry Kozlyuk > Acked-by: Tyler Retzlaff > --- > v3: PRIx16 -> PRIu16 > v2:

Re: [dpdk-dev] [PATCH v2] examples/rxtx_callbacks: fix port ID format specifier

2021-05-05 Thread Stephen Hemminger
On Thu, 6 May 2021 00:51:33 +0300 Dmitry Kozlyuk wrote: > This fixes -Wformat warning with clang 10.0.0 on Windows. > > Fixes: f8244c6399d9 ("ethdev: increase port id range") > Cc: sta...@dpdk.org > > Signed-off-by: Dmitry Kozlyuk > Acked-by: Tyler Retzlaff > --- > v2: use PRIx16 instead of

[dpdk-dev] 回复: [PATCH v1 3/4] net/mlx5: fix rebuild bug for Memory Region cache

2021-05-05 Thread Feifei Wang
Hi, Slava Would you have more comments about this patch? For my sight, only one wmb before "dev_gen" updating is enough to synchronize. Thanks very much for your attention. Best Regards Feifei > -邮件原件- > 发件人: Feifei Wang > 发送时间: 2021年4月20日 16:42 > 收件人: Slava Ovsiienko ; Matan Azrad >

Re: [dpdk-dev] [dpdk-stable] [DPDK] net/ice: fix txq mbuf release mode

2021-05-05 Thread Zhang, AlvinX
Hi Ferruh, Thanks for your advice. I'll update the commit log in V2. If the following changes are appropriate? net/ice: fix txq vector path selection If Tx vector mode is disabled, the txq vector mode should be disabled too. This patch adds checking of Tx vector mode before enabling txq vector m

[dpdk-dev] [PATCH v1] net/ice: remove redundant RSS configuration for GTPU

2021-05-05 Thread Wenjun Wu
Originally, the default RSS for GTPU is inner fields. Now, we hope outer RSS for GTPU to be the default. Since RSS for ipv4, RSS for ipv6, RSS for UDP and RSS for TCP can cover the cases of outer RSS for GTPU, this patch deletes redundant default RSS configurations for GTPU. Signed-off-by: Wenjun

Re: [dpdk-dev] [PATCH v2 1/2] app/testpmd: add link speed check before port start

2021-05-05 Thread Li, Xiaoyun
> -Original Message- > From: Huisong Li > Sent: Tuesday, May 4, 2021 09:46 > To: Li, Xiaoyun > Cc: Yigit, Ferruh ; dev@dpdk.org > Subject: Re: [dpdk-dev] [PATCH v2 1/2] app/testpmd: add link speed check > before port start > > > 在 2021/4/30 12:46, Li, Xiaoyun 写道: > > > >> -Origina

Re: [dpdk-dev] [PATCH] examples/timer: fix incorrect time interval

2021-05-05 Thread Chengchang Tang
On 2021/5/6 5:37, Thomas Monjalon wrote: > 15/04/2021 09:12, Min Hu (Connor): >> From: Chengchang Tang >> >> Timer sample example assumes that the frequency of the timer is about >> 2Ghz to control the period of calling rte_timer_manage(). But this >> assumption is easy to fail. For example. th

Re: [dpdk-dev] [dpdk-stable] [PATCH v2 2/5] net/txgbe: fix VF MTU limit setting

2021-05-05 Thread Jiawen Wu
On April 29, 2021 11:00 PM, Ferruh Yigit wrote: > On 4/29/2021 11:33 AM, Jiawen Wu wrote: > > Add the dev_started check to remove the limitation of VF MTU setting. > > When device is stopped, it is allowed to set MTU bigger than mbuf size. > > Scattered rx may be enabled in next starting, exempt fr

Re: [dpdk-dev] [dpdk-stable] [PATCH] examples/rxtx_callbacks: fix port ID format specifier

2021-05-05 Thread Thomas Monjalon
06/05/2021 00:45, Tyler Retzlaff: > as a side question, what is the projects stance on getting more warnings > clean? there are a few not enabled that i'd really like to see e.g. > format, conversion, truncation etc.. > > i looked at lib/eal previously and there are... hundreds? of instances so >

[dpdk-dev] [dpdk-announce] release candidate 21.05-rc2

2021-05-05 Thread Thomas Monjalon
A new DPDK release candidate is ready for testing: https://git.dpdk.org/dpdk/tag/?id=v21.05-rc2 There are 296 new patches in this snapshot, most of them adding or fixing features in drivers. Release notes: https://doc.dpdk.org/guides/rel_notes/release_21_05.html Please test and r

Re: [dpdk-dev] [dpdk-stable] [PATCH] examples/rxtx_callbacks: fix port ID format specifier

2021-05-05 Thread Tyler Retzlaff
On Wed, May 05, 2021 at 11:39:23PM +0200, Thomas Monjalon wrote: > 05/05/2021 18:00, Tyler Retzlaff: > > On Tue, May 04, 2021 at 09:48:22AM +0300, Dmitry Kozlyuk wrote: > > > > > > > > what was the -Wformat clang on windows complaint? > > > > > > PRIx16 would work, but I noticed that in other pla

[dpdk-dev] [PATCH v3] examples/rxtx_callbacks: fix port ID format specifier

2021-05-05 Thread Dmitry Kozlyuk
This fixes -Wformat warning with clang 10.0.0 on Windows. Fixes: f8244c6399d9 ("ethdev: increase port id range") Cc: sta...@dpdk.org Signed-off-by: Dmitry Kozlyuk Acked-by: Tyler Retzlaff --- v3: PRIx16 -> PRIu16 v2: u% and a cast -> PRIx16 (Tyler, Thomas) examples/rxtx_callbacks/main.c | 2 +-

Re: [dpdk-dev] [dpdk-ci] dpdk-next-crypto/master is failing tests

2021-05-05 Thread Thomas Monjalon
05/05/2021 23:31, Owen Hilyard: > The Intel 10G and Intel 40G systems at the community lab had test jobs > begin to crash a few hours ago. This was the result of DTS crashing > severely enough that the job was unable to continue. A brief investigation > makes it seem like the "-a" flag for testpmd

[dpdk-dev] [PATCH v2] examples/rxtx_callbacks: fix port ID format specifier

2021-05-05 Thread Dmitry Kozlyuk
This fixes -Wformat warning with clang 10.0.0 on Windows. Fixes: f8244c6399d9 ("ethdev: increase port id range") Cc: sta...@dpdk.org Signed-off-by: Dmitry Kozlyuk Acked-by: Tyler Retzlaff --- v2: use PRIx16 instead of %u and a cast (Tyler, Thomas). examples/rxtx_callbacks/main.c | 2 +- 1 fil

Re: [dpdk-dev] [PATCH] examples/vm_power_manager: remove vm channel number check

2021-05-05 Thread Thomas Monjalon
23/04/2021 10:56, David Hunt: > > On 21/4/2021 11:45 AM, Reshma Pattan wrote: > > VM channel number should not be validated against the > > host vm_power_manager coremask core indexes, as VM > > cores need not to be same as host cores. > > So remove this check, to allow all the vm channels > > to

Re: [dpdk-dev] [dpdk-stable] [PATCH] examples/rxtx_callbacks: fix port ID format specifier

2021-05-05 Thread Thomas Monjalon
05/05/2021 18:00, Tyler Retzlaff: > On Tue, May 04, 2021 at 09:48:22AM +0300, Dmitry Kozlyuk wrote: > > 2021-05-03 17:11 (UTC-0700), Tyler Retzlaff: > > > On Sun, May 02, 2021 at 05:56:56AM +0300, Dmitry Kozlyuk wrote: > > > > Use "%u" and a cast as in other places when port ID is formatted. > > >

Re: [dpdk-dev] [PATCH] examples/timer: fix incorrect time interval

2021-05-05 Thread Thomas Monjalon
15/04/2021 09:12, Min Hu (Connor): > From: Chengchang Tang > > Timer sample example assumes that the frequency of the timer is about > 2Ghz to control the period of calling rte_timer_manage(). But this > assumption is easy to fail. For example. the frequency of tsc on ARM64 > is much less than 2G

[dpdk-dev] dpdk-next-crypto/master is failing tests

2021-05-05 Thread Owen Hilyard
The Intel 10G and Intel 40G systems at the community lab had test jobs begin to crash a few hours ago. This was the result of DTS crashing severely enough that the job was unable to continue. A brief investigation makes it seem like the "-a" flag for testpmd has been removed, which is causing the t

Re: [dpdk-dev] [PATCH v3 2/2] eal: fix service core index validity

2021-05-05 Thread Thomas Monjalon
05/05/2021 12:43, Van Haaren, Harry: > From: Chengwen Feng > > > > This patch adds checking for service core index validity when parsing > > service corelist. > > > > Fixes: 7dbd7a6413ef ("service: add -S corelist option") > > Cc: sta...@dpdk.org > > > > Signed-off-by: Chengwen Feng > > Signed

Re: [dpdk-dev] [PATCH 1/2] raw/ntb: check spad user index

2021-05-05 Thread Thomas Monjalon
22/04/2021 07:05, Li, Xiaoyun: > > From: Chengwen Feng > > > > This patch adds checking spad user index validity when set or get attr. > > > > Fixes: 277310027965 ("raw/ntb: introduce NTB raw device driver") > > Cc: sta...@dpdk.org > > > > Signed-off-by: Chengwen Feng > > Signed-off-by: Min Hu

Re: [dpdk-dev] [PATCH v2] doc: fix Arm socs list

2021-05-05 Thread David Marchand
On Tue, May 4, 2021 at 2:43 PM David Marchand wrote: > > Keep the list of socs in a single place and include it so that the > documentation won't get outdated. > > Fixes: 8f5ea6a464ac ("config/arm: fix implementer and its SoCs") > Fixes: 1b4c86a721c9 ("config/arm: add Marvell CN10K") > Fixes: 7cf3

Re: [dpdk-dev] [PATCH 1/2] raw/ntb: check spad user index

2021-05-05 Thread Thomas Monjalon
21/04/2021 08:08, Min Hu (Connor): > 在 2021/4/21 12:54, Li, Xiaoyun 写道: > > From: Min Hu (Connor) > >> 在 2021/4/21 11:31, Li, Xiaoyun 写道: > >>> From: Min Hu (Connor) > From: Chengwen Feng > --- a/drivers/raw/ntb/ntb.c > +++ b/drivers/raw/ntb/ntb.c > index = a

Re: [dpdk-dev] [PATCH] eal: avoid side effects in RTE_ALIGN_MUL_NEAR(v, mul) for v and mul

2021-05-05 Thread David Marchand
On Wed, May 5, 2021 at 6:30 PM Tyler Retzlaff wrote: > > On Fri, Mar 12, 2021 at 09:07:22AM +0100, David Marchand wrote: > > On Thu, Mar 11, 2021 at 10:08 PM Tyler Retzlaff > > wrote: > > > > > > Avoid expanding v and mul parameters multiple times in the macro. based > > > on usage of the macro i

Re: [dpdk-dev] [dpdk-stable] [PATCH v2] examples: fix pkg-config override sequence

2021-05-05 Thread Thomas Monjalon
05/05/2021 17:30, Bruce Richardson: > On Wed, May 05, 2021 at 07:55:25PM +0530, jer...@marvell.com wrote: > > From: Jerin Jacob > > > > Move pkg-config override to beginning in the Makefile to allow > > use PKGCONF variable to detect the libdpdk availability. > > > > Fixes: fda34680eb9a ("exampl

Re: [dpdk-dev] [EXT] [PATCH] regex/octeontx2: remove unused include directory

2021-05-05 Thread Thomas Monjalon
> > The variable inc_dir is not defined in this file. > > > > Fixes: 4cd1c5fd9ed4 ("regex/octeontx2: introduce REE driver") > > Cc: sta...@dpdk.org > > > > Signed-off-by: Thomas Monjalon > > Acked-by: Guy Kaneti Applied

Re: [dpdk-dev] [PATCH] lib: restore developer mode checks

2021-05-05 Thread Thomas Monjalon
05/05/2021 17:34, Bruce Richardson: > On Wed, May 05, 2021 at 04:21:05PM +0200, David Marchand wrote: > > Most of the checks on developer_mode have been accidentally dropped. > > Restore them. > > > > Fixes: 7d611e35b077 ("lib: simplify main build file") > > > > Signed-off-by: David Marchand > >

Re: [dpdk-dev] [PATCH v2] hash: fix tuple adjustment

2021-05-05 Thread Stanislaw Kardach
On Tue, May 04, 2021 at 03:25:04PM +0100, Vladimir Medvedkin wrote: > rte_thash_adjust_tuple() uses random to generate a new subtuple if > fn() callback reports about collision. In some cases random changes > the subtuple in a way that after complementary bits are applied the > original tuple is ob

[dpdk-dev] [PATCH v13 09/10] app/testpmd: fix unused function warnings

2021-05-05 Thread Jie Zhou
Function print_fdir_mask and print_fdir_flex_payload is only called when either i40e or ixgbe presents. Add #if defined to remove "unused function" compilation warning. Signed-off-by: Jie Zhou Signed-off-by: Jie Zhou Acked-by: Tal Shnaiderman --- app/test-pmd/config.c | 82

[dpdk-dev] [PATCH v13 10/10] app/testpmd: enable building testpmd on Windows

2021-05-05 Thread Jie Zhou
From: Jie Zhou - Disable unsupported Apps on Windows - Enable building of testpmd on Windows Signed-off-by: Jie Zhou Signed-off-by: Jie Zhou Acked-by: Tal Shnaiderman --- app/meson.build| 4 app/pdump/meson.build | 6 ++ app/proc-info/meson.build

[dpdk-dev] [PATCH v13 07/10] app/testpmd: replace POSIX specific code

2021-05-05 Thread Jie Zhou
- Make printf format OS independent - Replace htons with RTE_BE16 - Replace POSIX specific inet_aton with OS independent inet_pton - Replace sleep with rte_delay_us_sleep - Repalce random with rte_rand - #ifndef mman related code for now Signed-off-by: Jie Zhou Signed-off-by: Jie Zhou ---

[dpdk-dev] [PATCH v13 05/10] app/testpmd: resolve name collisions

2021-05-05 Thread Jie Zhou
Resolve name collisions with Windows types Signed-off-by: Jie Zhou Signed-off-by: Jie Zhou Acked-by: Tal Shnaiderman --- app/test-pmd/cmdline_flow.c | 512 ++-- 1 file changed, 256 insertions(+), 256 deletions(-) diff --git a/app/test-pmd/cmdline_flow.c b/app/t

[dpdk-dev] [PATCH v13 06/10] app/testpmd: fix parse_fec_mode return type name

2021-05-05 Thread Jie Zhou
Replace parse_fec_mode misleading return type name mode with fec_capa Fixes: b19da32e3151 ("app/testpmd: add FEC command") Cc: sta...@dpdk.org Signed-off-by: Jie Zhou Signed-off-by: Jie Zhou --- app/test-pmd/cmdline.c | 6 +++--- app/test-pmd/config.c | 4 ++-- app/test-pmd/testpmd.h | 2 +-

[dpdk-dev] [PATCH v13 08/10] app/testpmd: fix headers inclusion

2021-05-05 Thread Jie Zhou
- Include rte_os_shim.h in testpmd.h - Remove redundant headers Signed-off-by: Jie Zhou Signed-off-by: Jie Zhou Acked-by: Tal Shnaiderman --- app/test-pmd/cmdline.c | 3 --- app/test-pmd/cmdline_flow.c | 2 -- app/test-pmd/parameters.c | 1 - app/test-pmd/testpmd.h | 1 + 4 files c

[dpdk-dev] [PATCH v13 02/10] eal/windows: add necessary macros

2021-05-05 Thread Jie Zhou
Add required macros by testpmd on Windows in rte_os_shim.h Signed-off-by: Jie Zhou Signed-off-by: Jie Zhou Acked-by: Tal Shnaiderman --- lib/eal/windows/include/rte_os_shim.h | 9 + 1 file changed, 9 insertions(+) diff --git a/lib/eal/windows/include/rte_os_shim.h b/lib/eal/windows/i

[dpdk-dev] [PATCH v13 04/10] eal/Windows: add clock_gettime on Windows

2021-05-05 Thread Jie Zhou
Add clock_gettime on Windows in rte_os_shim.h Signed-off-by: Jie Zhou Signed-off-by: Jie Zhou --- lib/eal/windows/include/rte_os_shim.h | 38 +++ 1 file changed, 38 insertions(+) diff --git a/lib/eal/windows/include/rte_os_shim.h b/lib/eal/windows/include/rte_os_shim.h

[dpdk-dev] [PATCH v13 00/10] app/testpmd: enable testpmd on Windows

2021-05-05 Thread Jie Zhou
This patchset is to enable testpmd on windows. It mainly includes: - Enable building libraries on Windows that testpmd depends on - Add necessary macros required by testpmd on Windows in rte_os_shim.h - Add device event stubs for Windows - Resolve name collisions with Windows types - Add clock_gett

[dpdk-dev] [PATCH v13 03/10] eal/windows: add device event stubs

2021-05-05 Thread Jie Zhou
Add device event stubs in eal_dev.c for Windows Signed-off-by: Jie Zhou Signed-off-by: Jie Zhou Acked-by: Tal Shnaiderman --- lib/eal/windows/eal_dev.c | 33 + lib/eal/windows/meson.build | 1 + 2 files changed, 34 insertions(+) create mode 100644 lib/eal/wi

[dpdk-dev] [PATCH v13 01/10] lib: build libraries that testpmd depends on

2021-05-05 Thread Jie Zhou
Enable building libraries that testpmd depends on for Windows Signed-off-by: Jie Zhou Signed-off-by: Jie Zhou --- lib/meson.build | 7 +++ 1 file changed, 7 insertions(+) diff --git a/lib/meson.build b/lib/meson.build index 64a59abab..3fd1ffa0b 100644 --- a/lib/meson.build +++ b/lib/meson.

[dpdk-dev] [PATCH] net/enic: enable GENEVE offload via VNIC configuration

2021-05-05 Thread John Daley
The admin-configured vNIC settings (i.e. via CIMC or UCSM) now include Geneve offload. Use that setting to decide whether to enable or disable Geneve offload and remove the devarg 'geneve-opt'. Also, the firmware now allows the driver to change the Geneve port number. So extend udp_tunnel_port_{ad

Re: [dpdk-dev] [PATCH v7 17/17] doc: update mlx5 support for conntrack

2021-05-05 Thread Ferruh Yigit
On 5/5/2021 1:23 PM, Bing Zhao wrote: > In the release notes and mlx5 NIC document, the support and > limitation of connection tracking are added. > A new NIC feature seems added, better to be acked beyond the mlx5 scope. > Signed-off-by: Bing Zhao > Acked-by: Viacheslav Ovsiienko > --- > doc

Re: [dpdk-dev] [PATCH v2] hash: fix tuple adjustment

2021-05-05 Thread Wang, Yipeng1
> -Original Message- > From: Medvedkin, Vladimir > Sent: Tuesday, May 4, 2021 7:25 AM > To: dev@dpdk.org > Cc: Ananyev, Konstantin ; Chilikin, Andrey > ; Kinsella, Ray ; Wang, > Yipeng1 ; Gobriel, Sameh > ; Richardson, Bruce > ; david.march...@redhat.com; > k...@semihalf.com; Medvedkin, Vl

Re: [dpdk-dev] [dpdk-stable] [PATCH v3] net/mlx5: fix tunnel offload private items location

2021-05-05 Thread Ferruh Yigit
On 5/4/2021 10:12 AM, Gregory Etelson wrote: > Hello Ferruh, > > [:snip:] > >> Hi Gregory, >> >> The were some questions around testpmd part of this patch in previous >> version, they are not answered and this version is dropping the testpmd >> part. >> > > The tunnel offload API allows applicat

[dpdk-dev] [PATCH v12 09/10] app/testpmd: fix unused function warnings

2021-05-05 Thread Jie Zhou
Function print_fdir_mask and print_fdir_flex_payload is only called when either i40e or ixgbe presents. Add #if defined to remove "unused function" compilation warning. Signed-off-by: Jie Zhou Signed-off-by: Jie Zhou Acked-by: Tal Shnaiderman --- app/test-pmd/config.c | 82

[dpdk-dev] [PATCH v12 05/10] app/testpmd: resolve name collisions

2021-05-05 Thread Jie Zhou
Resolve name collisions with Windows types Signed-off-by: Jie Zhou Signed-off-by: Jie Zhou Acked-by: Tal Shnaiderman --- app/test-pmd/cmdline_flow.c | 512 ++-- 1 file changed, 256 insertions(+), 256 deletions(-) diff --git a/app/test-pmd/cmdline_flow.c b/app/

[dpdk-dev] [PATCH v12 07/10] app/testpmd: replace POSIX specific code

2021-05-05 Thread Jie Zhou
- Make printf format OS independent - Replace htons with RTE_BE16 - Replace POSIX specific inet_aton with OS independent inet_pton - Replace sleep with rte_delay_us_sleep - Repalce random with rte_rand - #ifndef mman related code for now Signed-off-by: Jie Zhou Signed-off-by: Jie Zhou ---

[dpdk-dev] [PATCH v12 04/10] eal/Windows: add clock_gettime on Windows

2021-05-05 Thread Jie Zhou
Add clock_gettime on Windows in rte_os_shim.h Signed-off-by: Jie Zhou Signed-off-by: Jie Zhou --- lib/eal/windows/include/rte_os_shim.h | 38 +++ 1 file changed, 38 insertions(+) diff --git a/lib/eal/windows/include/rte_os_shim.h b/lib/eal/windows/include/rte_os_shim.h

[dpdk-dev] [PATCH v12 10/10] app/testpmd: enable building testpmd on Windows

2021-05-05 Thread Jie Zhou
From: Jie Zhou - Disable unsupported Apps on Windows - Enable building of testpmd on Windows Signed-off-by: Jie Zhou Signed-off-by: Jie Zhou Acked-by: Tal Shnaiderman --- app/meson.build| 4 app/pdump/meson.build | 6 ++ app/proc-info/meson.build

[dpdk-dev] [PATCH v12 08/10] app/testpmd: fix headers inclusion

2021-05-05 Thread Jie Zhou
- Include rte_os_shim.h in testpmd.h - Remove redundant headers Signed-off-by: Jie Zhou Signed-off-by: Jie Zhou Acked-by: Tal Shnaiderman --- app/test-pmd/cmdline.c | 3 --- app/test-pmd/cmdline_flow.c | 2 -- app/test-pmd/parameters.c | 1 - app/test-pmd/testpmd.h | 1 + 4 files c

[dpdk-dev] [PATCH v12 00/10] app/testpmd: enable testpmd on Windows

2021-05-05 Thread Jie Zhou
Future work: - Some issues discovered at validation need further investigations * Perf inconsistency: TPUT fluctuated significantly from runs * After traffic stop, port stats shows pps being 0 while bps not * mempool allocation only succeed with native. Other methods failed at rte_mem_loc

[dpdk-dev] [PATCH v12 06/10] app/testpmd: fix parse_fec_mode return type name

2021-05-05 Thread Jie Zhou
Replace parse_fec_mode misleading return type name mode with fec_capa Fixes: b19da32e3151 ("app/testpmd: add FEC command") Cc: sta...@dpdk.org Signed-off-by: Jie Zhou Signed-off-by: Jie Zhou --- app/test-pmd/cmdline.c | 6 +++--- app/test-pmd/config.c | 4 ++-- app/test-pmd/testpmd.h | 2 +-

[dpdk-dev] [PATCH v12 03/10] eal/windows: add device event stubs

2021-05-05 Thread Jie Zhou
Add device event stubs in eal_dev.c for Windows Signed-off-by: Jie Zhou Signed-off-by: Jie Zhou Acked-by: Tal Shnaiderman --- lib/eal/windows/eal_dev.c | 33 + lib/eal/windows/meson.build | 1 + 2 files changed, 34 insertions(+) create mode 100644 lib/eal/w

[dpdk-dev] [PATCH v12 01/10] lib: build libraries that testpmd depends on

2021-05-05 Thread Jie Zhou
Enable building libraries that testpmd depends on for Windows Signed-off-by: Jie Zhou Signed-off-by: Jie Zhou --- lib/meson.build | 7 +++ 1 file changed, 7 insertions(+) diff --git a/lib/meson.build b/lib/meson.build index c9a20f65b..2d499b238 100644 --- a/lib/meson.build +++ b/lib/meson.

[dpdk-dev] [PATCH v12 02/10] eal/windows: add necessary macros

2021-05-05 Thread Jie Zhou
Add required macros by testpmd on Windows in rte_os_shim.h Signed-off-by: Jie Zhou Signed-off-by: Jie Zhou Acked-by: Tal Shnaiderman --- lib/eal/windows/include/rte_os_shim.h | 9 + 1 file changed, 9 insertions(+) diff --git a/lib/eal/windows/include/rte_os_shim.h b/lib/eal/windows/

Re: [dpdk-dev] [PATCH v7 00/17] conntrack support in mlx5 PMD

2021-05-05 Thread Raslan Darawsheh
Hi, > -Original Message- > From: Bing Zhao > Sent: Wednesday, May 5, 2021 3:23 PM > To: Slava Ovsiienko ; Matan Azrad > ; NBU-Contact-Thomas Monjalon > > Cc: dev@dpdk.org; Ori Kam ; Raslan Darawsheh > > Subject: [PATCH v7 00/17] conntrack support in mlx5 PMD > > This patch set adds the

[dpdk-dev] [PATCH v11 10/10] app/testpmd: enable building testpmd on Windows

2021-05-05 Thread Jie Zhou
From: Jie Zhou - Disable unsupported Apps on Windows - Enable building of testpmd on Windows Signed-off-by: Jie Zhou Signed-off-by: Jie Zhou Acked-by: Tal Shnaiderman --- app/meson.build| 4 app/pdump/meson.build | 6 ++ app/proc-info/meson.build

[dpdk-dev] [PATCH v11 05/10] app/testpmd: resolve name collisions

2021-05-05 Thread Jie Zhou
Resolve name collisions with Windows types Signed-off-by: Jie Zhou Signed-off-by: Jie Zhou Acked-by: Tal Shnaiderman --- app/test-pmd/cmdline_flow.c | 512 ++-- 1 file changed, 256 insertions(+), 256 deletions(-) diff --git a/app/test-pmd/cmdline_flow.c b/app/

[dpdk-dev] [PATCH v11 09/10] app/testpmd: fix unused function warnings

2021-05-05 Thread Jie Zhou
Function print_fdir_mask and print_fdir_flex_payload is only called when either i40e or ixgbe presents. Add #if defined to remove "unused function" compilation warning. Signed-off-by: Jie Zhou Signed-off-by: Jie Zhou Acked-by: Tal Shnaiderman --- app/test-pmd/config.c | 82 +++

[dpdk-dev] [PATCH v11 07/10] app/testpmd: replace POSIX specific code

2021-05-05 Thread Jie Zhou
- Make printf format OS independent - Replace htons with RTE_BE16 - Replace POSIX specific inet_aton with OS independent inet_pton - Replace sleep with rte_delay_us_sleep - Repalce random with rte_rand - #ifndef mman related code for now Signed-off-by: Jie Zhou Signed-off-by: Jie Zhou ---

[dpdk-dev] [PATCH v11 08/10] app/testpmd: fix headers inclusion

2021-05-05 Thread Jie Zhou
- Include rte_os_shim.h in testpmd.h - Remove redundant headers Signed-off-by: Jie Zhou Signed-off-by: Jie Zhou Acked-by: Tal Shnaiderman --- app/test-pmd/cmdline.c | 3 --- app/test-pmd/cmdline_flow.c | 2 -- app/test-pmd/parameters.c | 1 - app/test-pmd/testpmd.h | 1 + 4 files

[dpdk-dev] [PATCH v11 06/10] app/testpmd: fix parse_fec_mode return type name

2021-05-05 Thread Jie Zhou
Replace parse_fec_mode misleading return type name mode with fec_capa Fixes: b19da32e3151 ("app/testpmd: add FEC command") Cc: sta...@dpdk.org Signed-off-by: Jie Zhou Signed-off-by: Jie Zhou --- app/test-pmd/cmdline.c | 6 +++--- app/test-pmd/config.c | 4 ++-- app/test-pmd/testpmd.h | 2 +-

[dpdk-dev] [PATCH v11 04/10] eal/Windows: add clock_gettime on Windows

2021-05-05 Thread Jie Zhou
Add clock_gettime on Windows in rte_os_shim.h Signed-off-by: Jie Zhou Signed-off-by: Jie Zhou --- lib/eal/windows/include/rte_os_shim.h | 38 +++ 1 file changed, 38 insertions(+) diff --git a/lib/eal/windows/include/rte_os_shim.h b/lib/eal/windows/include/rte_os_shim.h

[dpdk-dev] [PATCH v11 03/10] eal/windows: add device event stubs

2021-05-05 Thread Jie Zhou
Add device event stubs in eal_dev.c for Windows Signed-off-by: Jie Zhou Signed-off-by: Jie Zhou Acked-by: Tal Shnaiderman --- lib/eal/windows/eal_dev.c | 33 + lib/eal/windows/meson.build | 1 + 2 files changed, 34 insertions(+) create mode 100644 lib/eal/w

[dpdk-dev] [PATCH v11 00/10] app/testpmd: enable testpmd on Windows

2021-05-05 Thread Jie Zhou
This patchset is to enable testpmd on windows. It mainly includes: - Enable building libraries on Windows that testpmd depends on - Add necessary macros required by testpmd on Windows in rte_os_shim.h - Add device event stubs for Windows - Resolve name collisions with Windows types - Add clock_gett

[dpdk-dev] [PATCH v11 01/10] lib: build libraries that testpmd depends on

2021-05-05 Thread Jie Zhou
Enable building libraries that testpmd depends on for Windows Signed-off-by: Jie Zhou Signed-off-by: Jie Zhou --- lib/meson.build | 7 +++ 1 file changed, 7 insertions(+) diff --git a/lib/meson.build b/lib/meson.build index c9a20f65b..2d499b238 100644 --- a/lib/meson.build +++ b/lib/meson.

[dpdk-dev] [PATCH v11 02/10] eal/windows: add necessary macros

2021-05-05 Thread Jie Zhou
Add required macros by testpmd on Windows in rte_os_shim.h Signed-off-by: Jie Zhou Signed-off-by: Jie Zhou Acked-by: Tal Shnaiderman --- lib/eal/windows/include/rte_os_shim.h | 9 + 1 file changed, 9 insertions(+) diff --git a/lib/eal/windows/include/rte_os_shim.h b/lib/eal/windows/

[dpdk-dev] [dpdk-dev v1] crypto/snow3g: add support for digest appended ops

2021-05-05 Thread Kai Ji
This patch enable out-of-place auth-cipher operations where digest should be encrypted among with the rest of raw data. It also adds support for partially encrypted digest when using auth-cipher operations. Fixes: 7c87e2d7b359 ("crypto/snow3g: use IPsec library") Cc: pablo.de.lara.gua...@intel.com

Re: [dpdk-dev] [PATCH v1] test/ticketlock: use C11 atomic builtins for lcores sync

2021-05-05 Thread Tyler Retzlaff
On Wed, May 05, 2021 at 12:37:50AM +, Honnappa Nagarahalli wrote: > > > > > > > > > > > your subject line indicates the use of C11 which is a standard [1]. > > > > > > > > the patch itself uses gcc atomics builtins which are not part of C11 > > > > standard so the subject line is incorrect a

Re: [dpdk-dev] [PATCH v10 07/10] app/testpmd: replace POSIX specific code

2021-05-05 Thread Jie Zhou
On Wed, May 05, 2021 at 09:09:20AM -0700, Jie Zhou wrote: > On Wed, May 05, 2021 at 08:34:34AM +, Tal Shnaiderman wrote: > > > Subject: [PATCH v10 07/10] app/testpmd: replace POSIX specific code > > > > > > External email: Use caution opening links or attachments > > > > > > > > > - Make pr

Re: [dpdk-dev] [PATCH] eal: avoid side effects in RTE_ALIGN_MUL_NEAR(v, mul) for v and mul

2021-05-05 Thread Tyler Retzlaff
On Fri, Mar 12, 2021 at 09:07:22AM +0100, David Marchand wrote: > On Thu, Mar 11, 2021 at 10:08 PM Tyler Retzlaff > wrote: > > > > Avoid expanding v and mul parameters multiple times in the macro. based > > on usage of the macro it seems like side effects were not intended. > > > > For example: >

Re: [dpdk-dev] [PATCH] telemetry: fix accessing callbacks list using lock

2021-05-05 Thread David Marchand
On Wed, May 5, 2021 at 5:36 PM Bruce Richardson wrote: > > On Wed, May 05, 2021 at 03:22:48PM +, Ciara Power wrote: > > The list_commands() function accessed the callbacks list, > > but did not take the lock. This may have caused inconsistencies if > > callbacks were being registered at the sa

Re: [dpdk-dev] [PATCH] telemetry: remove internal symbol from public header

2021-05-05 Thread David Marchand
On Tue, May 4, 2021 at 6:19 PM Power, Ciara wrote: > >> Remove TELEMETRY_MAX_CALLBACKS symbol from public rte_telemetry.h > >> header file. > >> > >> Signed-off-by: Jerin Jacob > >Acked-by: Bruce Richardson > Acked-by: Ciara Power Applied, thanks. I'll post my suggestion as a patch aimed at 2

Re: [dpdk-dev] [PATCH v10 07/10] app/testpmd: replace POSIX specific code

2021-05-05 Thread Jie Zhou
On Wed, May 05, 2021 at 08:34:34AM +, Tal Shnaiderman wrote: > > Subject: [PATCH v10 07/10] app/testpmd: replace POSIX specific code > > > > External email: Use caution opening links or attachments > > > > > > - Make printf format OS independent > > - Replace htons with RTE_BE16 > > - Rep

Re: [dpdk-dev] [PATCH] examples/rxtx_callbacks: fix port ID format specifier

2021-05-05 Thread Tyler Retzlaff
On Tue, May 04, 2021 at 09:48:22AM +0300, Dmitry Kozlyuk wrote: > 2021-05-03 17:11 (UTC-0700), Tyler Retzlaff: > > On Sun, May 02, 2021 at 05:56:56AM +0300, Dmitry Kozlyuk wrote: > > > Use "%u" and a cast as in other places when port ID is formatted. > > > This fixes -Wformat warning with clang 10.

Re: [dpdk-dev] [PATCH v10 00/10] app/testpmd: enable testpmd on Windows

2021-05-05 Thread Jie Zhou
On Tue, May 04, 2021 at 09:31:08AM +0200, Thomas Monjalon wrote: > When you receive an ack in previous versions, > you should add it manually in your patch > so we keep track of it. > Will send out V11 with adding back the acked-by from V9. Thanks.

[dpdk-dev] [PATCH] doc: support IPsec Multi-buffer lib v1.0

2021-05-05 Thread Pablo de Lara
Updated AESNI MB and AESNI GCM, KASUMI, ZUC and SNOW3G PMD documentation guides with information about the latest Intel IPSec Multi-buffer library supported. Signed-off-by: Pablo de Lara --- doc/guides/cryptodevs/aesni_gcm.rst | 6 +++--- doc/guides/cryptodevs/aesni_mb.rst | 6 +++--- doc/guide

Re: [dpdk-dev] Questions about API with no parameter check

2021-05-05 Thread Tyler Retzlaff
On Tue, May 04, 2021 at 09:36:24AM +, Ananyev, Konstantin wrote: > > > > > > 2021-04-29 09:16 (UTC-0700), Tyler Retzlaff: > > > On Wed, Apr 07, 2021 at 05:10:00PM +0100, Ferruh Yigit wrote: > > > > On 4/7/2021 4:25 PM, Hemant Agrawal wrote: > > > > >>+1 > > > > >>But are we going to check al

Re: [dpdk-dev] [PATCH] app/testpmd: support meter color action in policy

2021-05-05 Thread Ori Kam
HI Haifei, > -Original Message- > From: Dumitrescu, Cristian > Sent: Monday, May 3, 2021 3:11 PM > Subject: RE: [PATCH] app/testpmd: support meter color action in policy > > > > > -Original Message- > > From: Yigit, Ferruh > > Sent: Friday, April 30, 2021 1:51 PM > > To: Haife

Re: [dpdk-dev] [PATCH] telemetry: fix accessing callbacks list using lock

2021-05-05 Thread Bruce Richardson
On Wed, May 05, 2021 at 03:22:48PM +, Ciara Power wrote: > The list_commands() function accessed the callbacks list, > but did not take the lock. This may have caused inconsistencies if > callbacks were being registered at the same time. > This is now fixed to lock before iterating the list, >

Re: [dpdk-dev] [PATCH] lib: restore developer mode checks

2021-05-05 Thread Bruce Richardson
On Wed, May 05, 2021 at 04:21:05PM +0200, David Marchand wrote: > Most of the checks on developer_mode have been accidentally dropped. > Restore them. > > Fixes: 7d611e35b077 ("lib: simplify main build file") > > Signed-off-by: David Marchand > --- Thanks. Acked-by: Bruce Richardson

Re: [dpdk-dev] [PATCH v2] examples: fix pkg-config override sequence

2021-05-05 Thread Bruce Richardson
On Wed, May 05, 2021 at 07:55:25PM +0530, jer...@marvell.com wrote: > From: Jerin Jacob > > Move pkg-config override to beginning in the Makefile to allow > use PKGCONF variable to detect the libdpdk availability. > > Fixes: fda34680eb9a ("examples: remove legacy sections of makefiles") > Cc: st

Re: [dpdk-dev] [EXT] [dpdk-dev v2] test/crypto: copy offset data to oop dst buffer

2021-05-05 Thread Akhil Goyal
> Copy over the offset data required for auth in out-of-place op > when auth offset and cipher offset are not aligned. > > Fixes: e847fc512817 ("test/crypto: add encrypted digest case for AES-CTR- > CMAC") > Cc: adamx.dybkow...@intel.com > > Signed-off-by: Kai Ji > --- > v2: >- rebase to hea

[dpdk-dev] [PATCH] telemetry: fix accessing callbacks list using lock

2021-05-05 Thread Ciara Power
The list_commands() function accessed the callbacks list, but did not take the lock. This may have caused inconsistencies if callbacks were being registered at the same time. This is now fixed to lock before iterating the list, and unlock afterwards. Fixes: f38748736eb2 ("telemetry: add default ca

Re: [dpdk-dev] [EXT] [PATCH] crypto/dpaa2_sec: fix the close and unint functions

2021-05-05 Thread Akhil Goyal
> The init function was calling the dpseci_open > while dpseci_close was called by the open function. > This is a mismatch un-init shall clean the init configurations and > close shall clear the configure function settings. > > This was causing issue with recent changes in test framework, where >

Re: [dpdk-dev] [EXT] [PATCH] crypto/dpaa_sec: fix to affine the thread portal affinity

2021-05-05 Thread Akhil Goyal
> DPAA requires the I/O shall be done in a HW portal context only. > The portal affinity is currently only being done in session create > and config APIs with the assumption that same thread will be used > for IO. This is causing issue. > This patch add support during I/O to check the HW portal aff

Re: [dpdk-dev] test/crypto: fix auth-cipher compare length in oop

2021-05-05 Thread Akhil Goyal
> > Subject: [dpdk-dev] test/crypto: fix auth-cipher compare length in oop > > > > For out-of-place operations, comparing expected ciphertext with > > the operation result should skip cipher_offset bytes, as those > > will not be copied from source to the destination buffer, making > > the tests fa

Re: [dpdk-dev] [EXT] [PATCH v2] app/crypto-perf: fix dereference of null return

2021-05-05 Thread Akhil Goyal
> Return value of a function 'rte_zmalloc' is dereferenced without > checking, and it may call segmentation fault. > > This patch fixed it. > > Fixes: f8be1786b1b8 ("app/crypto-perf: introduce performance test > application") > Cc: sta...@dpdk.org > > Signed-off-by: Min Hu (Connor) > --- > v2:

Re: [dpdk-dev] [PATCH] ppc64le: fix build with Clang and without glibc

2021-05-05 Thread David Marchand
On Tue, Mar 23, 2021 at 6:48 PM Piotr Kubaj wrote: > > Looks like I forgot to commit the change that includes headers for > sysctlbyname(). > > Patch attached. The mailing list drops attachments. Please submit a v2. Thanks. -- David Marchand

  1   2   3   >