Re: [dpdk-dev] [RFC PATCH v2 0/7] heterogeneous computing library

2021-07-31 Thread Jerin Jacob
On Fri, Jul 30, 2021 at 7:25 PM Thomas Monjalon wrote: > > From: Elena Agostini > > In heterogeneous computing system, processing is not only in the CPU. > Some tasks can be delegated to devices working in parallel. > > The goal of this new library is to enhance the collaboration between > DPDK,

[dpdk-dev] [Bug 766] Best sites to download free antivirus

2021-07-31 Thread bugzilla
https://bugs.dpdk.org/show_bug.cgi?id=766 Bug ID: 766 Summary: Best sites to download free antivirus Product: DPDK Version: 20.11 Hardware: All OS: All Status: UNCONFIRMED Severity: normal Priori

Re: [dpdk-dev] [RFC PATCH v2 0/7] heterogeneous computing library

2021-07-31 Thread Thomas Monjalon
31/07/2021 09:06, Jerin Jacob: > On Fri, Jul 30, 2021 at 7:25 PM Thomas Monjalon wrote: > > > > From: Elena Agostini > > > > In heterogeneous computing system, processing is not only in the CPU. > > Some tasks can be delegated to devices working in parallel. > > > > The goal of this new library i

Re: [dpdk-dev] [PATCH v1] test/func_reentrancy: free memzones after creating test case

2021-07-31 Thread Joyce Kong
> -Original Message- > From: Olivier Matz > Sent: Friday, July 30, 2021 7:27 PM > To: Joyce Kong > Cc: anatoly.bura...@intel.com; andrew.rybche...@oktetlabs.ru; > yipeng1.w...@intel.com; sameh.gobr...@intel.com; > bruce.richard...@intel.com; vladimir.medved...@intel.com; > konstantin.a

[dpdk-dev] [PATCH] net/bnxt: fix incorrect action record calculation

2021-07-31 Thread Ajit Khaparde
For Thor, the number of action records is being wrongly configured to 128 because of incorrect definition of divider. This results in an incorrect number of action records being negotiated with the FW. Remove the divider from the templates and delete the logic which uses the field in the resource m

Re: [dpdk-dev] [dpdk-stable] [PATCH] net/bnxt: fix incorrect action record calculation

2021-07-31 Thread Thomas Monjalon
31/07/2021 06:48, Ajit Khaparde: > For Thor, the number of action records is being wrongly configured > to 128 because of incorrect definition of divider. This results in > an incorrect number of action records being negotiated with the FW. > Remove the divider from the templates and delete the log

Re: [dpdk-dev] [EXT] [PATCH] crypto/mlx5: fix driver probing error flow

2021-07-31 Thread Thomas Monjalon
30/07/2021 21:23, Akhil Goyal: > > In crypto driver probing, there are two validations after context > > allocation. > > > > When one of them fails, the context structure was not freed what caused > > a memory leak. > > > > Free it. > > > > Fixes: debb27ea3442 ("crypto/mlx5: create login object

[dpdk-dev] DPDK 19.11.3 AWS perf issue

2021-07-31 Thread Nageswara Rao Penumarthy (npenumar)
Hi, We are running TCP/UDP based performance tests using teravm with DPDK 19.11. We see a performance drop of around 15% in DPDK 19.11 for AWS C5XL instance compared to DPDK 18.11 Are there any known issues? Please let us know. Thanks Nagesh

Re: [dpdk-dev] [PATCH v2] net/memif: fix abstract socket addr_len

2021-07-31 Thread Nathan Skrzypczak
Thanks ! Le ven. 30 juil. 2021 à 13:23, Thomas Monjalon a écrit : > > > This fixes using abstract sockets with memifs. > > > we were not passing the exact addr_len, which requires zeroing the > remaining > > > sun_path and doesn't appear well in other utilities (e.g. > > > lsof -U) > > > > > > S

Re: [dpdk-dev] [EXT] [PATCH 1/2] drivers/qat: fix wrong return value for invalid service

2021-07-31 Thread Thomas Monjalon
30/07/2021 21:18, Akhil Goyal: > > Subject: [EXT] [PATCH 1/2] drivers/qat: fix wrong return value for invalid > > service > > > Title changed as " drivers: fix return value for QAT PMDs " > Please check ./devtools/check-git-log.sh before sending patch. > > > Fix invalid value that is returned wh

[dpdk-dev] [Bug 767] Why do the Outlook rules not work in my account?

2021-07-31 Thread bugzilla
https://bugs.dpdk.org/show_bug.cgi?id=767 Bug ID: 767 Summary: Why do the Outlook rules not work in my account? Product: DPDK Version: unspecified Hardware: All OS: All Status: UNCONFIRMED Severity: normal

Re: [dpdk-dev] [PATCH 2/2] maint changes for marvell qlogic drivers

2021-07-31 Thread Thomas Monjalon
30/07/2021 08:27, Devendra Singh Rawat: > >Adding Marvell prefix for qlogic drivers. > >Removing myself as I'm no longer responsible for qede driver > > > >Signed-off-by: Igor Russkikh > >Signed-off-by: Rasesh Mody > > Acked. Applied, thanks.

[dpdk-dev] [PATCH v2] test/func_reentrancy: free memzones after creating test case

2021-07-31 Thread Joyce Kong
Function reentrancy test limits maximum number of iterations simultaneously, however it doesn't free the 'fr_test_once' memzones after the fact, so introduce freeing 'fr_test_once' in ring/mempool/hash/fbk/lpm_clean. Meanwhile, add the missing free for test case on main thread. Fixes: 104a92bd026

Re: [dpdk-dev] [PATCH] doc: announce renaming of mbuf offload flags

2021-07-31 Thread Andrew Rybchenko
On 7/30/21 6:57 PM, Olivier Matz wrote: The mbuf offload flags do not match the DPDK namespace (they are not prefixed by RTE_). Announce their rename in 21.11, and the removal of the old names in 22.11. A draft coccinelle script is provided to anticipate what the renaming will be. Signed-off-by

Re: [dpdk-dev] [PATCH v2 1/2] vhost: announce vDPA driver API marking as internal

2021-07-31 Thread Andrew Rybchenko
On 7/30/21 11:12 AM, Maxime Coquelin wrote: This patch announces the marking of all the vDPA driver APIs as internal. Acked-by: Chenbo Xia Signed-off-by: Maxime Coquelin Acked-by: Andrew Rybchenko

Re: [dpdk-dev] [PATCH v2 2/2] vhost: notice Vhost ops struct renaming

2021-07-31 Thread Andrew Rybchenko
On 7/30/21 11:12 AM, Maxime Coquelin wrote: This patch announces the renaming of struct vhost_device_ops to rte_vhost_device_ops in DPDK v21.11. Acked-by: Chenbo Xia Signed-off-by: Maxime Coquelin Acked-by: Andrew Rybchenko

Re: [dpdk-dev] [PATCH v3] net: fix Intel-specific Prepare the outer ipv4 hdr for checksum

2021-07-31 Thread Andrew Rybchenko
On 7/30/21 2:11 PM, Olivier Matz wrote: On Wed, Jul 28, 2021 at 06:46:53PM +0300, Andrew Rybchenko wrote: On 7/7/21 12:40 PM, Mohsin Kazmi wrote: Preparation the headers for the hardware offload misses the outer ipv4 checksum offload. It results in bad checksum computed by hardware NIC. This p

Re: [dpdk-dev] [RFC PATCH v2 0/7] heterogeneous computing library

2021-07-31 Thread Jerin Jacob
On Sat, Jul 31, 2021 at 1:51 PM Thomas Monjalon wrote: > > 31/07/2021 09:06, Jerin Jacob: > > On Fri, Jul 30, 2021 at 7:25 PM Thomas Monjalon wrote: > > > > > > From: Elena Agostini > > > > > > In heterogeneous computing system, processing is not only in the CPU. > > > Some tasks can be delegate

Re: [dpdk-dev] [PATCH v2] doc/sample_app_ug: use code snippets in sample app guides

2021-07-31 Thread Thomas Monjalon
> Currently the sample app user guides use hard coded code snippets, > this patch changes these to use literalinclude which will dynamically > update the snippets as changes are made to the code. > This was introduced in commit 413c75c33c40 ("doc: show how to include > code in guides"). Comments wi

Re: [dpdk-dev] [PATCH] doc: enhance arm64 profiling documentation

2021-07-31 Thread Thomas Monjalon
> > From: Jerin Jacob > > > > Documented the role of RTE_ARM_EAL_RDTSC_USE_PMU to enable PMU > > based rte_rdtsc(). > > > > Signed-off-by: Jerin Jacob > Acked-by: Ruifeng Wang Applied, thanks.

Re: [dpdk-dev] [EXT] [PATCH] doc: announce extension of crypto data-unit length

2021-07-31 Thread Thomas Monjalon
Ping for more votes. Please ack or reject. 17/05/2021 21:41, Akhil Goyal: > > The struct member dataunit_len is introduced in DPDK 21.05. > > It is limited to 16 bits to fit a padding hole in 32-bit build. > > This means the maximum data-unit length is 64 KB. > > Some use cases may benefit of a bi

Re: [dpdk-dev] [PATCH v3] doc/rel_notes: remove old deprecation notice for sched

2021-07-31 Thread Thomas Monjalon
14/06/2021 18:39, Andrew Rybchenko: > On 6/14/21 7:30 PM, Jasvinder Singh wrote: > > APIs and data structures hasve been modified as per deprecation > > note, so removing deprecation notice from the notes. > > > > Fixes: 85f52aa422d6 ("sched: add pipe config params to subport struct") > > > > Sig

Re: [dpdk-dev] [PATCH v2] doc/guides: add details for new test structure

2021-07-31 Thread Thomas Monjalon
16/07/2021 15:40, Ciara Power: > The testing guide is now updated to include details about > using sub-testsuites. Some example code is given to demonstrate how > they can be used. The trend is to avoid adding code in the doc, but include some existing code with literalinclude instead. Can it be a

[dpdk-dev] [PATCH] doc: announce SA config option struct changes

2021-07-31 Thread Akhil Goyal
From: Archana Muniganti Propose new fields to support offloads like - IPsec inner checksum(L3/L4) - IPsec tunnel header verification - TSO - etc in the structure ``rte_security_ipsec_sa_options``. Signed-off-by: Archana Muniganti Signed-off-by: Tejasree Kondoj Acked-by: Akhil Goyal --- Propos

Re: [dpdk-dev] [PATCH] app/test: flush stdout after forking

2021-07-31 Thread Thomas Monjalon
26/07/2021 14:27, Bruce Richardson: > On Mon, Jul 26, 2021 at 01:16:27PM +0100, John Levon wrote: > > meson test was not capturing the intended output from the child > > process; force a flush to ensure it reaches the test log. > > > > Signed-off-by: John Levon > > Acked-by: Bruce Richardson A

Re: [dpdk-dev] [PATCH] app/test: quieten noise while forking

2021-07-31 Thread Thomas Monjalon
26/07/2021 14:17, John Levon: > When closing file descriptors post-fork, ignore "." and ".." > directory entries. > > Signed-off-by: John Levon Please could you add an example of the noise in the commit explanation? Thanks

Re: [dpdk-dev] [EXT] [PATCH 01/10] security: add support for TSO on IPsec session

2021-07-31 Thread Akhil Goyal
> > Allow user to provision a per security session maximum segment size > > (MSS) for use when Transmit Segmentation Offload (TSO) is supported. > > The MSS value will be used when PKT_TX_TCP_SEG or PKT_TX_UDP_SEG > > ol_flags are specified in mbuf. > > > > Signed-off-by: Declan Doherty > > Signed

Re: [dpdk-dev] [PATCH] doc: announce IPsec SA config option struct changes

2021-07-31 Thread Akhil Goyal
> Proposing following new field for IPsec tunnel header verification > in the structure ``rte_security_ipsec_sa_options``. > enum rte_security_ipsec_tunnel_verify_mode tunnel_hdr_verify; > > It is used to indicate whether outer header verification > need to be done as part of inbound IPsec process

Re: [dpdk-dev] [PATCH] doc: announce SA config option struct changes

2021-07-31 Thread Akhil Goyal
> Proposing following two new fields for IPsec inner checksum > configuration in the structure ``rte_security_ipsec_sa_options``. > uint32_t ip_csum_enable : 1; > uint32_t l4_csum_enable : 1; > > With these config options, per SA, application can specify if > the inner checksum(compute/verify) t

Re: [dpdk-dev] [EXT] [PATCH] compress/mlx5: fix level translation in xform API

2021-07-31 Thread Akhil Goyal
> Compression Level is interpreted by each PMD differently. > However, lower numbers give faster compression > at the expense of compression ratio, while higher numbers > may give better compression ratios but are likely slower. > The level affects the block size, which affects performance, > the b

Re: [dpdk-dev] [dpdk-stable] [PATCH v5] doc: spell check

2021-07-31 Thread Thomas Monjalon
29/07/2021 18:48, Henry Nadeau: > Spell checked and corrected documentation. If there are any > errors, or I have changed something that wasn't an error please reach out to > me so I can update the dictionary. > > Cc: sta...@dpdk.org > > Signed-off-by: Henry Nadeau Applied, thanks.

[dpdk-dev] [PATCH 0/4] cryptodev and security ABI improvements

2021-07-31 Thread Akhil Goyal
This is a first series planned for ABI improvements in cryptodev and security library. Other planned improvements under development. - cryptodev: export driver interface as internal - cryptodev: split and hide struct rte_cryptodev, struct rte_cryptodev_data - cryptodev: hide struct rte_cryptodev_s

[dpdk-dev] [PATCH 1/4] cryptodev: remove LIST_END enumerators

2021-07-31 Thread Akhil Goyal
Remove *_LIST_END enumerators from asymmetric crypto lib to avoid ABI breakage for every new addition in enums. Signed-off-by: Akhil Goyal --- app/test/test_cryptodev_asym.c | 4 ++-- drivers/crypto/qat/qat_asym.c | 2 +- lib/cryptodev/rte_crypto_asym.h | 4 3 files changed, 3 insertions

[dpdk-dev] [PATCH 2/4] cryptodev: promote asym APIs to stable

2021-07-31 Thread Akhil Goyal
Asymmetric crypto APIs have been stable from quite some time, hence moving them from experimental to stable in DPDK 21.11 Signed-off-by: Akhil Goyal --- lib/cryptodev/rte_cryptodev.h | 10 -- lib/cryptodev/version.map | 24 +--- 2 files changed, 13 insertions(+),

[dpdk-dev] [PATCH 3/4] security: hide internal API

2021-07-31 Thread Akhil Goyal
rte_security_dynfield_register() is an internal API to be used by the driver, hence moving it to internal. Signed-off-by: Akhil Goyal --- lib/security/rte_security_driver.h | 2 +- lib/security/version.map | 7 ++- 2 files changed, 7 insertions(+), 2 deletions(-) diff --git a/lib/

[dpdk-dev] [PATCH 4/4] security: add reserved bitfields

2021-07-31 Thread Akhil Goyal
In struct rte_security_ipsec_sa_options, for every new option added, there is an ABI breakage, to avoid, a reserved_opts bitfield is added to for the remaining bits available in the structure. Now for every new sa option, these reserved_opts can be reduced and new option can be added. A correspondi

Re: [dpdk-dev] [PATCH 0/4] cryptodev and security ABI improvements

2021-07-31 Thread Akhil Goyal
> Subject: [PATCH 0/4] cryptodev and security ABI improvements > > This is a first series planned for ABI improvements > in cryptodev and security library. > > Other planned improvements under development. > - cryptodev: export driver interface as internal > - cryptodev: split and hide struct rte

[dpdk-dev] [PATCH] net/bnxt: fix incorrect action record calculation

2021-07-31 Thread Ajit Khaparde
For Thor, the number of action records is being wrongly configured to 128 because of incorrect definition of divider. This results in an incorrect number of action records being negotiated with the FW. Remove the divider from the templates and delete the logic which uses the field in the resource m

Re: [dpdk-dev] [dpdk-stable] [PATCH] net/bnxt: fix incorrect action record calculation

2021-07-31 Thread Ajit Khaparde
On Sat, Jul 31, 2021 at 2:25 AM Thomas Monjalon wrote: > > 31/07/2021 06:48, Ajit Khaparde: > > For Thor, the number of action records is being wrongly configured > > to 128 because of incorrect definition of divider. This results in > > an incorrect number of action records being negotiated with

Re: [dpdk-dev] [PATCH] net/bnxt: fix incorrect action record calculation

2021-07-31 Thread Ajit Khaparde
On Sat, Jul 31, 2021 at 7:49 AM Ajit Khaparde wrote: > > For Thor, the number of action records is being wrongly configured > to 128 because of incorrect definition of divider. This results in > an incorrect number of action records being negotiated with the FW. > Remove the divider from the templ

Re: [dpdk-dev] [PATCH v2] doc: update atomic operation deprecation

2021-07-31 Thread Thomas Monjalon
23/07/2021 11:49, Joyce Kong: > Update the incorrect description about atomic operations > with provided wrappers in deprecation doc[1]. > > [1]https://mails.dpdk.org/archives/dev/2021-July/21.html > > Fixes: 7518c5c4ae6a ("doc: announce adoption of C11 atomic operations > semantics") > Cc:

Re: [dpdk-dev] [PATCH] doc: announce removal of ABIs in PCI bus driver

2021-07-31 Thread Thomas Monjalon
27/07/2021 10:44, Bruce Richardson: > On Mon, Jul 26, 2021 at 05:56:17AM +, Xia, Chenbo wrote: > > From: Yigit, Ferruh > > > On 7/23/2021 8:39 AM, Xia, Chenbo wrote: > > > > From: dev On Behalf Of Chenbo Xia > > > >> +* pci: To reduce unnecessary ABIs exposed by DPDK bus driver, > > > "rte_bu

Re: [dpdk-dev] [PATCH] doc: announce common prefix for ethdev

2021-07-31 Thread Thomas Monjalon
> > > > > > Announce adding 'RTE_ETH_' prefix to all public ethdev macros/enums > > > on > > > > > > v21.11. > > > > > > Backward compatibility macros will be added on v21.11 and they will > > > > > > be > > > > > > removed on v22.11. > > > > > > > > > > > > Signed-off-by: Ferruh Yigit > > > > >

[dpdk-dev] [dpdk-announce] release candidate 21.08-rc3

2021-07-31 Thread Thomas Monjalon
A new DPDK release candidate is ready for testing: https://git.dpdk.org/dpdk/tag/?id=v21.08-rc3 There are 70 new patches in this snapshot. Release notes: https://doc.dpdk.org/guides/rel_notes/release_21_08.html Please check all pending announces of deprecations for 21.11. https:/

Re: [dpdk-dev] [PATCH] doc: announce extension of crypto data-unit length

2021-07-31 Thread Ajit Khaparde
On Wed, Apr 14, 2021 at 1:15 PM Thomas Monjalon wrote: > > The struct member dataunit_len is introduced in DPDK 21.05. > It is limited to 16 bits to fit a padding hole in 32-bit build. > This means the maximum data-unit length is 64 KB. > Some use cases may benefit of a bigger size as the proposed