[PATCH v10 00/13] preparation for the rte_flow offload of nfp PMD

This is the first patch series to add the support of rte_flow offload for nfp PMD, includes: Add the support of flower firmware application Add the support of representor port Add the flower service infrastructure Add the cmsg interactive channels between pmd and fw * Changes since v9 - Remove the

[PATCH v10 01/13] net/nfp: move app specific attributes to own struct

The NFP card can load different application firmwares. Currently only the CoreNIC application firmware is supported. This commit makes needed infrastructure changes in order to support other application firmwares too. Clearer separation is made between the PF device and any application firmware sp

[PATCH v10 02/13] net/nfp: simplify initialization and remove dead code

Calling nfp_net_init() is only done for the corenic firmware flavor and it is guaranteed to always be called from the primary process, so the explicit check for RTE_PROC_PRIMARY can be dropped. The calling graph of nfp_net_init() already guaranteed the free of resources when it fail, so remove the

[PATCH v10 03/13] net/nfp: move app specific init logic to own function

The NFP card can load different firmware applications. This commit move the init logic of corenic app of the secondary process into its own function. Signed-off-by: Chaoyong He Reviewed-by: Niklas Söderlund --- drivers/net/nfp/nfp_ethdev.c | 90 +--- 1 fi

[PATCH v10 04/13] net/nfp: add initial flower firmware support

Adds the basic probing infrastructure to support the flower firmware application. Adds the cpp service, used for some user tools. Signed-off-by: Chaoyong He Signed-off-by: Heinrich Kuhn Reviewed-by: Niklas Söderlund --- doc/guides/nics/nfp.rst| 13 + doc/guides/rel_notes/r

[PATCH v10 05/13] net/nfp: add flower PF setup logic

Adds the vNIC initialization logic for the flower PF vNIC. The flower firmware application exposes this vNIC for the purposes of fallback traffic in the switchdev use-case. Adds minimal dev_ops for this PF vNIC device. Because the device is being exposed externally to DPDK it needs to implements a

[PATCH v10 06/13] net/nfp: add flower ctrl VNIC related logics

Adds the setup/start logic for the ctrl vNIC. This vNIC is used by the PMD and flower firmware application as a communication channel between driver and firmware. In the case of OVS it is also used to communicate flow statistics from hardware to the driver. A rte_eth device is not exposed to DPDK

[PATCH v10 07/13] net/nfp: move common rxtx function for flower use

Move some common Rx and Tx logic to the header file so that they can be re-used by flower Tx and Rx logic. Signed-off-by: Chaoyong He Signed-off-by: Heinrich Kuhn Reviewed-by: Niklas Söderlund --- drivers/net/nfp/nfp_rxtx.c | 32 +--- drivers/net/nfp/nfp_rxtx.h | 31

[PATCH v10 08/13] net/nfp: add flower ctrl VNIC rxtx logic

Adds the Rx and Tx function for the ctrl VNIC. The logic is mostly identical to the normal Rx and Tx functionality of the NFP PMD. Make use of the ctrl VNIC service logic to service the ctrl vNIC Rx path. Signed-off-by: Chaoyong He Signed-off-by: Heinrich Kuhn Reviewed-by: Niklas Söderlund ---

[PATCH v10 09/13] net/nfp: add flower representor framework

Adds the framework to support flower representors. The number of VF representors are parsed from the command line. For physical port representors the current logic aims to create a representor for each physical port present on the hardware. An eth_dev is created for each physical port and VF, and

[PATCH v10 10/13] net/nfp: add flower PF related routines

Adds the start/stop/close routine of the flower PF vNIC. Signed-off-by: Chaoyong He Reviewed-by: Niklas Söderlund --- drivers/net/nfp/flower/nfp_flower.c | 210 drivers/net/nfp/flower/nfp_flower.h | 2 + drivers/net/nfp/flower/nfp_flower_repres

[PATCH v10 11/13] net/nfp: move rxtx function to header file

The flower firmware application makes use of the same Rx and Tx checksum logic as the normal PMD. Expose it so that flower firmware application also can make use of it. Signed-off-by: Chaoyong He Signed-off-by: Heinrich Kuhn Reviewed-by: Niklas Söderlund --- drivers/net/nfp/nfp_common.c|

[PATCH v10 12/13] net/nfp: add flower PF rxtx logic

For the Rx logic, fallback packets are multiplexed to the correct representor port based on the prepended metadata. For the Tx logic, because fallback packets are prepended metadata, the start of the packet has to be adjusted for in the Tx descriptor. Signed-off-by: Chaoyong He Reviewed-by: Nikl

[PATCH v10 13/13] net/nfp: add the representor port rxtx logic

For the Rx logic, the representor port decap packet from the corresponding ring. For the Tx logic, the representor port prepend the metadata into packet, and send to firmware through the queue 0 of pf vNIC. Signed-off-by: Chaoyong He Reviewed-by: Niklas Söderlund --- drivers/net/nfp/flower/nfp

RE: [PATCH] net/virtio: fix crash when dev is configured twice

> -Original Message- > From: Alexander Chernavin > Sent: Wednesday, August 31, 2022 4:54 PM > To: maxime.coque...@redhat.com; Xia, Chenbo > Cc: dev@dpdk.org; Alexander Chernavin > Subject: [PATCH] net/virtio: fix crash when dev is configured twice > > When first attempt to configure a d

[PATCH] net/octeon_ep: fix build for non-x86

A recent change removed errno.h from rte_common.h. x86 EAL headers seem to include it (probably via rte_spinlock/rte_cpuflags) but other architectures won't. Add an explicit inclusion. Fixes: d826133ae88e ("net/octeon_ep: support CN10K SoC") Signed-off-by: David Marchand --- Note: This patch

RE: [PATCH] vhost: enable CONFIG feature

> -Original Message- > From: Hao Chen > Sent: Tuesday, September 13, 2022 5:55 PM > To: maxime.coque...@redhat.com; Xia, Chenbo > Cc: dev@dpdk.org; ho...@yusur.tech; z...@yusur.tech; Hao Chen > > Subject: [PATCH] vhost: enable CONFIG feature > > Enable this feature, so that libvirt or q

RE: RE: [PATCH v2 2/2] vhost: introduce VHOST_USER_RESET_VRING

> -Original Message- > From: Xuan Zhuo > Sent: Friday, September 23, 2022 9:42 AM > To: Xia, Chenbo > Cc: dev@dpdk.org; hen...@linux.alibaba.com; jasonw...@redhat.com; > m...@redhat.com; Kangjie Xu ; > maxime.coque...@redhat.com > Subject: Re: RE: [PATCH v2 2/2] vhost: introduce VHOST_USE

facing _apply patch failure_ warning

Hi, I am facing _apply patch failure_ warning for one of my patchsets as below. https://lab.dpdk.org/results/dashboard/patchsets/23622/ http://mails.dpdk.org/archives/test-report/2022-September/309233.html Looks like the patch is being applied for dpdk branch, but the patchset is for "

RE: [EXT] [PATCH v2 1/5] test/crypto: add cryptodev_uadk_autotest

> Example: > sudo dpdk-test --vdev=crypto_uadk --log-level=6 > RTE>>cryptodev_uadk_autotest > RTE>>quit > > Signed-off-by: Zhangfei Gao > --- > app/test/test_cryptodev.c | 7 +++ > app/test/test_cryptodev.h | 1 + > 2 files changed, 8 insertions(+) > > diff --git a/app/test/test_cryptodev.c

Re: facing _apply patch failure_ warning

On Mon, Sep 26, 2022 at 9:54 AM Naga Harish K, S V wrote: > I am facing _apply patch failure_ warning for one of my patchsets as below. > > https://lab.dpdk.org/results/dashboard/patchsets/23622/ > > http://mails.dpdk.org/archives/test-report/2022-September/309233.html > > Looks like the patch i

RE: [PATCH v3 0/5] add remaining SGL support to AESNI_MB

Hi Ciara, > -Original Message- > From: Power, Ciara > Sent: Wednesday, September 21, 2022 2:51 PM > Cc: dev@dpdk.org; Ji, Kai ; Zhang, Roy Fan > ; De Lara Guarch, Pablo > ; Power, Ciara > Subject: [PATCH v3 0/5] add remaining SGL support to AESNI_MB > > Currently, the intel-ipsec-mb lib

[PATCH v2] net/virtio: fix crash when dev is configured twice

When first attempt to configure a device with RX interrupt enabled fails for some reason (e.g. because "Multiple intr vector not supported"), second attempt to configure the device with RX interrupt disabled and feature set unchanged will succeed but will leave virtio queues not allocated. Accessin

[PATCH v2] net/virtio: fix crash when dev is configured twice

When first attempt to configure a device with RX interrupt enabled fails for some reason (e.g. because "Multiple intr vector not supported"), second attempt to configure the device with RX interrupt disabled and feature set unchanged will succeed but will leave virtio queues not allocated. Accessin

RE: [EXT] [PATCH v2 2/5] crypto/uadk: introduce uadk crypto driver

> Introduce a new crypto PMD for hardware accelerators based on UADK [1]. > > UADK is a framework for user applications to access hardware accelerators. > UADK relies on IOMMU SVA (Shared Virtual Address) feature, which share > the same page table between IOMMU and MMU. > Thereby user application

[PATCH] raw/dpaa2_cmdif: remove driver

dpaa2_cmdif raw driver is no longer in use and has no known activer user. So this patch removes dpaa2_cmdif raw driver. Signed-off-by: Gagandeep Singh --- MAINTAINERS | 5 - doc/api/doxy-api-index.md | 1 - doc/api/doxy-api.conf.in

[RFC v2 0/3] enable hairpin queue

This patchset enables CPF for Intel Device ID of 0x1453 and supports hairpin queue based on idpf (Infrastructure Data Path Function) PMD [*EXPERIMENTAL*] in DPDK. This patchset is based on the idpf PMD code: https://patchwork.dpdk.org/project/dpdk/list/?series=24538&state=* v2: fix hairpin queue

[RFC v2 1/3] net/idpf: add support for CPF

Add CPF device id in idpf map. Signed-off-by: Junfeng Guo --- drivers/net/idpf/idpf_ethdev.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/net/idpf/idpf_ethdev.c b/drivers/net/idpf/idpf_ethdev.c index b3ca4e3326..08ada728b1 100644 --- a/drivers/net/idpf/idpf_ethdev.c +++ b/drivers/

[RFC v2 2/3] net/idpf: add support for hairpin queue

- Implement hairpin queue setup/confige/enable/disable. - Cross-vport hairpin queue implemented via hairpin_bind/unbind API. Test step: 1. Make sure no bug on CP side. 2. Add rule on IMC. - devmem 0x202920C100 64 0x804 - opcode=0x1303 prof_id=0x34 sub_prof_id=0x0 cookie=0xa2b87 key=0x18,\

[RFC v2 3/3] net/idpf: adjust RSS LUT to exclude hairpin queue

RSS should direct traffic only to the normal data Rx queues, so when hairpin queue configured, RSS LUT should be adjusted to exclude the hairpin queue. Signed-off-by: Xiao Wang Signed-off-by: Junfeng Guo --- drivers/net/idpf/idpf_ethdev.c | 24 1 file changed, 24 insert

Re: [EXT] [PATCH v2 2/5] crypto/uadk: introduce uadk crypto driver

Hi, Akhil On 2022/9/26 下午4:36, Akhil Goyal wrote: Introduce a new crypto PMD for hardware accelerators based on UADK [1]. UADK is a framework for user applications to access hardware accelerators. UADK relies on IOMMU SVA (Shared Virtual Address) feature, which share the same page table between

Re: [PATCH v2] eal: change alloc_sz calculation which may cause unnecessarily allocation

On Thu, Sep 22, 2022 at 12:52 AM Dmitry Kozlyuk wrote: > > 2022-09-21 15:35 (UTC+0200), David Marchand: > > On Thu, Jul 28, 2022 at 8:03 PM Dmitry Kozlyuk > > wrote: > > > > > > 2022-07-28 14:41 (UTC+0500), Fidaullah Noonari: > > > > The amount of memory to allocate from the system for heap expa

RE: [PATCH V3 2/7] table: add entry ID for learner tables

> -Original Message- > From: Thomas Monjalon > Sent: Saturday, September 24, 2022 9:25 AM > To: Dumitrescu, Cristian > Cc: dev@dpdk.org > Subject: Re: [PATCH V3 2/7] table: add entry ID for learner tables > > 30/08/2022 20:58, Cristian Dumitrescu: > > @@ -202,6 +213,7 @@ rte_swx_table

Re: [PATCH v2] eal: change alloc_sz calculation which may cause unnecessarily allocation

On Thu, Jul 28, 2022 at 8:03 PM Dmitry Kozlyuk wrote: > > 2022-07-28 14:41 (UTC+0500), Fidaullah Noonari: > > The amount of memory to allocate from the system for heap expansion > > was calculated in a way that may yield one page more than needed. > > This could hit the allocation limit from the s

RE: [PATCH 0/1] Add security perf application

Hi Akhil, Do you have any further comments? Thanks, Anoob > -Original Message- > From: Anoob Joseph > Sent: Friday, August 19, 2022 12:50 PM > To: Akhil Goyal > Cc: Aakash Sasidharan ; dev@dpdk.org; > techbo...@dpdk.org; Jerin Jacob Kollanukkaran ; Thomas > Monjalon ; Hemant Agrawal >

RE: [PATCH v7 1/4] eal: add lcore poll busyness telemetry

Hi Kevin, > >> Currently, there is no way to measure lcore poll busyness in a passive way, > >> without any modifications to the application. This patch adds a new EAL API > >> that will be able to passively track core polling busyness. > >> > >> The poll busyness is calculated by relying on the

Re: [PATCH v3] eal: update lcore-worker name due to high number of cores

Hello, On Fri, Sep 23, 2022 at 9:06 AM Abdullah Ömer Yamaç wrote: > > In this patch we suggest a new name for lcore-worker. > In case of more than 99 logical cores, name is truncated > (length is restricted to 16 characters, including the > terminating null byte ('\0')) and it makes hard to follo

RE: [PATCH v2] net/iavf: fix TSO offload for tunnel case

> -Original Message- > From: Zhichao Zeng > Sent: Monday, September 26, 2022 1:17 PM > To: dev@dpdk.org > Cc: Yang, Qiming ; Zhou, YidingX > ; Zhang, Qi Z ; Zeng, > ZhichaoX ; Wu, Jingjing ; > Xing, Beilei ; Sinha, Abhijit > ; > Doherty, Declan ; Nicolau, Radu > > Subject: [PATCH v2]

[PATCH v4] eal: fix thread names for high order lcores

In this patch we suggest a new name for lcore-worker. In case of more than 99 logical cores, name is truncated (length is restricted to 16 characters, including the terminating null byte ('\0')) and it makes hard to follow threads. Signed-off-by: Abdullah Ömer Yamaç Acked-by: Stephen Hemminger

[PATCH 0/2] fixes after sym session rework

1. Updated armv8 crypto PMD to adapt to new sym session rework. 2. Fixed UT test case issue found when testing armv8 crypto PMD. This patch set depends on the sym session framework rework series. http://patches.dpdk.org/project/dpdk/cover/20220924163542.4159562-1-gak...@marvell.com/ Depends-on: s

[PATCH 1/2] test/crypto: fix unexpected test abort

rte_cryptodev_sym_session_create now updates rte_errno. So rte_errno should be reset before executing current test case. This prevents current case from being aborted due to errno set by previous negative case. Signed-off-by: Ruifeng Wang --- app/test/test_cryptodev.c | 1 + 1 file changed, 1 in

[PATCH 2/2] crypto/armv8: adapt to updated sym session framework

Adapt to data structure changes. Signed-off-by: Ruifeng Wang --- drivers/crypto/armv8/armv8_pmd_private.h | 2 -- drivers/crypto/armv8/rte_armv8_pmd.c | 4 ++-- drivers/crypto/armv8/rte_armv8_pmd_ops.c | 5 ++--- 3 files changed, 4 insertions(+), 7 deletions(-) diff --git a/drivers/crypto/a

[PATCH v5] eal: fix thread names for high order lcores

In this patch we suggest a new name for lcore-worker. In case of higher order (greater than 99) logical cores, name is truncated (length is restricted to 16 characters, including the terminating null byte ('\0')) and it makes hard to follow threads. Ex: This issue can be generated using following

RE: [PATCH v3 0/6] crypto/security session framework rework

> -Original Message- > From: Akhil Goyal > Sent: Sunday, September 25, 2022 12:36 AM > To: dev@dpdk.org > Cc: tho...@monjalon.net; david.march...@redhat.com; hemant.agra...@nxp.com; > vattun...@marvell.com; ferruh.yi...@xilinx.com; andrew.rybche...@oktetlabs.ru; > konstantin.v.anan...@yand

RE: [PATCH v3 0/6] crypto/security session framework rework

> > Subject: [PATCH v3 0/6] crypto/security session framework rework > > > > This patchset reworks the symmetric crypto and security session data > > structure > to use a > > single virtual/physical contiguous buffer for symmetric crypto/security > > session > and driver > > private data. > > In

RE: [PATCH v2 1/3] net/bonding: support Tx prepare

Hi everyone, Sorry for late reply. > > The main problem is hard to design a tx_prepare for bonding device: > > 1. as Chas Williams said, there maybe twice hash calc to get target > > slave > >     devices. > > 2. also more important, if the slave devices have changes(e.g.

Re: [PATCH v3] eal: update lcore-worker name due to high number of cores

On 26/09/2022 10:39, David Marchand wrote: Hello, On Fri, Sep 23, 2022 at 9:06 AM Abdullah Ömer Yamaç wrote: In this patch we suggest a new name for lcore-worker. In case of more than 99 logical cores, name is truncated (length is restricted to 16 characters, including the terminating null by

[PATCH v3] examples/ipsec-secgw: free event vector mbufs

Free mbufs from event vector list when enqueue operation fails and during event port flush for cleanup. Signed-off-by: Rahul Bhansali --- v3: updated args of rte_pktmbuf_free_bulk as per v2 dependent series. v2: Added dependent series info. Depends-on: series-24761 ("[v2,1/3] eventdev: add elem

RE: [PATCH 1/3] eventdev: introduce event cryptodev vector type

> -Original Message- > From: Gujjar, Abhinandan S > Sent: Saturday, September 24, 2022 10:44 AM > To: Volodymyr Fialko ; dev@dpdk.org; Jerin Jacob > Kollanukkaran > ; Pavan Nikhilesh Bhagavatula ; > Shijith Thotton > ; Hemant Agrawal ; Sachin Saxena > ; Jayatheerthan, Jay > Cc: Akhil

[PATCH v2 0/3] Vector support for event crypto adapter

Introduce ability to aggregate crypto operations processed by event crypto adapter into single event containing rte_event_vector whose event type is RTE_EVENT_TYPE_CRYPTODEV_VECTOR. * Changes since v1 - Add missing newline/space issues - Add missing error messages - Remove double check of conf inp

[PATCH v2 1/3] eventdev: introduce event cryptodev vector type

Introduce ability to aggregate crypto operations processed by event crypto adapter into single event containing rte_event_vector whose event type is RTE_EVENT_TYPE_CRYPTODEV_VECTOR. Application should set RTE_EVENT_CRYPTO_ADAPTER_EVENT_VECTOR in rte_event_crypto_adapter_queue_conf::flag and provid

[PATCH v2 2/3] crypto/cnxk: add burst enqueue for event crypto

Added support for burst enqueue for cn10k event crypto adapter. Instruction will be grouped based on the queue pair and sent in a burst. Signed-off-by: Volodymyr Fialko --- drivers/crypto/cnxk/cn10k_cryptodev_ops.c | 147 -- drivers/crypto/cnxk/cn10k_cryptodev_ops.h | 7 +-

[PATCH v2 3/3] crypto/cnxk: add vectorization for event crypto

Add support for vector aggregation of crypto operations for cn10k. Crypto operations will be grouped by sub event type, flow id, scheduler type and queue id fields from rte_event_crypto_metadata::response_info. Signed-off-by: Volodymyr Fialko --- drivers/crypto/cnxk/cn10k_cryptodev_ops.c | 403

Re: [PATCH 0/3] mempool: apply deprecations

On Thu, Jul 28, 2022 at 01:38:04PM +0200, David Marchand wrote: > On Thu, Jul 28, 2022 at 11:36 AM Andrew Rybchenko > wrote: > > > > Remove deprecated macros. > > > > Andrew Rybchenko (3): > > mempool: remove deprecated macro to calculate header size > > mempool: remove deprecated macro to reg

Re: [PATCH] mbuf: remove deprecated offload flags

On Mon, Aug 22, 2022 at 05:26:34PM +0200, David Marchand wrote: > On Fri, Aug 5, 2022 at 10:03 AM Andrew Rybchenko > wrote: > > > > Remove deprecated ``PKT_*`` flags. Use corresponding flags with > > ``RTE_MBUF_F_`` prefix instead. > > > > Signed-off-by: Andrew Rybchenko Acked-by: Olivier Matz

Re: [PATCH v3 00/13] telemetry JSON escaping and other enhancements

On Fri, Sep 9, 2022 at 11:36 AM Bruce Richardson wrote: > > This patchset contains fixes for the problem of handling characters returned > by > telemetry callbacks which require escaping when encoded in JSON format. It > also > includes unit tests to validate the correct encoding in such scenari

RE: [PATCH v7 4/7] app/procinfo: add module eeprom info dump

> -Original Message- > From: Dongdong Liu > Subject: [PATCH v7 4/7] app/procinfo: add module eeprom info dump > > From: "Min Hu (Connor)" > > This patch add support for module eeprom info dump. > > The command is like: > dpdk-proc-info -a :xx:xx.x --file-prefix=xxx -- --show-mod

Re: [PATCH v5] usertools: rewrite pmdinfo

On Thu, Sep 22, 2022 at 01:58:02PM +0200, Robin Jarry wrote: > dpdk-pmdinfo.py does not produce any parseable output. The -r/--raw flag > merely prints multiple independent JSON lines which cannot be fed > directly to any JSON parser. Moreover, the script complexity is rather > high for such a simp

[PATCH] eventdev: fix name of Rx conf type

Rename configuration structure type to correspond definition. Fixes: dcc806c2638 ("eventdev: add eth Rx adapter API") Signed-off-by: Volodymyr Fialko --- lib/eventdev/rte_event_eth_rx_adapter.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/eventdev/rte_event_eth_rx_ada

RE: [PATCH v7 6/7] app/procinfo: fix some wrong doxygen syntax

> -Original Message- > From: Dongdong Liu > Signed-off-by: Dongdong Liu If no need to follow style of API comments I am fine with the changes. Acked-by: Reshma Pattan

Re: [PATCH 0/3] mempool: apply deprecations

On Mon, Sep 26, 2022 at 1:43 PM Olivier Matz wrote: > > On Thu, Jul 28, 2022 at 01:38:04PM +0200, David Marchand wrote: > > On Thu, Jul 28, 2022 at 11:36 AM Andrew Rybchenko > > wrote: > > > > > > Remove deprecated macros. > > > > > > Andrew Rybchenko (3): > > > mempool: remove deprecated macro

Re: [PATCH] mbuf: remove deprecated offload flags

On Mon, Sep 26, 2022 at 1:52 PM Olivier Matz wrote: > > On Mon, Aug 22, 2022 at 05:26:34PM +0200, David Marchand wrote: > > On Fri, Aug 5, 2022 at 10:03 AM Andrew Rybchenko > > wrote: > > > > > > Remove deprecated ``PKT_*`` flags. Use corresponding flags with > > > ``RTE_MBUF_F_`` prefix instead.

RE: [dpdk-dev] [PATCH v2 1/2] net/mlx5: fix use after free when releasing tx queues

Hi, Yunjian Could you, please, tell more details about problematic scenario? In bonding slave? It is not fully clean for me how mlx5_txq_release frees priv->txqs[idx] (BTW NULL is OK to free, it is safe). We have check for NULL here: > > - if (priv->txqs == NULL || (*priv->txqs)[idx] == NULL) p

Re: [PATCH 1/3] net: add MACsec header

Hi Akhil, Few comments below. On Mon, Aug 15, 2022 at 12:16:18AM +0530, Akhil Goyal wrote: > Added MACsec protocol header to be used for supporting > MACsec protocol offload in hardware or directly in the application. > > Signed-off-by: Akhil Goyal > --- > doc/api/doxy-api-index.md | 3 ++- >

Re: [PATCH v5] usertools: rewrite pmdinfo

Robin Jarry, Sep 22, 2022 at 13:58: > +# > > +def ld_so_path() -> Iterator[str]: > +""" > +Return the list of directories where dynamic libraries are loaded based > +on the contents of /etc/ld.so.conf/*.conf.

Re: DPDK 19.11.5 Legacy Memory Design Query

Thanks @Dmitry Kozlyuk for your suggestions I will try the following for DPDK pool creation My logic of calculating MBUF’s remains same Saw this code in DPDK testpmd where External heap memory is used case MP_ALLOC_XMEM_HUGE: {

Re: [PATCH v3] ethdev: add send to kernel action

On 9/19/22 18:50, Michael Savisko wrote: In some cases application may receive a packet that should have been received by the kernel. In this case application uses KNI or other means to transfer the packet to the kernel. With bifurcated driver we can have a rule to route packets matching a patte

Re: DPDK 19.11.5 Legacy Memory Design Query

Hi Dimitry We know If the application does unmap, DPDK native heaps are not getting cleaned up in “native: use regular DPDK memory” memory type. Can we get an API from DPDK community please to clean up the heaps given the VA and length to be freed ? Like below https://doc.dpdk.org/api/rte__m

Re: [PATCH v2] ethdev: queue rate parameter changed from 16b to 32b

On 9/23/22 16:45, skotesh...@marvell.com wrote: From: Satha Rao The rate parameter modified to uint32_t, so that it can work for more than 64 Gbps. Signed-off-by: Satha Rao Overall LGTM, but please update release notes and cleanup deprecation in the next version. However, the patch require

RE: [EXT] Re: [PATCH 1/3] net: add MACsec header

Hi Olivier, Thanks for your review. I will fix the issues in next version. > Hi Akhil, > > Few comments below. > > On Mon, Aug 15, 2022 at 12:16:18AM +0530, Akhil Goyal wrote: > > Added MACsec protocol header to be used for supporting > > MACsec protocol offload in hardware or directly in the a

[PATCH v6] usertools: rewrite pmdinfo

dpdk-pmdinfo.py does not produce any parseable output. The -r/--raw flag merely prints multiple independent JSON lines which cannot be fed directly to any JSON parser. Moreover, the script complexity is rather high for such a simple task: extracting PMD_INFO_STRING from .rodata ELF sections. Rewrit

Re: [PATCH V3 2/7] table: add entry ID for learner tables

26/09/2022 11:19, Dumitrescu, Cristian: > > > -Original Message- > > From: Thomas Monjalon > > Sent: Saturday, September 24, 2022 9:25 AM > > To: Dumitrescu, Cristian > > Cc: dev@dpdk.org > > Subject: Re: [PATCH V3 2/7] table: add entry ID for learner tables > > > > 30/08/2022 20:58, Cr

[PATCH] net/ice: check process type in reset operation

The secondary processes are not allowed to release shared resources. so the stop and reset operation are forbidden in a secondary process. Signed-off-by: Qiming Yang --- drivers/net/ice/ice_ethdev.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/drivers/net/ice/ice_ethdev.c b/drivers/

Re: [PATCH v5 1/3] ethdev: add IPsec SA expiry event subtypes

24/09/2022 15:57, Akhil Goyal: > From: Vamsi Attunuru > > Patch adds new event subtypes for notifying expiry > events upon reaching IPsec SA soft packet expiry and > hard packet/byte expiry limits. > > Signed-off-by: Vamsi Attunuru > Signed-off-by: Akhil Goyal > --- > --- a/lib/ethdev/rte_ethd

RE: [PATCH] eventdev: fix name of Rx conf type

Looks good to me. Thanks for catching this one! Acked-by: Jay Jayatheerthan > -Original Message- > From: Volodymyr Fialko > Sent: Monday, September 26, 2022 5:28 PM > To: dev@dpdk.org; Jayatheerthan, Jay ; Jerin > Jacob ; Rao, Nikhil > > Cc: gak...@marvell.com; ano...@marvell.com; Gu

[PATCH v5 00/10] dts: ssh connection to a node

All the necessary code needed to connect to a node in a topology with a bit more, such as basic logging and some extra useful methods. To run the code, modify the config file, conf.yaml and execute ./main.py from the root dts folder. Here's an example config: executions: - system_under_test: "SU

[PATCH v5 01/10] dts: add project tools config

Add configuration for Python tools used in DTS: Poetry, dependency and package manager Black, formatter Pylama, static analysis Isort, import sorting Add Python and DTS specifics to .gitignore and .editorconfig. Of note is the change of maximum line length of Python code to 88, which is a good com

[PATCH v5 03/10] dts: add config parser module

From: Owen Hilyard The configuration is split into two parts, one defining the parameters of the test run and the other defining the topology to be used. The format of the configuration is YAML. It is validated according to a json schema which also server as detailed documentation of the various

[PATCH v5 04/10] dts: add basic logging facility

The logging module provides loggers distinguished by two attributes, a custom format and a verbosity switch. The loggers log to both console and more verbosely to files. Signed-off-by: Owen Hilyard Signed-off-by: Juraj Linkeš --- dts/framework/__init__.py | 4 ++ dts/framework/logger.py | 1

[PATCH v5 02/10] dts: add developer tools

The Dockerfile contains basic image for CI and developers. There's also an integration of the Dockerfile with Visual Studio. The devtools that check Python code are Black and Isort to format the code and Pylama to do static analysis. Signed-off-by: Owen Hilyard Signed-off-by: Juraj Linkeš ---

[PATCH v5 05/10] dts: add remote session abstraction

The abstraction allows for easy switching of implementations of remote connections (ssh, telnet, etc.). It implements some common features, such as logging of commands and their outputs and history bookkeeping and defines methods that must be implemented by derived classes. Signed-off-by: Owen Hil

[PATCH v5 06/10] dts: add ssh connection module

The module uses the pexpect python library and implements connection to a node and two ways to interact with the node: 1. Send a string with specified prompt which will be matched after the string has been sent to the node. 2. Send a command to be executed. No prompt is specified here. Signed-o

[PATCH v5 07/10] dts: add node base class

The base class implements basic node management methods - connect and execute commands. Signed-off-by: Owen Hilyard Signed-off-by: Juraj Linkeš --- dts/framework/testbed_model/__init__.py | 8 +++ dts/framework/testbed_model/node.py | 83 + 2 files changed, 91 inser

[PATCH v5 08/10] dts: add dts workflow module

The module implements methods needed to run DTS. It handles the creation of objects and eventually the whole DTS workflow, such as running node setups, test gathering, setup and execution and various cleanups. Signed-off-by: Owen Hilyard Signed-off-by: Juraj Linkeš --- dts/framework/dts.py |

[PATCH v5 09/10] dts: add dts executable script

The script is an interface to run DTS. Signed-off-by: Owen Hilyard Signed-off-by: Juraj Linkeš --- dts/main.py | 24 1 file changed, 24 insertions(+) create mode 100755 dts/main.py diff --git a/dts/main.py b/dts/main.py new file mode 100755 index 00..a70070765

[PATCH v5 10/10] maintainers: add dts maintainers

Signed-off-by: Juraj Linkeš --- MAINTAINERS | 5 + 1 file changed, 5 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 32ffdd1a61..32a6493502 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -114,6 +114,11 @@ F: buildtools/symlink-drivers-solibs.py F: devtools/test-meson-builds.sh F:

Re: [dpdk-dev] [PATCH] ethdev: avoid unregistering a non-allocated callback

This patch is abandoned. Current behaviour is kept. 15/07/2021 11:06, Ferruh Yigit: > On 7/14/2021 4:42 PM, Thomas Monjalon wrote: > > 14/07/2021 16:16, Matan Azrad: > >> From: Thomas Monjalon > >>> 13/07/2021 15:42, Matan Azrad: > From: Thomas Monjalon > > When registering a new event c

Re: [dpdk-dev] [PATCH 1/2] lib/ethdev: introduce RTE_ETH_DEV_CAPA_FLOW_CREATE_BEFORE_START

There was no follow up to this patch. Ori, could you give your opinion please? 12/10/2021 14:02, Andrew Rybchenko: > Summary prefix should be just "ethdev: " and summary should > be human readable. It should not refer to defines etc. > > On 10/5/21 8:19 PM, Martin Havlik wrote: > > Not all PMDs

Re: [PATCH v2] ethdev: add GTP PSC QFI field

On 9/21/22 17:32, Dariusz Sosnowski wrote: This patch introduces the GTP PSC QFI modify field support and adds the testpmd CLI command support. An example for copying GTP QFI field using modify_field action: modify_field op set dst_type meta src

[PATCH v5 0/7] ethdev: separate metering and marking from policing

Extend Metering and Marking support in the Flow API: 1. Add METER_COLOR item to match Color Marker set by a Meter. 2. Add the ability to set Color Marker via modify_field Flow API. 3. Add Meter API to get profile/policy objects. 4. Add METER_MARK action to perform Meter color metering and marking.

[PATCH v5 1/7] ethdev: add meter color flow matching item

Provide an ability to use a Color Marker set by a Meter as a matching item in Flow API. The Color Marker reflects the metering result by setting the metadata for a packet to a particular codepoint: green, yellow or red. Signed-off-by: Alexander Kozyrev --- doc/guides/prog_guide/rte_flow.rst

[PATCH v5 2/7] ethdev: allow meter color marker modification

Extend modify_field Flow API with support of Meter Color Marker modifications. It allows setting the packet's metadata to any color marker: green, yellow or red. A user is able to specify an initial packet color for Meter API or create simple Metering and Marking flow rules based on his own colorin

[PATCH v5 4/7] ethdev: add meter color mark flow action

Create a new Flow API action: METER_MARK. It Meters a packet stream and marks its packets with colors. The marking is done on a metadata, not on a packet field. Unlike the METER action, it performs no policing at all. A user has the flexibility to create any policies with the help of the METER_COLO

[PATCH v5 3/7] ethdev: get meter profile/policy objects

Introduce a new Meter API to retrieve a Meter profile and policy objects using the profile/policy ID previously created with meter_profile_add() and meter_policy_create() functions. That allows to save the pointer and avoid any lookups in the corresponding lists for quick access during a flow rule

[PATCH v5 6/7] app/testpmd: allow meter color marker modification

Extend the list of available modify_field IDs to include recently added meter color marker item in testpmd CLI. Signed-off-by: Alexander Kozyrev --- app/test-pmd/cmdline_flow.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/app/test-pmd/cmdline_flow.c b/app/test-pmd/cmdline_

[PATCH v5 5/7] app/testpmd: add meter color flow matching item

Add testpmd command line to match on a meter color: flow create 0 ingress group 0 pattern meter color is green / end Signed-off-by: Alexander Kozyrev --- app/test-pmd/cmdline_flow.c | 83 + doc/guides/testpmd_app_ug/testpmd_funcs.rst | 4 + 2 files ch

[PATCH v5 7/7] app/testpmd: add meter color mark flow action

Add testpmd command line to match for METER_MARK action: flow create ... actions meter_mark mtr_profile 20 / end Signed-off-by: Alexander Kozyrev --- app/test-pmd/cmdline_flow.c | 153 app/test-pmd/config.c | 45 +- app/test

Re: [EXT] Re: [PATCH 1/6] ethdev: add trace points

@Jerin, I'd like to know what do you think about my question/thoughts below. On 9/13/22 10:18, Andrew Rybchenko wrote: On 9/13/22 09:48, Ankur Dwivedi wrote: Hi Andrew, -Original Message- From: Andrew Rybchenko Sent: Monday, September 12, 2022 4:30 PM To: Ankur Dwivedi ; dev@dpdk.org

[PATCH] app/eventdev: fix incorrect cleanup check

From: Pavan Nikhilesh During cleanup `rte_event_port_quiesce` should be called irrespective of whether an event has been dequeued or not to flush any prefetched events. Fixes: 7da008df0c8d ("app/eventdev: use port quiescing") Signed-off-by: Pavan Nikhilesh --- app/test-eventdev/test_pipeline_

Re: [PATCH v6] usertools: rewrite pmdinfo

On Mon, Sep 26, 2022 at 03:44:38PM +0200, Robin Jarry wrote: > dpdk-pmdinfo.py does not produce any parseable output. The -r/--raw flag > merely prints multiple independent JSON lines which cannot be fed > directly to any JSON parser. Moreover, the script complexity is rather > high for such a simp

  1   2   >