RE: [PATCH v3] net/ixgbe: add proper memory barriers for some Rx functions

2023-06-20 Thread Ruifeng Wang
> -Original Message- > From: Thomas Monjalon > Sent: Tuesday, June 20, 2023 11:53 PM > To: Zhang, Qi Z ; zhoumin ; > Ruifeng Wang > > Cc: dev@dpdk.org; m...@smartsharesystems.com; konstantin.v.anan...@yandex.ru; > Yang, Qiming > ; Wu, Wenjun1 ; > d...@linux.vnet.ibm.com; > roret...@lin

[PATCH] eal: add notes to SMP memory barrier APIs

2023-06-20 Thread Ruifeng Wang
The rte_smp_xx() APIs are deprecated. But it is not mentioned in the function header. Added notes in function header for clarification. Signed-off-by: Ruifeng Wang --- lib/eal/include/generic/rte_atomic.h | 15 +++ 1 file changed, 15 insertions(+) diff --git a/lib/eal/include/generi

Re: [PATCH v3 1/4] examples/fips_validation: fix external build

2023-06-20 Thread David Marchand
On Tue, Jun 20, 2023 at 4:30 PM David Marchand wrote: > > Added sources were not referenced in the makefile used when compiling > this example externally. > > Fixes: 36128a67c27e ("examples/fips_validation: add asymmetric validation") > Fixes: b455d261eb89 ("examples/fips_validation: validate ECDS

RE: release candidate 23.07-rc1

2023-06-20 Thread Xu, HailinX
> -Original Message- > From: Xu, HailinX > Sent: Thursday, June 15, 2023 3:18 PM > To: Thomas Monjalon ; annou...@dpdk.org; > dev@dpdk.org > Cc: Kovacevic, Marko ; Mcnamara, John > ; Richardson, Bruce > ; Ferruh Yigit > Subject: RE: release candidate 23.07-rc1 > > > -Original Message-

Re: [EXT] [PATCH v3] common/cnxk/roc_npa: unlock mbox in error cases

2023-06-20 Thread Jerin Jacob
On Wed, Jun 21, 2023 at 10:17 AM Ashwin Sekhar T K wrote: > > Acked-by: Ashwin Sekhar T K > > > -Original Message- > > From: Thierry Herbelot > > Sent: Tuesday, June 20, 2023 3:14 PM > > To: dev@dpdk.org > > Cc: Thierry Herbelot ; Thomas Monjalon > > ; Nithin Kumar Dabilpuram > > ; Satha

RE: [EXT] [PATCH v3] common/cnxk/roc_npa: unlock mbox in error cases

2023-06-20 Thread Ashwin Sekhar T K
Acked-by: Ashwin Sekhar T K > -Original Message- > From: Thierry Herbelot > Sent: Tuesday, June 20, 2023 3:14 PM > To: dev@dpdk.org > Cc: Thierry Herbelot ; Thomas Monjalon > ; Nithin Kumar Dabilpuram > ; Satha Koteswara Rao Kottidi > ; Jerin Jacob Kollanukkaran > > Subject: [EXT] [PATC

Re: [dpdk-dev] [PATCH] net/cnxk: fix incorrect queue index validation

2023-06-20 Thread Jerin Jacob
On Tue, Jun 20, 2023 at 4:32 PM wrote: > > From: Satheesh Paul > > When creating a flow rule directing traffic from PF > to VF, current validation is incorrectly checking the > RQ index in flow action with the PF's maximum > configured RQs. This validation is applicable only > when action target

Re: [PATCH v3] net/bonding: fix bond startup failure when NUMA is -1

2023-06-20 Thread humin (Q)
在 2023/6/20 22:10, Ferruh Yigit 写道: On 6/20/2023 2:15 PM, humin (Q) wrote: Hi, Niklas, Ferruh, 在 2023/6/20 19:03, Niklas Söderlund 写道: Hi Connor and Ferruh, On 2023-06-19 09:57:17 +0100, Ferruh Yigit wrote: On 6/16/2023 1:00 PM, humin (Q) wrote: Hi, 在 2023/6/16 15:20, Chaoyong He 写道: Fr

[PATCH] net/iavf: fix VLAN insertion in vector path

2023-06-20 Thread Wenzhuo Lu
As the VLAN insertion is partially supported in vector path, the behavior is different in scalar and vector path. For a VLAN packet, if using scalar path, the new VLAN tag will be inserted after the original VLAN tag. If using vector path, the new VLAN tag is inserted before the original VLAN tag.

[PATCH] net/iavf: fix SCTP tunnel packet forwarding issue

2023-06-20 Thread Wenzhuo Lu
The SCTP tunnel packets cannot be forwarded in AVX2 mode. As 2 features are developed in parallel, 5712bf9d6e14 ("net/iavf: add Tx AVX2 offload path") doesn't consider the impact of 4f8259df563a ("net/iavf: enable Tx outer checksum offload on AVX512"). So, the wrong TX path is selected. Fixes: 57

RE: [PATCH] ixgbe: Allow setting IVAR register for X550EM_a

2023-06-20 Thread Zhang, Qi Z
> -Original Message- > From: Joyce Kong > Sent: Monday, June 19, 2023 5:45 PM > To: Zhang, Qi Z ; Vladimir Ratnikov > ; Yang, Qiming ; Wu, > Wenjun1 > Cc: dev@dpdk.org; Ruifeng Wang > Subject: RE: [PATCH] ixgbe: Allow setting IVAR register for X550EM_a > > Hi All, > > It seems a fal

RE: [PATCH] net/igc: disable EEE by default on I225/226

2023-06-20 Thread Zhang, Qi Z
> -Original Message- > From: Matthew Smith > Sent: Tuesday, June 20, 2023 10:56 PM > To: Guo, Junfeng ; Su, Simei > Cc: dev@dpdk.org; Smith, Matthew > Subject: [PATCH] net/igc: disable EEE by default on I225/226 > > I226-V devices can hang when EEE is enabled. Explicitly disable EEE

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

2023-06-20 Thread Dmitry Kozlyuk
Sketch creation seeded the global PRNG using the supplied seed for hashing. The use of this seed by SKETCH set summary was not documented. SKETCH set summary does not require two independent hash seeds, unlike other set summary types. Seeding the global PRNG at sketch creation does not make the ske

Re: [PATCH] hash: add XOR32 hash function

2023-06-20 Thread Bili Dong
For the HW I have access to, I think so. That is actually one of the major final goals of this patch, so we can model this HW in P4, including all the hash functions it supports, through the DPDK SWX pipeline. Thanks, Bili On Sat, Jun 17, 2023 at 1:59 PM Stephen Hemminger < step...@networkplumbe

[PATCH v7 1/1] hash: add XOR32 hash function

2023-06-20 Thread Bili Dong
An XOR32 hash is needed in the Software Switch (SWX) Pipeline for its use case in P4. We implement it in this patch so it could be easily registered in the pipeline later. Signed-off-by: Bili Dong --- v7: * Simplified byte ordering conversion logic. (re Vladimir Medvedkin ) * Added test cases wi

[pull-request] next-crypto 23.07 rc2

2023-06-20 Thread Akhil Goyal
The following changes since commit 4645ed709d36bab64fbf2b8cff12054d39398af9: examples/l3fwd-graph: introduce mcore dispatch worker model (2023-06-19 21:27:05 +0200) are available in the Git repository at: http://git.dpdk.org/next/dpdk-next-crypto for you to fetch changes up to 4dc146d0e92

RE: [EXT] [PATCH v4 0/9] crypto/mlx5: support AES-GCM

2023-06-20 Thread Akhil Goyal
> AES-GCM provides both authenticated encryption and the ability to check > the integrity and authentication of additional authenticated data (AAD) > that is sent in the clear. > > The crypto operations are performed with crypto WQE. If the input > buffers(AAD, mbuf, digest) are not contiguous and

Re: [PATCH] net/nfp: fix add redundant read configure operations

2023-06-20 Thread Ferruh Yigit
On 6/14/2023 6:51 AM, Chaoyong He wrote: > From: Qin Ke > > Performance is reduced because of adding redundant read configure > operations in nfp_net_parse_ptype() called by nfp_net_recv_pkts(), > which cause every received packet to spend more time. > > Fix the performance issue by deleting the

RE: [dpdk-dev] [PATCH v1] doc: fix ip6 node API doxygen generation

2023-06-20 Thread Sunil Kumar Kori
> -Original Message- > From: jer...@marvell.com > Sent: Tuesday, June 20, 2023 3:26 PM > To: dev@dpdk.org; Nithin Kumar Dabilpuram ; > Amit Prakash Shukla ; Sunil Kumar Kori > > Cc: Jerin Jacob Kollanukkaran > Subject: [dpdk-dev] [PATCH v1] doc: fix ip6 node API doxygen generation > >

[PATCH v2 2/2] doc/contributing: guidelines for logging, tracing and telemetry

2023-06-20 Thread Bruce Richardson
As discussed by DPDK technical board [1], our contributor guide should include some details as to when to use logging vs tracing vs telemetry to provide the end user with information about the running process and the DPDK libraries it uses. [1] https://mails.dpdk.org/archives/dev/2023-March/265204

[PATCH v2 1/2] doc/contributing: provide coding details for dynamic logging

2023-06-20 Thread Bruce Richardson
While the section on dynamic logging in the contributors guide covered the details of the logging naming scheme, it failed to cover exactly how the component developer, i.e. the contributor, could actually use dynamic logging in their component. Fix this by splitting the details of the naming sche

[PATCH v2 0/2] Improve docs on getting info on running process

2023-06-20 Thread Bruce Richardson
Add details to our documentation on how to register logging for a component. Also provide guidelines on when to use logs vs tracing vs telemetry. V2: extended second patch based on the feedback provided. Hopefully, we can keep this section short and clear as the revisions roll! :-) Bruce

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

2023-06-20 Thread Ferruh Yigit
On 6/20/2023 11:07 AM, Jie Hai wrote: > On 2023/6/9 19:10, Ferruh Yigit wrote: >> On 6/9/2023 10:03 AM, Jie Hai wrote: >>> Here's how the problem arises. >>> step1: Start the app. >>> dpdk-testpmd -a :35:00.0 -l 0-3 -- -i --rxq=10 --txq=10 >>> >>> step2: Perform the following steps and sen

RE: [RFC PATCH v3] ethdev: advertise flow restore in mbuf

2023-06-20 Thread Slava Ovsiienko
> -Original Message- > From: David Marchand > Sent: Tuesday, June 20, 2023 2:10 PM > To: dev@dpdk.org > Cc: NBU-Contact-Thomas Monjalon (EXTERNAL) ; > i.maxim...@ovn.org; Ali Alnubani ; Aman Singh > ; Yuying Zhang ; > Matan Azrad ; Slava Ovsiienko > ; Ori Kam ; Suanming Mou > ; David Chris

Re: [PATCH] ethdev: rename functions checking queue validity

2023-06-20 Thread Ferruh Yigit
On 6/19/2023 10:41 AM, Andrew Rybchenko wrote: > On 6/14/23 19:44, Stephen Hemminger wrote: >> On Wed, 14 Jun 2023 17:20:59 +0200 >> Thomas Monjalon wrote: >> >>> Two functions helping to check Rx/Tx queues validity >>> were added in DPDK 23.07-rc1. >>> As the release is not closed, it is still ti

Re: [PATCH v3] net/ixgbe: add proper memory barriers for some Rx functions

2023-06-20 Thread Thomas Monjalon
13/06/2023 11:25, Ruifeng Wang: > From: Thomas Monjalon > > 12/06/2023 13:58, zhoumin: > > > On Mon, June 12, 2023 at 6:26PM, Thomas Monjalon wrote: > > > > 15/05/2023 04:10, Zhang, Qi Z: > > > >> From: Ruifeng Wang > > > >>> From: Min Zhou > > > --- > > > v3: > > > - Use rte_smp_

Re: [PATCH 0/9] Wangxun bug fixes

2023-06-20 Thread Ferruh Yigit
On 6/14/2023 3:34 AM, Jiawen Wu wrote: > Fix some bugs for txgbe and ngbe. > > Jiawen Wu (9): > net/txgbe: fix Tx failure with fiber hotplug > net/txgbe: fix interrupt enable mask > net/txgbe: fix issues caused by MNG veto bit setting > net/txgbe: fix to set autoneg for 1G speed > net/tx

Re: ring name length simplification in ipsec_mb_qp_create_processed_ops_ring

2023-06-20 Thread Ji, Kai
Can you fix the commit messages with signed-off and Fixes Acked-by: Kai Ji mailto:kai...@intel.com>> From: Stephen Hemminger Sent: 31 May 2023 00:34 To: Fan Zhang Cc: dev@dpdk.org Subject: ring name length simplification in ipsec_mb_qp_create_processed_ops_rin

Re: [PATCH v3] build: select optional libraries

2023-06-20 Thread Bruce Richardson
On Tue, Jun 20, 2023 at 04:33:15PM +0200, Thomas Monjalon wrote: > 20/06/2023 11:03, Bruce Richardson: > > On Tue, Jun 20, 2023 at 10:48:50AM +0200, David Marchand wrote: > > > On Tue, Jun 20, 2023 at 10:45 AM Bruce Richardson > > > wrote: > > > > > > > > I notice the change in behaviour for enabl

[PATCH] net/igc: disable EEE by default on I225/226

2023-06-20 Thread Matthew Smith
I226-V devices can hang when EEE is enabled. Explicitly disable EEE during initialization of the device, as the Linux and FreeBSD kernel drivers do. Signed-off-by: Matthew Smith --- drivers/net/igc/base/igc_i225.c | 4 1 file changed, 4 insertions(+) diff --git a/drivers/net/igc/base/igc_i

Re: [EXT] Re: [PATCH v2 8/8] crypto/ipsec_mb: set and use session ID

2023-06-20 Thread Thomas Monjalon
16/06/2023 11:38, Akhil Goyal: > > > > > > > > > > > > For info, this does not compile with > > > > > > https://urldefense.proofpoint.com/v2/url?u=https- > > 3A__git.gitlab.arm.com_arm-2Dreference-2Dsolutions_ipsec- > > 2Dmb&d=DwIFAg&c=nKjWec2b6R0mOyPaz7xtfQ&r=DnL7Si2wl_PRwpZ9TWey3e > > u68gBzn7DkP

Re: [PATCH v3] build: select optional libraries

2023-06-20 Thread Bruce Richardson
On Tue, Jun 20, 2023 at 04:33:15PM +0200, Thomas Monjalon wrote: > 20/06/2023 11:03, Bruce Richardson: > > On Tue, Jun 20, 2023 at 10:48:50AM +0200, David Marchand wrote: > > > On Tue, Jun 20, 2023 at 10:45 AM Bruce Richardson > > > wrote: > > > > > > > > I notice the change in behaviour for enabl

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

2023-06-20 Thread Thomas Monjalon
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 supports dump of ring information by its name. > >> An example using this command is shown below: > >> > >> --> /ring/info,MP_mb_pool_0 > >> { > >>"/ri

Re: [PATCH v3] build: select optional libraries

2023-06-20 Thread Thomas Monjalon
20/06/2023 11:03, Bruce Richardson: > On Tue, Jun 20, 2023 at 10:48:50AM +0200, David Marchand wrote: > > On Tue, Jun 20, 2023 at 10:45 AM Bruce Richardson > > wrote: > > > > > > > I notice the change in behaviour for enabling the deprecated > > > > > > > libs. Is there > > > > > > > any other ch

Re: [PATCH] ci: fix libabigail cache in GHA

2023-06-20 Thread Aaron Conole
David Marchand writes: > In repositories where multiple branches run the ABI checks using > different versions of libabigail (for example, a 22.11 branch using > libabigail-1.8 and a main branch using libabigail-2.1), a collision > happens on the libabigail binary cache entry. > As a single cache

Re: [PATCH v3] net/bonding: fix bond startup failure when NUMA is -1

2023-06-20 Thread Thomas Monjalon
20/06/2023 16:10, Ferruh Yigit: > On 6/20/2023 2:15 PM, humin (Q) wrote: > > Hi, Niklas, Ferruh, > > > > 在 2023/6/20 19:03, Niklas Söderlund 写道: > >> Hi Connor and Ferruh, > >> > >> On 2023-06-19 09:57:17 +0100, Ferruh Yigit wrote: > >>> On 6/16/2023 1:00 PM, humin (Q) wrote: > Hi, > > >

[PATCH v4 9/9] crypto/mlx5: enable AES-GCM capability

2023-06-20 Thread Suanming Mou
This commit generates AES-GCM capability based on the NIC attributes and enables AES-GCM algo. An new devarg "algo" is added to identify if the crypto PMD will be initialized as AES-GCM(algo=1) or AES-XTS(algo=0, default). Signed-off-by: Suanming Mou Acked-by: Matan Azrad --- doc/guides/crypto

[PATCH v4 8/9] crypto/mlx5: add enqueue and dequeue operations

2023-06-20 Thread Suanming Mou
The crypto operations are performed with crypto WQE. If the input buffers(AAD, mbuf, digest) are not contiguous and there is no enough headroom/tailroom for copying AAD/digest, as the requirement from FW, an UMR WQE is needed to generate contiguous address space for crypto WQE. The UMR WQE and cryp

[PATCH v4 7/9] crypto/mlx5: add queue pair setup for GCM

2023-06-20 Thread Suanming Mou
Crypto queue pair is for handling the encryption/decryption operations. As AES-GCM AEAD API provides AAD, mbuf, digest separately, low-level FW only accepts the data in a single contiguous memory region, two internal QPs are created for AES-GCM queue pair. One for organizing the memory to be conti

[PATCH v4 6/9] common/mlx5: add WQE-based QP synchronous basics

2023-06-20 Thread Suanming Mou
Nvidia HW provides a synchronous mechanism between QPs. When creating the QPs, user can set one as primary and another as follower. The follower QP's WQE execution can be controlled by primary QP via SEND_EN WQE. This commit introduces the SEND_EN WQE to improve the WQE execution sync-up between p

[PATCH v4 5/9] crypto/mlx5: add AES-GCM session configure

2023-06-20 Thread Suanming Mou
Sessions are used in symmetric transformations in order to prepare objects and data for packet processing stage. The AES-GCM session includes IV, AAD, digest(tag), DEK, operation mode information. Signed-off-by: Suanming Mou Acked-by: Matan Azrad --- drivers/common/mlx5/mlx5_prm.h| 12

[PATCH v4 4/9] crypto/mlx5: add AES-GCM encryption key

2023-06-20 Thread Suanming Mou
The crypto device requires the DEK(data encryption key) object for data encryption/decryption operation. This commit adds the AES-GCM DEK object management support. Signed-off-by: Suanming Mou Acked-by: Matan Azrad --- drivers/crypto/mlx5/mlx5_crypto.h | 17 - drivers/crypto/mlx5/mlx5

[PATCH v4 2/9] crypto/mlx5: split AES-XTS

2023-06-20 Thread Suanming Mou
As there will be other crypto algo be supported. This commit splits AES-XTS code to another *_xts.c file. The mlx5_crypto.c file will just contain the common code. Signed-off-by: Suanming Mou Acked-by: Matan Azrad --- drivers/crypto/mlx5/meson.build | 1 + drivers/crypto/mlx5/mlx5_crypt

[PATCH v4 3/9] crypto/mlx5: add AES-GCM query and initialization

2023-06-20 Thread Suanming Mou
AES-GCM provides both authenticated encryption and the ability to check the integrity and authentication of additional authenticated data (AAD) that is sent in the clear. This commit adds the AES-GCM attributes query and initialization function. Signed-off-by: Suanming Mou Acked-by: Matan Azrad

[PATCH v4 1/9] common/mlx5: export memory region lookup by address

2023-06-20 Thread Suanming Mou
In case user provides the address without mempool. Export the function to lookup the address without mempool is required. Signed-off-by: Suanming Mou Acked-by: Matan Azrad --- drivers/common/mlx5/mlx5_common_mr.c | 2 +- drivers/common/mlx5/mlx5_common_mr.h | 4 drivers/common/mlx5/version

[PATCH v4 0/9] crypto/mlx5: support AES-GCM

2023-06-20 Thread Suanming Mou
AES-GCM provides both authenticated encryption and the ability to check the integrity and authentication of additional authenticated data (AAD) that is sent in the clear. The crypto operations are performed with crypto WQE. If the input buffers(AAD, mbuf, digest) are not contiguous and there is no

[PATCH v3 1/4] examples/fips_validation: fix external build

2023-06-20 Thread David Marchand
Added sources were not referenced in the makefile used when compiling this example externally. Fixes: 36128a67c27e ("examples/fips_validation: add asymmetric validation") Fixes: b455d261eb89 ("examples/fips_validation: validate ECDSA") Cc: sta...@dpdk.org Signed-off-by: David Marchand Acked-by:

[PATCH v3 0/4] Test examples compilation externally

2023-06-20 Thread David Marchand
As DPDK provides examples compiled with makefiles, we need some tests in the CI. So far, a few maintainers have been testing them but a simple issue has been missed for some time and there was no way to try to build all examples that were built with meson. Additionnally, this series can help in id

Re: [PATCH v3] net/bonding: fix bond startup failure when NUMA is -1

2023-06-20 Thread Ferruh Yigit
On 6/20/2023 2:15 PM, humin (Q) wrote: > Hi, Niklas, Ferruh, > > 在 2023/6/20 19:03, Niklas Söderlund 写道: >> Hi Connor and Ferruh, >> >> On 2023-06-19 09:57:17 +0100, Ferruh Yigit wrote: >>> On 6/16/2023 1:00 PM, humin (Q) wrote: Hi, 在 2023/6/16 15:20, Chaoyong He 写道: > From: Zer

[PATCH v3 2/4] examples/server_node_efd: simplify build configuration

2023-06-20 Thread David Marchand
The server_node_efd example contains various binaries whose sources are in sub directories that do not match their name. Rename sub directories in this example and stop overriding the binaries names in meson.build. This makes a next change easier in the build scripts used by CI. Signed-off-by: Da

[PATCH v3 3/4] devtools: build all examples externally

2023-06-20 Thread David Marchand
So far, users of test-meson-builds.sh had to define their own set of examples to build externally. This is not that great because users need to maintain this list when examples are removed/added. Rework the script so that the 'all' value triggers an automatic discovery based on what was configured

[PATCH v3 4/4] ci: build examples externally

2023-06-20 Thread David Marchand
Enhance our CI coverage by building examples against an installed DPDK. Signed-off-by: David Marchand Acked-by: Bruce Richardson Acked-by: Aaron Conole --- Changes since v2: - dropped unneeded -e in sed cmdline, Changes since v1: - reworked built examples discovery, - added comment for people

Re: [PATCH v5 2/3] common/sfc_efx/base: add support to enable VLAN stripping

2023-06-20 Thread Andrew Rybchenko
On 6/20/23 16:10, Artemii Morozov wrote: On 6/20/23 15:50, Andrew Rybchenko wrote: On 6/20/23 12:55, Artemii Morozov wrote: On 6/19/23 14:28, Andrew Rybchenko wrote: On 6/13/23 18:12, Artemii Morozov wrote: To enable VLAN stripping, two conditions must be met: the corresponding flag must be

RE: [PATCH v2 0/9] crypto/mlx5: support AES-GCM

2023-06-20 Thread Matan Azrad
From: Suanming Mou > > -Original Message- > > From: Akhil Goyal > > Sent: Tuesday, June 20, 2023 5:59 PM > > To: Suanming Mou > > Cc: Raslan Darawsheh ; dev@dpdk.org > > Subject: RE: [PATCH v2 0/9] crypto/mlx5: support AES-GCM > > > > Hi Suanming, > > > Hi Akhil, > > > > > > Maybe due t

[PATCH] ci: fix libabigail cache in GHA

2023-06-20 Thread David Marchand
In repositories where multiple branches run the ABI checks using different versions of libabigail (for example, a 22.11 branch using libabigail-1.8 and a main branch using libabigail-2.1), a collision happens on the libabigail binary cache entry. As a single cache entry is used, the content of the

RE: [PATCH] test/pdcp: remove side affect from assertion

2023-06-20 Thread Akhil Goyal
> > Remove side affect from test assertion, to avoid different behaviour in non- > > debug build. Issues reported by coverity scan. > > > > Coverity issue: 385419, 385420 > > Fixes: 6f00462979ad ("test/pdcp: add timer expiry cases") > > > > Signed-off-by: Volodymyr Fialko > > Acked-by: Anoob Jose

RE: [PATCH] test/security: fix event inline IPsec reassembly tests

2023-06-20 Thread Akhil Goyal
> -Original Message- > From: Rahul Bhansali > Sent: Tuesday, June 6, 2023 10:37 AM > To: dev@dpdk.org; Akhil Goyal ; Anoob Joseph > ; Volodymyr Fialko > Cc: Rahul Bhansali > Subject: [PATCH] test/security: fix event inline IPsec reassembly tests > > Event Inline IPsec test suite stuc

Re: [PATCH v3] net/bonding: fix bond startup failure when NUMA is -1

2023-06-20 Thread humin (Q)
Hi, Niklas, Ferruh, 在 2023/6/20 19:03, Niklas Söderlund 写道: Hi Connor and Ferruh, On 2023-06-19 09:57:17 +0100, Ferruh Yigit wrote: On 6/16/2023 1:00 PM, humin (Q) wrote: Hi, 在 2023/6/16 15:20, Chaoyong He 写道: From: Zerun Fu After the mainline Linux kernel commit "fe205d984e7730f4d21f6f8e

Re: [PATCH v5 2/3] common/sfc_efx/base: add support to enable VLAN stripping

2023-06-20 Thread Artemii Morozov
On 6/20/23 15:50, Andrew Rybchenko wrote: On 6/20/23 12:55, Artemii Morozov wrote: On 6/19/23 14:28, Andrew Rybchenko wrote: On 6/13/23 18:12, Artemii Morozov wrote: To enable VLAN stripping, two conditions must be met: the corresponding flag must be set and the appropriate RX prefix should

RE: [EXT] [PATCH] doc: update IPsec Multi-buffer version for Arm

2023-06-20 Thread Akhil Goyal
> Updated SNOW3G and ZUC PMD documentation guides with information > about the latest library version supported. > > Signed-off-by: Ruifeng Wang Applied to dpdk-next-crypto Thanks.

Re: [PATCH 1/2] test: add graph tests

2023-06-20 Thread Kevin Traynor
On 20/06/2023 12:52, David Marchand wrote: On Tue, Jun 20, 2023 at 12:32 PM Kevin Traynor wrote: On 20/06/2023 07:56, David Marchand wrote: On Tue, Jun 20, 2023 at 8:50 AM Jerin Jacob wrote: On Tue, Jun 20, 2023 at 2:17 AM David Marchand wrote: We forgot to add graph unit tests to the C

RE: [EXT] [PATCH v2] doc: updated libcrypto dependencies in QAT guide

2023-06-20 Thread Akhil Goyal
> The documentation needs extra steps for customers to explicitly > show how to install libcrypto, which is needed for QAT. > It requires the commands to install libcrypto for > Ubuntu and RHEL. > > Signed-off-by: Samina Arshad Applied to dpdk-next-crypto Thanks.

RE: [EXT] [PATCH v7] crypto/qat: default to IPsec MB for computations

2023-06-20 Thread Akhil Goyal
> Pre and post computations currently use the OpenSSL library by default. > This patch changes the default option to Intel IPsec MB library version > 1.4 for the required computations. If this version of IPsec is not met > it will fallback to use OpenSSL. ARM will always default to using > OpenSSL

RE: [PATCH v2 0/5] net/mlx5: introduce Tx datapath tracing

2023-06-20 Thread Raslan Darawsheh
Hi, > -Original Message- > From: Viacheslav Ovsiienko > Sent: Tuesday, June 13, 2023 7:59 PM > To: dev@dpdk.org > Subject: [PATCH v2 0/5] net/mlx5: introduce Tx datapath tracing > > The mlx5 provides the send scheduling on specific moment of time, > and for the related kind of applicatio

Re: [PATCH 1/2] test: add graph tests

2023-06-20 Thread David Marchand
On Tue, Jun 20, 2023 at 12:32 PM Kevin Traynor wrote: > > On 20/06/2023 07:56, David Marchand wrote: > > On Tue, Jun 20, 2023 at 8:50 AM Jerin Jacob wrote: > >> > >> On Tue, Jun 20, 2023 at 2:17 AM David Marchand > >> wrote: > >>> > >>> We forgot to add graph unit tests to the CI testsuites. > >

[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-06-20 Thread bugzilla
https://bugs.dpdk.org/show_bug.cgi?id=1256 Bug ID: 1256 Summary: drivers/common/mlx5: mlx5_malloc() called on invalid socket ID when global MR cache is full and rte_extmem_* API is used Product: DPDK Version: 21.1

Re: [PATCH v5 2/3] common/sfc_efx/base: add support to enable VLAN stripping

2023-06-20 Thread Andrew Rybchenko
On 6/20/23 12:55, Artemii Morozov wrote: On 6/19/23 14:28, Andrew Rybchenko wrote: On 6/13/23 18:12, Artemii Morozov wrote: To enable VLAN stripping, two conditions must be met: the corresponding flag must be set and the appropriate RX prefix should be requested. RX -> Rx VLAN stripping is

Re: [PATCH v2 0/3] net/sfc: support FEC feature

2023-06-20 Thread Ferruh Yigit
On 6/15/2023 9:38 AM, Denis Pryazhennikov wrote: > This patch series adds setting and querying of Forward error > correction (FEC). AUTO, BASER and RS modes are supported > by EF10/EF100 hardware. LLRS mode is not supported. > > Changes in v2: > * Applied review notes in [2/3] and [3/3]; > * Added

Re: [PATCH 1/3] net/sfc: split link update function

2023-06-20 Thread Andrew Rybchenko
On 6/20/23 14:25, Ferruh Yigit wrote: On 6/2/2023 9:52 AM, Andrew Rybchenko wrote: On 6/2/23 01:23, Denis Pryazhennikov wrote: Separate the original link update function into two functions: state retrieval and update. This improves code clarity and maintainability. Signed-off-by: Denis Pryazhe

Re: [PATCH v2 2/3] common/sfc_efx/base: add FEC related macros

2023-06-20 Thread Ferruh Yigit
On 6/15/2023 9:38 AM, Denis Pryazhennikov wrote: > Added new macros to simplify working with FEC bits. > > Signed-off-by: Denis Pryazhennikov > Reviewed-by: Andy Moreton > Acked-by: Andrew Rybchenko > --- > drivers/common/sfc_efx/base/efx.h | 9 + > 1 file changed, 9 insertions(+) > >

Re: [PATCH v2 3/3] net/sfc: support FEC feature

2023-06-20 Thread Ferruh Yigit
On 6/19/2023 10:39 AM, Andrew Rybchenko wrote: > On 6/15/23 11:38, Denis Pryazhennikov wrote: >> Support ethdev methods to query and set FEC information. >> Limitations: ignoring rte_eth_fec_get_capability() results >> can lead to NOFEC if the device is not strated. >> >> Signed-off-by: Denis Pryaz

Re: [PATCH 1/3] net/sfc: split link update function

2023-06-20 Thread Ferruh Yigit
On 6/2/2023 9:52 AM, Andrew Rybchenko wrote: > On 6/2/23 01:23, Denis Pryazhennikov wrote: >> Separate the original link update function into >> two functions: state retrieval and update. >> This improves code clarity and maintainability. >> >> Signed-off-by: Denis Pryazhennikov >> Reviewed-by: Iv

Re: [PATCH] net/bonding: fix destroy dedicated queues flow

2023-06-20 Thread Ferruh Yigit
On 6/20/2023 4:02 AM, humin (Q) wrote: > 在 2023/6/8 10:59, Chaoyong He 写道: >> From: Long Wu >> >> Bonding port in mode 4 enables dedicated queues, we >> will create a flow for this feature. So we need to >> destroy this flow when we remove the member port. >> >> Furthermore if we don't destroy th

RE: [RFC] lib/ethdev: introduce table driven APIs

2023-06-20 Thread Ori Kam
Hi > -Original Message- > From: Jerin Jacob > Sent: Tuesday, June 20, 2023 8:07 AM > > On Tue, Jun 20, 2023 at 7:22 AM Zhang, Qi Z wrote: > > > > > > > > > -Original Message- > > > From: Jerin Jacob > > > Sent: Monday, June 19, 2023 5:52 PM > > > To: Zhang, Qi Z > > > Cc: Dumi

[RFC PATCH v3] ethdev: advertise flow restore in mbuf

2023-06-20 Thread David Marchand
As reported by Ilya [1], unconditionally calling rte_flow_get_restore_info() impacts an application performance for drivers that do not provide this ops. It could also impact processing of packets that require no call to rte_flow_get_restore_info() at all. Register a dynamic mbuf flag when an appl

[Bug 1255] [asan] net/mlx5: memory leak on port cleanup

2023-06-20 Thread bugzilla
https://bugs.dpdk.org/show_bug.cgi?id=1255 Bug ID: 1255 Summary: [asan] net/mlx5: memory leak on port cleanup Product: DPDK Version: unspecified Hardware: All OS: All Status: UNCONFIRMED Severity: normal

Re: [PATCH v3] net/bonding: fix bond startup failure when NUMA is -1

2023-06-20 Thread Niklas Söderlund
Hi Connor and Ferruh, On 2023-06-19 09:57:17 +0100, Ferruh Yigit wrote: > On 6/16/2023 1:00 PM, humin (Q) wrote: > > Hi, > > > > 在 2023/6/16 15:20, Chaoyong He 写道: > >> From: Zerun Fu > >> > >> After the mainline Linux kernel commit > >> "fe205d984e7730f4d21f6f8ebc60f0698404ac31" (ACPI: Remove s

[dpdk-dev] [PATCH] net/cnxk: fix incorrect queue index validation

2023-06-20 Thread psatheesh
From: Satheesh Paul When creating a flow rule directing traffic from PF to VF, current validation is incorrectly checking the RQ index in flow action with the PF's maximum configured RQs. This validation is applicable only when action target is an RQ of the PF. The validation has been modified ac

RE: [PATCH] crypto/openssl: do not build useless workaround

2023-06-20 Thread Akhil Goyal
Applied to dpdk-next-crypto Thanks.

RE: [PATCH] crypto/openssl: fix memory leak in auth processing function

2023-06-20 Thread Akhil Goyal
Applied to dpdk-next-crypto Thanks.

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

2023-06-20 Thread Ferruh Yigit
On 6/20/2023 11:07 AM, Jie Hai wrote: > On 2023/6/9 19:10, Ferruh Yigit wrote: >> On 6/9/2023 10:03 AM, Jie Hai wrote: >>> Here's how the problem arises. >>> step1: Start the app. >>> dpdk-testpmd -a :35:00.0 -l 0-3 -- -i --rxq=10 --txq=10 >>> >>> step2: Perform the following steps and sen

[PATCH v7] crypto/qat: default to IPsec MB for computations

2023-06-20 Thread Brian Dooley
Pre and post computations currently use the OpenSSL library by default. This patch changes the default option to Intel IPsec MB library version 1.4 for the required computations. If this version of IPsec is not met it will fallback to use OpenSSL. ARM will always default to using OpenSSL as ARM IPs

RE: [PATCH 1/3] examples/ipsec-secgw: avoid error pkt prints in fast path

2023-06-20 Thread Akhil Goyal
> Subject: [PATCH 1/3] examples/ipsec-secgw: avoid error pkt prints in fast path > > Avoid printing errors due to bad packets in fast path as it > effects performance. Make them RTE_LOG_DP() instead. > Also print the actual ptype that is used to classify the pkt. > > Signed-off-by: Nithin Dabilpu

[PATCH v4 3/3] app/test: add rsa none padding tests

2023-06-20 Thread Arkadiusz Kusztal
Added RSA NONE padding test cases. Signed-off-by: Arkadiusz Kusztal --- app/test/test_cryptodev_asym.c | 205 +++ app/test/test_cryptodev_rsa_test_vectors.h | 220 + 2 files changed, 425 insertions(+) diff --git a/app/test/test_cryptodev_asym.c b/

[PATCH v4 2/3] app/test: add modexp and modinv functions

2023-06-20 Thread Arkadiusz Kusztal
Added modular exponentiation and modular inverse tests to the asymmetric crypto testsuite. Signed-off-by: Arkadiusz Kusztal --- app/test/test_cryptodev_asym.c | 152 +++-- app/test/test_cryptodev_mod_test_vectors.h | 123 + 2 files changed, 263 inserti

[PATCH v4 1/3] app/test: remove obsolete test function

2023-06-20 Thread Arkadiusz Kusztal
Removed function `one by one` as it was not enough scalable and robust. Signed-off-by: Arkadiusz Kusztal --- app/test/test_cryptodev_asym.c | 418 -- app/test/test_cryptodev_mod_test_vectors.h | 925 - app/test/test_cryptodev_rsa_test_vectors.h | 224 -

[PATCH v4 0/3] Replace obsolote test cases.

2023-06-20 Thread Arkadiusz Kusztal
This patchset removes obsolete test cases for RSA, MOD EXP, MOD INV. Doing that, new way of handling ut_setup and ut_teardown was proposed. Now both behave like constructor/desctuctor in the unit tests. It frees particular alghorithm functions from any kind of responsibility to free resources for

RE: [PATCH] examples/ipsec-secgw: fix AES-CTR IV length

2023-06-20 Thread Akhil Goyal
> Subject: [PATCH] examples/ipsec-secgw: fix AES-CTR IV length > > Set AES-CTR IV length to 8 in SA config as it is > used for per packet IV length and set it to 16 > in xform since the application populates 16B IV > in the datapath. AES-CTR requires 16B IV > constructed from nonce and counter. >

Re: [PATCH 1/2] test: add graph tests

2023-06-20 Thread Kevin Traynor
On 20/06/2023 07:56, David Marchand wrote: On Tue, Jun 20, 2023 at 8:50 AM Jerin Jacob wrote: On Tue, Jun 20, 2023 at 2:17 AM David Marchand wrote: We forgot to add graph unit tests to the CI testsuites. Signed-off-by: David Marchand Could we add Fixes: ? Either way, Acked-by: Jerin Jac

RE: [EXT] [PATCH v4] crypto/qat: add SM3 HMAC to gen4 devices

2023-06-20 Thread Akhil Goyal
> This commit adds SM3 HMAC to Intel QuickAssist Technology PMD > generation 3 and 4 devices. > > Signed-off-by: Arkadiusz Kusztal > Acked-by: Ciara Power Fix your .gitconfig to have uniform author name and signoff names. Applied to dpdk-next-crypto Thanks.

RE: [PATCH v3 0/8] fixes and improvements to CNXK crypto PMD

2023-06-20 Thread Akhil Goyal
> Subject: [PATCH v3 0/8] fixes and improvements to CNXK crypto PMD > > This series adds SM4, raw cryptodev API support and > fixes to CNXK crypto PMD. > > v3: > * Updated feature file for cn10k Series applied to dpdk-next-crypto Had just applied your v2 with the doc changes and few title update

[PATCH v3 8/8] crypto/cnxk: fix order of ECFPM parameters

2023-06-20 Thread Tejasree Kondoj
From: Gowrishankar Muthukrishnan Fix order of ECFPM parameters. Fixes: 76618fc4bef ("crypto/cnxk: fix order of ECFPM parameters") Signed-off-by: Gowrishankar Muthukrishnan --- drivers/crypto/cnxk/cnxk_ae.h | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/crypto/cn

[PATCH v3 7/8] crypto/cnxk: add support for sm4

2023-06-20 Thread Tejasree Kondoj
From: Vidya Sagar Velumuri Add support for SM4 cipher Support for modes: SM4_CBC, SM4_ECB, SM4_CTR, SM4_OFB, SM4_CFB Signed-off-by: Vidya Sagar Velumuri --- doc/guides/cryptodevs/cnxk.rst| 1 + doc/guides/cryptodevs/features/cn10k.ini | 5 + doc/guides/rel_notes/releas

[PATCH v3 6/8] test/crypto: enable raw crypto tests for crypto_cn10k

2023-06-20 Thread Tejasree Kondoj
From: Anoob Joseph Enable raw crypto tests with crypto_cn10k. Signed-off-by: Anoob Joseph --- app/test/test_cryptodev.c | 8 1 file changed, 8 insertions(+) diff --git a/app/test/test_cryptodev.c b/app/test/test_cryptodev.c index fb2af40b99..2ba37ed4bd 100644 --- a/app/test/test_cryp

[PATCH v3 5/8] crypto/cnxk: add support for raw APIs

2023-06-20 Thread Tejasree Kondoj
From: Anoob Joseph Add crypto RAW API support in cnxk PMD Enable the flag to allow execution of raw test suite. Signed-off-by: Vidya Sagar Velumuri Signed-off-by: Anoob Joseph --- doc/guides/cryptodevs/features/cn10k.ini | 1 + doc/guides/rel_notes/release_23_07.rst| 1 + drivers/cry

[PATCH v3 4/8] crypto/cnxk: enable context cache for 103XX

2023-06-20 Thread Tejasree Kondoj
Enabling context cache for SE instructions on 106B0 and 103XX. Signed-off-by: Tejasree Kondoj --- drivers/crypto/cnxk/cnxk_cryptodev_ops.c | 6 +++--- drivers/crypto/cnxk/cnxk_cryptodev_ops.h | 8 2 files changed, 11 insertions(+), 3 deletions(-) diff --git a/drivers/crypto/cnxk/cnxk_c

[PATCH v3 3/8] crypto/cnxk: use pt inst for null cipher with null auth

2023-06-20 Thread Tejasree Kondoj
From: Aakash Sasidharan Use passthrough instruction for NULL cipher with NULL auth combination. Signed-off-by: Aakash Sasidharan --- drivers/crypto/cnxk/cnxk_cryptodev_ops.c | 20 drivers/crypto/cnxk/cnxk_se.h| 59 2 files changed, 50 insertions(+)

[PATCH v3 2/8] crypto/cnxk: remove packet length checks in crypto offload

2023-06-20 Thread Tejasree Kondoj
From: Anoob Joseph When performing crypto offload, the packet length of the input/output buffer does not matter. The length that matters is the cipher/authentication range specified in crypto_op. Since application can request for ciphering of a small portion of the buffer, the extra comparison of

[PATCH v3 1/8] crypto/cnxk: check for null pointer

2023-06-20 Thread Tejasree Kondoj
Checking for NULL pointer dereference. Signed-off-by: Tejasree Kondoj --- drivers/crypto/cnxk/cnxk_se.h | 14 -- 1 file changed, 8 insertions(+), 6 deletions(-) diff --git a/drivers/crypto/cnxk/cnxk_se.h b/drivers/crypto/cnxk/cnxk_se.h index c66ab80749..a85e4c5170 100644 --- a/drive

[PATCH v3 0/8] fixes and improvements to CNXK crypto PMD

2023-06-20 Thread Tejasree Kondoj
This series adds SM4, raw cryptodev API support and fixes to CNXK crypto PMD. v3: * Updated feature file for cn10k v2: * Fixed documentation build failure * Updated release notes Aakash Sasidharan (1): crypto/cnxk: use pt inst for null cipher with null auth Anoob Joseph (3): crypto/cnxk: re

  1   2   >