RE: [PATCH v3 1/2] net/i40e: replace put function

2023-02-16 Thread Morten Brørup
> From: Lu, Wenzhuo [mailto:wenzhuo...@intel.com] > Sent: Friday, 17 February 2023 03.03 > > Hi Kamalakshitha, > > > From: Kamalakshitha Aligeri > > Sent: Tuesday, February 14, 2023 2:18 AM > > > > Integrated zero-copy put API in mempool cache in i40e PMD. > > On Ampere Altra server, l3fwd singl

RE: [PATCH v3 4/4] vhost: stop using mempool for IOTLB cache

2023-02-16 Thread Yuan, DukaiX
Hi David, I get an error message when I use qemu-6.2.0 to relaunch dpdk-l3fwd-power with Asan, I need your help to confirm this issue. For more information please refer to https://bugs.dpdk.org/show_bug.cgi?id=1135. Waiting for your reply. Best regards, Dukai,Yuan > -Original Message- >

[RFC v3 10/10] net/gve: add AdminQ command to verify driver compatibility

2023-02-16 Thread Junfeng Guo
Check whether the driver is compatible with the device presented. Change gve_driver_info fields to report DPDK as OS type and DPDK RTE version as OS version, reserving driver_version fields for GVE driver version based on features. Signed-off-by: Rushil Gupta Signed-off-by: Joshua Washington Si

[RFC v3 09/10] net/gve: support jumbo frame for GQI

2023-02-16 Thread Junfeng Guo
Add multi-segment support to enable GQI Rx Jumbo Frame. Signed-off-by: Rushil Gupta Signed-off-by: Junfeng Guo Signed-off-by: Jeroen de Borst --- drivers/net/gve/gve_ethdev.h | 8 ++ drivers/net/gve/gve_rx.c | 137 +-- 2 files changed, 108 insertions(+), 3

[RFC v3 08/10] net/gve: enable Tx checksum offload for DQO

2023-02-16 Thread Junfeng Guo
Enable Tx checksum offload once any flag of L4 checksum is set. Signed-off-by: Junfeng Guo Signed-off-by: Rushil Gupta Signed-off-by: Jordan Kimbrough Signed-off-by: Jeroen de Borst --- drivers/net/gve/gve_ethdev.h | 4 drivers/net/gve/gve_tx_dqo.c | 5 + 2 files changed, 9 insertion

[RFC v3 07/10] net/gve: support basic stats for DQO

2023-02-16 Thread Junfeng Guo
Add basic stats support for DQO. Signed-off-by: Junfeng Guo Signed-off-by: Rushil Gupta Signed-off-by: Jordan Kimbrough Signed-off-by: Jeroen de Borst --- drivers/net/gve/gve_ethdev.c | 2 ++ drivers/net/gve/gve_rx_dqo.c | 12 +++- drivers/net/gve/gve_tx_dqo.c | 6 ++ 3 files ch

[RFC v3 06/10] net/gve: support basic Rx data path for DQO

2023-02-16 Thread Junfeng Guo
Add basic Rx data path support for DQO. Signed-off-by: Junfeng Guo Signed-off-by: Rushil Gupta Signed-off-by: Jordan Kimbrough Signed-off-by: Jeroen de Borst --- drivers/net/gve/gve_ethdev.c | 1 + drivers/net/gve/gve_ethdev.h | 3 + drivers/net/gve/gve_rx_dqo.c | 128

[RFC v3 05/10] net/gve: support basic Tx data path for DQO

2023-02-16 Thread Junfeng Guo
Add basic Tx data path support for DQO. Signed-off-by: Junfeng Guo Signed-off-by: Rushil Gupta Signed-off-by: Jordan Kimbrough Signed-off-by: Jeroen de Borst --- drivers/net/gve/gve_ethdev.c | 1 + drivers/net/gve/gve_ethdev.h | 4 + drivers/net/gve/gve_tx_dqo.c | 141

[RFC v3 04/10] net/gve: support queue release and stop for DQO

2023-02-16 Thread Junfeng Guo
Add support for queue operations: - gve_tx_queue_release_dqo - gve_rx_queue_release_dqo - gve_stop_tx_queues_dqo - gve_stop_rx_queues_dqo Signed-off-by: Junfeng Guo Signed-off-by: Rushil Gupta Signed-off-by: Jordan Kimbrough Signed-off-by: Jeroen de Borst --- drivers/net/gve/gve_ethdev.c

[RFC v3 03/10] net/gve: support device start and close for DQO

2023-02-16 Thread Junfeng Guo
Add device start and close support for DQO. Signed-off-by: Junfeng Guo Signed-off-by: Rushil Gupta Signed-off-by: Jordan Kimbrough Signed-off-by: Jeroen de Borst --- drivers/net/gve/base/gve_adminq.c | 12 - drivers/net/gve/gve_ethdev.c | 43 ++- 2 fil

[RFC v3 02/10] net/gve: add Rx queue setup for DQO

2023-02-16 Thread Junfeng Guo
Add support for rx_queue_setup_dqo ops. Signed-off-by: Junfeng Guo Signed-off-by: Rushil Gupta Signed-off-by: Jordan Kimbrough Signed-off-by: Jeroen de Borst --- drivers/net/gve/gve_ethdev.c | 1 + drivers/net/gve/gve_ethdev.h | 14 drivers/net/gve/gve_rx_dqo.c | 154 +

[RFC v3 01/10] net/gve: add Tx queue setup for DQO

2023-02-16 Thread Junfeng Guo
Add support for tx_queue_setup_dqo ops. DQO format has submission and completion queue pair for each Tx/Rx queue. Note that with DQO format all descriptors and doorbells, as well as counters are written in little-endian. Signed-off-by: Junfeng Guo Signed-off-by: Rushil Gupta Signed-off-by: Jord

[RFC v3 00/10] gve PMD enhancement

2023-02-16 Thread Junfeng Guo
This patch set includs three main enhancements for gve PMD: - support basic data path for DQO queue format - support jumbo frame for GQI queue format - add new AdminQ cmd to verify driver compatibility v3: - refactor the dev_ops override for DQO. - add support for L4 Tx checksum. - update dr

RE: [PATCH v11 2/6] ethdev: add trace points for ethdev (part one)

2023-02-16 Thread Li, WeiyuanX
Hi Ankur Dwivedi, This patch has been merged into the DPDK23.03. At present, we have failed to test the fuzz case error display global-buffer-overflow. Could you please hava a look at it , also submitted a Bugzilla ticket: https://bugs.dpdk.org/show_bug.cgi?id=1162. Regards, Li, Weiyuan > ---

Re: [PATCH] app/testpmd: fix crash on cleanup

2023-02-16 Thread Singh, Aman Deep
On 2/6/2023 9:19 PM, David Marchand wrote: If allocating the ports[] array fails, a crash will occur when shutting down testpmd since ethdev emits RTE_ETH_EVENT_DESTROY events. Move init_port() before registering ethdev event handler. Looks ok to me. Avoids seg_fault, due to Null pointer acce

[Bug 1162] [dpdk23.03] [fuzzing test] fuzzing/*: launch dpdk-fuzz as global-buffer-overflow error.

2023-02-16 Thread bugzilla
https://bugs.dpdk.org/show_bug.cgi?id=1162 Bug ID: 1162 Summary: [dpdk23.03] [fuzzing test] fuzzing/*: launch dpdk-fuzz as global-buffer-overflow error. Product: DPDK Version: 23.03 Hardware: All OS: All

RE: [PATCH v5 1/2] ethdev: introduce the Tx map API for aggregated ports

2023-02-16 Thread Jiawei(Jonny) Wang
> -Original Message- > From: Thomas Monjalon > Sent: Friday, February 17, 2023 1:42 AM > To: Jiawei(Jonny) Wang > Cc: Slava Ovsiienko ; Ori Kam ; > andrew.rybche...@oktetlabs.ru; Aman Singh ; > Yuying Zhang ; Ferruh Yigit ; > dev@dpdk.org; Raslan Darawsheh > Subject: Re: [PATCH v5 1/2

RE: [PATCH v5 2/2] ethdev: add Aggregated affinity match item

2023-02-16 Thread Jiawei(Jonny) Wang
Hi, > -Original Message- > From: Thomas Monjalon > Sent: Friday, February 17, 2023 1:46 AM > To: Jiawei(Jonny) Wang > Cc: Slava Ovsiienko ; Ori Kam ; > andrew.rybche...@oktetlabs.ru; Aman Singh ; > Yuying Zhang ; Ferruh Yigit ; > dev@dpdk.org; Raslan Darawsheh > Subject: Re: [PATCH v5 2

RE: [PATCH v5 1/2] ethdev: introduce the Tx map API for aggregated ports

2023-02-16 Thread Jiawei(Jonny) Wang
> -Original Message- > From: Ferruh Yigit > Sent: Friday, February 17, 2023 1:58 AM > To: Jiawei(Jonny) Wang ; Slava Ovsiienko > ; Ori Kam ; NBU-Contact- > Thomas Monjalon (EXTERNAL) ; > andrew.rybche...@oktetlabs.ru; Aman Singh ; > Yuying Zhang > Cc: dev@dpdk.org; Raslan Darawsheh > S

Re: Multiple Tx-Queues not working as expected

2023-02-16 Thread Rajasekhar Pulluru
Ok Stephen, thanks for the information, I can try that. One of the problems I see with single Tx Queue mode is that Ixia reports packet drops, though I confirmed with the help of counters (before invoking tx burst) that all packets are being sent-out. Dumping HW counters don't report any drops in

[Bug 1161] [dpdk-23.03]virtio_user_as_exceptional_path/vhost_exception_path_with_virtio_user: launch testpmd as virtio-user failed

2023-02-16 Thread bugzilla
https://bugs.dpdk.org/show_bug.cgi?id=1161 Bug ID: 1161 Summary: [dpdk-23.03]virtio_user_as_exceptional_path/vhost_exce ption_path_with_virtio_user: launch testpmd as virtio-user failed Product: DPDK Version: 22.0

[PATCH 3/3] net/nfp: get rid of the usage of RTE log macro

2023-02-16 Thread Chaoyong He
Replace the usage of RTE_LOG* macro with PMD specific logging. Signed-off-by: Chaoyong He Reviewed-by: Niklas Söderlund --- drivers/net/nfp/flower/nfp_flower.c| 10 ++--- drivers/net/nfp/flower/nfp_flower_ctrl.c | 2 +- drivers/net/nfp/nfp_cpp_bridge.c | 48 ++--

[PATCH 2/3] net/nfp: get rid of the usage of RTE log level type

2023-02-16 Thread Chaoyong He
Register the own RX/TX debug log level type, and get rid of the usage of RTE_LOGTYPE_*. Then we can control the log by a independent switch. Signed-off-by: Chaoyong He Reviewed-by: Niklas Söderlund --- drivers/net/nfp/nfp_logs.c | 10 ++ drivers/net/nfp/nfp_logs.h | 8 ++-- 2 files

[PATCH 1/3] net/nfp: add the log source file

2023-02-16 Thread Chaoyong He
Prepare for adding more log functionality by moving the existing log functionality to its own file. Signed-off-by: Chaoyong He Reviewed-by: Niklas Söderlund --- drivers/net/nfp/meson.build | 1 + drivers/net/nfp/nfp_common.c | 3 --- drivers/net/nfp/nfp_logs.c | 10 ++ 3 files chan

[PATCH 0/3] refactor the nfp log subsystem

2023-02-16 Thread Chaoyong He
Follow the advice from community reviewer, we get rid of the use of RTE log level type and RTE_LOG_*() macro, and also wrap the rte_log() with our own log macro. Chaoyong He (3): net/nfp: add the log source file net/nfp: get rid of the usage of RTE log level type net/nfp: get rid of the usag

Re: [External] Re: [PATCH] malloc: fix malloc performance may becomes worse as the number of malloc increases

2023-02-16 Thread Fengnan Chang
Liang Ma 于2023年2月16日周四 22:04写道: > > On Wed, Feb 15, 2023 at 12:10:23PM +0100, Morten Brørup wrote: > > +CC: Fidaullah Noonari , your name also shows > > up in the git log; perhaps you can help review this patch. > > > > > > I gave up reviewing in depth, because the existing code is not easy to >

RE: [PATCH v3 1/2] net/i40e: replace put function

2023-02-16 Thread Lu, Wenzhuo
Hi Kamalakshitha, > -Original Message- > From: Kamalakshitha Aligeri > Sent: Tuesday, February 14, 2023 2:18 AM > To: Zhang, Yuying ; Xing, Beilei > ; Matz, Olivier ; > andrew.rybche...@oktetlabs.ru; Richardson, Bruce > ; m...@smartsharesystems.com; > konstantin.anan...@huawei.com; honnap

RE: [PATCH v3] net/iavf: enable Tx outer checksum offload on avx512

2023-02-16 Thread Xu, Ke1
> -Original Message- > From: Zeng, ZhichaoX > Sent: Friday, February 17, 2023 9:49 AM > To: dev@dpdk.org > Cc: Yang, Qiming ; Zhou, YidingX > ; Zhang, Qi Z ; Xu, Ke1 > ; Zeng, ZhichaoX ; Wu, > Jingjing ; Xing, Beilei ; > Richardson, Bruce ; Konstantin Ananyev > > Subject: [PATCH v3] ne

[PATCH v3] net/iavf: enable Tx outer checksum offload on avx512

2023-02-16 Thread Zhichao Zeng
This patch is to enable outer checksum offload on avx512 Tx path for tunnel packet by adding Tx path with context descriptor and adjusting path select logic. Signed-off-by: Yiding Zhou Signed-off-by: Zhichao Zeng --- v3: fix tunneling parameters --- v2: fix path select logic --- drivers/net/ia

Re: [PATCH] ethdev: add link speed 400G

2023-02-16 Thread Thomas Monjalon
16/02/2023 13:30, Ferruh Yigit: > On 2/16/2023 10:43 AM, Thomas Monjalon wrote: > > There are some devices supporting 400G speed, > > and it is well standardized in IEEE. > > > > Signed-off-by: Thomas Monjalon > > --- > > app/test-pmd/cmdline.c | 2 ++ > > app/test-pmd/config.c

Re: [PATCH v11 1/4] lib: add generic support for reading PMU events

2023-02-16 Thread Konstantin Ananyev
16/02/2023 17:54, Tomasz Duszynski пишет: Add support for programming PMU counters and reading their values in runtime bypassing kernel completely. This is especially useful in cases where CPU cores are isolated i.e run dedicated tasks. In such cases one cannot use standard perf utility without

Re: TCP stack support on DPDK

2023-02-16 Thread Liang Ma
On Thu, Feb 16, 2023 at 03:01:28PM +, Van Haaren, Harry wrote: > > -Original Message- > > From: Liang Ma > > Sent: Thursday, February 16, 2023 2:40 PM > > To: Sam Kirubakaran > > Cc: dev@dpdk.org > > Subject: Re: TCP stack support on DPDK > > > > On Tue, Jan 24, 2023 at 01:24:38PM +0

Re: [PATCH v5 0/4] add support for self monitoring

2023-02-16 Thread Liang Ma
PMU is kind of MSR. Precisely , that's MSR per core. All MSR reading will lead to IPI(Please reference the kernel implementation of MSR driver). The IPI will disturb the DPDK application because the userspace/kernel context switch, which has impact to the tail latency.

Re: [PATCH 1/2] net/gve: change offloading capability

2023-02-16 Thread Rushil Gupta
Makes sense. The virtual device only does L4 checksum. Acked-by: Rushil Gupta On Thu, Feb 16, 2023 at 10:58 AM Levend Sayar wrote: > Google Virtual NIC is not doing IPv4 checksummimg. > Removed that capability from PMD. > > Signed-off-by: Levend Sayar > --- > drivers/net/gve/gve_ethdev.c | 1

Re: Multiple Tx-Queues not working as expected

2023-02-16 Thread Stephen Hemminger
On Thu, 16 Feb 2023 23:40:27 +0530 Rajasekhar Pulluru wrote: > Hi Team, > > I am trying to set-up 8 Tx-Queues (but only 1 Rx-Queue) and burst traffic > out of different Tx-Queues from the same cpu core on an ixgbe nic (10G). > Although transmitted packets reach the peer, reading the statistics >

[PATCH 2/2] net/gve: add extended statistics

2023-02-16 Thread Levend Sayar
Google Virtual NIC PMD is enriched with extended statistics info. eth_dev_ops callback names are also synched with eth_dev_ops field names Signed-off-by: Levend Sayar --- drivers/net/gve/gve_ethdev.c | 152 ++- drivers/net/gve/gve_rx.c | 8 +- 2 files change

[PATCH 1/2] net/gve: change offloading capability

2023-02-16 Thread Levend Sayar
Google Virtual NIC is not doing IPv4 checksummimg. Removed that capability from PMD. Signed-off-by: Levend Sayar --- drivers/net/gve/gve_ethdev.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/net/gve/gve_ethdev.c b/drivers/net/gve/gve_ethdev.c index 06d1b796c8..fef2458a16 100644 ---

[PATCH v2 4/4] net/gve: add standard and extended statistics

2023-02-16 Thread Levend Sayar
NIC statistics are enhanced with rx/tx queue errors. Signed-off-by: Levend Sayar --- drivers/net/gve/gve_ethdev.c | 49 +++- 1 file changed, 43 insertions(+), 6 deletions(-) diff --git a/drivers/net/gve/gve_ethdev.c b/drivers/net/gve/gve_ethdev.c index 735847ede7

[PATCH v2 2/4] net/gve: add standard and extended statistics

2023-02-16 Thread Levend Sayar
Google Virtual NIC PMD is enriched with statistics info. Signed-off-by: Levend Sayar --- drivers/net/gve/gve_ethdev.c | 147 ++- drivers/net/gve/gve_ethdev.h | 11 +++ drivers/net/gve/gve_rx.c | 18 - drivers/net/gve/gve_tx.c | 6 ++ 4 files change

[PATCH v2 3/4] net/gve: add standard and extended statistics

2023-02-16 Thread Levend Sayar
TX side statistics is updated. Signed-off-by: Levend Sayar --- drivers/net/gve/gve_tx.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/net/gve/gve_tx.c b/drivers/net/gve/gve_tx.c index d4e52e3ea5..5c8a3b4379 100644 --- a/drivers/net/gve/gve_tx.c +++ b/drivers

Multiple Tx-Queues not working as expected

2023-02-16 Thread Rajasekhar Pulluru
Hi Team, I am trying to set-up 8 Tx-Queues (but only 1 Rx-Queue) and burst traffic out of different Tx-Queues from the same cpu core on an ixgbe nic (10G). Although transmitted packets reach the peer, reading the statistics indicates only the Tx-q[0] has non-zero packets and bytes count, the rest

[PATCH v2 1/4] net/gve: change offloading capability

2023-02-16 Thread Levend Sayar
Google Virtual NIC is not doing IPv4 checksummimg. Removed that capability from PMD. Signed-off-by: Levend Sayar --- drivers/net/gve/gve_ethdev.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/net/gve/gve_ethdev.c b/drivers/net/gve/gve_ethdev.c index 97781f0ed3..e357f16e16 100644 ---

RE: [PATCH v11 0/4] add support for self monitoring

2023-02-16 Thread Ruifeng Wang
> -Original Message- > From: Tomasz Duszynski > Sent: Friday, February 17, 2023 1:55 AM > To: dev@dpdk.org > Cc: roret...@linux.microsoft.com; Ruifeng Wang ; > bruce.richard...@intel.com; jer...@marvell.com; mattias.ronnb...@ericsson.com; > m...@smartsharesystems.com; tho...@monjalon.net;

Re: [PATCH v5 1/2] ethdev: introduce the Tx map API for aggregated ports

2023-02-16 Thread Ferruh Yigit
On 2/14/2023 3:48 PM, Jiawei Wang wrote: > When multiple ports are aggregated into a single DPDK port, > (example: Linux bonding, DPDK bonding, failsafe, etc.), > we want to know which port use for Tx via a queue. > > This patch introduces the new ethdev API > rte_eth_dev_map_aggr_tx_affinity(), i

[PATCH v11 4/4] eal: add PMU support to tracing library

2023-02-16 Thread Tomasz Duszynski
In order to profile app one needs to store significant amount of samples somewhere for an analysis latern on. Since trace library supports storing data in a CTF format lets take adventage of that and add a dedicated PMU tracepoint. Signed-off-by: Tomasz Duszynski Acked-by: Morten Brørup --- app

[PATCH v11 2/4] pmu: support reading ARM PMU events in runtime

2023-02-16 Thread Tomasz Duszynski
Add support for reading ARM PMU events in runtime. Signed-off-by: Tomasz Duszynski Acked-by: Morten Brørup Acked-by: Ruifeng Wang --- app/test/test_pmu.c | 4 ++ lib/pmu/meson.build | 7 +++ lib/pmu/pmu_arm64.c | 94 + lib/pmu/rte_p

[PATCH v11 3/4] pmu: support reading Intel x86_64 PMU events in runtime

2023-02-16 Thread Tomasz Duszynski
Add support for reading Intel x86_64 PMU events in runtime. Signed-off-by: Tomasz Duszynski Acked-by: Morten Brørup --- app/test/test_pmu.c | 2 ++ lib/pmu/meson.build | 1 + lib/pmu/rte_pmu.h| 2 ++ lib/pmu/rte_pmu_pmc_x86_64.h | 24 4

[PATCH v11 1/4] lib: add generic support for reading PMU events

2023-02-16 Thread Tomasz Duszynski
Add support for programming PMU counters and reading their values in runtime bypassing kernel completely. This is especially useful in cases where CPU cores are isolated i.e run dedicated tasks. In such cases one cannot use standard perf utility without sacrificing latency and performance. Signed

[PATCH v11 0/4] add support for self monitoring

2023-02-16 Thread Tomasz Duszynski
This series adds self monitoring support i.e allows to configure and read performance measurement unit (PMU) counters in runtime without using perf utility. This has certain advantages when application runs on isolated cores running dedicated tasks. Events can be read directly using rte_pmu_read()

Re: [PATCH v5 2/2] ethdev: add Aggregated affinity match item

2023-02-16 Thread Thomas Monjalon
For the title, I suggest ethdev: add flow matching of aggregated port 14/02/2023 16:48, Jiawei Wang: > When multiple ports are aggregated into a single DPDK port, > (example: Linux bonding, DPDK bonding, failsafe, etc.), > we want to know which port is used for Rx and Tx. > > This patch allows to

Re: [PATCH v5 1/2] ethdev: introduce the Tx map API for aggregated ports

2023-02-16 Thread Thomas Monjalon
For the title, I suggest ethdev: add Tx queue mapping of aggregated ports 14/02/2023 16:48, Jiawei Wang: > When multiple ports are aggregated into a single DPDK port, > (example: Linux bonding, DPDK bonding, failsafe, etc.), > we want to know which port use for Tx via a queue. > > This patch intr

Re: [PATCH] net/nfp: only flush specified port flow table

2023-02-16 Thread Ferruh Yigit
On 2/16/2023 6:36 AM, Chaoyong He wrote: > From: Jin Liu > > The flow flush function will destroy all the flow table on all the > port of NFP nic rather than the provide port. Modify logic, only > destroy the flow table on the corresponding port. > > Fixes: 30ecce522732 ("net/nfp: support flow A

Re: [PATCH v2 2/2] app/testpmd: add support for bonding port's LACP negotiation

2023-02-16 Thread Ferruh Yigit
On 2/16/2023 8:32 AM, Chaoyong He wrote: > From: Long Wu > > If bonding port is mode4 with disabling dedicated queue and there > are no other packets, forward loop will not call port's TX function > and bonding port will not send LACP packets. > > Add sending LACP packets periodically in forward

Re: [PATCH] net/nfp: remove the unneeded check logic

2023-02-16 Thread Ferruh Yigit
On 2/16/2023 6:28 AM, Chaoyong He wrote: > Remove the unneeded check logic of ingress and egress flow attribute. > After the OVS commit c7da49bc ("netdev-offload-dpdk: Fix transfer flows"), > the flow can not offload anymore if we don't do this modification. > > Signed-off-by: Chaoyong He > Revie

Re: [PATCH v11 1/3] ethdev: skip congestion management configuration

2023-02-16 Thread Ferruh Yigit
On 2/16/2023 2:51 PM, Rakesh Kudurumalla wrote: > Introduce new flow action to skip congestion management configuration > This feature helps to skip the congestion management processing > based on per flow or the packet color identified by > rte_flow meter object. For example, If one Rx queue confi

Re: [PATCH v10 1/2] ethdev: add query and update sync and async function calls

2023-02-16 Thread Ferruh Yigit
On 2/2/2023 1:47 PM, Gregory Etelson wrote: > Current API allows either query or update indirect flow action. > If indirect action must be conditionally updated according to it's > present state application must first issue action query then > analyze returned data and if needed issue update reques

RE: TCP stack support on DPDK

2023-02-16 Thread Van Haaren, Harry
> -Original Message- > From: Liang Ma > Sent: Thursday, February 16, 2023 2:40 PM > To: Sam Kirubakaran > Cc: dev@dpdk.org > Subject: Re: TCP stack support on DPDK > > On Tue, Jan 24, 2023 at 01:24:38PM +0530, Sam Kirubakaran wrote: > > Hi Liang, > > > > Thank you so much for your respon

[PATCH v11 1/3] ethdev: skip congestion management configuration

2023-02-16 Thread Rakesh Kudurumalla
Introduce new flow action to skip congestion management configuration This feature helps to skip the congestion management processing based on per flow or the packet color identified by rte_flow meter object. For example, If one Rx queue configured as RED congestion and application wants tobypass t

[PATCH v11 3/3] net/cnxk: skip red drop for ingress policer

2023-02-16 Thread Rakesh Kudurumalla
Dropping of packets is based on action configured to meter.If both skip_red and drop actions are configured then tail dropping in invoked else if only drop action is configured then RED drop is invoked.This action is supported only when RED is configured using rte_eth_cman_config_set() Signed-off-

[PATCH v11 2/3] app/testpmd: add skip cman support for testpmd

2023-02-16 Thread Rakesh Kudurumalla
added support for testpmd application to accept skip_cman action while configuring policy action Below is sample command given after configuring meter profile for which congestion configuration is applied for red and yellow color packets and bypassed for green color packets in dpdk-testpmd applica

RE: [PATCH v10 1/4] lib: add generic support for reading PMU events

2023-02-16 Thread Tomasz Duszynski
[...] >> + >> +if (rte_pmu_init() < 0) >> +return TEST_FAILED; > >Can we return TEST_SKIPPED here? >On aarch64, this feature requires kernel version >= 5.17. CI setups doesn't >meet this requirement >will start to report failure when running fast_tests. > Okay. I think that's goo

Re: TCP stack support on DPDK

2023-02-16 Thread Liang Ma
On Tue, Jan 24, 2023 at 01:24:38PM +0530, Sam Kirubakaran wrote: > Hi Liang, > > Thank you so much for your response. > I would like to convert our in-house tool into a high performance traffic > generator and it is written in Python. > Is it possible to use any of the user space TCP stack by comp

[PATCH 3/3] examples/ipsec-secgw: refactor inline capability check

2023-02-16 Thread Akhil Goyal
In cases of inline IPsec, the supported ol_flags are retrieved from security capability of device. Now that capability checks are added before creating the session, ol_flags can be retrieved from the same function call. Signed-off-by: Akhil Goyal --- examples/ipsec-secgw/ipsec.c | 65 ++-

[PATCH 2/3] examples/ipsec-secgw: check capabilities before session create

2023-02-16 Thread Akhil Goyal
Currently, sessions are created without checking the device capabilities, which may result in failure at a later stage. Device capabilities are now checked before creating the security/crypto session. Signed-off-by: Akhil Goyal --- examples/ipsec-secgw/ipsec.c | 208

[PATCH 1/3] examples/ipsec-secgw: fix auth IV length

2023-02-16 Thread Akhil Goyal
Currently, cipher IV length is getting used to set auth xform IV length. Auth IV is needed for AES-GMAC case, and in all other cases, auth IV should be 0. Used a separate auth IV length to separate out cipher and auth cases. Fixes: 9413c3901f31 ("examples/ipsec-secgw: support additional algorithms

[PATCH v2 4/4] doc: update supported devices for NVIDIA NICs

2023-02-16 Thread Raslan Darawsheh
This update the supproted list of devices to include missing ConnectX-7 in some drivers and add BlueFeild-3 Signed-off-by: Raslan Darawsheh --- doc/guides/compressdevs/mlx5.rst | 5 +++-- doc/guides/cryptodevs/mlx5.rst | 12 doc/guides/nics/mlx5.rst | 33 +

[PATCH v2 2/4] config/arm: add NVIDIA BlueField-3 platform

2023-02-16 Thread Raslan Darawsheh
Add meson build configuration for NVIDIA BlueField-3 platform with Cortex-A78AE cores. Signed-off-by: Raslan Darawsheh --- config/arm/meson.build | 16 1 file changed, 16 insertions(+) diff --git a/config/arm/meson.build b/config/arm/meson.build index 6442ec9596..89d8f327f1 100

[PATCH v2 3/4] crypto/mlx5: add Bluefield-3 and ConnectX-7 device IDs

2023-02-16 Thread Raslan Darawsheh
This adds the Bluefield-3 and ConnectX-7 device IDs to the list of supported NVIDIA devices that run the MLX5 compress PMD. The BlueField-3 device is still in development stage. Signed-off-by: Raslan Darawsheh --- drivers/crypto/mlx5/mlx5_crypto.c | 8 1 file changed, 8 insertions(+) d

[PATCH v2 0/4] support NVIDIA Bluefield-3 device

2023-02-16 Thread Raslan Darawsheh
this series is intentioned to update missing docs, drivers and config for supporting NVIDIA BlueField-3 device. --- v2: send cover letter fix patches order --- Raslan Darawsheh (4): mlx: update PCI device IDs names for NVIDIA BlueField NICs config/arm: add NVIDIA BlueField-3 platform c

[PATCH v2 1/4] mlx: update PCI device IDs names for NVIDIA BlueField NICs

2023-02-16 Thread Raslan Darawsheh
This updates the PCI device names for all NVIDIA BlueField family. Signed-off-by: Raslan Darawsheh --- drivers/common/mlx5/mlx5_common.h | 8 drivers/common/mlx5/mlx5_common_pci.c | 2 +- drivers/compress/mlx5/mlx5_compress.c | 4 ++-- drivers/crypto/mlx5/mlx5_crypto.c | 2 +

Re: [PATCH] malloc: fix malloc performance may becomes worse as the number of malloc increases

2023-02-16 Thread Liang Ma
On Wed, Feb 15, 2023 at 12:10:23PM +0100, Morten Brørup wrote: > +CC: Fidaullah Noonari , your name also shows > up in the git log; perhaps you can help review this patch. > > > I gave up reviewing in depth, because the existing code is not easy to > quickly understand, and it would take too lo

[PATCH 2/4] crypto/mlx5: add Bluefield-3 and ConnectX-7 device IDs

2023-02-16 Thread Raslan Darawsheh
This adds the Bluefield-3 and ConnectX-7 device IDs to the list of supported NVIDIA devices that run the MLX5 compress PMD. The BlueField-3 device is still in development stage. Signed-off-by: Raslan Darawsheh --- drivers/crypto/mlx5/mlx5_crypto.c | 8 1 file changed, 8 insertions(+) d

[PATCH 4/4] mlx: update PCI device IDs names for NVIDIA BlueField NICs

2023-02-16 Thread Raslan Darawsheh
This updates the PCI device names for all NVIDIA BlueField family. Signed-off-by: Raslan Darawsheh --- drivers/common/mlx5/mlx5_common.h | 8 drivers/common/mlx5/mlx5_common_pci.c | 2 +- drivers/compress/mlx5/mlx5_compress.c | 4 ++-- drivers/crypto/mlx5/mlx5_crypto.c | 4 +

[PATCH 3/4] doc: update supported devices for NVIDIA NICs

2023-02-16 Thread Raslan Darawsheh
This update the supproted list of devices to include missing ConnectX-7 in some drivers and add BlueFeild-3 Signed-off-by: Raslan Darawsheh --- doc/guides/compressdevs/mlx5.rst | 5 +++-- doc/guides/cryptodevs/mlx5.rst | 12 doc/guides/nics/mlx5.rst | 33 +

[PATCH 1/4] config/arm: add NVIDIA BlueField-3 platform

2023-02-16 Thread Raslan Darawsheh
Add meson build configuration for NVIDIA BlueField-3 platform with Cortex-A78AE cores. Signed-off-by: Raslan Darawsheh --- config/arm/meson.build | 16 1 file changed, 16 insertions(+) diff --git a/config/arm/meson.build b/config/arm/meson.build index 6442ec9596..89d8f327f1 100

Re: [PATCH v6] net/af_xdp: support CNI Integration

2023-02-16 Thread Ferruh Yigit
On 2/15/2023 4:30 PM, Shibin Koikkara Reeny wrote: > Integrate support for the AF_XDP CNI and device plugin [1] so that the > DPDK AF_XDP PMD can work in an unprivileged container environment. > Part of the AF_XDP PMD initialization process involves loading > an eBPF program onto the given netdev.

Re: [PATCH v5 2/2] ethdev: support xstats reset telemetry command

2023-02-16 Thread Bruce Richardson
On Thu, Feb 16, 2023 at 12:54:20PM +, Bruce Richardson wrote: > On Thu, Feb 16, 2023 at 08:42:34PM +0800, fengchengwen wrote: > > On 2023/2/16 20:06, Ferruh Yigit wrote: > > > On 2/16/2023 11:53 AM, fengchengwen wrote: > > >> On 2023/2/15 11:19, Dongdong Liu wrote: > > >>> Hi Chengwen > > >>> >

Re: [PATCH v5 2/2] ethdev: support xstats reset telemetry command

2023-02-16 Thread Bruce Richardson
On Thu, Feb 16, 2023 at 08:42:34PM +0800, fengchengwen wrote: > On 2023/2/16 20:06, Ferruh Yigit wrote: > > On 2/16/2023 11:53 AM, fengchengwen wrote: > >> On 2023/2/15 11:19, Dongdong Liu wrote: > >>> Hi Chengwen > >>> > >>> On 2023/2/9 10:32, Chengwen Feng wrote: > The xstats reset is useful

Re: [PATCH v5 2/2] ethdev: support xstats reset telemetry command

2023-02-16 Thread fengchengwen
On 2023/2/16 20:06, Ferruh Yigit wrote: > On 2/16/2023 11:53 AM, fengchengwen wrote: >> On 2023/2/15 11:19, Dongdong Liu wrote: >>> Hi Chengwen >>> >>> On 2023/2/9 10:32, Chengwen Feng wrote: The xstats reset is useful for debugging, so add it to the ethdev telemetry command lists. >

[PATCH] net/virtio: deduce IP length for Virtio TSO checksum

2023-02-16 Thread Boleslav Stankevich
The length of TSO payload could not fit into 16 bits provided by the IPv4 total length and IPv6 payload length fields. Thus, deduce it from the length of the packet. Signed-off-by: Boleslav Stankevich Reviewed-by: Andrew Rybchenko --- drivers/net/virtio/virtio_rxtx.c | 25 --

[PATCH] net/virtio: deduce IP length for Virtio TSO checksum

2023-02-16 Thread Boleslav Stankevich
The length of TSO payload could not fit into 16 bits provided by the IPv4 total length and IPv6 payload length fields. Thus, deduce it from the length of the packet. Signed-off-by: Boleslav Stankevich Reviewed-by: Andrew Rybchenko --- drivers/net/virtio/virtio_rxtx.c | 25 --

Re: [PATCH] ethdev: add link speed 400G

2023-02-16 Thread Ferruh Yigit
On 2/16/2023 10:43 AM, Thomas Monjalon wrote: > There are some devices supporting 400G speed, > and it is well standardized in IEEE. > > Signed-off-by: Thomas Monjalon > --- > app/test-pmd/cmdline.c | 2 ++ > app/test-pmd/config.c | 2 ++ > app/test-pmd/parameters.c

RE: [PATCH v4 10/12] app/mldev: enable support for inference validation

2023-02-16 Thread Anup Prabhu
> -Original Message- > From: Srikanth Yalavarthi > Sent: Tuesday, February 7, 2023 9:20 PM > To: Srikanth Yalavarthi > Cc: dev@dpdk.org; Shivah Shankar Shankar Narayan Rao > ; Jerin Jacob Kollanukkaran > ; Anup Prabhu ; Prince Takkar > ; Parijat Shukla > Subject: [PATCH v4 10/12] app/m

Re: [PATCH v10 2/3] app/testpmd: add skip cman support for testpmd

2023-02-16 Thread Ferruh Yigit
On 2/15/2023 5:42 PM, Rakesh Kudurumalla wrote: > added support for testpmd application to accept > skip_cman action while configuring policy action > > Signed-off-by: Rakesh Kudurumalla A comment seems missed, please check v6 & v8 [1], can you please send a new version with it? [1] https://inb

Re: [PATCH v5 2/2] ethdev: support xstats reset telemetry command

2023-02-16 Thread Ferruh Yigit
On 2/16/2023 11:53 AM, fengchengwen wrote: > On 2023/2/15 11:19, Dongdong Liu wrote: >> Hi Chengwen >> >> On 2023/2/9 10:32, Chengwen Feng wrote: >>> The xstats reset is useful for debugging, so add it to the ethdev >>> telemetry command lists. >>> >>> Signed-off-by: Chengwen Feng >> This patch lo

Re: [PATCH v5 2/2] ethdev: support xstats reset telemetry command

2023-02-16 Thread fengchengwen
On 2023/2/15 11:19, Dongdong Liu wrote: > Hi Chengwen > > On 2023/2/9 10:32, Chengwen Feng wrote: >> The xstats reset is useful for debugging, so add it to the ethdev >> telemetry command lists. >> >> Signed-off-by: Chengwen Feng > This patch looks good, so > Reviewed-by: Dongdong Liu > > A min

Re: [PATCH v3 6/6] test/dmadev: add tests for stopping and restarting dev

2023-02-16 Thread fengchengwen
Acked-by: Chengwen Feng On 2023/2/16 19:09, Bruce Richardson wrote: > Validate device operation when a device is stopped or restarted. > > The only complication - and gap in the dmadev ABI specification - is > what happens to the job ids on restart. Some drivers reset them to 0, > while others c

Re: [PATCH v3 4/6] test/dmadev: check result for device stop

2023-02-16 Thread fengchengwen
Acked-by: Chengwen Feng On 2023/2/16 19:09, Bruce Richardson wrote: > The DMA device stop API can return an error value so check that return > value when running dmadev unit tests. > > Signed-off-by: Bruce Richardson > Reviewed-by: Conor Walsh > Acked-by: Kevin Laatz > --- > app/test/test_dm

Re: [PATCH] ethdev: add link speed 400G

2023-02-16 Thread fengchengwen
Hi Thomas, Higher rates are generally implemented with multiple lanes. The lane configuration is critical to link negotiation success. The number of lanes configured using the ethtool is provided [1]. Currently, the community uses only one u32 bit (link_speed) to indicate a rate, which is n

Re: [PATCH] ethdev: add link speed 400G

2023-02-16 Thread Andrew Rybchenko
On 2/16/23 13:43, Thomas Monjalon wrote: There are some devices supporting 400G speed, and it is well standardized in IEEE. Signed-off-by: Thomas Monjalon Acked-by: Andrew Rybchenko

Re: [PATCH] net/nfp: support 48-bit DMA address for firmware with NFDk

2023-02-16 Thread Ferruh Yigit
On 2/16/2023 11:11 AM, Nole Zhang wrote: > > >> -Original Message- >> From: Ferruh Yigit >> Sent: Thursday, February 16, 2023 7:00 PM >> To: Chaoyong He ; Niklas Soderlund >> ; Kevin Traynor >> Cc: Xueming(Steven) Li ; dev@dpdk.org; Luca >> Boccassi ; oss-drivers ; Nole >> Zhang ; Kevin

RE: [PATCH] net/nfp: support 48-bit DMA address for firmware with NFDk

2023-02-16 Thread Nole Zhang
> -Original Message- > From: Ferruh Yigit > Sent: Thursday, February 16, 2023 7:00 PM > To: Chaoyong He ; Niklas Soderlund > ; Kevin Traynor > Cc: Xueming(Steven) Li ; dev@dpdk.org; Luca > Boccassi ; oss-drivers ; Nole > Zhang ; Kevin Liu ; > sta...@dpdk.org > Subject: Re: [PATCH] net/n

[PATCH v3 6/6] test/dmadev: add tests for stopping and restarting dev

2023-02-16 Thread Bruce Richardson
Validate device operation when a device is stopped or restarted. The only complication - and gap in the dmadev ABI specification - is what happens to the job ids on restart. Some drivers reset them to 0, while others continue where things left off. Take account of both possibilities in the test ca

[PATCH v3 5/6] test/dmadev: create separate function for single copy test

2023-02-16 Thread Bruce Richardson
The copy tests for dmadev had separate blocks in the test function for single copy and burst copies. Separate out the single-copy block to its own function so that it can be re-used if necessary. Signed-off-by: Bruce Richardson Acked-by: Kevin Laatz Acked-by: Chengwen Feng --- app/test/test_dm

[PATCH v3 4/6] test/dmadev: check result for device stop

2023-02-16 Thread Bruce Richardson
The DMA device stop API can return an error value so check that return value when running dmadev unit tests. Signed-off-by: Bruce Richardson Reviewed-by: Conor Walsh Acked-by: Kevin Laatz --- app/test/test_dmadev.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/app/tes

[PATCH v3 3/6] dma/ioat: fix incorrect error reporting on restart

2023-02-16 Thread Bruce Richardson
When the DMA device was stopped and restarted by the driver, the control register specifying the behaviour on error was not getting correctly reset. This caused unit tests to fail as explicitly introduced errors were got getting reported back. Fix by moving the setting of the register to the start

[PATCH v3 2/6] dma/ioat: fix incorrectly set indexes after restart

2023-02-16 Thread Bruce Richardson
As part of the process of restarting a dma instance, the IOAT driver will reset the HW addresses and state values. The read and write indexes for SW use need to be similarly reset to keep HW and SW in sync. Fixes: 583f046dd404 ("dma/ioat: add start and stop") Cc: conor.wa...@intel.com Cc: sta...@d

[PATCH v3 0/6] dma/ioat: fix issues with stopping and restarting device

2023-02-16 Thread Bruce Richardson
This patchset fixes a couple of problems with stopping and restarting an ioat DMA device. Following the three fixes, a series of improvements are made to the dmadev unit tests to properly validate that dmadevs work correctly as they are started and stopped, and ensure that no other or future driver

[PATCH v3 1/6] dma/ioat: fix device stop if no copies done

2023-02-16 Thread Bruce Richardson
The HW DMA devices supported by IOAT driver do not transition to the "active" state until the first operation is started by the HW. Therefore, if the user calls "rte_dma_stop()" on a device without triggering any operations, the sequence of commands to be sent to the HW is different, as is the fina

Re: [PATCH] net/nfp: support 48-bit DMA address for firmware with NFDk

2023-02-16 Thread Ferruh Yigit
On 2/16/2023 10:41 AM, Chaoyong He wrote: > > >> -Original Message- >> From: Niklas Soderlund >> Sent: Thursday, February 16, 2023 6:37 PM >> To: Kevin Traynor >> Cc: Ferruh Yigit ; Xueming(Steven) Li >> ; Chaoyong He ; >> dev@dpdk.org; Luca Boccassi ; oss-drivers > driv...@corigine.com

  1   2   >