[PATCH v3] doc: add iavf live migration guide

2023-07-06 Thread Lingyu Liu
Add iavf live migration steps based on KVM VFIO migration. Signed-off-by: Lingyu Liu --- v2: Fixed CI. Added brief introduction about live migration. Clarified this is iavf feature. v3: Added intro and link about vfio live migration. Added description about kernel boot parameters.

RE: [PATCH] doc: support IPsec Multi-buffer lib v1.4

2023-07-06 Thread De Lara Guarch, Pablo
> -Original Message- > From: Power, Ciara > Sent: Wednesday, July 5, 2023 3:34 PM > To: dev@dpdk.org > Cc: Ji, Kai ; De Lara Guarch, Pablo > ; Power, Ciara > Subject: [PATCH] doc: support IPsec Multi-buffer lib v1.4 > > Updated AESNI MB and AESNI GCM, KASUMI, ZUC, SNOW3G and > CHACHA2

[PATCH v2 2/2] vhost: fix vduse features negotiation

2023-07-06 Thread Maxime Coquelin
The series introducing VDUSE support missed the application capability to disable supported features. This results in TSO being negotiated while not supported by the application. Fixes: 0adb8eccc6a6 ("vhost: add VDUSE device creation and destruction") Signed-off-by: Maxime Coquelin --- lib/vho

[PATCH v2 0/2] VDUSE fixes for v23.07

2023-07-06 Thread Maxime Coquelin
This small series brings a couple of VDUSE fixes for v23.07, discovered during testing with OVS-DPDK. Changes in v2: == - Define a common set of features to highlight delta between Vhsot and VDUSE (David) - Change patches order for simplification Maxime Coquelin (2): vduse: fix mi

[PATCH v2 1/2] vduse: fix missing event index features

2023-07-06 Thread Maxime Coquelin
This features was mistakenly removed, add it back. Fixes: 0adb8eccc6a6 ("vhost: add VDUSE device creation and destruction") Signed-off-by: Maxime Coquelin --- lib/vhost/vduse.c | 1 + 1 file changed, 1 insertion(+) diff --git a/lib/vhost/vduse.c b/lib/vhost/vduse.c index a509daf80c..b9514e6c29

Re: [PATCH v2 0/2] VDUSE fixes for v23.07

2023-07-06 Thread David Marchand
On Thu, Jul 6, 2023 at 10:13 AM Maxime Coquelin wrote: > > This small series brings a couple of VDUSE fixes > for v23.07, discovered during testing with OVS-DPDK. > > Changes in v2: > == > - Define a common set of features to highlight delta > between Vhsot and VDUSE (David) > - Chan

Re: [dpdk-dev] [PATCH v5 0/4] improve options help

2023-07-06 Thread Thomas Monjalon
29/06/2023 18:27, Stephen Hemminger: > On Mon, 5 Apr 2021 21:39:50 +0200 > Thomas Monjalon wrote: > > > After v4, this series is split in several parts. > > The remaining 4 patches of this series are low priority. > > > > Patches 1 and 3 are simple improvements. > > > > Patches 2 and 4 lead to

[Bug 1256] drivers/common/mlx5: mlx5_malloc() called on invalid socket ID when global MR cache is full and rte_extmem_* API is used

2023-07-06 Thread bugzilla
https://bugs.dpdk.org/show_bug.cgi?id=1256 Raslan Darawsheh (rasl...@nvidia.com) changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED Resoluti

RE: [PATCH v2 1/2] vduse: fix missing event index features

2023-07-06 Thread Xia, Chenbo
> -Original Message- > From: Maxime Coquelin > Sent: Thursday, July 6, 2023 4:12 PM > To: dev@dpdk.org; Xia, Chenbo ; > david.march...@redhat.com > Cc: Maxime Coquelin > Subject: [PATCH v2 1/2] vduse: fix missing event index features > > This features was mistakenly removed, add it back.

RE: [PATCH v2 2/2] vhost: fix vduse features negotiation

2023-07-06 Thread Xia, Chenbo
> -Original Message- > From: Maxime Coquelin > Sent: Thursday, July 6, 2023 4:12 PM > To: dev@dpdk.org; Xia, Chenbo ; > david.march...@redhat.com > Cc: Maxime Coquelin > Subject: [PATCH v2 2/2] vhost: fix vduse features negotiation > > The series introducing VDUSE support missed the > ap

Re: [PATCH v4 3/3] ring: add telemetry cmd for ring info

2023-07-06 Thread David Marchand
On Tue, Jul 4, 2023 at 4:11 PM Thomas Monjalon wrote: > > 04/07/2023 10:04, Jie Hai: > > On 2023/6/20 22:34, Thomas Monjalon wrote: > > > 20/06/2023 10:14, Jie Hai: > > >> On 2023/2/20 20:55, David Marchand wrote: > > >>> On Fri, Feb 10, 2023 at 3:50 AM Jie Hai wrote: > > > > This patch

[PATCH v3] net/mlx5: fix RSS expansion inner buffer overflow.

2023-07-06 Thread Maayan Kashani
The stack which used for RSS expansion was overflowed and trashed RSS expansion data. (buf->entry[MLX5_RSS_EXP_ELT_N]). Due to this overflow, packets such as ARP or LACP with overwritten RSS types due to the overflow will be dropped. This increases the buffer size to avoid such overflows and add

RE: [PATCH v2 1/2] net/virtio: fix legacy device IO port map in secondary process

2023-07-06 Thread Xia, Chenbo
> -Original Message- > From: David Marchand > Sent: Monday, July 3, 2023 3:48 PM > To: Li, Miao > Cc: dev@dpdk.org; sta...@dpdk.org; Maxime Coquelin > ; Xia, Chenbo > Subject: Re: [PATCH v2 1/2] net/virtio: fix legacy device IO port map in > secondary process > > On Thu, Jun 29, 2023 at

Re: [PATCH] vhost: fix build with gcc 4.8

2023-07-06 Thread Maxime Coquelin
On 7/3/23 10:21, Ali Alnubani wrote: Adds braces around initializer to resolve the following false-positive build error with gcc 4.8.5 on CentOS: lib/vhost/vduse.c:441:9: error: missing braces around initializer [-Werror=missing-braces] Fixes: 653327e191f0 ("vhost: add multiqueue support t

Re: [PATCH v2 0/2] VDUSE fixes for v23.07

2023-07-06 Thread Maxime Coquelin
On 7/6/23 10:12, Maxime Coquelin wrote: This small series brings a couple of VDUSE fixes for v23.07, discovered during testing with OVS-DPDK. Changes in v2: == - Define a common set of features to highlight delta between Vhsot and VDUSE (David) - Change patches order for simpli

[Bug 1256] drivers/common/mlx5: mlx5_malloc() called on invalid socket ID when global MR cache is full and rte_extmem_* API is used

2023-07-06 Thread bugzilla
https://bugs.dpdk.org/show_bug.cgi?id=1256 Marius-Cristian Baciu (baciumariuscrist...@yahoo.com) changed: What|Removed |Added Resolution|FIXED |---

RE: [EXT] [PATCH] ipsec: fix NAT-T length calculation

2023-07-06 Thread Konstantin Ananyev
Hi Akhil, > > Hi Konstantin, > Can you review this patch? > > > UDP header length is included in sa->hdr_len. Take care of that in > > L3 header and pakcet length calculation. > > > > Fixes: 01eef5907fc3 ("ipsec: support NAT-T") > > > > Signed-off-by: Xiao Liang > > --- > > lib/ipsec/esp_out

Re: [PATCH v2 1/6] baseband/fpga_5gnr_fec: fix possible div by zero

2023-07-06 Thread Maxime Coquelin
On 5/25/23 20:28, Hernan Vargas wrote: Add fix to have an early exit when z_c is zero to prevent a possible division by zero. Fixes: 44dc6faa796f ("baseband/fpga_5gnr_fec: add LDPC processing functions") Cc: sta...@dpdk.org Signed-off-by: Hernan Vargas Reviewed-by: Maxime Coquelin --- dr

Re: [PATCH v2 1/6] baseband/fpga_5gnr_fec: fix possible div by zero

2023-07-06 Thread Maxime Coquelin
On 5/25/23 20:28, Hernan Vargas wrote: Add fix to have an early exit when z_c is zero to prevent a possible division by zero. Fixes: 44dc6faa796f ("baseband/fpga_5gnr_fec: add LDPC processing functions") Cc: sta...@dpdk.org Signed-off-by: Hernan Vargas Reviewed-by: Maxime Coquelin --- dr

Re: [PATCH v2 0/6] changes for 23.07

2023-07-06 Thread Maxime Coquelin
On 5/25/23 20:28, Hernan Vargas wrote: v2: Targeting 23.11. Update in commits 1,2 based on review comments. v1: Targeting 23.07 if possible. Add support for AGX100 (N6000) and corner case fixes. Hernan Vargas (6): baseband/fpga_5gnr_fec: fix possible div by zero baseband/fpga_5gnr_fec:

Re: [PATCH v1 1/1] bbdev: extend range of allocation function

2023-07-06 Thread Maxime Coquelin
On 6/2/23 04:04, Nicolas Chautru wrote: Realigning the argument to unsigned int to align with number support by underlying rte_mempool_get_bulk function. Signed-off-by: Nicolas Chautru --- lib/bbdev/rte_bbdev_op.h | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/l

Re: [PATCH v11 1/2] mempool cache: add zero-copy get and put functions

2023-07-06 Thread Konstantin Ananyev
05/07/2023 18:18, Kamalakshitha Aligeri пишет: From: Morten Brørup Zero-copy access to mempool caches is beneficial for PMD performance. Furthermore, having a zero-copy mempool API is considered a precondition for fixing a certain category of bugs, present in some PMDs: For performance reasons,

Re: [PATCH v11 2/2] net/i40e: replace put function

2023-07-06 Thread Konstantin Ananyev
05/07/2023 18:18, Kamalakshitha Aligeri пишет: Integrated zero-copy put API in mempool cache in i40e PMD. On Ampere Altra server, l3fwd single core's performance improves by 5% with the new API Signed-off-by: Kamalakshitha Aligeri Reviewed-by: Ruifeng Wang Reviewed-by: Feifei Wang --- .mail

RE: [PATCH] app/testpmd: revert primary process polling all queues fix

2023-07-06 Thread Jiale, SongX
> -Original Message- > From: Ferruh Yigit > Sent: Wednesday, July 5, 2023 10:32 PM > To: Singh, Aman Deep ; Zhang, Yuying > ; Burakov, Anatoly ; > Jie Hai > Cc: dev@dpdk.org; Thomas Monjalon ; David > Marchand ; sta...@dpdk.org; Jiale, SongX > ; Yang, Qiming > Subject: [PATCH] app/testpm

Re: [PATCH 1/7] net/mlx5: fix the modify field check of tag

2023-07-06 Thread Thomas Monjalon
03/07/2023 15:31, Bing Zhao: > Hi Stephen, > If I understand correctly, do you mean that the internal value and rte_flow > API value may have some conflict? > All the MLX5 internal enum values start from INT_MIN. When treating it as a > int value, it would not have the same value with rte_flow en

Re: [PATCH] devtools: fix mailmap check for parentheses

2023-07-06 Thread Thomas Monjalon
04/07/2023 01:40, Stephen Hemminger: > On Mon, 26 Jun 2023 12:24:03 +0200 > Thomas Monjalon wrote: > > > When checking names having parentheses, the grep matching was failing. > > It is fixed by escaping the open parenthesis. > > > > Also, the mailmap path was relative to the root directory. > >

[PATCH v7 0/4] Recycle mbufs from Tx queue to Rx queue

2023-07-06 Thread Feifei Wang
Currently, the transmit side frees the buffers into the lcore cache and the receive side allocates buffers from the lcore cache. The transmit side typically frees 32 buffers resulting in 32*8=256B of stores to lcore cache. The receive side allocates 32 buffers and stores them in the receive side so

[PATCH v7 1/4] ethdev: add API for mbufs recycle mode

2023-07-06 Thread Feifei Wang
Add 'rte_eth_recycle_rx_queue_info_get' and 'rte_eth_recycle_mbufs' APIs to recycle used mbufs from a transmit queue of an Ethernet device, and move these mbufs into a mbuf ring for a receive queue of an Ethernet device. This can bypass mempool 'put/get' operations hence saving CPU cycles. For eac

[PATCH v7 2/4] net/i40e: implement mbufs recycle mode

2023-07-06 Thread Feifei Wang
Define specific function implementation for i40e driver. Currently, mbufs recycle mode can support 128bit vector path and avx2 path. And can be enabled both in fast free and no fast free mode. Suggested-by: Honnappa Nagarahalli Signed-off-by: Feifei Wang Reviewed-by: Ruifeng Wang Reviewed-by: H

[PATCH v7 3/4] net/ixgbe: implement mbufs recycle mode

2023-07-06 Thread Feifei Wang
Define specific function implementation for ixgbe driver. Currently, recycle buffer mode can support 128bit vector path. And can be enabled both in fast free and no fast free mode. Suggested-by: Honnappa Nagarahalli Signed-off-by: Feifei Wang Reviewed-by: Ruifeng Wang Reviewed-by: Honnappa Naga

[PATCH v7 4/4] app/testpmd: add recycle mbufs engine

2023-07-06 Thread Feifei Wang
Add recycle mbufs engine for testpmd. This engine forward pkts with I/O forward mode. But enable mbufs recycle feature to recycle used txq mbufs for rxq mbuf ring, which can bypass mempool path and save CPU cycles. Suggested-by: Jerin Jacob Signed-off-by: Feifei Wang Reviewed-by: Ruifeng Wang -

RE: [PATCH 1/7] net/mlx5: fix the modify field check of tag

2023-07-06 Thread Bing Zhao
Thank you. > -Original Message- > From: Thomas Monjalon > Sent: Thursday, July 6, 2023 5:37 PM > To: Stephen Hemminger ; Bing Zhao > > Cc: dev@dpdk.org; Matan Azrad ; Slava Ovsiienko > ; Ori Kam ; Suanming Mou > ; Raslan Darawsheh ; > dev@dpdk.org; Michael Baum > Subject: Re: [PATCH 1/7

RE: [PATCH] app/testpmd: revert primary process polling all queues fix

2023-07-06 Thread Ali Alnubani
> -Original Message- > From: Ferruh Yigit > Sent: Wednesday, July 5, 2023 5:32 PM > To: Aman Singh ; Yuying Zhang > ; Anatoly Burakov ; > Jie Hai > Cc: dev@dpdk.org; NBU-Contact-Thomas Monjalon (EXTERNAL) > ; David Marchand ; > sta...@dpdk.org; songx.ji...@intel.com; qiming.y...@intel.com

Re: [EXT] [PATCH] ipsec: fix NAT-T length calculation

2023-07-06 Thread Radu Nicolau
On 06-Jul-23 10:08 AM, Konstantin Ananyev wrote: Hi Akhil, Hi Konstantin, Can you review this patch? UDP header length is included in sa->hdr_len. Take care of that in L3 header and pakcet length calculation. Fixes: 01eef5907fc3 ("ipsec: support NAT-T") Signed-off-by: Xiao Liang --- li

Re: [PATCH] app/testpmd: revert primary process polling all queues fix

2023-07-06 Thread Ferruh Yigit
On 7/6/2023 10:24 AM, Jiale, SongX wrote: >> -Original Message- >> From: Ferruh Yigit >> Sent: Wednesday, July 5, 2023 10:32 PM >> To: Singh, Aman Deep ; Zhang, Yuying >> ; Burakov, Anatoly ; >> Jie Hai >> Cc: dev@dpdk.org; Thomas Monjalon ; David >> Marchand ; sta...@dpdk.org; Jiale, Son

Re: [PATCH v2] app/pdump: exit if no device specified

2023-07-06 Thread Thomas Monjalon
03/07/2023 08:29, fengchengwen: > Acked-by: Chengwen Feng > > On 2023/7/1 10:16, Stephen Hemminger wrote: > > Simpler version of earlier patch which had a good idea, was just > > implemented with more code than necessary. > > If no device is specified don't start the capture loop. > > > > Report

RE: [PATCH v2] pcap: support MTU set

2023-07-06 Thread Ido Goshen
I've suggested 2 ways to do it 1. Data path enforcement by pcap pmd [PATCH v4] http://patches.dpdk.org/project/dpdk/patch/20220606162147.57218-1-...@cgstowernetworks.com/ 2. Control path only sets the underlying OS network interface MTU [PATCH v8] http://patches.dpdk.org/project/dpdk/cover/2022062

[PATCH v2] app/crypto-perf: fix socket ID default value

2023-07-06 Thread Ciara Power
Due to recent changes to the default device socket ID, before being used as an index for session mempool list, the socket ID should be set to 0 if unknown (-1). Fixes: 7dcd73e37965 ("drivers/bus: set device NUMA node to unknown by default") Fixes: 64c469b9e7d8 ("app/crypto-perf: check range of soc

RE: [PATCH] doc: announce deprecation for security ops

2023-07-06 Thread Power, Ciara
Hi Akhil, > -Original Message- > From: Akhil Goyal > Sent: Tuesday 4 July 2023 20:45 > To: dev@dpdk.org > Cc: tho...@monjalon.net; david.march...@redhat.com; > jer...@marvell.com; ano...@marvell.com; ndabilpu...@marvell.com; De > Lara Guarch, Pablo ; > hemant.agra...@nxp.com; g.si...@nxp.

RE: [PATCH v3] net/mlx5: fix RSS expansion inner buffer overflow.

2023-07-06 Thread Raslan Darawsheh
Hi, > -Original Message- > From: Maayan Kashani > Sent: Thursday, July 6, 2023 11:56 AM > To: dev@dpdk.org > Cc: Maayan Kashani ; Ori Kam ; > Raslan Darawsheh ; Matan Azrad > ; Slava Ovsiienko ; Suanming > Mou > Subject: [PATCH v3] net/mlx5: fix RSS expansion inner buffer overflow. > >

[PATCH v2] examples/ipsec-secgw: fix of socket id default value

2023-07-06 Thread Kai Ji
Due to recent changes to the default device socket ID, before being used as an index for session mempool list, set socket ID to 0 if unknown (-1). Fixes: 7dcd73e37965 ("drivers/bus: set device NUMA node to unknown by default") Cc: olivier.m...@6wind.com Cc: sta...@dpdk.org Signed-off-by: Kai Ji

RE: [PATCH] service: avoid worker lcore exit deadlock

2023-07-06 Thread Van Haaren, Harry
> -Original Message- > From: Mattias Rönnblom > Sent: Tuesday, July 4, 2023 10:44 PM > To: Van Haaren, Harry ; Stephen Hemminger > > Cc: hof...@lysator.liu.se; dev@dpdk.org; Suanming Mou > ; tho...@monjalon.net; > david.march...@redhat.com; mattias.ronnblom > ; sta...@dpdk.org > Subject:

RE: [PATCH v2] app/testpmd: add IP length field matching

2023-07-06 Thread Ori Kam
> -Original Message- > From: Bing Zhao > Sent: Saturday, July 1, 2023 12:27 PM > Subject: [PATCH v2] app/testpmd: add IP length field matching > > Added support of parsing IPv4 total length and IPv6 payload length > in the command line. The value of L3 length can be passed to the > rte_

[PATCH] net/ice: allow setting CIR

2023-07-06 Thread markus . theil
From: Michael Rossberg ice only allowed to set peak information rate (PIR), while the hardware also supports setting committed information rate (CIR). In many use cases both values are needed, therefore add support for CIR. Signed-off-by: Michael Rossberg --- drivers/net/ice/ice_tm.c | 39

[PATCH v2] net/mlx5: support symmetric RSS hash function

2023-07-06 Thread Xueming Li
This patch supports symmetric hash function that creating same hash result for bi-direction traffic which having reverse source and destination IP and L4 port. Since the hash algorithom is different than spec(XOR), leave a warning in validation. Signed-off-by: Xueming Li --- doc/guides/nics/mlx

[PATCH] net/mlx5: fix query for NIC flow cap

2023-07-06 Thread Ori Kam
Add query for nic flow table support bit. Fixes: 5f44fb1958e5 ("common/mlx5: query capability of registers") Cc: bi...@nvidia.com Signed-off-by: Ori Kam Acked-by: Suanming Mou suanmi...@nvidia.com --- drivers/common/mlx5/mlx5_devx_cmds.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drive

Re: [PATCH] doc: ensure sphinx output is reproducible

2023-07-06 Thread Christian Ehrhardt
On Mon, Jul 3, 2023 at 5:29 PM Thomas Monjalon wrote: > > 29/06/2023 14:58, christian.ehrha...@canonical.com: > > From: Christian Ehrhardt > > > > By adding -j we build in parallel, to make building on multiprocessor > > machines more effective. While that works it does also break > > reproducibl

[PATCH v1] crypto/ipsec_mb: remove unused defines

2023-07-06 Thread Brian Dooley
removed AESNI_MB_DOCSIS_SEC_ENABLED defines as they are no longer used. Fixes: fda5216fba55 ("crypto/aesni_mb: support DOCSIS protocol") Cc: david.co...@intel.com Signed-off-by: Brian Dooley --- drivers/crypto/ipsec_mb/ipsec_mb_private.c | 4 drivers/crypto/ipsec_mb/pmd_aesni_mb.c |

RE: [PATCH v7 1/4] ethdev: add API for mbufs recycle mode

2023-07-06 Thread Morten Brørup
> From: Feifei Wang [mailto:feifei.wa...@arm.com] > Sent: Thursday, 6 July 2023 11.50 > > Add 'rte_eth_recycle_rx_queue_info_get' and 'rte_eth_recycle_mbufs' > APIs to recycle used mbufs from a transmit queue of an Ethernet device, > and move these mbufs into a mbuf ring for a receive queue of an

RE: [PATCH v11 2/2] net/i40e: replace put function

2023-07-06 Thread Morten Brørup
> From: Kamalakshitha Aligeri [mailto:kamalakshitha.alig...@arm.com] > Sent: Wednesday, 5 July 2023 19.18 > > Integrated zero-copy put API in mempool cache in i40e PMD. > On Ampere Altra server, l3fwd single core's performance improves by 5% > with the new API > > Signed-off-by: Kamalakshitha Ali

Re: [PATCH v2] app/testpmd: add IP length field matching

2023-07-06 Thread Ferruh Yigit
On 7/6/2023 12:38 PM, Ori Kam wrote: > > >> -Original Message- >> From: Bing Zhao >> Sent: Saturday, July 1, 2023 12:27 PM >> Subject: [PATCH v2] app/testpmd: add IP length field matching >> >> Added support of parsing IPv4 total length and IPv6 payload length >> in the command line. The

Re: [PATCH] fib: fix adding a default route

2023-07-06 Thread Thomas Monjalon
03/07/2023 17:43, Vladimir Medvedkin: > Fixed an issue that occurs when > adding a default route as the first route. > > Bugzilla ID: 1160 > Fixes: 7dc7868b200d ("fib: add DIR24-8 dataplane algorithm") > Cc: sta...@dpdk.org > > Signed-off-by: Vladimir Medvedkin Applied, thanks.

Re: [PATCH v3] eal: fix prompt info when remap_segment failed

2023-07-06 Thread Thomas Monjalon
05/07/2023 03:33, fengchengwen: > This is bugfix, suggest add Cc. > > With above add, Acked-by: Chengwen Feng > > On 2023/7/4 20:17, Fengnan Chang wrote: > > When there is not enough space to memsegs, we should prompt > > which configuration should be modified instead of printing > > some number

Re: [dpdk-dev] [PATCH v5 0/4] improve options help

2023-07-06 Thread Stephen Hemminger
On Thu, 06 Jul 2023 10:29:04 +0200 Thomas Monjalon wrote: > 29/06/2023 18:27, Stephen Hemminger: > > On Mon, 5 Apr 2021 21:39:50 +0200 > > Thomas Monjalon wrote: > > > > > After v4, this series is split in several parts. > > > The remaining 4 patches of this series are low priority. > > > >

Re: [PATCH v2 1/2] hash: fix reading unaligned bits implementation

2023-07-06 Thread Thomas Monjalon
30/06/2023 19:09, Vladimir Medvedkin: > Fixes: 28ebff11c2dc ("hash: add predictable RSS") > Cc: sta...@dpdk.org > > Acked-by: Konstantin Ananyev > Tested-by: Konstantin Ananyev > Signed-off-by: Vladimir Medvedkin I've just merged another patch from you where the explanation is useless. Here th

RE: [v3 1/5] net/mlx5/hws: remove unneeded new line for DR_LOG

2023-07-06 Thread Raslan Darawsheh
Hi, > -Original Message- > From: Itamar Gozlan > Sent: Tuesday, July 4, 2023 7:05 PM > To: Alex Vesker ; Slava Ovsiienko > ; Matan Azrad ; NBU- > Contact-Thomas Monjalon (EXTERNAL) ; Suanming > Mou ; Ori Kam > Cc: dev@dpdk.org > Subject: [v3 1/5] net/mlx5/hws: remove unneeded new line fo

RE: [PATCH v2] net/mlx5: support symmetric RSS hash function

2023-07-06 Thread Matan Azrad
From: Xueming(Steven) Li > This patch supports symmetric hash function that creating same hash result > for bi-direction traffic which having reverse source and destination IP and L4 > port. > > Since the hash algorithom is different than spec(XOR), leave a warning in > validation. > > Signed-

RE: [PATCH v2] net/mlx5: support symmetric RSS hash function

2023-07-06 Thread Raslan Darawsheh
Hi, > -Original Message- > From: Xueming Li > Sent: Thursday, July 6, 2023 2:56 PM > To: Matan Azrad ; Slava Ovsiienko > ; Ori Kam ; Suanming Mou > > Cc: Xueming(Steven) Li ; dev@dpdk.org > Subject: [PATCH v2] net/mlx5: support symmetric RSS hash function > > This patch supports symmetr

[PATCH] hash: fix segfault by adding param name NULL check

2023-07-06 Thread Conor Fogarty
Add NULL pointer check to params->name, which is later copied into the hash datastructure. Without this check the code segfaults on the strlcpy() of a NULL pointer. Fixes: 48a399119619 ("hash: replace with cuckoo hash implementation") Signed-off-by: Conor Fogarty --- Cc: pablo.de.lara.gua...@in

Re: [dpdk-dev] [PATCH v5 0/4] improve options help

2023-07-06 Thread Thomas Monjalon
06/07/2023 16:44, Stephen Hemminger: > Trying to reach 500 patches by 23.08 release. Impossible. It is 23.07 :)

RE: [PATCH v2] examples/ipsec-secgw: fix of socket id default value

2023-07-06 Thread Power, Ciara
> -Original Message- > From: Kai Ji > Sent: Thursday 6 July 2023 12:01 > To: dev@dpdk.org > Cc: gak...@marvell.com; sta...@dpdk.org; Ji, Kai ; Matz, > Olivier > Subject: [PATCH v2] examples/ipsec-secgw: fix of socket id default value > > Due to recent changes to the default device soc

[PATCH v2 0/2] remove unused defines

2023-07-06 Thread Brian Dooley
This series removes some unused defines throughout common qat drivers and crypto ipsec mb drivers. It also removes some defines that should have been removed previously. v2: more defines removed in additional patch and changed fixline Brian Dooley (2): crypto/ipsec_mb: remove unused defines c

[PATCH v2 1/2] crypto/ipsec_mb: remove unused defines

2023-07-06 Thread Brian Dooley
removed AESNI_MB_DOCSIS_SEC_ENABLED defines as they are no longer used. Fixes: 66a9d8d0bc6d ("crypto/qat: remove security library presence checks") Cc: maxime.coque...@redhat.com Signed-off-by: Brian Dooley --- drivers/crypto/ipsec_mb/ipsec_mb_private.c | 4 drivers/crypto/ipsec_mb/pmd_a

[PATCH v2 2/2] common/qat: change define header

2023-07-06 Thread Brian Dooley
change define from RTE_LIB_SECURITY to BUILD_QAT_SYM as RTE_ETHER_CRC_LEN value is protected by BUILD_QAT_SYM. Fixes: ce7a737c8f02 ("crypto/qat: support cipher-CRC offload") Cc: kevin.osulli...@intel.com Signed-off-by: Brian Dooley --- drivers/common/qat/qat_qp.c | 2 +- 1 file changed, 1 inser

RE: [PATCH v2 2/2] common/qat: change define header

2023-07-06 Thread Power, Ciara
> -Original Message- > From: Brian Dooley > Sent: Thursday 6 July 2023 17:05 > To: Ji, Kai > Cc: dev@dpdk.org; gak...@marvell.com; Dooley, Brian > ; O'Sullivan, Kevin > Subject: [PATCH v2 2/2] common/qat: change define header > > change define from RTE_LIB_SECURITY to BUILD_QAT_SYM a

RE: [PATCH v2 1/2] crypto/ipsec_mb: remove unused defines

2023-07-06 Thread Power, Ciara
Hi Brian, > -Original Message- > From: Brian Dooley > Sent: Thursday 6 July 2023 17:05 > To: Ji, Kai ; De Lara Guarch, Pablo > > Cc: dev@dpdk.org; gak...@marvell.com; Dooley, Brian > ; maxime.coque...@redhat.com > Subject: [PATCH v2 1/2] crypto/ipsec_mb: remove unused defines > > remove

Re: [PATCH] ethtool: remove a redundant call to rte_eth_dev_stop()

2023-07-06 Thread Thomas Monjalon
04/07/2023 00:31, Stephen Hemminger: > On Thu, 18 Aug 2022 15:18:36 +0500 > Usman Tanveer wrote: > > > Hi, > > > > Can you please have a look and update the status? > > Looks OK to me. > > Acked-by: Stephen Hemminger better title: examples/ethtool: remove stop before start Applied, thanks.

Re: [PATCH] member: fix PRNG seed reset in NitroSketch mode

2023-07-06 Thread Thomas Monjalon
03/07/2023 17:54, Stephen Hemminger: > On Wed, 21 Jun 2023 00:17:20 +0300 > Dmitry Kozlyuk wrote: > > > Seeding the global PRNG at sketch creation > > does not make the sketch operation deterministic: > > it uses rte_rand() later, the PRNG may be seeded again by that point. > > On the other hand,

[PATCH v3 0/2] remove unused defines

2023-07-06 Thread Brian Dooley
This series removes some unused defines throughout common qat drivers and crypto ipsec mb drivers. It also removes some defines that should have been removed previously. v3: Incorrect fixline v2: more defines removed in additional patch and changed fixline Brian Dooley (2): crypto/ipsec_mb: re

[PATCH v3 1/2] crypto/ipsec_mb: remove unused defines

2023-07-06 Thread Brian Dooley
removed AESNI_MB_DOCSIS_SEC_ENABLED defines as they are no longer used. Fixes: 798f9d134519 ("crypto/ipsec_mb: remove security lib presence checks") Cc: maxime.coque...@redhat.com Signed-off-by: Brian Dooley --- drivers/crypto/ipsec_mb/ipsec_mb_private.c | 4 drivers/crypto/ipsec_mb/pmd_

[PATCH v3 2/2] common/qat: change define header

2023-07-06 Thread Brian Dooley
change define from RTE_LIB_SECURITY to BUILD_QAT_SYM as RTE_ETHER_CRC_LEN value is protected by BUILD_QAT_SYM. Fixes: ce7a737c8f02 ("crypto/qat: support cipher-CRC offload") Cc: kevin.osulli...@intel.com Signed-off-by: Brian Dooley --- drivers/common/qat/qat_qp.c | 2 +- 1 file changed, 1 inser

RE: [PATCH v5 0/4] net/mlx5: introduce Tx datapath tracing

2023-07-06 Thread Raslan Darawsheh
Hi, > -Original Message- > From: Slava Ovsiienko > Sent: Wednesday, July 5, 2023 6:31 PM > To: dev@dpdk.org > Cc: jer...@marvell.com; Raslan Darawsheh > Subject: [PATCH v5 0/4] net/mlx5: introduce Tx datapath tracing > > The mlx5 provides the send scheduling on specific moment of time,

Re: [PATCH 1/1] app/mldev: add check for model and filelist option

2023-07-06 Thread Thomas Monjalon
23/03/2023 17:21, Srikanth Yalavarthi: > Application currently doesn't check for empty models list and > filelist entries. This causes the app to report an incorrect > error messages and test status when the lists are empty. > > Fixes: bbd272edcb14 ("app/mldev: add ordered inferences") > Fixes: f6

Re: [PATCH v2] app/mldev: fix code formatting and typos

2023-07-06 Thread Thomas Monjalon
23/04/2023 06:58, Srikanth Yalavarthi: > Updated ML application source files to have uniform code formatting > style across. Remove extra blank lines. Fix typos in application help. > > Fixes: 8cb22a545447 ("app/mldev: fix debug build") > Fixes: da6793390596 ("app/mldev: support inference validati

Re: [PATCH] member: fix PRNG seed reset in NitroSketch mode

2023-07-06 Thread Stephen Hemminger
On Thu, 06 Jul 2023 18:20:19 +0200 Thomas Monjalon wrote: > > > > This raises a more global issue. > > rte_srand() overrides the system seed which is set during startup. > > This is a bad thing, it reduces the entropy in the random number generator. > > > > There are two possible solutions to t

Re: [PATCH 0/1] mempool: implement index-based per core cache

2023-07-06 Thread Stephen Hemminger
On Thu, 13 Jan 2022 05:31:18 + Dharmik Thakkar wrote: > Hi, > > Thank you for your valuable review comments and suggestions! > > I will be sending out a v2 in which I have increased the size of the mempool > to 32GB by using division by sizeof(uintptr_t). > However, I am seeing ~5% perform

[PATCH v3 00/14] Use rte_pktmbuf_mtod_offset() where possible

2023-07-06 Thread Stephen Hemminger
Run the coccinelle script for rte_pktmbuf_mtod_offset against current main branch. v3 - rebase to cover gro changes Stephen Hemminger (14): gro: use rte_pktmbuf_mtod_offset gso: use rte_pktmbuf_mtod_offset testpmd: use rte_pktmbuf_mtod_offset test: cryptodev use rte_pktmbuf_mtod_offset

[PATCH v3 01/14] gro: use rte_pktmbuf_mtod_offset

2023-07-06 Thread Stephen Hemminger
Use rte_pktmbuf_mtod_offset. Change was automatically generated by cocci/mtod-offset.cocci. Signed-off-by: Stephen Hemminger --- lib/gro/gro_tcp.h| 2 +- lib/gro/gro_tcp4.c | 2 +- lib/gro/gro_udp4.c | 4 ++-- lib/gro/gro_vxlan_tcp4.c | 4 ++-- lib/gro/gro_vxlan_udp4.c | 4 ++

[PATCH v3 02/14] gso: use rte_pktmbuf_mtod_offset

2023-07-06 Thread Stephen Hemminger
Use the rte_pktmbuf_mtod_offset macro. Change was automatically generated by cocci/mtod-offset.cocci. Signed-off-by: Stephen Hemminger --- lib/gso/gso_common.h | 11 +-- lib/gso/gso_tcp4.c| 8 lib/gso/gso_tunnel_tcp4.c | 12 ++-- lib/gso/gso_tunnel_udp4.c |

[PATCH v3 03/14] testpmd: use rte_pktmbuf_mtod_offset

2023-07-06 Thread Stephen Hemminger
Use helper macro. Signed-off-by: Stephen Hemminger --- app/test-pmd/ieee1588fwd.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/app/test-pmd/ieee1588fwd.c b/app/test-pmd/ieee1588fwd.c index 386d9f10e642..3371771751dd 100644 --- a/app/test-pmd/ieee1588fwd.c +++ b/app/tes

[PATCH v3 04/14] test: cryptodev use rte_pktmbuf_mtod_offset

2023-07-06 Thread Stephen Hemminger
Based off patch generated by cocci/mtod-offset.cocci. With some cleanup to shorten lines by using conditional with omitted operand. Signed-off-by: Stephen Hemminger --- app/test/test_cryptodev.c | 66 +-- 1 file changed, 36 insertions(+), 30 deletions(-) diff

[PATCH v3 05/14] examples: use rte_pktmbuf_mtod_offset

2023-07-06 Thread Stephen Hemminger
Automatically generated from cocci/mtod-offset.cocci Signed-off-by: Stephen Hemminger --- examples/l2fwd-crypto/main.c | 16 +--- examples/ptpclient/ptpclient.c | 18 +- 2 files changed, 18 insertions(+), 16 deletions(-) diff --git a/examples/l2fwd-crypto/main.c b/

[PATCH v3 06/14] net/tap: use rte_pktmbuf_mtod_offset

2023-07-06 Thread Stephen Hemminger
Automatically generated by cocci/mbuf-offset.cocci Signed-off-by: Stephen Hemminger --- drivers/net/tap/rte_eth_tap.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/net/tap/rte_eth_tap.c b/drivers/net/tap/rte_eth_tap.c index bf98f7555990..ebddbae9fe9f 100644 --- a/

[PATCH v3 07/14] net/nfp: use rte_pktmbuf_mtod_offset

2023-07-06 Thread Stephen Hemminger
Automatically generated by cocci/mtod-offset.cocci. Signed-off-by: Stephen Hemminger --- drivers/net/nfp/flower/nfp_flower_cmsg.h | 3 ++- drivers/net/nfp/flower/nfp_flower_ctrl.c | 4 ++-- 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/drivers/net/nfp/flower/nfp_flower_cmsg.h b

[PATCH v3 09/14] crypto/qat: use rte_pktmbuf_mtod_offset

2023-07-06 Thread Stephen Hemminger
Auto generated with cocci/mtod-offset.cocci Signed-off-by: Stephen Hemminger --- drivers/crypto/qat/dev/qat_crypto_pmd_gens.h | 9 + drivers/crypto/qat/qat_sym.h | 9 + 2 files changed, 10 insertions(+), 8 deletions(-) diff --git a/drivers/crypto/qat/dev/qat_cryp

[PATCH v3 08/14] crypto/ipsec_mb: use rte_pktmbuf_mtod_offset

2023-07-06 Thread Stephen Hemminger
Initial patch generated with cocci/mtod-offset. Additional manual cleanups to indentation and remove unnecessary parenthesis. Signed-off-by: Stephen Hemminger --- drivers/crypto/ipsec_mb/pmd_kasumi.c | 16 ++--- drivers/crypto/ipsec_mb/pmd_snow3g.c | 35 +++- driv

[PATCH v3 10/14] crypto/cnxk: use rte_ptkmbuf_mtod_offset

2023-07-06 Thread Stephen Hemminger
Autogenerated with cocci/mtod-offset.cocci Signed-off-by: Stephen Hemminger --- drivers/crypto/cnxk/cn9k_cryptodev_ops.c | 2 +- drivers/crypto/cnxk/cnxk_se.h| 5 ++--- 2 files changed, 3 insertions(+), 4 deletions(-) diff --git a/drivers/crypto/cnxk/cn9k_cryptodev_ops.c b/drivers/

[PATCH v3 11/14] common/cpt: use rte_pktmbuf_mtod_offset

2023-07-06 Thread Stephen Hemminger
Autogenerated with cocci/mtod-offset.cocci Signed-off-by: Stephen Hemminger --- drivers/common/cpt/cpt_ucode.h | 10 -- 1 file changed, 4 insertions(+), 6 deletions(-) diff --git a/drivers/common/cpt/cpt_ucode.h b/drivers/common/cpt/cpt_ucode.h index b393be4cf661..87a3ac80b9da 100644 --

[PATCH v3 12/14] crypto/caam_jr: use rte_pktmbuf_mtod_offset

2023-07-06 Thread Stephen Hemminger
Autogenerated with cocci/mtod-offset.cocci. Signed-off-by: Stephen Hemminger --- drivers/crypto/caam_jr/caam_jr.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/crypto/caam_jr/caam_jr.c b/drivers/crypto/caam_jr/caam_jr.c index b55258689b49..9c96fd21a48d 10064

[PATCH v3 13/14] net/mlx4: use rte_pktmbuf_mtod_offset

2023-07-06 Thread Stephen Hemminger
Autogenerated with cocci/mtod-offset.cocci. Signed-off-by: Stephen Hemminger --- drivers/net/mlx4/mlx4_rxtx.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/net/mlx4/mlx4_rxtx.c b/drivers/net/mlx4/mlx4_rxtx.c index 059e432a63fc..d5feeb7f7e6d 100644 --- a/driver

[PATCH v3 14/14] baseband/fpga_5gnr: use rte_pktmbu_mtod_offset

2023-07-06 Thread Stephen Hemminger
Autogenerated with cocci/mtod-offset.cocci. Signed-off-by: Stephen Hemminger --- drivers/baseband/fpga_5gnr_fec/rte_fpga_5gnr_fec.c | 7 +++ 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/drivers/baseband/fpga_5gnr_fec/rte_fpga_5gnr_fec.c b/drivers/baseband/fpga_5gnr_fec/rte_

Re: [PATCH v1] examples/l3fwd: fix for coverity scan

2023-07-06 Thread Stephen Hemminger
On Wed, 01 Feb 2023 18:28:44 +0100 Thomas Monjalon wrote: > 10/01/2023 15:56, Mohammad Iqbal Ahmad: > > This patch fixes (Logically dead code) coverity issue. > > This patch also fixes (Uninitialized scalar variable) coverity issue. > > > > Coverity issue: 381687 > > Coverity issue: 381686 > > F

RE: [EXT] [PATCH v2] app/crypto-perf: fix socket ID default value

2023-07-06 Thread Akhil Goyal
> Due to recent changes to the default device socket ID, > before being used as an index for session mempool list, > the socket ID should be set to 0 if unknown (-1). > > Fixes: 7dcd73e37965 ("drivers/bus: set device NUMA node to unknown by > default") > Fixes: 64c469b9e7d8 ("app/crypto-perf: chec

RE: [PATCH v2] examples/ipsec-secgw: fix of socket id default value

2023-07-06 Thread Akhil Goyal
> > Subject: [PATCH v2] examples/ipsec-secgw: fix of socket id default value > > > > Due to recent changes to the default device socket ID, before being used as > > an index for session mempool list, set socket ID to 0 if unknown (-1). > > > > Fixes: 7dcd73e37965 ("drivers/bus: set device NUMA node

Re: [PATCH v3] usertools: add check for IOMMU support in dpdk-devbind

2023-07-06 Thread Stephen Hemminger
On Mon, 21 Mar 2022 17:27:27 +0500 Fidaullah Noonari wrote: > binding with vfio driver, when IOMMU is disabled, causes program to crash. > this patch adds a flag for noiommmu-mode. when this is set, if IOMMU is > disabled, it changes vfio into unsafe noiommu mode and prints warning > message. >

RE: [EXT] [PATCH v3 0/2] remove unused defines

2023-07-06 Thread Akhil Goyal
> This series removes some unused defines throughout common qat drivers > and crypto ipsec mb drivers. It also removes some defines that should > have been removed previously. > > v3: > Incorrect fixline Series applied to dpdk-next-crypto Thanks

RE: [EXT] [PATCH] test/ipsec: fix TAP default MAC address

2023-07-06 Thread Akhil Goyal
> default TAP mac address was changed in commit id: > c3006be2acab49c6b77ae9c9ef04b061e5dacbd6 > reflect changes in ipsec test scripts. > > Fixes: c3006be2acab ("net/tap: set locally administered bit for fixed MAC > address") > Cc: d...@linux.vnet.ibm.com > Cc: sta...@dpdk.org > > Signed-off-by:

RE: [PATCH] doc: announce deprecation for security ops

2023-07-06 Thread Akhil Goyal
> > Subject: [PATCH] doc: announce deprecation for security ops > > > > Structure rte_security_ops and rte_security_ctx are meant to be used by > > rte_security library and the PMDs associated. > > These will be moved to an internal header in DPDK 23.11 release. > > > > Signed-off-by: Akhil Goyal

Re: [dpdk-dev] [PATCH v5 2/4] eal: improve options usage text

2023-07-06 Thread Stephen Hemminger
On Mon, 5 Apr 2021 21:39:52 +0200 Thomas Monjalon wrote: > The description of the EAL options was printed before the application > description provided via the hook. > It is better to let the application print the global syntax > and describes the detail of the EAL options below. > > Also, some

Re: [dpdk-dev] [PATCH] interrupts: fix error log level

2023-07-06 Thread Stephen Hemminger
On Mon, 1 Nov 2021 23:14:47 +0530 Harman Kalra wrote: > Fixing the error logs level, as currently default level is > set to debug. Due to this failure is not getting captured. > > Fixes: b7c984291611 ("interrupts: add allocator and accessors") > > Signed-off-by: Harman Kalra > --- > lib/eal/c

  1   2   >