[PATCH v4 2/2] test/ipsec: add unit test for stateless processing

2024-10-03 Thread Aakash Sasidharan
Add unit test for IPsec stateless processing. Signed-off-by: Aakash Sasidharan --- app/test/test_ipsec.c | 113 -- 1 file changed, 99 insertions(+), 14 deletions(-) diff --git a/app/test/test_ipsec.c b/app/test/test_ipsec.c index 6cb1bac1e7..ac63c3b6d3 10

[PATCH v4 1/2] ipsec: allow stateless IPsec processing

2024-10-03 Thread Aakash Sasidharan
Introduce stateless packet preparation API for IPsec processing. The new API would allow preparation of IPsec packets without altering the internal state of an IPsec session. For outbound IPsec processing, the change enables user to provide sequence number to be used for the IPsec operation. Sign

Re: [PATCH 0/3] support load firmware from flash

2024-10-03 Thread Ferruh Yigit
On 9/3/2024 6:52 AM, Chaoyong He wrote: > This patch series add the needed logic to support load firmware from the > flash in card. > > Chaoyong He (3): > net/nfp: fix potential problem on certain version BSP > net/nfp: add two APIs of the NSP module > net/nfp: support load firmware from fla

[PATCH v2 2/2] cryptodev: move RSA padding information into xform

2024-10-03 Thread Gowrishankar Muthukrishnan
RSA padding information could be a xform entity rather than part of crypto op, as it seems associated with hashing algorithm used for the entire crypto session, where this algorithm is used in message digest itself. Even in virtIO standard spec, this info is associated in the asymmetric session cre

[PATCH v2 1/2] cryptodev: fix RSA xform for ASN.1 syntax

2024-10-03 Thread Gowrishankar Muthukrishnan
As per ASN.1 syntax (RFC 3447 Appendix A.1.2), RSA private key would need specification of quintuple along with private exponent. It is up to the implementation to internally handle, but not at RTE itself to make them exclusive each other. Removing union on them allows asymmetric implementation in

[PATCH v2 0/2] cryptodev: fix RSA xform to support VirtIO standard

2024-10-03 Thread Gowrishankar Muthukrishnan
In this series of patches, RSA crypto xform is fixed to support VirtIO standard. Changes: v2: - Decoupled spec related patches into this series from v1. Gowrishankar Muthukrishnan (2): cryptodev: fix RSA xform for ASN.1 syntax cryptodev: move RSA padding information into xform app/test/t

[PATCH v5 6/6] app/crypto-perf: support EDDSA

2024-10-03 Thread Gowrishankar Muthukrishnan
Added support for EDDSA 25519 curve SIGN and VERIFY operations. Signed-off-by: Gowrishankar Muthukrishnan --- app/test-crypto-perf/cperf_ops.c | 52 app/test-crypto-perf/cperf_options.h | 2 + app/test-crypto-perf/cperf_options_parsing.c | 9 +++- app/t

[PATCH v5 4/6] test/crypto: add asymmetric EDDSA test cases

2024-10-03 Thread Gowrishankar Muthukrishnan
Add test cases to validate EDDSA sign and verify ops, as per RFC 8032. Signed-off-by: Gowrishankar Muthukrishnan --- app/test/test_cryptodev_asym.c | 345 +++- app/test/test_cryptodev_ecdh_test_vectors.h | 94 +++- app/test/test_cryptodev_ecdsa_test_vectors.h | 4 + app

[PATCH v5 5/6] examples/fips_validation: support EDDSA

2024-10-03 Thread Gowrishankar Muthukrishnan
Add EDDSA support in fips_validation app. Signed-off-by: Gowrishankar Muthukrishnan --- examples/fips_validation/fips_validation.c| 2 + examples/fips_validation/fips_validation.h| 23 ++ .../fips_validation/fips_validation_eddsa.c | 307 + examples/fips_validation/m

[PATCH v5 3/6] crypto/cnxk: support EDDSA

2024-10-03 Thread Gowrishankar Muthukrishnan
Support EDDSA crypto algorithm in CNXK PMD. Signed-off-by: Gowrishankar Muthukrishnan --- v5: - long line fixed. --- doc/guides/cryptodevs/features/cn10k.ini | 1 + drivers/common/cnxk/hw/cpt.h | 3 +- drivers/common/cnxk/roc_ae.c | 52 +- drivers/com

[PATCH v5 2/6] crypto/openssl: support EDDSA

2024-10-03 Thread Gowrishankar Muthukrishnan
Support EDDSA crypto algorithm in OpenSSL PMD. Signed-off-by: Gowrishankar Muthukrishnan --- v5: - fixed compilation error due to RTE_LOG_CHECK_NO_NEWLINE. --- doc/guides/cryptodevs/features/openssl.ini | 1 + drivers/crypto/openssl/openssl_pmd_private.h | 13 ++ drivers/crypto/openssl/rte

[PATCH v5 1/6] cryptodev: add EDDSA asymmetric crypto algorithm

2024-10-03 Thread Gowrishankar Muthukrishnan
Add support for asymmetric EDDSA in cryptodev, as referenced in RFC: https://datatracker.ietf.org/doc/html/rfc8032 Signed-off-by: Gowrishankar Muthukrishnan --- doc/guides/cryptodevs/features/default.ini | 1 + doc/guides/prog_guide/cryptodev_lib.rst| 2 +- lib/cryptodev/rte_crypto_asym.h

Re: [PATCH] net/nfp: reserve BAR for expansion ROM

2024-10-03 Thread Ferruh Yigit
On 9/3/2024 3:34 AM, Chaoyong He wrote: > For some platform, the warm restart of host doesn't trigger > the reset of NIC, which causes the initialize process of NIC > not executed with the right expansion ROM mapping. Consequently, > the PXE boot won't work in this case. > > Now reserve BAR 2.0 wh

Re: [RFC PATCH] app/testpmd: display TM parameters when adding nodes

2024-10-03 Thread Ferruh Yigit
On 8/12/2024 2:26 PM, Bruce Richardson wrote: > The commands to add TM nodes and shapers take many parameters without > any descriptive words in between to identify what parameter is what. To > help make it clear what a command is actually doing, and to help catch > any issues with parameters put i

Re: [PATCH v4 3/6] eventdev: add SW event preschedule hint

2024-10-03 Thread Jerin Jacob
On Wed, Oct 2, 2024 at 2:19 AM wrote: > > From: Pavan Nikhilesh > > Add a new eventdev API to provide a hint to the eventdev PMD to > pre-schedule the next event into the event port, without releasing > the current flow context. > Event device that support this feature advertises the capability >

Re: [PATCH v2 0/3] app/testpmd: improve sse based macswap

2024-10-03 Thread Ferruh Yigit
On 8/21/2024 3:38 PM, Vipin Varghese wrote: > Goal of the patch series is to improve SSE macswap on x86_64 by > reducing the stalls in backend engine. Original implementation of > the SSE-mac-swap makes loop call to multiple load, shuffle & store. > > Using SIMD ISA interleaving, register variable

Re: [PATCH v2 1/3] app/testpmd: add register keyword

2024-10-03 Thread Ferruh Yigit
On 9/6/2024 2:02 PM, Varghese, Vipin wrote: > [AMD Official Use Only - AMD Internal Distribution Only] > > >   >> > > >> --- a/app/test-pmd/macswap_sse.h >> > > >> +++ b/app/test-pmd/macswap_sse.h >> > > >> @@ -16,13 +16,13 @@ do_macswap(struct rte_mbuf *pkts[], uint16_t >> nb, >> > > >>    u

Re: [PATCH v4 2/6] eventdev: add event port pre-schedule modify

2024-10-03 Thread Jerin Jacob
On Tue, Oct 1, 2024 at 7:14 PM wrote: > > From: Pavan Nikhilesh > > Some event devices allow pre-schedule types to be modified at > runtime on an event port. > Add `RTE_EVENT_DEV_CAP_EVENT_PER_PORT_PRESCHEDULE` capability > to indicate that the event device supports this feature. > > Add `rte_eve

Re: [PATCH v2] app/testpmd: show output of commands read from file

2024-10-03 Thread Ferruh Yigit
On 8/22/2024 11:41 AM, Bruce Richardson wrote: > Testpmd supports the "--cmdline-file" parameter to read a set of initial > commands from a file. However, the only indication that this has been > done successfully on startup is a single-line message, no output from > the commands is seen. > > To i

Re: [PATCH] app/testpmd: show output of commands read from file

2024-10-03 Thread Ferruh Yigit
On 10/4/2024 5:55 AM, Ferruh Yigit wrote: > On 8/22/2024 11:36 AM, Bruce Richardson wrote: >> Testpmd supports the "--cmdline-file" parameter to read a set of initial >> commands from a file. However, the only indication that this has been >> done successfully on startup is a single-line message, n

Re: [PATCH] app/testpmd: show output of commands read from file

2024-10-03 Thread Ferruh Yigit
On 8/22/2024 11:36 AM, Bruce Richardson wrote: > Testpmd supports the "--cmdline-file" parameter to read a set of initial > commands from a file. However, the only indication that this has been > done successfully on startup is a single-line message, no output from > the commands is seen. > > To i

Re: [PATCH] net/gve: always attempt Rx refill on DQ

2024-10-03 Thread Ferruh Yigit
On 10/2/2024 12:45 AM, Joshua Washington wrote: > Before this patch, gve_rx_refill_dqo() is only called if the number of > packets received in a cycle is non-zero. However, in a > memory-constrained scenario, this doesn't behave well, as this could be > a potential source of lockup, if there is no

Re: [PATCH v4 1/6] eventdev: introduce event pre-scheduling

2024-10-03 Thread Jerin Jacob
On Tue, Oct 1, 2024 at 7:44 PM wrote: > > From: Pavan Nikhilesh > > Event pre-scheduling improves scheduling performance by assigning events > to event ports in advance when dequeues are issued. > The dequeue operation initiates the pre-schedule operation, which completes > in parallel without af

Re: [PATCH] net/gve: fix mbuf allocation memory leak for DQ Rx

2024-10-03 Thread Ferruh Yigit
On 10/2/2024 12:48 AM, Joshua Washington wrote: > Currently, gve_rxq_mbufs_alloc_dqo() allocates RING_SIZE buffers, but > only posts RING_SIZE - 1 of them, inevitably leaking a buffer every > time queues are stopped/started. This could eventually lead to running > out of mbufs if an application sto

Re: [PATCH 0/4] Support new card using NFP 3800 chip

2024-10-03 Thread Ferruh Yigit
On 9/3/2024 2:41 AM, Chaoyong He wrote: > This patch series add support of a new card which using the NFP 3800 chip. > Because it has different configure BAR size and flow steering rules > limit, we also do some refactor to the related logic. > > Chaoyong He (4): > net/nfp: add a new flag to ind

[PATCH] ethdev: convert string initialization

2024-10-03 Thread Ferruh Yigit
) 15.0.0 20241003 (experimental) [2] ../lib/ethdev/rte_flow.h:906:36: error: initializer-string for array of ‘unsigned char’ is too long [-Werror=unterminated-string-initialization] 906 | .hdr.dst_addr.addr_bytes = "\xff\xff\xff\xff\xf

Re: [PATCH] doc: update TAP device features

2024-10-03 Thread Ferruh Yigit
On 10/4/2024 3:26 AM, Stephen Hemminger wrote: > On Fri, 4 Oct 2024 02:48:21 +0100 > Ferruh Yigit wrote: > >> On 9/4/2024 4:42 PM, Stephen Hemminger wrote: >>> The TAP device does have per-queue stats and handles multi-process. >>> >>> Signed-off-by: Stephen Hemminger >>> --- >>> doc/guides/nic

Re: [PATCH] [RFC] cryptodev: replace LIST_END enumerators with APIs

2024-10-03 Thread Ferruh Yigit
On 9/23/2024 9:41 PM, Akhil Goyal wrote: > Hi Morten, > > Apologies for delayed response. >> Maybe a combination, returning the lowest end of the two versions of the >> list, >> would work... >> >> -- >> Common header file (rte_common.h): >> ---

Re: [EXTERNAL] Re: [PATCH] [RFC] cryptodev: replace LIST_END enumerators with APIs

2024-10-03 Thread Ferruh Yigit
On 9/6/2024 8:45 AM, Akhil Goyal wrote: >>> >>> Here's an idea... >>> >>> We can introduce a generic design pattern where we keep the _LIST_END enum >> value at the end, somehow marking it private (and not part of the API/ABI), >> and >> move the _list_end() function inside the C file, so it uses

Re: [PATCH] [RFC] cryptodev: replace LIST_END enumerators with APIs

2024-10-03 Thread Ferruh Yigit
On 9/5/2024 11:14 AM, Akhil Goyal wrote: > Replace *_LIST_END enumerators from asymmetric crypto > lib to avoid ABI breakage for every new addition in > enums with inline APIs. > > Signed-off-by: Akhil Goyal > --- > This patch was discussed in ML long time back. > https://patches.dpdk.org/project

Re: [v2 2/2] examples: move vendor specific apps to drivers sub directory

2024-10-03 Thread Stephen Hemminger
On Wed, 5 Jul 2023 15:26:26 +0530 Hemant Agrawal wrote: > As approved by DPDK technical board on 2021-03-24 NTB > raw driver example application will be moved to > example/rawdev/ to avoid PMD specific example application > to show up in examples directory. > > Signed-off-by: Hemant Agrawal A

Re: [v2 1/2] doc: add policy for adding vendor PMD specific examples

2024-10-03 Thread Stephen Hemminger
On Wed, 5 Jul 2023 15:26:25 +0530 Hemant Agrawal wrote: > This patch update the dpdk sample app policy to support > vendor PMD specific test examples in the respective drivers > sub-directory in examples. > > Signed-off-by: Hemant Agrawal > Acked-by: Jerin Jacob Acked-by: Stephen Hemminger

Re: [PATCH v5] bitmap: add scan from offset function

2024-10-03 Thread Stephen Hemminger
On Mon, 3 Jul 2023 14:39:08 +0200 Volodymyr Fialko wrote: > +/** > + * Bitmap initialize internal scan pointers at the given position for the > scan function. > + * > + * Note: for private/internal use, for public: > + * @see rte_bitmap_scan_from_offset() > + * > + * @param bmp > + * Handle to

[PATCH v3] app/proc-info: add rte_eal_cleanup() to avoid memory leak

2024-10-03 Thread Stephen Hemminger
From: Fidaullah Noonari when app is launched with -m proc-info exit without rte_eal_cleanup() causing memory leakage. This commit resolves the memory leakage issue and closes app properly. Bugzilla id: 898 Signed-off-by: Fidaullah Noonari Acked-by: Stephen Hemminger --- v3 - handle eventdev_xs

Re: [PATCH v2 2/2] usertools/telemetry: add new telemetry client

2024-10-03 Thread Stephen Hemminger
On Wed, 31 Aug 2022 12:52:50 +0100 Conor Walsh wrote: > A lot of DPDK apps do not provide feedback to their users while > running, but many interesting metrics are available through the > app's telemetry socket. Currently, the main way to view these > metrics is to query them using the dpdk-telem

Re: [PATCH] doc: update TAP device features

2024-10-03 Thread Stephen Hemminger
On Fri, 4 Oct 2024 02:48:21 +0100 Ferruh Yigit wrote: > On 9/4/2024 4:42 PM, Stephen Hemminger wrote: > > The TAP device does have per-queue stats and handles multi-process. > > > > Signed-off-by: Stephen Hemminger > > --- > > doc/guides/nics/features/tap.ini | 2 ++ > > 1 file changed, 2 inse

Re: [PATCH v4] vhost: support CPU copy for small packets

2024-10-03 Thread Stephen Hemminger
On Mon, 29 Aug 2022 08:56:58 +0800 Wenwu Ma wrote: > Offloading small packets to DMA degrades throughput 10%~20%, > and this is because DMA offloading is not free and DMA is not > good at processing small packets. In addition, control plane > packets are usually small, and assign those packets to

Re: [PATCH] doc: update TAP device features

2024-10-03 Thread Ferruh Yigit
On 9/4/2024 4:42 PM, Stephen Hemminger wrote: > The TAP device does have per-queue stats and handles multi-process. > > Signed-off-by: Stephen Hemminger > --- > doc/guides/nics/features/tap.ini | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/doc/guides/nics/features/tap.ini > b/doc/

[PATCH] net/gve: add IO memory barriers before reading descriptors

2024-10-03 Thread Joshua Washington
Without memory barriers, there is no guarantee that the CPU will actually wait until after the descriptor has been fully written before loading descriptor data. In this case, it is possible that stale data is read and acted on by the driver when processing TX or RX completions. This change adds re

[PATCH] net/gve: fix refill logic causing memory corruption

2024-10-03 Thread Joshua Washington
There is a seemingly mundane error in the RX refill path which can lead to major issues and ultimately program crashing. This error occurs as part of an edge case where the exact number of buffers the refill causes the ring to wrap around to 0. The current refill logic is split into two conditions

Re: [dpdk-dev v1] doc/multi-process: fixed grammar and rephrasing

2024-10-03 Thread Stephen Hemminger
On Wed, 1 Jun 2022 17:57:19 +0800 Kai Ji wrote: > Update and rephrasing some sentences, small improvements > made to the multi-process sample application user guide > > Fixes: d0dff9ba445e ("doc: sample application user guide") > Cc: bernard.iremon...@intel.com > > Signed-off-by: Kai Ji It's

Re: [PATCH dpdk v2 01/16] net: split raw checksum functions in separate header

2024-10-03 Thread Stephen Hemminger
On Tue, 1 Oct 2024 10:17:13 +0200 Robin Jarry wrote: > The checksum functions are used by both ipv4 and ipv6 functions. In > preparation of moving ipv6 symbols to a new header, move the checksum > related symbols to another dedicated header. > > Signed-off-by: Robin Jarry > --- Acked-by: Step

Re: [PATCH dpdk v2 03/16] net: add structure for ipv6 addresses

2024-10-03 Thread Stephen Hemminger
On Tue, 1 Oct 2024 10:17:15 +0200 Robin Jarry wrote: > There is currently no structure defined for IPv6 addresses. Introduce > one that is simply a uint8_t array of 16 elements without any union. The > idea is to ensure this structure alignment is 1 so that it can be mapped > directly on unalign

Re: [PATCH dpdk v2 04/16] net: use ipv6 structure for header addresses

2024-10-03 Thread Stephen Hemminger
On Tue, 1 Oct 2024 10:17:16 +0200 Robin Jarry wrote: > diff --git a/app/test/packet_burst_generator.c > b/app/test/packet_burst_generator.c > index 2cd34abc1a65..76a409a56452 100644 > --- a/app/test/packet_burst_generator.c > +++ b/app/test/packet_burst_generator.c > @@ -149,8 +149,8 @@ initial

Re: [PATCH dpdk v2 05/16] fib6,rib6,lpm6: use ipv6 addr struct

2024-10-03 Thread Stephen Hemminger
On Tue, 1 Oct 2024 10:17:17 +0200 Robin Jarry wrote: > Replace ad-hoc uint8_t[16] array types in the API of rte_fib6, rte_rib6 > and rte_lpm6. Update all code accordingly. > > Unfortunately, these three libraries are strongly intertwined, it is not > possible to split this change in separate co

Re: [PATCH dpdk v2 09/16] pipeline: use ipv6 addr struct

2024-10-03 Thread Stephen Hemminger
On Tue, 1 Oct 2024 10:17:21 +0200 Robin Jarry wrote: > diff --git a/examples/ip_pipeline/cli.c b/examples/ip_pipeline/cli.c > index e8269ea90c11..66fdeac8f501 100644 > --- a/examples/ip_pipeline/cli.c > +++ b/examples/ip_pipeline/cli.c > @@ -3540,8 +3540,8 @@ parse_table_action_encap(char **toke

Re: [PATCH dpdk v2 06/16] net: add ipv6 address utilities

2024-10-03 Thread Stephen Hemminger
On Tue, 1 Oct 2024 10:17:18 +0200 Robin Jarry wrote: > +/** > + * Copy an IPv6 address into another one. > + * > + * @param dst > + * The address into which to copy data. > + * @param src > + * The address from which to copy. > + */ > +static inline void > +rte_ipv6_addr_cpy(struct rte_ipv6_

Re: [PATCH] Add bus master enable/disable APIs for CDX devices

2024-10-03 Thread Ferruh Yigit
On 10/3/2024 11:16 PM, Stephen Hemminger wrote: > On Fri, 27 Oct 2023 17:22:11 +0100 > Ferruh Yigit wrote: > >> From: Shubham Rohila >> >> Define rte_cdx_vfio_bm_enable and rte_cdx_vfio_bm_disable to >> enable or disable bus master functionality for cdx devices. >> >> Signed-off-by: Shubham Rohi

Re: [PATCH] drivers: remove compile-time option for IEEE 1588

2024-10-03 Thread Ferruh Yigit
On 10/3/2024 8:12 PM, Stephen Hemminger wrote: > On Wed, 17 Apr 2024 14:24:00 +0100 > Ferruh Yigit wrote: > >>> :) remove compile flag in 24.11 LTS. >>> >>> But we may end up in same situation in 24.11, some drivers not being >>> ready, so we should merge this patch very early in the 24.11 to giv

[PATCH] app/dumpcap: fix handling of jumbo frames

2024-10-03 Thread Stephen Hemminger
If dumpcap (in legacy pcap mode) tried to handle a large segmented frame it would core dump because rte_pktmbuf_read() would return NULL. Fix by using same logic as in pcap PMD. Fixes: cbb44143be74 ("app/dumpcap: add new packet capture application") Reported-by: Tianli Lai Signed-off-by: Stephen

RE: [RFC v2 0/2] Add high-performance timer facility

2024-10-03 Thread Morten Brørup
> From: Stephen Hemminger [mailto:step...@networkplumber.org] > Sent: Thursday, 3 October 2024 20.37 > > On Wed, 15 Mar 2023 18:03:40 +0100 > Mattias Rönnblom wrote: > > > This patchset is an attempt to introduce a high-performance, highly > > scalable timer facility into DPDK. > > > > More spec

[PATCH v2 10/10] baseband/acc: cosmetic changes

2024-10-03 Thread Hernan Vargas
Cosmetic code changes. No functional impact. Signed-off-by: Hernan Vargas --- drivers/baseband/acc/rte_acc100_pmd.c | 2 +- drivers/baseband/acc/rte_vrb_pmd.c| 62 +++ 2 files changed, 44 insertions(+), 20 deletions(-) diff --git a/drivers/baseband/acc/rte_acc100_pm

[PATCH v2 08/10] baseband/acc: remove check on HARQ memory

2024-10-03 Thread Hernan Vargas
Automatically reset HARQ memory to prevent errors and simplify usage. In a way we can assume that the HARQ output operation will always overwrite the buffer, so we can reset this from the driver to prevent an error being reported when application fails to do this explicitly. Signed-off-by: Hernan

[PATCH v2 09/10] baseband/acc: reset ring data valid bit

2024-10-03 Thread Hernan Vargas
Reset only the valid bit to keep info ring data notably for dumping. Signed-off-by: Hernan Vargas --- drivers/baseband/acc/rte_vrb_pmd.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/baseband/acc/rte_vrb_pmd.c b/drivers/baseband/acc/rte_vrb_pmd.c index 27c8bdca3d08

[PATCH v2 07/10] baseband/acc: algorithm tuning for LDPC decoder

2024-10-03 Thread Hernan Vargas
Reverting to MS1 version of the algorithm to improve MU1 fading conditions. Signed-off-by: Hernan Vargas --- drivers/baseband/acc/rte_vrb_pmd.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/baseband/acc/rte_vrb_pmd.c b/drivers/baseband/acc/rte_vrb_pmd.c index

[PATCH v2 05/10] baseband/acc: enhance SW ring alignment

2024-10-03 Thread Hernan Vargas
Calculate the aligned total size required for queue rings, ensuring that the size is a power of two for proper memory allocation. Signed-off-by: Hernan Vargas --- drivers/baseband/acc/acc_common.h | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/drivers/baseband/acc/acc

[PATCH v2 06/10] baseband/acc: remove soft output bypass

2024-10-03 Thread Hernan Vargas
Removing soft output bypass capability due to device limitations. Signed-off-by: Hernan Vargas --- drivers/baseband/acc/rte_vrb_pmd.c | 7 +++ 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/drivers/baseband/acc/rte_vrb_pmd.c b/drivers/baseband/acc/rte_vrb_pmd.c index 26335d55

[PATCH v2 04/10] baseband/acc: future proof structure comparison

2024-10-03 Thread Hernan Vargas
Some implementation in the PMD is based on some size assumption from the bbdev structure, which should use sizeof instead to be more future proof in case these structures change. Signed-off-by: Hernan Vargas --- drivers/baseband/acc/acc_common.h | 4 ++-- 1 file changed, 2 insertions(+), 2 delet

[PATCH v2 03/10] baseband/acc: configure max queues per device

2024-10-03 Thread Hernan Vargas
Configure max_queues based on the number of queue groups and numbers of AQS per device variant. Signed-off-by: Hernan Vargas --- drivers/baseband/acc/rte_vrb_pmd.c | 9 +++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/drivers/baseband/acc/rte_vrb_pmd.c b/drivers/baseband/

[PATCH v2 02/10] baseband/acc: queue allocation refactor

2024-10-03 Thread Hernan Vargas
Refactor to manage queue memory per operation more flexibly for VRB devices. Signed-off-by: Hernan Vargas --- drivers/baseband/acc/acc_common.h | 5 + drivers/baseband/acc/rte_vrb_pmd.c | 214 - 2 files changed, 157 insertions(+), 62 deletions(-) diff --git a/driv

[PATCH v2 01/10] baseband/acc: fix access to deallocated mem

2024-10-03 Thread Hernan Vargas
Prevent op_addr access during queue_stop operation, as this memory may have been deallocated. Fixes: e640f6cdfa84 ("baseband/acc200: add LDPC processing") Cc: sta...@dpdk.org Signed-off-by: Hernan Vargas Reviewed-by: Maxime Coquelin --- drivers/baseband/acc/rte_acc100_pmd.c | 36 --

[PATCH v2 00/10] acc baseband PMD fix and updates for 24.11

2024-10-03 Thread Hernan Vargas
This series targets 24.11. v2: Rebased to the latest next-baseband-for-main which includes needed rte_bbdev lib updates. v1: It includes a memory access fix, refactoring of queue allocation and general improvements. Hernan Vargas (10): baseband/acc: fix access to deallocated mem baseband/acc

[PATCH v14 3/3] event/dsw: add capability for independent enqueue

2024-10-03 Thread Abdullah Sevincer
To use independent enqueue capability applications need to set flag RTE_EVENT_PORT_CFG_INDEPENDENT_ENQ during port setup only if the capability RTE_EVENT_DEV_CAP_INDEPENDENT_ENQ exists. Hence, this commit adds the capability of independent enqueue to the DSW driver. Signed-off-by: Abdullah Sevince

[PATCH v14 2/3] event/dlb2: add support for independent enqueue

2024-10-03 Thread Abdullah Sevincer
DLB devices need events to be enqueued in the same order they are dequeued. Applications are not suppose to change event order between dequeue and to enqueue. Since Eventdev standard does not add such restrictions independent enqueue support is needed for DLB PMD so that it restores dequeue order o

[PATCH v14 0/3] Independent Enqueue Support

2024-10-03 Thread Abdullah Sevincer
v14: Address comments. v13: Address comments. v12: Address comments. v11: Address comments. v10: Add acked-by reviewer name. v9: Address comments. v8: Address build issues. v7: Address documentation reviews. v6: Update patch with more documentation. v5: Address build issues. v4: Address comments. v

[PATCH v14 1/3] eventdev: add support for independent enqueue

2024-10-03 Thread Abdullah Sevincer
This commit adds support for independent enqueue feature and updates Event Device and PMD feature list. A new capability RTE_EVENT_DEV_CAP_INDEPENDENT_ENQ is introduced. It allows out-of-order enqueuing of RTE_EVENT_OP_FORWARD or RELEASE type events on an event port where this capability is enable

[PATCH v13 3/3] event/dsw: add capability for independent enqueue

2024-10-03 Thread Abdullah Sevincer
To use independent enqueue capability applications need to set flag RTE_EVENT_PORT_CFG_INDEPENDENT_ENQ during port setup only if the capability RTE_EVENT_DEV_CAP_INDEPENDENT_ENQ exists. Hence, this commit adds the capability of independent enqueue to the DSW driver. Signed-off-by: Abdullah Sevince

[PATCH v13 2/3] event/dlb2: add support for independent enqueue

2024-10-03 Thread Abdullah Sevincer
DLB devices need events to be enqueued in the same order they are dequeued. Applications are not suppose to change event order between dequeue and to enqueue. Since Eventdev standard does not add such restrictions independent enqueue support is needed for DLB PMD so that it restores dequeue order o

[PATCH v13 1/3] eventdev: add support for independent enqueue

2024-10-03 Thread Abdullah Sevincer
This commit adds support for independent enqueue feature and updates Event Device and PMD feature list. A new capability RTE_EVENT_DEV_CAP_INDEPENDENT_ENQ is introduced. It allows out-of-order enqueuing of RTE_EVENT_OP_FORWARD or RELEASE type events on an event port where this capability is enable

[PATCH v13 0/3] Independent Enqueue Support

2024-10-03 Thread Abdullah Sevincer
v13: Address comments. v12: Address comments. v11: Address comments. v10: Add acked-by reviewer name. v9: Address comments. v8: Address build issues. v7: Address documentation reviews. v6: Update patch with more documentation. v5: Address build issues. v4: Address comments. v3: Fix CI/build issues.

[PATCH v2] drivers: ignore non-critical syndromes for Tx queues

2024-10-03 Thread Alexander Kozyrev
Only 3 syndromes are considered critical and warrant a queue restart. All other syndromes can be safely ignored. We ignore them for Rx queues. Skip non-critical error CQEs for Tx queues as well. Fixes: 957e45fb7b ("net/mlx5: handle Tx completion with error") Cc: sta...@dpdk.org Signed-off-by: Ale

[PATCH v2] drivers: fix error CQE handling

2024-10-03 Thread Alexander Kozyrev
The completion queue element size can be independently configured to report either 64 or 128 bytes CQEs by programming cqe_sz parameter at CQ creation. This parameter depends on the cache line size and affects both regular CQEs and error CQEs. But the error handling assumes that an error CQE is 64

Re: [PATCH] drivers: fix error CQE handling

2024-10-03 Thread Stephen Hemminger
On Tue, 2 May 2023 21:49:22 +0300 Alexander Kozyrev wrote: > The completion queue element size can be independently configured > to report either 64 or 128 bytes CQEs by programming cqe_sz parameter > at CQ creation. This parameter depends on the cache line size and > affects both regular CQEs an

Re: [PATCH] drivers: remove compile-time option for IEEE 1588

2024-10-03 Thread Stephen Hemminger
On Wed, 17 Apr 2024 14:24:00 +0100 Ferruh Yigit wrote: > > :) remove compile flag in 24.11 LTS. > > > > But we may end up in same situation in 24.11, some drivers not being > > ready, so we should merge this patch very early in the 24.11 to give > > time to drivers to fix if they were not ready

[PATCH v4 4/6] test/crypto: add asymmetric EDDSA test cases

2024-10-03 Thread Gowrishankar Muthukrishnan
Add test cases to validate EDDSA sign and verify ops, as per RFC 8032. Signed-off-by: Gowrishankar Muthukrishnan --- app/test/test_cryptodev_asym.c | 345 +++- app/test/test_cryptodev_ecdh_test_vectors.h | 94 +++- app/test/test_cryptodev_ecdsa_test_vectors.h | 4 + app

[PATCH v4 1/6] cryptodev: add EDDSA asymmetric crypto algorithm

2024-10-03 Thread Gowrishankar Muthukrishnan
Add support for asymmetric EDDSA in cryptodev, as referenced in RFC: https://datatracker.ietf.org/doc/html/rfc8032 Signed-off-by: Gowrishankar Muthukrishnan --- doc/guides/cryptodevs/features/default.ini | 1 + doc/guides/prog_guide/cryptodev_lib.rst| 2 +- lib/cryptodev/rte_crypto_asym.h

[PATCH v4 2/6] crypto/openssl: support EDDSA

2024-10-03 Thread Gowrishankar Muthukrishnan
Support EDDSA crypto algorithm in OpenSSL PMD. Signed-off-by: Gowrishankar Muthukrishnan --- v4: - Restrict eddsa session creation only for openssl 3.3.0 and above. Lower v3.x may support pure instances, but not context and prehash instances. --- doc/guides/cryptodevs/features/openssl

[PATCH v4 5/6] examples/fips_validation: support EDDSA

2024-10-03 Thread Gowrishankar Muthukrishnan
Add EDDSA support in fips_validation app. Signed-off-by: Gowrishankar Muthukrishnan --- examples/fips_validation/fips_validation.c| 2 + examples/fips_validation/fips_validation.h| 23 ++ .../fips_validation/fips_validation_eddsa.c | 307 + examples/fips_validation/m

[PATCH v4 6/6] app/crypto-perf: support EDDSA

2024-10-03 Thread Gowrishankar Muthukrishnan
Added support for EDDSA 25519 curve SIGN and VERIFY operations. Signed-off-by: Gowrishankar Muthukrishnan --- app/test-crypto-perf/cperf_ops.c | 52 app/test-crypto-perf/cperf_options.h | 2 + app/test-crypto-perf/cperf_options_parsing.c | 9 +++- app/t

[PATCH v4 3/6] crypto/cnxk: support EDDSA

2024-10-03 Thread Gowrishankar Muthukrishnan
Support EDDSA crypto algorithm in CNXK PMD. Signed-off-by: Gowrishankar Muthukrishnan --- doc/guides/cryptodevs/features/cn10k.ini | 1 + drivers/common/cnxk/hw/cpt.h | 3 +- drivers/common/cnxk/roc_ae.c | 52 +- drivers/common/cnxk/roc_ae.h

[PATCH v3 3/3] crypto/ipsec_mb: add SM4 algorithm support

2024-10-03 Thread Brian Dooley
This patch introduces SM4 CBC, SM4 ECB and SM4 CTR algorithm support to the AESNI_MB PMD. SM4 CTR is available in the v2.0 release of Intel IPsec MB. Signed-off-by: Brian Dooley --- v2: Add Intel IPsec MB version check to capabilities Add SM4 CTR algorithm support v3: Fix

[PATCH v3 2/3] crypto/ipsec_mb: add HMAC SM3 algorithm support

2024-10-03 Thread Brian Dooley
This patch introduces HMAC SM3 algorithm support to the AESNI_MB PMD. Signed-off-by: Brian Dooley --- v3: Add documentation --- doc/guides/cryptodevs/aesni_mb.rst | 1 + doc/guides/cryptodevs/features/aesni_mb.ini | 1 + doc/guides/rel_notes/release_24_11.rst | 1 + driv

[PATCH v3 1/3] crypto/ipsec_mb: add SM3 algorithm support

2024-10-03 Thread Brian Dooley
This patch introduces SM3 algorithm support to the AESNI_MB PMD. Signed-off-by: Brian Dooley --- v3: Add ipsec mb version check Add documentation --- doc/guides/cryptodevs/aesni_mb.rst | 1 + doc/guides/cryptodevs/features/aesni_mb.ini | 1 + doc/guides/rel_notes/relea

Re: [PATCH v12 1/7] buildtools/chkincs: relax C linkage requirement

2024-10-03 Thread Robin Jarry
Mattias Rönnblom, Sep 20, 2024 at 12:47: Relax chkincs requirement of all DPDK header files having to contain 'extern "C"'. Instructing a C++ toolchain to use C linkage is only necessarily if the header file declares symbols (i.e., functions or global variables). With this change, chkincs tries

[PATCH v1 1/1] dma/cnxk: support dma queue priority configuration

2024-10-03 Thread Vamsi Krishna
From: Vamsi Attunuru Allow configuration of DPI DMA queue priority through mailbox request. Signed-off-by: Vamsi Attunuru --- Depends-on: patch-144960 ("dmadev: support priority configuration") v1 changes: * Addressed v0 comments. drivers/common/cnxk/roc_dpi.c | 2 ++ drivers/common/cnx

Re: [dpdk-dev] [PATCH 10/10] net/bonding: fix configuration assignment overflow

2024-10-03 Thread Stephen Hemminger
On Mon, 19 Apr 2021 21:34:49 +0800 "Min Hu (Connor)" wrote: > From: Chengchang Tang > > The expression may cause an overflow. > > This patch fix the codeDEX static check warning "INTEGER_OVERFLOW". > > Fixes: 46fb43683679 ("bond: add mode 4") > Cc: sta...@dpdk.org > > Signed-off-by: Chengcha

Re: [PATCH v2] app/test:subprocess synchronization of parameters

2024-10-03 Thread Stephen Hemminger
On Wed, 26 Jul 2023 10:39:47 +0800 Kaisen You wrote: > In meson_test, because the child process does not synchronize > the NIC startup parameters of the parent process at startup, > it uses all NICs bound by vfio as startup parameters by default, > and an exception occurs in the subsequent hugefi

Re: [PATCH] eventdev: replace zero-length-array

2024-10-03 Thread Jerin Jacob
On Wed, Oct 2, 2024 at 3:39 AM Stephen Hemminger wrote: > > Zero Length Array's (ZLA) are a Gcc extension, and standard > Variable Length Array (VLA) should be used instead. > > Signed-off-by: Stephen Hemminger Isn't this patch same https://patches.dpdk.org/project/dpdk/patch/20240725171609.142

Re: [PATCH v3 00/18] add Marvell cn20k SOC support for mempool and net

2024-10-03 Thread Jerin Jacob
On Wed, Oct 2, 2024 at 4:49 AM Nithin Dabilpuram wrote: > > This series adds support for Marvell cn20k SOC for mempool and > net PMD's. > > This series also adds few net/cnxk PMD updates to expose IPsec > features supported by HW that are very custom in nature and > some enhancements for cn10k. >

RE: [EXTERNAL] Re: [PATCH v0 1/1] dma/cnxk: support dma queue priority configuration

2024-10-03 Thread Vamsi Krishna Attunuru
>-Original Message- >From: Jerin Jacob >Sent: Thursday, October 3, 2024 7:26 PM >To: Vamsi Krishna Attunuru >Cc: fengcheng...@huawei.com; dev@dpdk.org; tho...@monjalon.net; >kevin.la...@intel.com; bruce.richard...@intel.com; Jerin Jacob > >Subject: [EXTERNAL] Re: [PATCH v0 1/1] dma/cnx

Re: [PATCH v2] service: fix deadlock on worker lcore exit

2024-10-03 Thread Van Haaren, Harry
> From: David Marchand > Sent: Thursday, October 3, 2024 10:13 AM > To: Mattias Rönnblom ; Van Haaren, Harry > > Cc: dev@dpdk.org ; step...@networkplumber.org > ; suanmi...@nvidia.com ; > tho...@monjalon.net ; sta...@dpdk.org ; > Tyler Retzlaff ; Aaron Conole > > Subject: Re: [PATCH v2] ser

RE: [EXTERNAL] [PATCH v4 1/1] dmadev: support priority configuration

2024-10-03 Thread Vamsi Krishna Attunuru
>-Original Message- >From: Anoob Joseph >Sent: Thursday, October 3, 2024 7:59 PM >To: Vamsi Krishna Attunuru ; >tho...@monjalon.net; fengcheng...@huawei.com; >bruce.richard...@intel.com; m...@smartsharesystems.com >Cc: dev@dpdk.org; kevin.la...@intel.com; Jerin Jacob ; >conor.wa...@intel

RE: [EXTERNAL] [PATCH v2] cryptodev: add ec points to sm2 op

2024-10-03 Thread Akhil Goyal
> In the case when PMD cannot support the full process of the SM2, > but elliptic curve computation only, additional fields > are needed to handle such a case. > > Points C1, kP therefore were added to the SM2 crypto operation struct. > > Signed-off-by: Arkadiusz Kusztal > --- > lib/cryptodev/r

RE: [EXTERNAL] [PATCH v4 1/1] dmadev: support priority configuration

2024-10-03 Thread Anoob Joseph
Hi Vamsi, Please see below. Thanks, Anoob > -Original Message- > From: Vamsi Krishna Attunuru > Sent: Thursday, October 3, 2024 6:46 PM > To: Anoob Joseph ; tho...@monjalon.net; > fengcheng...@huawei.com; bruce.richard...@intel.com; > m...@smartsharesystems.com > Cc: dev@dpdk.org; kevin

Re: [PATCH v5 2/2] timer: allow platform to override cpu TSC frequency

2024-10-03 Thread Bruce Richardson
On Thu, Oct 03, 2024 at 03:26:04PM +0300, Isaac Boukris wrote: > The CPU provided value is often not accurate, allow overriding it > based on info from the host OS. > > On Linux X86, if the tsc_known_freq cpu flag is missing, it means > the kernel doesn't trust it and calculates its own. We should

Re: [PATCH v5 1/2] timer: lower rounding of TSC estimation to 100KHz

2024-10-03 Thread Bruce Richardson
On Thu, Oct 03, 2024 at 03:26:03PM +0300, Isaac Boukris wrote: > In practice, the frequency is often not a nice round number, while > the estimation results are rather accurate, just a couple of KHz > away from the kernel's tsc_khz value, so it should suffice. > > Rounding to 10MHz can cause a sig

Re: [PATCH v0 1/1] dma/cnxk: support dma queue priority configuration

2024-10-03 Thread Jerin Jacob
On Thu, Oct 3, 2024 at 7:19 PM Vamsi Krishna wrote: > > From: Vamsi Attunuru > > Allow configuration of dpi dma queue priority through dpi > dev open mbox request. dpi-> DPI dma -> DMA dpi dev open mbox request-> mailbox request. > > Signed-off-by: Vamsi Attunuru > --- > Depends-on: patch-144

[PATCH v3 2/2] test/ipsec: add unit test for stateless processing

2024-10-03 Thread Aakash Sasidharan
Add unit test for IPsec stateless processing. Signed-off-by: Aakash Sasidharan --- app/test/test_ipsec.c | 111 +- 1 file changed, 98 insertions(+), 13 deletions(-) diff --git a/app/test/test_ipsec.c b/app/test/test_ipsec.c index 6cb1bac1e7..f65b3eedc5 10

[PATCH v3 1/2] ipsec: allow stateless IPsec processing

2024-10-03 Thread Aakash Sasidharan
Introduce stateless packet preparation API for IPsec processing. The new API would allow preparation of IPsec packets without altering the internal state of an IPsec session. For outbound IPsec processing, the change enables user to provide sequence number to be used for the IPsec operation. Sign

[PATCH v0 1/1] dma/cnxk: support dma queue priority configuration

2024-10-03 Thread Vamsi Krishna
From: Vamsi Attunuru Allow configuration of dpi dma queue priority through dpi dev open mbox request. Signed-off-by: Vamsi Attunuru --- Depends-on: patch-144960 ("dmadev: support priority configuration") drivers/common/cnxk/roc_dpi.c | 2 ++ drivers/common/cnxk/roc_dpi.h | 1 + driv

  1   2   >