[PATCH] doc: add sunset clause for experimental tag

The experimental tag is intended as temporary to avoid having all of DPDK APIs as experimental and thereby rendering the stable policy as worthless. Add some wording into the existing policy in doc. Signed-off-by: Stephen Hemminger --- doc/guides/contributing/abi_policy.rst | 10 ++ 1 f

Re: [PATCH] doc: add sunset clause for experimental tag

On Wed, Nov 29, 2023 at 07:27:38AM -0800, Stephen Hemminger wrote: > The experimental tag is intended as temporary to avoid having > all of DPDK APIs as experimental and thereby rendering the > stable policy as worthless. > > Add some wording into the existing policy in doc. > > Signed-off-by: St

Re: [PATCH] eal: introduce missing rte_thread wrappers

On Tue, 28 Nov 2023 09:54:02 +0100 David Vodak wrote: > On 11/27/23 18:27, Tyler Retzlaff wrote: > > > On Mon, Nov 27, 2023 at 10:25:02AM +0100, David Vodak wrote: > >> Function rte_ctrl_thread_create has been replaced by > >> rte_thread_create_control, > >> encouraging Linux users to switch

[PATCH 24.03 v2 0/5] add new command line argument parsing library

A recent thread on the mailing list[1] discussed corelist and coremask parsing and the idea of a new library dedicated to command line parsing was mentioned[2]. This patchset adds the library, along with the new APIs, edits to EAL functions to call the API instead of implementing their own, and cha

[PATCH 24.03 v2 2/5] arg_parser: add new coremask parsing API

Add new coremask parsing API. This API behaves similarly to the corelist parsing API, parsing the coremask string, filling its values into the cores array. The API also returns a 'count' which corresponds to the total number of cores in the coremask string. Signed-off-by: Euan Bourke --- lib/ar

[PATCH 24.03 v2 3/5] eal: add support for new arg parsing library

Update to EAL functions relating to corelist and coremask parsing to support the new arg parsing library. Functions now call the API instead of implementing their own version. Signed-off-by: Euan Bourke --- lib/eal/common/eal_common_options.c | 114 lib/eal/meson.bui

[PATCH 24.03 v2 4/5] eal: update to service core related parsers

Updates to the parse service cores functions in EAL to call the arg parser API instead of implementing its own versions. Signed-off-by: Euan Bourke --- lib/eal/common/eal_common_options.c | 177 1 file changed, 47 insertions(+), 130 deletions(-) diff --git a/lib/eal

[PATCH 24.03 v2 1/5] arg_parser: new library for command line parsing

Add a new library to make it easier for eal and other libraries to parse command line arguments. The first function in this library is one to parse a corelist string into an array of individual core ids. The function will then return the total number of cores described in the corelist. Signed-off

[PATCH 24.03 v2 5/5] event/dlb2: add new arg parsing library API support

Switched the dlb2 driver to call the new arg parsing library instead of eal for coremask parsing, and updated the resource probe function to support the changed formatting of the API. Signed-off-by: Euan Bourke --- drivers/event/dlb2/dlb2_priv.h | 4 +- drivers/event/dlb2/pf/base/dl

[PATCH v1 1/3] cryptodev: add EDDSA asymmetric crypto algorithm

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 v1 2/3] crypto/openssl: add EDDSA support

Add EDDSA support in OpenSSL PMD. Signed-off-by: Gowrishankar Muthukrishnan --- For openssl library support, refer: https://github.com/openssl/openssl/commit/4f8b7c2319523f8e83b8b2fa31127832fa092552 --- drivers/crypto/openssl/openssl_pmd_private.h | 6 + drivers/crypto/openssl/rte_openssl_pmd.

[PATCH v1 3/3] test/crypto: add asymmetric EDDSA test cases

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 | 339 +++ app/test/test_cryptodev_eddsa_test_vectors.h | 330 ++ 2 files changed, 669 insertions(+)

[PATCH] version: 24.03-rc0

Start a new release cycle with empty release notes. Bump version and ABI minor. Bump libabigail from 2.1 to 2.4 and enable ABI checks. Signed-off-by: David Marchand --- .ci/linux-build.sh | 7 +- .github/workflows/build.yml| 6 +- ABI_VERSION

[PATCH v2] examples/ptp: replace terms master and slave

The IEEE 1588 standard has been updated to remove the use of master-slave terminolgy. Use the terms recommended by IEEE 1588g-2022 amendment: In place of the term “master”, use the term “timeTransmitter”. In place of the term “slave”, use the term “timeReceiver”. Signed-off-by: Stephen Hemmin

[PATCH v2] test: replace use of term segregate

Change comment based on inclusive naming recommendations. https://inclusivenaming.org/word-lists/tier-3/ Signed-off-by: Stephen Hemminger Acked-by: Tyler Retzlaff --- v2 - fix typo in commit message app/test/test_hash_readwrite_lf_perf.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

[PATCH v5 00/19] remove use of the term "sanity check"

This set of patches removes all use of the term sanity check in DPDK. It does not address base drivers which come from vendors. The term sanity-check and related formats are on the Inclusive Naming Tier 2 (Strongly Consider Replacing) word list. https://inclusivenaming.org/word-lists/tier-2/sanity

[PATCH v5 01/19] mbuf: replace term sanity check

Replace rte_mbuf_sanity_check() with rte_mbuf_verify() to match the similar macro RTE_VERIFY() in rte_debug.h The term sanity check is on the Tier 2 list of words that should be replaced. Note: the rte_mbuf_verify() function is added without the experimental tag since it is intended as direct rep

[PATCH v5 02/19] eal: replace use of sanity check in comments and messages

Sanity check is on the Tier 2 non-inclusive list. Replace or remove it. Signed-off-by: Stephen Hemminger Acked-by: Anatoly Burakov --- lib/eal/common/eal_common_memory.c | 2 +- lib/eal/common/eal_common_proc.c | 3 ++- lib/eal/common/eal_common_trace.c | 2 +- lib/eal/common/eal_memcfg.h

[PATCH v5 03/19] test: replace use word sanity

The word "sanity" is on the not-allowed inclusive naming list. Replace the unnecessary usage in tests. Signed-off-by: Stephen Hemminger --- app/test/test_bitmap.c | 4 +-- app/test/test_bpf.c| 6 ++-- app/test/test_common.c | 2 +- app/test/test_distrib

[PATCH v5 04/19] examples: remove term sanity

Do not use non-inclusive terms. Signed-off-by: Stephen Hemminger --- examples/ipsec-secgw/event_helper.c | 2 +- examples/qos_sched/args.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/examples/ipsec-secgw/event_helper.c b/examples/ipsec-secgw/event_helper.c i

[PATCH v5 05/19] lib: replace use of sanity check in comments and messages

Sanity check is on the Tier 2 non-inclusive list. Replace or remove it. Signed-off-by: Stephen Hemminger --- lib/graph/graph.c | 2 +- lib/graph/graph_stats.c | 2 +- lib/graph/node.c| 2 +- lib/jobstats/rte_jobstats.c | 6 +++--- lib/metrics/rte_metrics.c | 2 +- lib

[PATCH v5 06/19] doc/eventdev_pipeline: remove sanity

The word sanity check is unnecessary here. Signed-off-by: Stephen Hemminger --- doc/guides/gpus/cuda.rst | 2 +- doc/guides/sample_app_ug/eventdev_pipeline.rst | 2 +- doc/guides/tools/testbbdev.rst | 8 3 files changed, 6 insertions(+), 6 deletions

[PATCH v5 07/19] net/ring: replace use of sanity

Don't use term sanity check Signed-off-by: Stephen Hemminger --- drivers/net/ring/rte_eth_ring.c | 7 ++- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/drivers/net/ring/rte_eth_ring.c b/drivers/net/ring/rte_eth_ring.c index 48953dd7a059..710fd9ebe526 100644 --- a/drivers/net/

[PATCH v5 08/19] net/fm10k, net/ixgbe: remove word sanity

Remove non-inclusive term sanity from these Intel drivers. Signed-off-by: Stephen Hemminger --- drivers/net/fm10k/fm10k_ethdev.c | 2 +- drivers/net/fm10k/fm10k_rxtx.c | 2 +- drivers/net/ixgbe/ixgbe_fdir.c | 2 +- drivers/net/ixgbe/ixgbe_ipsec.c | 2 +- drivers/net/ixgbe/ixgbe_rxtx.c | 2

[PATCH v5 09/19] net/mlx[45]: remove word sanity

The term "sanity" is on non-inclusive naming list. Remove it from the Nvidia drivers. Signed-off-by: Stephen Hemminger --- drivers/common/mlx5/linux/mlx5_common_os.c | 2 +- drivers/net/mlx4/mlx4.c| 2 +- drivers/net/mlx4/mlx4_flow.c | 6 +++--- drivers/net/mlx5

[PATCH v5 10/19] net/sfc: remove term "sanity check"

Remove term sanity check in comment. Signed-off-by: Stephen Hemminger Acked-by: Andrew Rybchenko --- drivers/net/sfc/sfc_dp_rx.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/sfc/sfc_dp_rx.h b/drivers/net/sfc/sfc_dp_rx.h index 9f9bf2898881..dfb4fb98844b 100644

[PATCH v5 11/19] net/ark: replace use of term sanity

Do not use non inclusive terms. Signed-off-by: Stephen Hemminger --- drivers/net/ark/ark_ethdev.c| 8 drivers/net/ark/ark_ethdev_rx.c | 2 +- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/net/ark/ark_ethdev.c b/drivers/net/ark/ark_ethdev.c index 0ffd4b9e9e4a

[PATCH v5 12/19] net/bnxt: replace use of term sanity

Do not use non-inclusive terms. Signed-off-by: Stephen Hemminger --- drivers/net/bnxt/bnxt_hwrm.c | 2 +- drivers/net/bnxt/hsi_struct_def_dpdk.h | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/net/bnxt/bnxt_hwrm.c b/drivers/net/bnxt/bnxt_hwrm.c index 06f1

[PATCH v5 13/19] net/bnx2x: remove reference to sanity

Replace "sanity check" Signed-off-by: Stephen Hemminger --- drivers/net/bnx2x/bnx2x.c | 2 +- drivers/net/bnx2x/bnx2x_stats.c | 8 drivers/net/bnx2x/ecore_sp.c| 2 +- drivers/net/bnx2x/elink.c | 4 ++-- 4 files changed, 8 insertions(+), 8 deletions(-) diff --git a/drive

[PATCH v5 14/19] cnxk: replace term sanity

Do not use non-inclusive naming. Signed-off-by: Stephen Hemminger --- drivers/common/cnxk/roc_nix_tm_ops.c | 2 +- drivers/common/cnxk/roc_npa.c| 4 ++-- drivers/net/cnxk/cnxk_ethdev.c | 4 ++-- 3 files changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/common/cnxk/roc_

[PATCH v5 15/19] event/opdl: remove term sanity

Don't use non-inclusive term. Signed-off-by: Stephen Hemminger --- drivers/event/opdl/opdl_evdev.c | 2 +- drivers/event/opdl/opdl_evdev_init.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/event/opdl/opdl_evdev.c b/drivers/event/opdl/opdl_evdev.c index 0ccc

[PATCH v5 16/19] net/txgbe: replace term sanity

Don't use term sanity. Signed-off-by: Stephen Hemminger --- drivers/net/txgbe/txgbe_ipsec.c | 2 +- drivers/net/txgbe/txgbe_rxtx.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/net/txgbe/txgbe_ipsec.c b/drivers/net/txgbe/txgbe_ipsec.c index f9f8108fb894..cadb094

[PATCH v5 17/19] net/cxgbe: remove use of term sanity

Remove non-inclusive terminolgy. Signed-off-by: Stephen Hemminger --- drivers/net/cxgbe/cxgbe_ethdev.c | 10 ++ drivers/net/cxgbe/cxgbevf_main.c | 4 ++-- 2 files changed, 4 insertions(+), 10 deletions(-) diff --git a/drivers/net/cxgbe/cxgbe_ethdev.c b/drivers/net/cxgbe/cxgbe_ethdev.c

[PATCH v5 18/19] crypto/bcmfs: replace term sanity check

Do not use non-inclusive naming here. Signed-off-by: Stephen Hemminger --- drivers/crypto/bcmfs/hw/bcmfs4_rm.c | 6 +++--- drivers/crypto/bcmfs/hw/bcmfs5_rm.c | 6 +++--- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/crypto/bcmfs/hw/bcmfs4_rm.c b/drivers/crypto/bcmfs/hw

[PATCH v5 19/19] drivers: remove use of term sanity check

Replace non-inclusive language with better comments. Covers multiple drivers Signed-off-by: Stephen Hemminger Acked-by: Hemant Agrawal --- drivers/bus/fslmc/qbman/qbman_sys_decl.h | 2 +- drivers/common/cnxk/roc_npa.c | 2 +- drivers/common/dpaax/caamflib/desc.h | 2 +- driver

Re: [PATCH v2] eal/linux: force iova-mode va without pa available

On 11/24/23 2:09 AM, christian.ehrha...@canonical.com wrote: From: David Wilder When using --no-huge option physical address are not guaranteed to be persistent. This change effectively makes "--no-huge" the same as "--no-huge --iova-mode=va". When --no-huge is used (or any other condition

Re: [PATCH 24.03 v2 1/5] arg_parser: new library for command line parsing

On Tue, 28 Nov 2023 14:07:41 + Euan Bourke wrote: > +struct core_bits { > + uint8_t bits[(UINT16_MAX + 1)/CHAR_BIT]; > + uint16_t max_bit_set; > + uint16_t min_bit_set; > + uint32_t total_bits_set; > +}; Looks like a good candidate for flex array

Re: [PATCH 24.03 v2 1/5] arg_parser: new library for command line parsing

On Tue, 28 Nov 2023 14:07:41 + Euan Bourke wrote: > +static inline bool > +get_core_bit(struct core_bits *mask, uint16_t idx) > +{ > + return !!(mask->bits[idx / 8] & (1 << (idx % 8))); > +} > + You used CHAR_BIT above, why not here?

Re: [PATCH 24.03 v2 1/5] arg_parser: new library for command line parsing

On Tue, 28 Nov 2023 14:07:41 + Euan Bourke wrote: > +int > +rte_parse_corelist(const char *corelist, uint16_t *cores, uint32_t cores_len) > +{ > + int32_t min = -1; > + char *end = NULL; > + > + struct core_bits *mask = malloc(sizeof(struct core_bits)); > + if (mask == NULL) >

RE: [PATCH v5 19/19] drivers: remove use of term sanity check

Looks good to me, thanks. Acked-by: Chaoyong He > -Original Message- > From: Stephen Hemminger > Sent: Thursday, November 30, 2023 1:26 AM > To: dev@dpdk.org > Cc: Stephen Hemminger ; Hemant Agrawal > ; Sachin Saxena ; > Nithin Dabilpuram ; Kiran Kumar K > ; Sunil Kumar Kori ; Satha > R

RE: [PATCH v5 05/10] net/nfp: replace zero length array with flex array

Looks good to me, thanks. Acked-by: Chaoyong He > -Original Message- > From: Stephen Hemminger > Sent: Wednesday, November 29, 2023 10:39 AM > To: dev@dpdk.org > Cc: Stephen Hemminger ; Niklas Söderlund > ; Tyler Retzlaff > ; Chaoyong He > Subject: [PATCH v5 05/10] net/nfp: replace zer

RE: [EXT] [PATCH v5 19/19] drivers: remove use of term sanity check

>-Original Message- >From: Stephen Hemminger >Sent: Wednesday, November 29, 2023 10:56 PM >To: dev@dpdk.org >Cc: Stephen Hemminger ; Hemant Agrawal >; Sachin Saxena ; Nithin >Kumar Dabilpuram ; Kiran Kumar Kokkilagadda >; Sunil Kumar Kori ; Satha >Koteswara Rao Kottidi ; Andrew Rybchenk

[v21.11] net/iavf: support rte flow with mask for FDIR

Add mask support for FDIR, including eth/ipv4/ipv6/tcp/udp flow items. This patch is based on DPDK v21.11 LTS [45cef8185a4fcb3aea4279711850c16e580b9a36], for customer cherry-pick. Signed-off-by: Zhichao Zeng --- drivers/common/iavf/virtchnl.h | 40 ++- drivers/net/iavf/iavf_fdir.c | 435

Re: [PATCH v2] examples/ptp: replace terms master and slave

On Wed, Nov 29, 2023 at 10:14 PM Stephen Hemminger wrote: > > The IEEE 1588 standard has been updated to remove the use > of master-slave terminolgy. Use the terms recommended by > IEEE 1588g-2022 amendment: > > In place of the term “master”, use the term “timeTransmitter”. > In place of the t

Re: [PATCH v5 12/19] net/bnxt: replace use of term sanity

On Wed, Nov 29, 2023 at 11:01 PM Stephen Hemminger wrote: > > Do not use non-inclusive terms. > > Signed-off-by: Stephen Hemminger > --- > drivers/net/bnxt/bnxt_hwrm.c | 2 +- > drivers/net/bnxt/hsi_struct_def_dpdk.h | 2 +- > 2 files changed, 2 insertions(+), 2 deletions(-) > > diff -

Re: [PATCH v3] windows/virt2phys: fix block MDL not updated

2023-09-12 19:17 (UTC+0800), Ric Li: > +virt2phys_is_valid_block(struct virt2phys_block *block, PVOID base) Nit: "virt" would be more consistent with the rest of the code than "base". Reviewed-by: Dmitry Kozlyuk

RE: [EXT] [PATCH v5 19/19] drivers: remove use of term sanity check

> >-Original Message- > >From: Stephen Hemminger > >Sent: Wednesday, November 29, 2023 10:56 PM > >To: dev@dpdk.org > >Cc: Stephen Hemminger ; Hemant > Agrawal > >; Sachin Saxena ; > Nithin > >Kumar Dabilpuram ; Kiran Kumar Kokkilagadda > >; Sunil Kumar Kori ; > Satha > >Koteswara Rao Kott

RE: [PATCH v5 03/19] test: replace use word sanity

> -Original Message- > From: Stephen Hemminger > Sent: Wednesday, November 29, 2023 10:56 PM > To: dev@dpdk.org > Cc: Stephen Hemminger ; Cristian > Dumitrescu ; Konstantin Ananyev > ; David Hunt ; > Chengwen Feng ; Kevin Laatz > ; Bruce Richardson ; > Yipeng Wang ; Sameh Gobriel > ; Vla

RE: [EXT] Re: [PATCH v5 0/1] devtools: add tracepoint check in checkpatch

>-Original Message- >From: Thomas Monjalon >Sent: Tuesday, November 28, 2023 9:25 PM >To: Ankur Dwivedi >Cc: dev@dpdk.org; Jerin Jacob Kollanukkaran >Subject: Re: [EXT] Re: [PATCH v5 0/1] devtools: add tracepoint check in >checkpatch > >28/11/2023 15:07, Ankur Dwivedi: >> >07/03/2023

[PATCH v2] lib/ethdev: modified the definition of 'NVGRE_ENCAP'

Fix the issue of incorrect definition of 'NVGRE_ENCAP', and modified the error comments of 'rte_flow_action_nvgre_encap'. Fixes: c2beb1d469d2 ("ethdev: add missing items/actions to flow object converter") Fixes: 3850cf0c8c37 ("ethdev: add tunnel encap/decap actions") Cc: sta...@dpdk.org Signed-of

[PATCH] test/crypto: add ext mbuf test for aes-gcm aead algo

Add external mbuf test for AES GCM aead algo. Signed-off-by: Aakash Sasidharan --- app/test/test_cryptodev.c | 335 app/test/test_cryptodev_aead_test_vectors.h | 5 +- 2 files changed, 202 insertions(+), 138 deletions(-) diff --git a/app/test/test_crypto