[dpdk-dev] [PATCH v6 4/5] net/ice: add input set byte number check

2020-07-02 Thread Wei Zhao
This patch add the total input set byte number check, as there is a hardware requirement for the total number of 32 byte. Fixes: 47d460d63233 ("net/ice: rework switch filter") Cc: sta...@dpdk.org Signed-off-by: Wei Zhao --- drivers/net/ice/ice_switch_filter.c | 43 +++--

[dpdk-dev] Questions about rte_flow_create APIs

2020-07-02 Thread oulijun
Hi, Guys I am analyzing rte_flow_create API and test and is testing server parametric scenario. When use testpmd and the following flow creat cmd: testpmd> flow create 0 ingress pattern end actions rss func simple_xor / end why the contents of rte_flow_action_rss is not same acquisition m

[dpdk-dev] [PATCH v6 3/5] net/ice: support switch flow for specific L4 type

2020-07-02 Thread Wei Zhao
This patch add more specific tunnel type for ipv4/ipv6 packet, it enable tcp/udp layer of ipv4/ipv6 as L4 payload but without L4 dst/src port number as input set for the switch filter rule. Fixes: 47d460d63233 ("net/ice: rework switch filter") Cc: sta...@dpdk.org Signed-off-by: Wei Zhao --- dri

[dpdk-dev] [PATCH v6 2/5] net/ice: fix tunnel type for switch rule

2020-07-02 Thread Wei Zhao
This patch add check for protocol type of IPv4 packet, it need to update tunnel type when NVGRE is in payload. Fixes: 6bc7628c5e0b ("net/ice: change default tunnel type") Cc: sta...@dpdk.org Signed-off-by: Wei Zhao --- drivers/net/ice/ice_switch_filter.c | 7 ++- 1 file changed, 6 insertion

[dpdk-dev] [PATCH v6 5/5] net/ice: fix typo

2020-07-02 Thread Wei Zhao
fix typo of "valid". Fixes: 8f5d8e74fb38 ("net/ice: support flow for AH ESP and L2TP") Fixes: 66ff8851792f ("net/ice: support ESP/AH/L2TP") Fixes: 45b53ed3701d ("net/ice: support IPv6 NAT-T") Cc: sta...@dpdk.org Signed-off-by: Wei Zhao --- drivers/net/ice/ice_switch_filter.c | 76 ++

[dpdk-dev] [PATCH v6 1/5] net/ice: add support more PPPoE packeat type for switch

2020-07-02 Thread Wei Zhao
This patch add more support for switch parser of pppoe packet, it enable parse tcp/udp L4 layer and ipv4/ipv6 L3 layer parser for pppoe payload, so we can use L4 dst/src port and L3 ip address as input set for switch filter pppoe related rule. Signed-off-by: Wei Zhao --- doc/guides/rel_notes/rel

[dpdk-dev] [PATCH v6 0/5] enable more PPPoE packet type for switch

2020-07-02 Thread Wei Zhao
1. add more support for switch parser of pppoe packet. 2. add check for NVGRE protocol 3. support flow for specific L4 type 4. add input set byte number check 5. fix typo This patchset is based on: [1] https://patches.dpdk.org/cover/70762/ : net/ice: base code update Depends-on: series-10300 v2:

Re: [dpdk-dev] [PATCH v2 1/7] ethdev: introduce sample action for rte flow

2020-07-02 Thread Jerin Jacob
On Fri, Jul 3, 2020 at 12:13 AM Jiawei Wang wrote: > > When using full offload, all traffic will be handled by the HW, and > directed to the requested vf or wire, the control application loses > visibility on the traffic. > So there's a need for an action that will enable the control application >

Re: [dpdk-dev] [PATCH v3 1/2] eal: fix memory leak when removing event_cb

2020-07-02 Thread Jeff Guo
hi, yunjian On 7/2/2020 7:47 PM, wangyunjian wrote: From: Yunjian Wang The event_cb->dev_name is not freed when freeing event_cb, and this causes a memory leak. Fixes: a753e53d517b ("eal: add device event monitor framework") Cc: sta...@dpdk.org Signed-off-by: Yunjian Wang --- lib/librte_e

Re: [dpdk-dev] [PATCH v3 4/5] net/i40e: enable flow query RSS

2020-07-02 Thread Jeff Guo
On 7/1/2020 4:24 PM, Chenxu Di wrote: This patch enables flow query function to get the configuration of the specified rule. Signed-off-by: Chenxu Di --- doc/guides/rel_notes/release_20_08.rst | 1 + drivers/net/i40e/i40e_flow.c | 49 ++ 2 files changed,

Re: [dpdk-dev] [PATCH v3 3/5] app/testpmd: re-implement commands by using private API

2020-07-02 Thread Jeff Guo
hi, chenxu Please use --in-relpy-to  next time to help reviewers check all relevant comments, and some minor comment, after fix it you could bring my ACK in the coming version, thanks. On 7/1/2020 4:24 PM, Chenxu Di wrote: The legacy filter API will be superseded. This patch use private ap

[dpdk-dev] [PATCH v2 8/8] test/crypto: use generic testsuite for OCTEON TX/TX2

2020-07-02 Thread Ankur Dwivedi
From: Anoob Joseph This patch enables the generic crypto tests for OCTEON TX and OCTEON TX2 PMDs. Removes the PMD specific tests. Signed-off-by: Ankur Dwivedi Signed-off-by: Anoob Joseph --- app/test/test_cryptodev.c | 398 +- 1 file changed, 2 inse

Re: [dpdk-dev] [PATCH v3 1/5] net/i40e: add private APIs

2020-07-02 Thread Jeff Guo
On 7/1/2020 4:24 PM, Chenxu Di wrote: The legacy filter API will be superseded by rte_flow. There are also several small features which can not be implemented in rte_flow. This patch re-implemented these features as private API. Three APIs are added: rte_pmd_i40e_get_fdir_info. rte_pmd_i40e_get

Re: [dpdk-dev] [PATCH v3 2/5] net/ixgbe: add private APIs

2020-07-02 Thread Jeff Guo
On 7/1/2020 4:24 PM, Chenxu Di wrote: The legacy filter API will be superseded by rte_flow. There are also several small features which can not be implemented in rte_flow. This patch re-implemented these features as private API. Two APIs are added: rte_pmd_ixgbe_get_fdir_info. rte_pmd_ixgbe_get

[dpdk-dev] [PATCH v2 3/8] crypto/octeontx: set session private data

2020-07-02 Thread Ankur Dwivedi
This patch sets the first 32 bytes of session private data to zero. This prevents garbage data to be used in code logic. Signed-off-by: Ankur Dwivedi --- drivers/crypto/octeontx/otx_cryptodev_ops.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/crypto/octeontx/otx_cryptodev_ops.c

[dpdk-dev] [PATCH v2 4/8] crypto/octeontx2: set session private data

2020-07-02 Thread Ankur Dwivedi
This patch sets the first 32 bytes of session private data to zero. This prevents garbage data to be used in code logic. Signed-off-by: Ankur Dwivedi --- drivers/crypto/octeontx2/otx2_cryptodev_ops.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/crypto/octeontx2/otx2_cryptodev_o

[dpdk-dev] [PATCH v2 6/8] crypto/octeontx2: revert non-byte aligned data feature

2020-07-02 Thread Ankur Dwivedi
This reverts commit 51f3e107aca23a1cbc1a5ad9fdce7921340307b5. For SNOW and ZUC algos the offset value for enryption and decryption is converted to bytes. Hence RTE_CRYPTODEV_FF_NON_BYTE_ALIGNED_DATA feature is not supported by the octeontx2 crypto pmd. Fixes: 51f3e107aca2 ("crypto/octeontx2: enab

[dpdk-dev] [PATCH v2 7/8] test/crypto: remove PMD specific tests

2020-07-02 Thread Ankur Dwivedi
From: Anoob Joseph This patch removes the OCTEON TX and OCTEON TX2 PMDs specific test cases related to null cipher. Signed-off-by: Ankur Dwivedi Signed-off-by: Anoob Joseph --- app/test/test_cryptodev.c | 371 -- 1 file changed, 371 deletions(-) di

[dpdk-dev] [PATCH v2 5/8] crypto/octeontx: revert non-byte aligned data feature

2020-07-02 Thread Ankur Dwivedi
This reverts commit 32b8f26adf8b26a55230408ff6adffd4b2327e52. For SNOW and ZUC algos the offset value for enryption and decryption is converted to bytes. Hence RTE_CRYPTODEV_FF_NON_BYTE_ALIGNED_DATA feature is not supported by the octeontx crypto pmd. Fixes: 32b8f26adf8b ("crypto/octeontx: enable

[dpdk-dev] [PATCH v2 1/8] common/cpt: fix encryption offset

2020-07-02 Thread Ankur Dwivedi
In case of gmac auth the encryption offset should be set to zero. Fixes: b74652f3a91f ("common/cpt: add microcode interface for encryption") Fixes: 177b41ceee61 ("common/cpt: add microcode interface for decryption") Cc: sta...@dpdk.org Signed-off-by: Ankur Dwivedi --- drivers/common/cpt/cpt_uco

[dpdk-dev] [PATCH v2 2/8] crypto/octeontx: remove stats callback

2020-07-02 Thread Ankur Dwivedi
The stats get and reset functions for octeontx crypto PMD are unimplemented. So removing them. Signed-off-by: Ankur Dwivedi --- drivers/crypto/octeontx/otx_cryptodev_ops.c | 17 ++--- 1 file changed, 2 insertions(+), 15 deletions(-) diff --git a/drivers/crypto/octeontx/otx_cryptodev

[dpdk-dev] 17.11.10 fails to build from source on arm64

2020-07-02 Thread Christian Ehrhardt
Hi, 17.11.10 contains: [1] 4cd784b0 "net/mlx5: fix Rx CQ doorbell synchronization on aarch64" Which is a backport from 94a24aaf But if you build on arm64 with MLX5 enabled that breaks like: mlx5_rxtx_vec_neon.h:1035: undefined reference to `rte_cio_wmb' I found and confirmed that adding the f

[dpdk-dev] [dpdk-dev v4 4/4] app/testpmd: add GTPU to RSS hash commands

2020-07-02 Thread Jeff Guo
Add testpmd cmdline support for GTPU, it could be used to configure gtpu teid hash. The commands as below: testpmd> flow create 0 ingress pattern eth / ipv4 / udp / gtpu / \ ipv4 / end actions rss types gtpu end key_len 0 queues end / end Signed-off-by: Jeff Guo --- app/test-pmd/cmdline.c | 8

[dpdk-dev] [dpdk-dev v4 2/4] net/ice: support hash for new GTPU protocols

2020-07-02 Thread Jeff Guo
Support hash flow for the new protoclos of GTPU_EH UPLINK, GTPU_EH DOWNLINK and GTPU_IP. Signed-off-by: Jeff Guo --- drivers/net/ice/ice_hash.c | 67 ++ 1 file changed, 54 insertions(+), 13 deletions(-) diff --git a/drivers/net/ice/ice_hash.c b/drivers/net/ic

[dpdk-dev] [dpdk-dev v4 3/4] net/ice: enable new input set for rss hash

2020-07-02 Thread Jeff Guo
PF could add or delete a RSS rule base on the PF's hash capability. Some new rss input set will be supported, the protocols as below: eth/vlan/l2tpv3/esp/ah/pfcp. Signed-off-by: Jeff Guo --- drivers/net/ice/ice_hash.c | 104 + 1 file changed, 104 insertions(+)

[dpdk-dev] [dpdk-dev v4 0/4] enable new hash flow for pf

2020-07-02 Thread Jeff Guo
PF could add or delete a RSS rule base on the PF's hash capability. Some new rss input set will be supported, the protocols as below: eth/vlan/l2tpv3/esp/ah/pfcp/gtpu downlink/gtpu uplink. Jeff Guo (4): net/ice: refactor for pf hash flow net/ice: support hash for new GTPU protocols net/ice:

[dpdk-dev] [dpdk-dev v4 1/4] net/ice: refactor for pf hash flow

2020-07-02 Thread Jeff Guo
Refactor hash flow by change the action parser and clean some code. Signed-off-by: Jeff Guo --- drivers/net/ice/ice_hash.c | 562 +++-- 1 file changed, 355 insertions(+), 207 deletions(-) diff --git a/drivers/net/ice/ice_hash.c b/drivers/net/ice/ice_hash.c index

[dpdk-dev] [dpdk-dev v4 3/3] net/iavf: enable some new hash flow

2020-07-02 Thread Jeff Guo
Some new hash flow will be supported to expend the flow hash capability, the input set are the session id for NAT-T ESP protocol, the l3 src/dst and the teid for GTPU_IP protocol. Signed-off-by: Jeff Guo --- drivers/net/iavf/iavf_hash.c | 76 ++-- 1 file changed,

[dpdk-dev] [dpdk-dev v4 1/3] net/iavf: refactor for hash flow

2020-07-02 Thread Jeff Guo
Refactor hash flow by change the process of the pattern parser and the action parser, and refine the lookup table for regular IP and GTPU_EH. Signed-off-by: Jeff Guo --- drivers/net/iavf/iavf_hash.c | 1658 -- 1 file changed, 1163 insertions(+), 495 deletions(-)

[dpdk-dev] [dpdk-dev v4 2/3] net/iavf: enable 5 tuple rss hash

2020-07-02 Thread Jeff Guo
Previous iavf not support 5 tuple hash, this patch aims to enable it for regular ip pattern and also GTPU inner ip pattern, the 5 tuple involves ip src and ip dst, tcp sport and tcp dport, udp sport and udp dport and protocol id. Signed-off-by: Jeff Guo --- drivers/net/iavf/iavf_hash.c | 186 +++

[dpdk-dev] [dpdk-dev v4 0/3] enable new hash flow for VF

2020-07-02 Thread Jeff Guo
Some new hash flow will be supported to expend the flow hash capability, the input set are the 5 tuple for regular ip pattern and also GTPU inner ip pattern, and the session id for NAT-T ESP protocol, the l3 src/dst and the teid for GTPU_IP protocol. Jeff Guo (3): net/iavf: refactor for hash flo

Re: [dpdk-dev] [dpdk-dev v3 2/2] net/ice: enable new input set for rss hash

2020-07-02 Thread Jeff Guo
hi, qi Please expect the separate patch on next version and hope it would help to review again, thanks. On 7/1/2020 9:02 PM, Zhang, Qi Z wrote: Please also separate this patch into code refactor purpose and new features -Original Message- From: Guo, Jia Sent: Sunday, June 21, 2020

Re: [dpdk-dev] [dpdk-dev v3] net/iavf: add inner 5 tuple hash for GTPU

2020-07-02 Thread Jeff Guo
hi, qi Let me separate the patch and thanks your advise. On 7/1/2020 8:56 PM, Zhang, Qi Z wrote: -Original Message- From: Guo, Jia Sent: Sunday, June 21, 2020 10:03 PM To: Xing, Beilei ; Zhang, Qi Z ; Wu, Jingjing Cc: Ye, Xiaolong ; dev@dpdk.org; Guo, Junfeng ; Guo, Jia Subject: [d

[dpdk-dev] [PATCH v4 50/51] net/bnxt: add support for count action in flow query

2020-07-02 Thread Ajit Khaparde
From: Somnath Kotur Use the flow counter manager to fetch the accumulated stats for a flow. Signed-off-by: Somnath Kotur Signed-off-by: Venkat Duvvuru Reviewed-by: Ajit Khaparde --- drivers/net/bnxt/tf_ulp/bnxt_ulp_flow.c | 45 +++- drivers/net/bnxt/tf_ulp/ulp_fc_mgr.c| 141

[dpdk-dev] [PATCH v4 49/51] net/bnxt: add ULP Flow counter Manager

2020-07-02 Thread Ajit Khaparde
From: Somnath Kotur The Flow counter manager allocates memory to hold the software view of the counters where the on-chip counter data will be accumulated along with another memory block that will be shadowing the on-chip counter data i.e where the raw counter data will be DMAed into from the chi

[dpdk-dev] [PATCH v4 51/51] doc: update release notes

2020-07-02 Thread Ajit Khaparde
Update release notes with enhancements in Broadcom PMD. Signed-off-by: Ajit Khaparde --- doc/guides/rel_notes/release_20_08.rst | 11 ++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --git a/doc/guides/rel_notes/release_20_08.rst b/doc/guides/rel_notes/release_20_08.rst index 5

[dpdk-dev] [PATCH v4 46/51] net/bnxt: create default flow rules for the VF-rep

2020-07-02 Thread Ajit Khaparde
From: Somnath Kotur Invoked 3 new APIs for the default flow create/destroy and to get the action ptr for a default flow. Changed ulp_intf_update() to accept rte_eth_dev as input and invoke the same from the VF rep start function. ULP Mark Manager will indicate if the cfa_code returned in the Rx c

[dpdk-dev] [PATCH v4 41/51] net/bnxt: enhancements for port db

2020-07-02 Thread Ajit Khaparde
From: Venkat Duvvuru 1. Add "enum bnxt_ulp_intf_type” as the second parameter for the port & func helper functions 2. Return vfrep related port & func information in the helper functions 3. Allocate phy_port_list dynamically based on port count 4. Introduce ulp_func_id_tbl array for book keepi

[dpdk-dev] [PATCH v4 47/51] net/bnxt: add port default rules for ingress and egress

2020-07-02 Thread Ajit Khaparde
From: Venkat Duvvuru ingress & egress port default rules are needed to send the packet from port_to_dpdk & dpdk_to_port respectively. Signed-off-by: Venkat Duvvuru Reviewed-by: Somnath Kotur Reviewed-by: Ajit Khaparde --- drivers/net/bnxt/bnxt_ethdev.c | 76 +-

[dpdk-dev] [PATCH v4 42/51] net/bnxt: manage VF to VFR conduit

2020-07-02 Thread Ajit Khaparde
From: Venkat Duvvuru When VF-VFR conduits are created, a mark is added to the mark database. mark_flag indicates whether the mark is valid and has VFR information (VFR_ID bit in mark_flag). Rx path was checking for this VFR_ID bit. However, while adding the mark to the mark database, VFR_ID bit i

[dpdk-dev] [PATCH v4 44/51] net/bnxt: fill mapper parameters with default rules

2020-07-02 Thread Ajit Khaparde
From: Venkat Duvvuru Default rules are needed for the packets to be punted between the following entities in the non-offloaded path 1. Device PORT to DPDK App 2. DPDK App to Device PORT 3. VF Representor to VF 4. VF to VF Representor This patch fills all the relevant information in the computed

[dpdk-dev] [PATCH v4 48/51] net/bnxt: fill cfa action in the Tx descriptor

2020-07-02 Thread Ajit Khaparde
From: Venkat Duvvuru Currently, only vfrep transmit requires cfa_action to be filled in the tx buffer descriptor. However with truflow, dpdk(non vfrep) to port also requires cfa_action to be filled in the tx buffer descriptor. This patch uses the correct cfa_action pointer while transmitting the

[dpdk-dev] [PATCH v4 38/51] net/bnxt: add support for internal exact match entries

2020-07-02 Thread Ajit Khaparde
From: Kishore Padmanabha Added support for the internal exact match entries. Signed-off-by: Kishore Padmanabha Signed-off-by: Venkat Duvvuru Reviewed-by: Somnath Kotur Reviewed-by: Ajit Khaparde --- drivers/net/bnxt/tf_ulp/bnxt_ulp.c| 38 +++ drivers/net/bnxt/tf_

[dpdk-dev] [PATCH v4 43/51] net/bnxt: parse reps along with other dev-args

2020-07-02 Thread Ajit Khaparde
From: Somnath Kotur Representor dev-args need to be parsed during pci probe as they determine subsequent probe of VF representor ports as well. Signed-off-by: Venkat Duvvuru Signed-off-by: Somnath Kotur Reviewed-by: Ajit Khaparde --- drivers/net/bnxt/bnxt_ethdev.c | 2 ++ 1 file changed, 2 i

[dpdk-dev] [PATCH v4 31/51] net/bnxt: add support for EEM System memory

2020-07-02 Thread Ajit Khaparde
From: Peter Spreadborough - Select EEM Host or System memory via config parameter - Add EEM system memory support for kernel memory - Dependent on DPDK changes that add support for the HWRM_OEM_CMD. Signed-off-by: Peter Spreadborough Reviewed-by: Randy Schacher Signed-off-by: Venkat Duvvuru -

[dpdk-dev] [PATCH v4 39/51] net/bnxt: add support for conditional execution of mapper tables

2020-07-02 Thread Ajit Khaparde
From: Kishore Padmanabha Added support for conditional execution of the mapper tables so that actions like count will have table processed only if action count is configured. Signed-off-by: Kishore Padmanabha Signed-off-by: Venkat Duvvuru Reviewed-by: Somnath Kotur Reviewed-by: Ajit Khaparde

Re: [dpdk-dev] [PATCH v2 3/4] eventdev: remove redundant code

2020-07-02 Thread Dharmik Thakkar
Hi Phil, Looks good. > On Jul 2, 2020, at 12:26 AM, Phil Yang wrote: > > There is no thread will access these impl_opaque data after timer > canceled. When new timer armed, it got refilled. So the cleanup > process is unnecessary. > > Signed-off-by: Phil Yang > --- > lib/librte_eventdev/rte_ev

[dpdk-dev] [PATCH v4 40/51] net/bnxt: enable port MAC qcfg command for trusted VF

2020-07-02 Thread Ajit Khaparde
From: Venkat Duvvuru Issue HWRM_PORT_MAC_QCFG command on trusted vf to fetch the port count. Signed-off-by: Venkat Duvvuru Reviewed-by: Somnath Kotur Reviewed-by: Ajit Khaparde --- drivers/net/bnxt/bnxt_hwrm.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/ne

[dpdk-dev] [PATCH v4 35/51] net/bnxt: disable Tx vector mode if truflow is enabled

2020-07-02 Thread Ajit Khaparde
From: Kishore Padmanabha The vector mode in the tx handler is disabled when truflow is enabled since truflow now requires bd action record support. Signed-off-by: Kishore Padmanabha Signed-off-by: Somnath Kotur Signed-off-by: Venkat Duvvuru Reviewed-by: Ajit Khaparde --- drivers/net/bnxt/bn

[dpdk-dev] [PATCH v4 36/51] net/bnxt: add index opcode and operand to mapper table

2020-07-02 Thread Ajit Khaparde
From: Kishore Padmanabha Extended the regfile and computed field operations to a common index opcode operation and added globlal resource operations are also part of the index opcode operation. Signed-off-by: Kishore Padmanabha Signed-off-by: Venkat Duvvuru Reviewed-by: Somnath Kotur Reviewed

[dpdk-dev] [PATCH v4 37/51] net/bnxt: add support for global resource templates

2020-07-02 Thread Ajit Khaparde
From: Kishore Padmanabha Added support for the global resource templates, so that they can be reused by the other regular templates. Signed-off-by: Kishore Padmanabha Signed-off-by: Venkat Duvvuru Reviewed-by: Somnath Kotur Reviewed-by: Ajit Khaparde --- drivers/net/bnxt/tf_ulp/ulp_mapper.c

[dpdk-dev] [PATCH v4 33/51] net/bnxt: add support for internal encap records

2020-07-02 Thread Ajit Khaparde
From: Mike Baucom Modifications to allow internal encap records to be supported: - Modified the mapper index table processing to handle encap without an action record - Modified the session open code to reserve some 64 Byte internal encap records on tx - Modified the blob encap swap to suppor

[dpdk-dev] [PATCH v4 29/51] net/bnxt: add TF register and unregister

2020-07-02 Thread Ajit Khaparde
From: Michael Wildt - Add TF register/unregister support. Session got session clients to keep track of the ctrl-channels/function. - Add support code to tfp layer Signed-off-by: Michael Wildt Signed-off-by: Venkat Duvvuru Reviewed-by: Randy Schacher Reviewed-by: Ajit Khaparde --- drivers/

[dpdk-dev] [PATCH v4 34/51] net/bnxt: add support for if table processing

2020-07-02 Thread Ajit Khaparde
From: Kishore Padmanabha Added support for if table processing in the ulp mapper layer. This enables support for the default partition action record pointer interface table. Signed-off-by: Kishore Padmanabha Signed-off-by: Venkat Duvvuru Reviewed-by: Somnath Kotur Reviewed-by: Ajit Khaparde

[dpdk-dev] [PATCH v4 28/51] net/bnxt: implement IF tables set and get

2020-07-02 Thread Ajit Khaparde
From: Jay Ding - Implement set/get for PROF_SPIF_CTXT, LKUP_PF_DFLT_ARP, PROF_PF_ERR_ARP with tunneled HWRM messages - Add IF table for PROF_PARIF_DFLT_ARP - Fix page size offset in the HCAPI code - Fix Entry offset calculation Signed-off-by: Jay Ding Signed-off-by: Venkat Duvvuru Reviewed-b

[dpdk-dev] [PATCH v4 27/51] net/bnxt: align CFA resources with RM

2020-07-02 Thread Ajit Khaparde
From: Randy Schacher - HCAPI resources need to align for Resource Manager - Clean up unnecessary debug messages Signed-off-by: Randy Schacher Signed-off-by: Venkat Duvvuru Reviewed-by: Ajit Khaparde --- drivers/net/bnxt/tf_core/cfa_resource_types.h | 250 +- drivers/net/bnxt/

[dpdk-dev] [PATCH v4 32/51] net/bnxt: integrate with the latest tf core changes

2020-07-02 Thread Ajit Khaparde
From: Kishore Padmanabha ULP changes to integrate with the latest session open interface in tf_core Signed-off-by: Kishore Padmanabha Signed-off-by: Somnath Kotur Signed-off-by: Venkat Duvvuru Reviewed-by: Ajit Khaparde --- drivers/net/bnxt/tf_ulp/bnxt_ulp.c | 46 +++

[dpdk-dev] [PATCH v4 30/51] net/bnxt: add global config set and get APIs

2020-07-02 Thread Ajit Khaparde
From: Jay Ding - Add support to update global configuration for ACT_TECT and ACT_ABCR. - Add support to allow Tunnel and Action global configuration. - Remove register read and write operations. - Remove the register read and write support. Signed-off-by: Jay Ding Signed-off-by: Venkat Duvvur

[dpdk-dev] [PATCH v4 26/51] net/bnxt: add external action alloc and free

2020-07-02 Thread Ajit Khaparde
From: Jay Ding - Link external action alloc and free to new hcapi interface - Add parameter range checking - Fix issues with index allocation check Signed-off-by: Jay Ding Signed-off-by: Venkat Duvvuru Reviewed-by: Randy Schacher Reviewed-by: Ajit Khaparde --- drivers/net/bnxt/tf_core/tf_co

[dpdk-dev] [PATCH v4 19/51] net/bnxt: update identifier with remap support

2020-07-02 Thread Ajit Khaparde
From: Michael Wildt - Add Identifier L2 CTXT Remap to the P4 device and updated the cfa_resource_types.h to get the type support. Signed-off-by: Michael Wildt Signed-off-by: Venkat Duvvuru Reviewed-by: Randy Schacher Reviewed-by: Ajit Khaparde --- drivers/net/bnxt/tf_core/cfa_resource_typ

[dpdk-dev] [PATCH v4 25/51] net/bnxt: remove table scope from session

2020-07-02 Thread Ajit Khaparde
From: Peter Spreadborough - Remove table scope data from session. Added to EEM. - Complete move to RM of table scope base and range. - Fix some err messaging strings. - Fix the tcam logging message. Signed-off-by: Peter Spreadborough Reviewed-by: Randy Schacher Signed-off-by: Venkat Duvvuru R

[dpdk-dev] [PATCH v4 24/51] net/bnxt: update RM to support HCAPI only

2020-07-02 Thread Ajit Khaparde
From: Peter Spreadborough - For the EM Module there is a need to only allocate the EM Records in HCAPI RM but the storage control is requested to be outside of the RM DB. - Add TF_RM_ELEM_CFG_HCAPI_BA. - Return error when the number of reserved entries for wc tcam is odd number in tf_tcam_bin

[dpdk-dev] [PATCH v4 18/51] net/bnxt: multiple device implementation

2020-07-02 Thread Ajit Khaparde
From: Michael Wildt Implement the Identifier, Table Type and the Resource Manager modules. Integrate Resource Manager with HCAPI. Update open/close session. Move to direct msgs for qcaps and resv messages. Signed-off-by: Michael Wildt Signed-off-by: Venkat Duvvuru Reviewed-by: Randy Schacher

[dpdk-dev] [PATCH v4 21/51] net/bnxt: support two level priority for TCAMs

2020-07-02 Thread Ajit Khaparde
From: Jay Ding Allow TCAM indexes to be allocated from top or bottom. If the priority is set to 0, allocate from the lowest tcam indexes i.e. from top. Any other value, allocate it from the highest tcam indexes i.e. from bottom. Signed-off-by: Jay Ding Signed-off-by: Venkat Duvvuru Reviewed-by

[dpdk-dev] [PATCH v4 17/51] net/bnxt: implement support for TCAM access

2020-07-02 Thread Ajit Khaparde
From: Jay Ding Implement TCAM alloc, free, bind, and unbind functions Update tf_core, tf_msg, etc. Signed-off-by: Jay Ding Signed-off-by: Venkat Duvvuru Reviewed-by: Randy Schacher Reviewed-by: Ajit Khaparde --- drivers/net/bnxt/tf_core/tf_core.c | 258 ++--- drivers/ne

[dpdk-dev] [PATCH v4 20/51] net/bnxt: update RM with residual checker

2020-07-02 Thread Ajit Khaparde
From: Michael Wildt - Add residual checker to the TF Host RM as well as new RM APIs. On close it will scan the DB and check of any remaining elements. If found they will be logged and FW msg sent for FW to scrub that specific type of resources. - Update the module bind to be aware of the mo

[dpdk-dev] [PATCH v4 16/51] net/bnxt: add core changes for EM and EEM lookups

2020-07-02 Thread Ajit Khaparde
From: Randy Schacher - Move External Exact and Exact Match to device module using HCAPI to add and delete entries - Make EM active through the device interface. Signed-off-by: Randy Schacher Signed-off-by: Venkat Duvvuru Reviewed-by: Shahaji Bhosle Reviewed-by: Ajit Khaparde --- drivers/n

[dpdk-dev] [PATCH v4 15/51] net/bnxt: add HCAPI interface support

2020-07-02 Thread Ajit Khaparde
From: Peter Spreadborough Add new hardware shim APIs to support multiple device generations Signed-off-by: Peter Spreadborough Signed-off-by: Venkat Duvvuru Reviewed-by: Randy Schacher Reviewed-by: Ajit Khaparde --- drivers/net/bnxt/hcapi/Makefile | 10 + drivers/net/bnxt/hcapi/h

[dpdk-dev] [PATCH v4 13/51] net/bnxt: update multi device design support

2020-07-02 Thread Ajit Khaparde
From: Michael Wildt - Implement the modules RM, Device (WH+), Identifier. - Update Session module. - Implement new HWRMs for RM direct messaging. - Add new parameter check macro's and clean up the header includes for i.e. tfp such that bnxt.h is not directly included in the new modules. - Add c

[dpdk-dev] [PATCH v4 14/51] net/bnxt: support two-level priority for TCAMs

2020-07-02 Thread Ajit Khaparde
From: Shahaji Bhosle Allow TCAM indexes to be allocated from top or bottom. If the priority is set to 0, allocate from the lowest tcam indexes i.e. from top. Any other value, allocate it from the highest tcam indexes i.e. from bottom. Signed-off-by: Shahaji Bhosle Signed-off-by: Venkat Duvvuru

[dpdk-dev] [PATCH v4 12/51] net/bnxt: support bulk table get and mirror

2020-07-02 Thread Ajit Khaparde
From: Shahaji Bhosle - Add new bulk table type get using FW to DMA the data back to host. - Add flag to allow records to be cleared if possible - Set mirror using tf_alloc_tbl_entry Signed-off-by: Shahaji Bhosle Signed-off-by: Venkat Duvvuru Reviewed-by: Ajit Khaparde --- drivers/net/bnxt/

[dpdk-dev] [PATCH v4 11/51] net/bnxt: add multi device support

2020-07-02 Thread Ajit Khaparde
From: Michael Wildt Introduce new modules for Device, Resource Manager, Identifier, Table Types, and TCAM for multi device support. Signed-off-by: Michael Wildt Reviewed-by: Randy Schacher Signed-off-by: Venkat Duvvuru Reviewed-by: Ajit Khaparde --- drivers/net/bnxt/meson.build

[dpdk-dev] [PATCH v4 04/51] net/bnxt: initialize parent PF information

2020-07-02 Thread Ajit Khaparde
From: Lance Richardson Add support to query parent PF information (MAC address, function ID, port ID and default VNIC) from firmware. Current firmware returns zero for parent default vnic, a temporary Wh+-specific workaround is included until that can be fixed. Signed-off-by: Lance Richardson

[dpdk-dev] [PATCH v4 06/51] net/bnxt: get port and function info

2020-07-02 Thread Ajit Khaparde
From: Venkat Duvvuru add helper functions to get port & function related information like parif, physical port id & vport id. Signed-off-by: Venkat Duvvuru Signed-off-by: Somnath Kotur Reviewed-by: Kalesh AP Reviewed-by: Kishore Padmanabha Reviewed-by: Ajit Khaparde --- drivers/net/bnxt/bn

[dpdk-dev] [PATCH v4 10/51] net/bnxt: modify EM insert and delete to use HWRM direct

2020-07-02 Thread Ajit Khaparde
From: Peter Spreadborough Modify Exact Match insert and delete to use the HWRM messages directly. Remove tunneled EM insert and delete message types. Signed-off-by: Peter Spreadborough Signed-off-by: Venkat Duvvuru Reviewed-by: Randy Schacher Reviewed-by: Ajit Khaparde --- drivers/net/bnxt/

[dpdk-dev] [PATCH v4 05/51] net/bnxt: modify port db dev interface

2020-07-02 Thread Ajit Khaparde
From: Venkat Duvvuru Modify ulp_port_db_dev_port_intf_update prototype to take "struct rte_eth_dev *" as the second parameter. Signed-off-by: Venkat Duvvuru Signed-off-by: Somnath Kotur Reviewed-by: Kishore Padmanabha Reviewed-by: Ajit Khaparde --- drivers/net/bnxt/tf_ulp/bnxt_ulp.c| 4

[dpdk-dev] [PATCH v4 08/51] net/bnxt: modify port db to handle more info

2020-07-02 Thread Ajit Khaparde
From: Venkat Duvvuru Apart from func_svif, func_id & vnic, port_db now stores and retrieves func_spif, func_parif, phy_port_id, port_svif, port_spif, port_parif, port_vport. New helper functions have been added to support the same. Signed-off-by: Venkat Duvvuru Reviewed-by: Somnath Kotur Revie

[dpdk-dev] [PATCH v4 07/51] net/bnxt: add support for hwrm port phy qcaps

2020-07-02 Thread Ajit Khaparde
From: Venkat Duvvuru Issue HWRM_PORT_PHY_QCAPS to the firmware to get the physical port count of the device. Signed-off-by: Venkat Duvvuru Reviewed-by: Somnath Kotur Reviewed-by: Kalesh AP Reviewed-by: Kishore Padmanabha Reviewed-by: Ajit Khaparde --- drivers/net/bnxt/bnxt.h| 1 +

[dpdk-dev] [PATCH v4 00/51] add features for host-based flow management

2020-07-02 Thread Ajit Khaparde
v1->v2: - update commit message - rebase patches against latest changes in the tree - fix signed-off-by tags - update release notes v2->v3: - fix compilation issues v3->v4: - rebase against latest dpdk-next-net Ajit Khaparde (1): doc: update release notes Jay Ding (5): net/bnxt: imple

[dpdk-dev] [PATCH v4 02/51] net/bnxt: add support for VF-reps data path

2020-07-02 Thread Ajit Khaparde
From: Somnath Kotur Added code to support Tx/Rx from a VF representor port. The VF-reps use the RX/TX rings of the Trusted VF/PF. For each VF-rep, the Trusted VF/PF driver issues a VFR_ALLOC FW cmd that returns "cfa_code" and "cfa_action" values. The FW sets up the filter tables in such a way tha

[dpdk-dev] [PATCH v4 01/51] net/bnxt: add basic infrastructure for VF reps

2020-07-02 Thread Ajit Khaparde
From: Somnath Kotur Defines data structures and code to init/uninit VF representors during pci_probe and pci_remove respectively. Most of the dev_ops for the VF representor are just stubs for now and will be will be filled out in next patch. To create a representor using testpmd: testpmd -c 0xff

[dpdk-dev] [PATCH v4 03/51] net/bnxt: get IDs for VF-Rep endpoint

2020-07-02 Thread Ajit Khaparde
From: Somnath Kotur Use 'first_vf_id' and the 'vf_id' that is input as part of adding a representor to obtain the PCI function ID(FID) of the VF(VFR endpoint). Use the FID as an input to FUNC_QCFG HWRM cmd to obtain the default vnic ID of the VF. Along with getting the default vNIC ID by supplyin

[dpdk-dev] 回复: [PATCH v2 2/2] test/ring: rearrange names for ring stress tests

2020-07-02 Thread Feifei Wang
> -邮件原件- > 发件人: Honnappa Nagarahalli > 发送时间: 2020年7月3日 10:38 > 收件人: Feifei Wang ; Konstantin Ananyev > ; dev@dpdk.org > 抄送: nd ; Honnappa Nagarahalli > ; nd > 主题: RE: [PATCH v2 2/2] test/ring: rearrange names for ring stress tests > > > > > > 主题: [PATCH v2 2/2] test/ring: rearrange nam

Re: [dpdk-dev] [PATCH v5 0/5] enable more PPPoE packet type for switch

2020-07-02 Thread Lu, Nannan
-Original Message- From: Zhao1, Wei Sent: Monday, June 29, 2020 1:10 PM To: dev@dpdk.org Cc: sta...@dpdk.org; Zhang, Qi Z ; Lu, Nannan Subject: [PATCH v5 0/5] enable more PPPoE packet type for switch 1. add more support for switch parser of pppoe packet. 2. add check for NVGRE protocol

Re: [dpdk-dev] [PATCH v2 2/2] test/ring: rearrange names for ring stress tests

2020-07-02 Thread Honnappa Nagarahalli
> > 主题: [PATCH v2 2/2] test/ring: rearrange names for ring stress tests > > > > Rename test_ring_peek_stress to test_ring_mt_peek_stress to keep same > > naming conventions for ST and MT test cases. > > > > Signed-off-by: Konstantin Ananyev > > --- > > app/test/Makefile

[dpdk-dev] [PATCH 2/2] net/enic: add new flow manager actions to the dump function

2020-07-02 Thread Hyong Youb Kim
The following commit introduced several new actions. Make the dump function to print those actions. commit 6faf81f1d2c3 ("net/enic: update flow manager API") Signed-off-by: Hyong Youb Kim Reviewed-by: John Daley --- drivers/net/enic/enic_fm_flow.c | 7 +++ 1 file changed, 7 insertions(+)

[dpdk-dev] [PATCH 1/2] net/enic: support VLAN push and pop flow actions

2020-07-02 Thread Hyong Youb Kim
Flow manager API includes push/pop actions, so support corresponding DPDK flow actions. Signed-off-by: Hyong Youb Kim Reviewed-by: John Daley --- doc/guides/rel_notes/release_20_08.rst | 3 ++ drivers/net/enic/enic_fm_flow.c| 61 ++ 2 files changed, 64 insertion

[dpdk-dev] [PATCH 0/2] net/enic: minor flow manager changes

2020-07-02 Thread Hyong Youb Kim
This series enables VLAN push/pop actions. The first patch adds code to translate rte_flow push/pop actions to flow manager actions. The second patch extends the debug-print function to print recently added flow manager actions, including the ones used by the first patch. Thanks. -Hyong Hyong You

[dpdk-dev] 回复: [PATCH v2 2/2] test/ring: rearrange names for ring stress tests

2020-07-02 Thread Feifei Wang
> -邮件原件- > 发件人: Konstantin Ananyev > 发送时间: 2020年7月2日 22:10 > 收件人: dev@dpdk.org > 抄送: Honnappa Nagarahalli ; Feifei Wang > ; Konstantin Ananyev > > 主题: [PATCH v2 2/2] test/ring: rearrange names for ring stress tests > > Rename test_ring_peek_stress to test_ring_mt_peek_stress to keep sam

Re: [dpdk-dev] [PATCH v2] app/testpmd: fix missing set for current forward engine

2020-07-02 Thread Phil Yang
> > IMPORTANT NOTICE: The contents of this email and any attachments are > confidential and may also be privileged. If you are not the intended > recipient, > please notify the sender immediately and do not disclose the contents to any > other person, use it for any purpose, or store or copy the

Re: [dpdk-dev] [PATCH v4 2/2] ethdev: fix VLAN offloads set if no relative capabilities

2020-07-02 Thread Wei Hu (Xavier)
Hi, Heinrich Kuhn & Jerin Jacob & Nithin Dabilpuram & Kiran Kumar K & Rasesh Mody & Shahed Shaikh & Qi Zhang & Xiao Wang & Beilei Xing & Jeff Guo. Could you please help to review this patch and give 'Acked-by' if there are no objects. Thanks, Xavier On 2020/6/29 9:34, Wei Hu (Xavier) wr

Re: [dpdk-dev] [PATCH v4 5/7] net/ixgbe: return unknown speed in status

2020-07-02 Thread Zhao1, Wei
> -Original Message- > From: Ivan Dyukov > Sent: Thursday, July 2, 2020 9:22 PM > To: dev@dpdk.org; i.dyu...@samsung.com; v.kurams...@samsung.com; > tho...@monjalon.net; david.march...@redhat.com; Yigit, Ferruh > ; arybche...@solarflare.com; Zhao1, Wei > ; Guo, Jia ; Xing, Beilei > ; Yang

[dpdk-dev] [PATCH] doc: announce changes to eventdev public data structures

2020-07-02 Thread McDaniel, Timothy
From: "McDaniel, Timothy" Signed-off-by: McDaniel, Timothy --- doc/guides/rel_notes/deprecation.rst | 28 1 file changed, 28 insertions(+) diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst index d1034f6..6af9b40 100644 ---

Re: [dpdk-dev] [PATCH 1/3] eventdev: fix race condition on timer list counter

2020-07-02 Thread Honnappa Nagarahalli
> > > > > > > > > > > > > > Hi Phil, > > > > > > > > > > Good catch - thanks for the fix. I've commented in-line: > > > > > > > > > > > -Original Message- > > > > > > From: Phil Yang > > > > > > Sent: Friday, June 12, 2020 6:20 AM > > > > > > To: dev@dpdk.org; Carrillo, Erik G > > > >

Re: [dpdk-dev] [PATCH v2] build: check functionality rather than binutils version

2020-07-02 Thread Thomas Monjalon
18/06/2020 13:56, Bruce Richardson: > Rather than checking the binutils version number, which can lead to > unnecessary disabling of AVX512 if fixes have been backported to distro > versions, we can instead check the output of "as" from binutils to see if > it is correct. > > The check in the scri

Re: [dpdk-dev] [PATCH v2 0/4] improve runtime loading of shared drivers

2020-07-02 Thread Thomas Monjalon
22/06/2020 16:33, Bruce Richardson: > Bruce Richardson (4): > eal: remove unnecessary null-termination Maybe add scope of the change with "in plugin path" ? > eal: only load shared libs from driver plugin directory I suggest: "eal: load only shared libraries from plugin directory" > eal:

Re: [dpdk-dev] [PATCH 1/3] eventdev: fix race condition on timer list counter

2020-07-02 Thread Carrillo, Erik G
> -Original Message- > From: Honnappa Nagarahalli > Sent: Wednesday, July 1, 2020 10:56 PM > To: Phil Yang ; Carrillo, Erik G > ; dev@dpdk.org > Cc: d...@linux.vnet.ibm.com; Ruifeng Wang ; > Dharmik Thakkar ; sta...@dpdk.org; nd > ; jer...@marvell.com; Honnappa Nagarahalli > ; nd > Sub

Re: [dpdk-dev] [PATCH v2 0/4] improve runtime loading of shared drivers

2020-07-02 Thread Thomas Monjalon
22/06/2020 16:33, Bruce Richardson: > This set includes a number of small improvements for handling the loading > of drivers at runtime using the EAL -d flag. > > It limits the loading of files to only those files which end in .so, which > means that one can pass in the whole "drivers/" subfolder

Re: [dpdk-dev] [PATCH] test/ring: code rework to reduce compilation time

2020-07-02 Thread Honnappa Nagarahalli
> > Rework test code to reduce code complexity for the compiler and bring down > compilation time and memory consumption. > > Signed-off-by: Konstantin Ananyev Thomas has a nit [1]. Otherwise, Acked-by: Honnappa Nagarahalli [1] http://mails.dpdk.org/archives/dev/2020-May/166325.html > ---

Re: [dpdk-dev] [PATCH] test/ring: fix long compilation time

2020-07-02 Thread Honnappa Nagarahalli
> > Hi everyone, > > > > > > > Konstantin, Honnappa, > > > Any update about a fix for compilation time please? > > > > +1 > > A good amount of time waiting for builds is on this code. > > > > Between 69567 and 69559: > http://patches.dpdk.org/patch/69567/ > http://patches.dpdk.org/patch/69559/

  1   2   3   >