Re: [PATCH v18 2/8] eal: add thread attributes

2022-02-04 Thread Tyler Retzlaff
On Fri, Feb 04, 2022 at 07:21:10PM +, Ananyev, Konstantin wrote: > > Implement thread attributes for: > > * thread affinity > > * thread priority > > Implement functions for managing thread attributes. > > > > Priority is represented through an enum that allows for two levels: > > - RTE_TH

[PATCH 2/2] net/iavf: count continuous DD bits for Arm in flex Rx

2022-02-04 Thread Kathleen Capella
On Arm platforms, reading of descriptors may be re-ordered causing the status of DD bits to be discontinuous. Add logic to only process continuous descriptors by checking DD bits. Fixes: b8b4c54ef9b0 ("net/iavf: support flexible Rx descriptor in normal path") Cc: sta...@dpdk.org Signed-off-by: Ka

[PATCH 1/2] net/iavf: count continuous DD bits for Arm

2022-02-04 Thread Kathleen Capella
On Arm platforms, reading of descriptors may be re-ordered causing the status of DD bits to be discontinuous. Add logic to only process continuous descriptors by checking DD bits. Fixes: 1060591eada5 ("net/avf: enable bulk allocate Rx") Cc: sta...@dpdk.org Signed-off-by: Kathleen Capella Reviewe

[PATCH 0/2] Add logic to IAVF to count continuous DD bits for Arm

2022-02-04 Thread Kathleen Capella
This patchset introduces a fix for Arm platforms to the IAVF driver that was added to the i40e driver in a previous patchset [1]. The driver determines which descriptors in the HW ring reference packets that are ready to be received by counting those descriptors whose DD bit is set to 1. On Arm,

Re: [PATCH v5 0/2] Add config file support for l3fwd

2022-02-04 Thread Stephen Hemminger
On Fri, 4 Feb 2022 19:59:03 + Sean Morrissey wrote: > This patchset introduces config file support for l3fwd > and its lookup methods LPM, FIB, and EM, similar to > that of l3fwd-acl. This allows for route rules to be > defined in configuration files and edited there instead > of in each of

RE: [PATCH v4 1/3] ethdev: introduce IP reassembly offload

2022-02-04 Thread Akhil Goyal
> Subject: [PATCH v4 1/3] ethdev: introduce IP reassembly offload > > IP Reassembly is a costly operation if it is done in software. > The operation becomes even more costlier if IP fragments are encrypted. > However, if it is offloaded to HW, it can considerably save application > cycles. > > He

[PATCH v4 3/3] security: add IPsec option for IP reassembly

2022-02-04 Thread Akhil Goyal
A new option is added in IPsec to enable and attempt reassembly of inbound packets. Signed-off-by: Akhil Goyal Change-Id: I6f66f0b5a659550976a32629130594070cb16cb1 --- devtools/libabigail.abignore | 14 ++ lib/security/rte_security.h | 12 +++- 2 files changed, 25 insertions

[PATCH v4 2/3] ethdev: add mbuf dynfield for incomplete IP reassembly

2022-02-04 Thread Akhil Goyal
Hardware IP reassembly may be incomplete for multiple reasons like reassembly timeout reached, duplicate fragments, etc. To save application cycles to process these packets again, a new mbuf dynflag is added to show that the mbuf received is not reassembled properly. Now if this dynflag is set, ap

[PATCH v4 1/3] ethdev: introduce IP reassembly offload

2022-02-04 Thread Akhil Goyal
IP Reassembly is a costly operation if it is done in software. The operation becomes even more costlier if IP fragments are encrypted. However, if it is offloaded to HW, it can considerably save application cycles. Hence, a new offload feature is exposed in eth_dev ops for devices which can attemp

[PATCH v4 0/3] ethdev: introduce IP reassembly offload

2022-02-04 Thread Akhil Goyal
As discussed in the RFC[1] sent in 21.11, a new offload is introduced in ethdev for IP reassembly. This patchset add the IP reassembly RX offload. Currently, the offload is tested along with inline IPsec processing. It can also be updated as a standalone offload without IPsec, if there are some ha

[PATCH v5 2/2] examples/l3fwd: add config file support for EM

2022-02-04 Thread Sean Morrissey
Add support to define ipv4 and ipv6 forwarding tables from reading from a config file for EM with a format similar to l3fwd-acl one. Users can now use the default hardcoded route tables or optionally config files for 'l3fwd_em'. Default config files have been provided for use with EM. Related l3f

[PATCH v5 1/2] examples/l3fwd: add config file support for LPM/FIB

2022-02-04 Thread Sean Morrissey
Add support to define ipv4 and ipv6 forwarding tables from reading from a config file for LPM and FIB, with format similar to l3fwd-acl one. Users can now use the default hardcoded route tables or optionally config files. Default config files have been provided for use with LPM and FIB. Signed-of

[PATCH v5 0/2] Add config file support for l3fwd

2022-02-04 Thread Sean Morrissey
This patchset introduces config file support for l3fwd and its lookup methods LPM, FIB, and EM, similar to that of l3fwd-acl. This allows for route rules to be defined in configuration files and edited there instead of in each of the lookup methods hardcoded route tables. V4: * Fix nondeterministi

Re: [PATCH v5 1/1] eal: add internal function to get base address

2022-02-04 Thread Dmitry Kozlyuk
2022-01-18 05:33 (UTC-0800), Srikanth Yalavarthi: > Added an internal helper to get OS-specific EAL mapping base address > > This helper can be used by the drivers to program offload / accelerator > devices, where the base address can be used as a reference address by > the accelerator to access t

Re: [PATCH v18 2/8] eal: add thread attributes

2022-02-04 Thread Ananyev, Konstantin
> Implement thread attributes for: > * thread affinity > * thread priority > Implement functions for managing thread attributes. > > Priority is represented through an enum that allows for two levels: > - RTE_THREAD_PRIORITY_NORMAL > - RTE_THREAD_PRIORITY_REALTIME_CRITICAL > > Affinit

Re: [PATCH v18 1/8] eal: add basic threading functions

2022-02-04 Thread Ananyev, Konstantin
> Use a portable, type-safe representation for the thread identifier. > Add functions for comparing thread ids and obtaining the thread id > for the current thread. > > Signed-off-by: Narcisa Vasile > --- > lib/eal/common/meson.build| 1 + > lib/eal/{unix => common}/rte_thread.c | 5

[dpdk-dev v6 10/10] crypto/qat: support out of place SG list

2022-02-04 Thread Kai Ji
This patch adds the SGL out of place support to QAT PMD Signed-off-by: Kai Ji --- drivers/crypto/qat/dev/qat_crypto_pmd_gen3.c | 28 -- drivers/crypto/qat/dev/qat_crypto_pmd_gen4.c | 14 - drivers/crypto/qat/dev/qat_sym_pmd_gen1.c| 55 +--- 3 files changed, 83 ins

[dpdk-dev v6 09/10] crypto/qat: raw dp api integration

2022-02-04 Thread Kai Ji
This patch "unifies" QAT's raw dp api implementations to share the same enqueue/dequeue methods as the crypto operation enqueue/dequeue methods. In addition, different QAT generation specific implementations are done respectively. The qat_sym_hw_dp.c is removed as no longer required. Signed-off-by

[dpdk-dev v6 08/10] crypto/qat: op burst data path rework

2022-02-04 Thread Kai Ji
This patch enable op_build_request function in qat_enqueue_op_burst, and qat_dequeue_process_response function in qat_dequeue_op_burst. The op_build_request invoked in crypto build request op is based on crypto operations setup during session init. Signed-off-by: Kai Ji --- drivers/common/qat/qa

[dpdk-dev v6 07/10] crypto/qat: unify qat asym pmd apis

2022-02-04 Thread Kai Ji
This patch removes qat_asym_pmd.c and integrates all the functions into qat_asym.c. The unified/integrated asym crypto pmd apis should make them easier to maintain. Signed-off-by: Kai Ji --- drivers/common/qat/meson.build| 2 +- drivers/crypto/qat/qat_asym.c | 180 +

[dpdk-dev v6 06/10] crypto/qat: unify qat sym pmd apis

2022-02-04 Thread Kai Ji
This patch removes qat_sym_pmd.c and integrates all the apis into qat_sym.c. The unified/integrated qat sym crypto pmd apis should make them easier to maintain. Signed-off-by: Kai Ji --- drivers/common/qat/meson.build | 4 +- drivers/common/qat/qat_device.c | 4 +- drivers/common/

[dpdk-dev v6 05/10] crypto/qat: rework asymmetric crypto build operation

2022-02-04 Thread Kai Ji
This patch reworks the asymmetric crypto data path implementation to QAT driver. The change includes separation of different QAT generations' asymmetric crypto data path implementations and shrink the device capabilities declaration code size. Signed-off-by: Kai Ji --- drivers/common/qat/qat_qp.

[dpdk-dev v6 04/10] crypto/qat: rework session APIs

2022-02-04 Thread Kai Ji
This patch introduces the set_session methods for different generations of QAT. In addition, the patch replaces 'min_qat_dev_gen_id' with 'qat_dev_gen'. Thus, the session no longer allow to be created by one generation of QAT used by another generation. Signed-off-by: Kai Ji --- drivers/crypto/q

[dpdk-dev v6 03/10] crypto/qat: qat generation specific enqueue

2022-02-04 Thread Kai Ji
This patch add in specific aead & auth build op enqueue functions for QAT Gen3 & Gen4 Signed-off-by: Kai Ji --- drivers/crypto/qat/dev/qat_crypto_pmd_gen3.c | 117 ++- drivers/crypto/qat/dev/qat_crypto_pmd_gen4.c | 34 +- 2 files changed, 149 insertions(+), 2 deletions(-) d

[dpdk-dev v6 02/10] crypto/qat: sym build op request specific implementation

2022-02-04 Thread Kai Ji
This patch adds common inline functions for QAT symmetric crypto driver to process crypto op and the build op request function pointer implementation for QAT generation 1. Signed-off-by: Kai Ji --- drivers/crypto/qat/dev/qat_crypto_pmd_gens.h | 832 ++- drivers/crypto/qat/dev/qat

[dpdk-dev v6 01/10] common/qat: define build op request and dequeue op

2022-02-04 Thread Kai Ji
This patch introduce build request op and dequeue op function pointers to qat queue pair implementation. Those two functions are used to be assigned during qat session generation based on crypto operation Signed-off-by: Kai Ji --- drivers/common/qat/qat_qp.c | 10 -- drivers/common/

[dpdk-dev v6 00/10] drivers/qat: QAT symmetric crypto datapatch rework

2022-02-04 Thread Kai Ji
This patch reworks QAT symmetric crypto datapatch implementation where each generation request building separated and the crypto operation under the raw datapath api implementation are unified. In addtion this patchset also enables QAT OOP support in raw datapath api implementation. v6: - fix of

Re: [PATCH 6/7] vhost: remove non-C++ compatible includes

2022-02-04 Thread Bruce Richardson
On Fri, Feb 04, 2022 at 05:42:08PM +, Bruce Richardson wrote: > Some of the linux header includes are explicitly noted as being > incompatible with C++. However, these headers can included by C files > directly, or by internal headers, to avoid polluting the public DPDK > headers with non-C++ s

Re: [PATCH 3/7] graph: add explicit cast to allow C++ build

2022-02-04 Thread Stephen Hemminger
On Fri, 4 Feb 2022 17:42:05 + Bruce Richardson wrote: > C++ does not have automatic casting to/from void pointers, so need > explicit cast if header is to be included in C++ code > > Signed-off-by: Bruce Richardson > --- > lib/graph/rte_graph_worker.h | 2 +- > 1 file changed, 1 insertion

[dpdk-dev v1] crypto/openssl: openssl EVP MAC routine api update

2022-02-04 Thread Kai Ji
This patch update the EVP MAC routine in crypto openssl pmd to adopt openssl 3.0 library. Signed-off-by: Kai Ji --- drivers/crypto/openssl/compat.h | 12 ++ drivers/crypto/openssl/openssl_pmd_private.h | 4 + drivers/crypto/openssl/rte_openssl_pmd.c | 129 ++-

[PATCH 7/7] buildtools/chkincs: test headers for C++ compatibility

2022-02-04 Thread Bruce Richardson
Add support for checking each of our headers for issues when included in a C++ file. Signed-off-by: Bruce Richardson --- buildtools/chkincs/main.cpp| 4 buildtools/chkincs/meson.build | 20 2 files changed, 24 insertions(+) create mode 100644 buildtools/chkincs/ma

[PATCH 6/7] vhost: remove non-C++ compatible includes

2022-02-04 Thread Bruce Richardson
Some of the linux header includes are explicitly noted as being incompatible with C++. However, these headers can included by C files directly, or by internal headers, to avoid polluting the public DPDK headers with non-C++ safe includes. Signed-off-by: Bruce Richardson --- drivers/net/vhost/rte

[PATCH 5/7] table: add explicit casts for C++ compatibility

2022-02-04 Thread Bruce Richardson
Since C++ doesn't support automatic casting from void * to other types, we need to explicitly add the casts to any header files in DPDK. Signed-off-by: Bruce Richardson --- lib/table/rte_table_hash_func.h | 32 1 file changed, 16 insertions(+), 16 deletions(-) d

[PATCH 4/7] ipsec: add explicit cast to allow C++ build

2022-02-04 Thread Bruce Richardson
C++ does not have automatic casting to/from void pointers, so need explicit cast if header is to be included in C++ code Signed-off-by: Bruce Richardson --- lib/ipsec/rte_ipsec_group.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/ipsec/rte_ipsec_group.h b/lib/ipsec

[PATCH 3/7] graph: add explicit cast to allow C++ build

2022-02-04 Thread Bruce Richardson
C++ does not have automatic casting to/from void pointers, so need explicit cast if header is to be included in C++ code Signed-off-by: Bruce Richardson --- lib/graph/rte_graph_worker.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/graph/rte_graph_worker.h b/lib/graph/r

[PATCH 2/7] eventdev: fix header build with C++

2022-02-04 Thread Bruce Richardson
The eventdev headers had issues when used from C++ * Missing closing "}" for the extern "C" block * No automatic casting to/from void * Signed-off-by: Bruce Richardson --- lib/eventdev/rte_event_timer_adapter.h | 3 +++ lib/eventdev/rte_eventdev.h| 2 +- 2 files changed, 4 insertion

[PATCH 1/7] eal: fix header build with C++

2022-02-04 Thread Bruce Richardson
C++ files could not include some headers because: * "new" is a keyword in C++, so can't be a variable name * there is no automatic casting to/from void * Signed-off-by: Bruce Richardson --- lib/eal/include/generic/rte_ticketlock.h | 14 +++--- lib/eal/include/rte_trace_point.h|

[PATCH 0/7] Verify C++ compatibility of public headers

2022-02-04 Thread Bruce Richardson
This patchset expands upon the exiting chkincs infrastructure to build a C++ test application as well as the existing C test app, which helps catch any build errors when the public DPDK headers are included in C++ code. The rest of this patchset fixes the errors caught by this additional check - mo

RE: [PATCH v5] eal: fix unaligned loads/stores in rte_memcpy_generic

2022-02-04 Thread Ananyev, Konstantin
> Calls to rte_memcpy_generic could result in unaligned loads/stores for > 1 < n < 16. This is undefined behavior according to the C standard, > and it gets flagged by the clang undefined behavior sanitizer. > > rte_memcpy_generic is called with unaligned src and dst addresses. > When 1 < n < 16

Re: [PATCH v5] eal: fix unaligned loads/stores in rte_memcpy_generic

2022-02-04 Thread Luc Pelletier
Hi, It's been several days and I haven't received any additional feedback on this patch (and the other patch related to this one: http://mails.dpdk.org/archives/dev/2022-January/232491.html). I would welcome any kind of feedback. Ideally, I'm hoping an authoritative voice would indicate if there's

Re: [v1 2/4] ethdev: support GRE optional fields

2022-02-04 Thread Ferruh Yigit
On 2/1/2022 12:57 PM, Ori Kam wrote: Hi Sean, Please add this feature to the release note. Hi Sean, Reminder of this change request, waiting for new version. Also please remember to add the missing doc in mlx patch (4/4) in next version. Thanks, ferruh -Original Message- From: S

Re: [PATCH v2 0/8] net/bonding: fixes and LACP short timeout

2022-02-04 Thread Ferruh Yigit
On 12/21/2021 7:57 PM, Robert Sanford wrote: This patchset makes the following changes to net/bonding: - Clean up minor errors in spelling, whitespace, C++ wrappers, and comments. - Replace directly overwriting of slave port's rte_eth_conf by copying it, but only updating it via rte_eth_dev

Re: [PATCH v2 1/8] net/bonding: fix typos and whitespace

2022-02-04 Thread Ferruh Yigit
On 12/21/2021 7:57 PM, Robert Sanford wrote: - Clean up minor typos in comments, strings, and private names. - Fix whitespace in log messages and function formatting (new line before open brace). - Move closing C++ wrapper to the end of rte_eth_bond_8023ad.h. Signed-off-by: Robert Sanford --

Re: [PATCH v2 8/8] net/bonding: add LACP short timeout tests

2022-02-04 Thread Ferruh Yigit
On 12/21/2021 7:57 PM, Robert Sanford wrote: - Add "set bonding lacp timeout_ctrl on|off" to testpmd. - Add "test_mode4_lacp_timeout_control" to dpdk-test. - Remove call to rte_eth_dev_mac_addr_remove from add_slave, as it always fails and prints an error. Signed-off-by: Robert Sanford ---

Re: [PATCH v2 7/8] net/ring: add promisc and all-MC stubs

2022-02-04 Thread Bruce Richardson
On Fri, Feb 04, 2022 at 02:36:47PM +, Ferruh Yigit wrote: > On 12/21/2021 7:57 PM, Robert Sanford wrote: > > Add promiscuous_enable, promiscuous_disable, allmulticast_enable, > > and allmulticast_disable API stubs. > > This helps clean up errors in dpdk-test link_bonding_mode4_autotest. > > >

Re: [PATCH v2 5/8] net/bonding: add bond_8023ad and bond_alb to doc

2022-02-04 Thread Ferruh Yigit
On 12/21/2021 7:57 PM, Robert Sanford wrote: - Add bond_8023ad and bond_alb to API documentation. Signed-off-by: Robert Sanford --- doc/api/doxy-api-index.md | 2 ++ 1 file changed, 2 insertions(+) diff --git a/doc/api/doxy-api-index.md b/doc/api/doxy-api-index.md index 4245b96..830235c 100

Re: [PATCH v2 4/8] net/bonding: support enabling LACP short timeout

2022-02-04 Thread Ferruh Yigit
On 12/21/2021 7:57 PM, Robert Sanford wrote: - Add support for enabling LACP short timeout, i.e., link partner can use fast periodic time interval between transmits. Signed-off-by: Robert Sanford --- drivers/net/bonding/eth_bond_8023ad_private.h | 3 ++- drivers/net/bonding/rte_eth_bond_

Re: [PATCH v2 7/8] net/ring: add promisc and all-MC stubs

2022-02-04 Thread Ferruh Yigit
On 12/21/2021 7:57 PM, Robert Sanford wrote: Add promiscuous_enable, promiscuous_disable, allmulticast_enable, and allmulticast_disable API stubs. This helps clean up errors in dpdk-test link_bonding_mode4_autotest. Signed-off-by: Robert Sanford --- drivers/net/ring/rte_eth_ring.c | 28 ++

Re: [PATCH v2] net/af_xdp: re-enable secondary process support

2022-02-04 Thread Ferruh Yigit
On 2/4/2022 12:54 PM, Ciara Loftus wrote: Secondary process support had been disabled for the AF_XDP PMD because there was no logic in place to share the AF_XDP socket file descriptors between the processes. This commit introduces this logic using the IPC APIs. Since AF_XDP rings are single-prod

Re: [PATCH v1] drivers/net: use internal API to get eth dev from name

2022-02-04 Thread Ferruh Yigit
On 2/3/2022 12:31 PM, Ferruh Yigit wrote: On 2/3/2022 8:24 AM, Kumara Parameshwaran wrote: From: Kumara Parameshwaran Make changes in PMDs to use the new function where rte_eth_dev_get_port_by_name is used to get port_id to access rte_eth_devices Signed-off-by: Kumara Parameshwaran --- v1 *

Re: [PATCH v6 0/2] Add functions to calculate UDP/TCP cksum in mbuf

2022-02-04 Thread Ferruh Yigit
On 1/24/2022 12:28 PM, Xiaoyun Li wrote: Added functions to calculate UDP/TCP checksum for packets which may be over multi-segments and called the functions in testpmd csum forwarding mode to support UDP/TCP sotfware checksum over multi-segments. Xiaoyun Li (2): net: add functions to calculat

Re: [PATCH v6 2/2] app/testpmd: enable L4 SW csum over multi segments

2022-02-04 Thread Ferruh Yigit
On 1/24/2022 12:28 PM, Xiaoyun Li wrote: Csum forwarding mode only supports software UDP/TCP csum calculation for single segment packets when hardware offload is not enabled. This patch enables software UDP/TCP csum calculation over multiple segments. Signed-off-by: Xiaoyun Li Tested-by: Sunil

[PATCH v2] net/af_xdp: re-enable secondary process support

2022-02-04 Thread Ciara Loftus
Secondary process support had been disabled for the AF_XDP PMD because there was no logic in place to share the AF_XDP socket file descriptors between the processes. This commit introduces this logic using the IPC APIs. Since AF_XDP rings are single-producer single-consumer, rx/tx in the secondary

Re: [PATCH 1/1] vhost: Move fds outside of VhostUserMessage

2022-02-04 Thread Maxime Coquelin
Hi Christophe, On 1/23/22 12:59, Christophe Fontaine wrote: FDs at the end of the VhostUserMessage structure limits the size of the payload. Move them to an other englobing structure, before the header & payload of a VhostUserMessage. Also removes a reference to fds in the VHUMsg structure defin

Re: [dpdk-dev] [PATCH 2/3] app/testpmd: fix slave device isn't released

2022-02-04 Thread Ferruh Yigit
On 10/25/2021 7:39 AM, Min Hu (Connor) wrote: From: Huisong Li Currently, some eth devices are added to bond device, these devices are not released when the quit command is executed in testpmd. This patch adds the release operation for all active slaves under a bond device. 'close_port()' fu

Re: [dpdk-dev] [PATCH 1/3] app/testpmd: fix port status of active slave device

2022-02-04 Thread Ferruh Yigit
On 10/25/2021 7:39 AM, Min Hu (Connor) wrote: From: Huisong Li Stopping a bond device also stops all active slaves under the bond device. If this port is bond device, we need to modify the port status of all slaves from RTE_PORT_STARTED to RTE_PORT_STOPPED. Fixes: 0e545d3047fe ("app/testpmd: c

RE:[PATCH] vdpa/sfc: fix null pointer dereference

2022-02-04 Thread Weiguo Li
The macro sfc_vdpa_info(sva, ...) will invoke sva->logtype_main, It'll trigger a NULL dereference when sva is NULL. This patch fix it. BTW this patch is not superseded by another one: [v2] vdpa/sfc: fix null dereference (http://dpdk.org/patch/106720) They looks alike in title, but deal with d

Re: [PATCH v2 0/2] vhost: fix async address mapping

2022-02-04 Thread Maxime Coquelin
On 1/19/22 16:10, xuan.d...@intel.com wrote: From: Xuan Ding This patchset fixes the issue of incorrect DMA mapping in PA mode. Due to the ambiguity of host_phys_addr naming in the guest page struct, rename it to host_iova. v2: * Change the order of patch. Xuan Ding (2): vhost: rename f

Re: [PATCH v2 2/2] vhost: fix physical address mapping

2022-02-04 Thread Maxime Coquelin
On 1/19/22 16:10, xuan.d...@intel.com wrote: From: Xuan Ding When choosing IOVA as PA mode, IOVA is likely to be discontinuous, which requires page by page mapping for DMA devices. To be consistent, this patch implements page by page mapping instead of mapping at the region granularity for b

Re: [PATCH v2 0/2] vhost: fix async address mapping

2022-02-04 Thread Maxime Coquelin
On 2/1/22 12:29, Kevin Traynor wrote: On 01/02/2022 08:28, Maxime Coquelin wrote: On 1/19/22 16:10, xuan.d...@intel.com wrote: From: Xuan Ding This patchset fixes the issue of incorrect DMA mapping in PA mode. Due to the ambiguity of host_phys_addr naming in the guest page struct, rename

RE: [PATCH 4/4] crypto: reorganize endianness comments, add crypto uint

2022-02-04 Thread Zhang, Roy Fan
> -Original Message- > From: Kusztal, ArkadiuszX > Sent: Wednesday, February 2, 2022 6:51 AM > To: dev@dpdk.org > Cc: gak...@marvell.com; Zhang, Roy Fan ; > rb...@marvell.com; Kusztal, ArkadiuszX > Subject: [PATCH 4/4] crypto: reorganize endianness comments, add crypto > uint > > This pa

RE: [PATCH 3/4] crypto: use rte macro instead of direct attribute

2022-02-04 Thread Zhang, Roy Fan
> -Original Message- > From: Kusztal, ArkadiuszX > Sent: Wednesday, February 2, 2022 6:50 AM > To: dev@dpdk.org > Cc: gak...@marvell.com; Zhang, Roy Fan ; > rb...@marvell.com; Kusztal, ArkadiuszX > Subject: [PATCH 3/4] crypto: use rte macro instead of direct attribute > > This commit rep

RE: [PATCH 0/4] Clarify asymmetric random, add 'k', uint

2022-02-04 Thread Zhang, Roy Fan
> -Original Message- > From: Kusztal, ArkadiuszX > Sent: Wednesday, February 2, 2022 6:50 AM > To: dev@dpdk.org > Cc: gak...@marvell.com; Zhang, Roy Fan ; > rb...@marvell.com; Kusztal, ArkadiuszX > Subject: [PATCH 0/4] Clarify asymmetric random, add 'k', uint > > This patchset introduces

RE: [PATCH 2/4] crypto: clarify usage of random numbers in asym

2022-02-04 Thread Zhang, Roy Fan
> -Original Message- > From: Kusztal, ArkadiuszX > Sent: Wednesday, February 2, 2022 6:50 AM > To: dev@dpdk.org > Cc: gak...@marvell.com; Zhang, Roy Fan ; > rb...@marvell.com; Kusztal, ArkadiuszX > Subject: [PATCH 2/4] crypto: clarify usage of random numbers in asym > > This commit clari

RE: [PATCH 1/4] crypto: add dsa random number k

2022-02-04 Thread Zhang, Roy Fan
> -Original Message- > From: Kusztal, ArkadiuszX > Sent: Wednesday, February 2, 2022 6:50 AM > To: dev@dpdk.org > Cc: gak...@marvell.com; Zhang, Roy Fan ; > rb...@marvell.com; Kusztal, ArkadiuszX > Subject: [PATCH 1/4] crypto: add dsa random number k > > This commit adds random number 'k

RE: [PATCH v1] drivers/net: use internal API to get eth dev from name

2022-02-04 Thread Singh, Jasvinder
> -Original Message- > From: Yigit, Ferruh > Sent: Thursday, February 3, 2022 12:36 PM > To: Singh, Jasvinder ; Dumitrescu, Cristian > > Cc: Kumara Parameshwaran ; Kumara > Parameshwaran ; dev@dpdk.org > Subject: Re: [PATCH v1] drivers/net: use internal API to get eth dev from > name >

RE: [PATCH v6] eventdev/eth_rx: fix memory leak when token parsing finished

2022-02-04 Thread Jayatheerthan, Jay
Looks good. Thanks Weiguo for posting this patch and addressing all the comments! Acked-by: Jay Jayatheerthan < jay.jayatheert...@intel.com> -Jay > -Original Message- > From: Weiguo Li > Sent: Friday, February 4, 2022 1:51 PM > To: Jayatheerthan, Jay > Cc: Kundapura, Ganapati ; Naga

[PATCH v6] eventdev/eth_rx: fix memory leak when token parsing finished

2022-02-04 Thread Weiguo Li
The memory get from strdup should be freed when parameter parsing finished, and also should be freed when error occurs. Fixes: 814d01709328 ("eventdev/eth_rx: support telemetry") Fixes: 9e583185318f ("eventdev/eth_rx: support telemetry") Signed-off-by: Weiguo Li Acked-by: Ganapati Kundapura --