[dpdk-dev] [PATCH v2 54/58] net/bnxt: add ICMPv6 parser to ULP

2021-06-12 Thread Ajit Khaparde
From: Kishore Padmanabha This patch adds support for parsing rte_flow items for ICMPv6 flows. Signed-off-by: Kishore Padmanabha Signed-off-by: Venkat Duvvuru Reviewed-by: Randy Schacher Reviewed-by: Ajit Khaparde --- drivers/net/bnxt/tf_ulp/ulp_rte_handler_tbl.c | 4 +- drivers/net/bnxt/tf

[dpdk-dev] [PATCH v2 51/58] net/bnxt: process resource lists before session open

2021-06-12 Thread Ajit Khaparde
From: Mike Baucom Shared sessions require both named and unnamed resources to be requested during a tf_open_session. ULP uses named resources for global resources that are pre-allocated and remain through the life of the application. Unnamed resources are generally per flow resources and allocate

[dpdk-dev] [PATCH v2 55/58] net/bnxt: add context list for timers

2021-06-12 Thread Ajit Khaparde
From: Kishore Padmanabha Add context in ULP for timers. The alarm callback needs to have a valid context pointer when it is invoked. The context could become invalid if the port goes down and the callback is invoked before it is cancelled. Signed-off-by: Kishore Padmanabha Signed-off-by: Venkat

[dpdk-dev] [PATCH v2 53/58] net/bnxt: add HA support in ULP

2021-06-12 Thread Ajit Khaparde
From: Mike Baucom Add the ability for cooperative applications to share resources and use the high availability functionality in the HW. Signed-off-by: Mike Baucom Signed-off-by: Venkat Duvvuru Reviewed-by: Shahaji Bhosle Reviewed-by: Randy Schacher Reviewed-by: Ajit Khaparde --- drivers/n

[dpdk-dev] [PATCH v2 44/58] net/bnxt: refactor ULP mapper

2021-06-12 Thread Ajit Khaparde
From: Kishore Padmanabha 1. The internal and external exact match table resource types is combined since the resource handle contains the encoded type whether it is internal or external exact match entry. 2. When a flow doesn't hit the offloaded rules, the default action is to send it to the ker

[dpdk-dev] [PATCH v2 41/58] net/bnxt: add support for wild card pattern match

2021-06-12 Thread Ajit Khaparde
From: Kishore Padmanabha The computational field is enabled for wild card pattern support. The template checks the computational field to add a flow as wild card entry or exact match entry. Signed-off-by: Kishore Padmanabha Reviewed-by: Venkat Duvvuru Reviewed-by: Ajit Khaparde --- drivers/n

[dpdk-dev] [PATCH v2 42/58] net/bnxt: add support for GRE flows

2021-06-12 Thread Ajit Khaparde
From: Venkat Duvvuru This patch does the following to support GRE flows: 1. RTE_FLOW_ITEM_TYPE_ANY & RTE_FLOW_ITEM_TYPE_GRE processing 2. Calculate the absolute function ID from the logical VF ID passed as part of RTE_FLOW_ACTION_TYPE_VF action. 3. Move bnxt_get_bp API to bnxt_ethdev.c Signed

[dpdk-dev] [PATCH v2 48/58] net/bnxt: add shared session support to ULP

2021-06-12 Thread Ajit Khaparde
From: Mike Baucom Shared session permits cooperative sharing of prescribed resources between applications. - devargs added for app-id in order to enable sharing session resources across applications - shared session management added - TRUFLOW resource reservations are now app ID and device dep

[dpdk-dev] [PATCH v2 40/58] net/bnxt: add partial header field processing

2021-06-12 Thread Ajit Khaparde
From: Kishore Padmanabha For support for wild card TCAM, some of the header fields have to be partially written, hence this new opcode is added. Signed-off-by: Kishore Padmanabha Signed-off-by: Venkat Duvvuru Reviewed-by: Mike Baucom Reviewed-by: Ajit Khaparde --- drivers/net/bnxt/tf_ulp/ul

[dpdk-dev] [PATCH v2 36/58] net/bnxt: set shared handle for generic table

2021-06-12 Thread Ajit Khaparde
From: Kishore Padmanabha The shared handle is set in the mapper params when generic resource are created, this shall be used by application as a handle to the shared resource like mirror handle. Signed-off-by: Kishore Padmanabha Signed-off-by: Venkat Duvvuru Reviewed-by: Mike Baucom Reviewed-

[dpdk-dev] [PATCH v2 32/58] net/bnxt: modify table processing

2021-06-12 Thread Ajit Khaparde
From: Kishore Padmanabha 1. Added interface table specific opcode to process interface table entry creation and reuse. This allows reuse of the interface table entry for multiple flows. Changed the regfile apis to store the data in big endian format. 2. The result blob creation being done in tca

[dpdk-dev] [PATCH v2 29/58] net/bnxt: add conditional processing of templates

2021-06-12 Thread Ajit Khaparde
From: Mike Baucom Conditional execution and rejection processing added for templates and tables. This allows the mapper to skip tables and reject templates based on the content without having to hard code rules. Signed-off-by: Mike Baucom Signed-off-by: Venkat Duvvuru Reviewed-by: Kishore Pad

[dpdk-dev] [PATCH v2 28/58] net/bnxt: add support for mapper flow database opcodes

2021-06-12 Thread Ajit Khaparde
From: Kishore Padmanabha Added support for mapper flow database opcode to enable shared resources like mirror action. This allows mapper to conditionally populate flow database based on template content. Signed-off-by: Kishore Padmanabha Signed-off-by: Venkat Duvvuru Reviewed-by: Mike Baucom

[dpdk-dev] [PATCH v2 25/58] net/bnxt: add API to clear TCAM regions

2021-06-12 Thread Ajit Khaparde
From: Farah Smith Add TRUFLOW API to clear either the hi or the low regions in WildCard TCAM for ungraceful exit cleanup. Signed-off-by: Farah Smith Signed-off-by: Randy Schacher Signed-off-by: Venkat Duvvuru Reviewed-by: Jay Ding Reviewed-by: Peter Spreadborough Reviewed-by: Ajit Khaparde

[dpdk-dev] [PATCH v2 24/58] net/bnxt: add support for WC TCAM shared session

2021-06-12 Thread Ajit Khaparde
From: Jay Ding If the session shares WC TCAM entries with others, specify it in the session name by attach "-wc_tcam". Firmware will flush the shared WC TCAM entries if the last shared session using them is closed. Signed-off-by: Jay Ding Signed-off-by: Randy Schacher Signed-off-by: Venkat Duv

[dpdk-dev] [PATCH v2 26/58] net/bnxt: check FW capability to support TRUFLOW

2021-06-12 Thread Ajit Khaparde
From: Venkat Duvvuru Currently, a devarg (host-based-truflow) is passed while launching the app to enable TRUFLOW feature. However, this mechanism adds an extra step in enabling TRUFLOW. This doesn't give a seamless experience when flow offloads has to work with FW that doesn't/does support TRUFL

[dpdk-dev] [PATCH v2 30/58] net/bnxt: modify TCAM opcode processing

2021-06-12 Thread Ajit Khaparde
From: Kishore Padmanabha Added TCAM table specific opcode to process TCAM entry creation and reuse. This change removes the TCAM cache mechanism and uses the generic table mechanism for reuse of TCAM entries. Signed-off-by: Kishore Padmanabha Signed-off-by: Venkat Duvvuru Reviewed-by: Mike Bau

[dpdk-dev] [PATCH v2 31/58] net/bnxt: modify VXLAN decap for multichannel mode

2021-06-12 Thread Ajit Khaparde
From: Venkat Duvvuru The driver is using physical port id as the index into the tunnel inner flow table. However, this will not work in case of multichannel mode where multiple physical functions are going to share the same physical port id. When tunnel inner flow offload request comes before tu

[dpdk-dev] [PATCH v2 21/58] net/bnxt: add API to get shared table increments

2021-06-12 Thread Ajit Khaparde
From: Farah Smith Add TRUFLOW API to get the shared table increment value for a given TRUFLOW table type. The API is being added for Wh+ and Thor devices. Signed-off-by: Farah Smith Signed-off-by: Venkat Duvvuru Reviewed-by: Randy Schacher Reviewed-by: Ajit Khaparde --- drivers/net/bnxt/tf_

[dpdk-dev] [PATCH v2 19/58] net/bnxt: cleanup logs in session handling paths

2021-06-12 Thread Ajit Khaparde
From: Farah Smith Cleanup some of the log messages in the session open and close paths. Signed-off-by: Farah Smith Signed-off-by: Jay Ding Signed-off-by: Venkat Duvvuru Reviewed-by: Randy Schacher Reviewed-by: Peter Spreadborough Reviewed-by: Ajit Khaparde --- drivers/net/bnxt/tf_core/tf_

[dpdk-dev] [PATCH v2 23/58] net/bnxt: cleanup WC TCAM shared pool

2021-06-12 Thread Ajit Khaparde
From: Farah Smith Clean up all allocated hi or lo pool TCAM regions on close. Along with message cleanup and remove unnecessary multi-slice options Also make sure that find next free entry should start from 0 first time. Signed-off-by: Farah Smith Signed-off-by: Venkat Duvvuru Reviewed-by: Ran

[dpdk-dev] [PATCH v2 20/58] net/bnxt: add WC TCAM management support

2021-06-12 Thread Ajit Khaparde
From: Farah Smith - Add new API to move WC TCAM regions from the hi pool to the low pool. - Enable shared TCAM get/set functions on Thor. Signed-off-by: Farah Smith Signed-off-by: Randy Schacher Signed-off-by: Venkat Duvvuru Reviewed-by: Ajit Khaparde --- drivers/net/bnxt/tf_core/tf_core.

[dpdk-dev] [PATCH v2 22/58] net/bnxt: refactor host session failure cleanup

2021-06-12 Thread Ajit Khaparde
From: Jay Ding - Close FW session if session open fails after Fw session open. - Additional WC TCAM debug info to help in future debug - Reduce key/mask buffer sizes for performance - When a 64b counter is freed, clear the entry Signed-off-by: Jay Ding Signed-off-by: Farah Smith Signed-off-by:

[dpdk-dev] [PATCH v2 18/58] net/bnxt: shared TCAM region support

2021-06-12 Thread Ajit Khaparde
From: Farah Smith - switch to single slice management on Wh+ - Support of shared session WC_TCAM_HIGH and WC_TCAM_LOW regions - Enable/disable using TF_TCAM_SHARED flag in tf_core.h - Fix empty session module DBs in the case that none are allocated for a given module type Signed-off-by: Farah

[dpdk-dev] [PATCH v2 17/58] net/bnxt: modify resource reservation strategy

2021-06-12 Thread Ajit Khaparde
From: Jay Ding Allow an application to only reserve resources for one direction. Signed-off-by: Jay Ding Signed-off-by: Randy Schacher Signed-off-by: Venkat Duvvuru Reviewed-by: Ajit Khaparde --- drivers/net/bnxt/tf_core/tf_core.c| 4 -- drivers/net/bnxt/tf_core/tf_em_common.c |

[dpdk-dev] [PATCH v2 15/58] net/bnxt: add dpool allocator for EM allocation

2021-06-12 Thread Ajit Khaparde
From: Peter Spreadborough The dpool allocator supports variable size entries and also supports defragmentation of the allocation space. EM will by default use the fixed size stack allocator. The dynamic allocator may be selected at build time. The dpool allocator supports variable size entries an

[dpdk-dev] [PATCH v2 13/58] net/bnxt: change RM database type

2021-06-12 Thread Ajit Khaparde
From: Jay Ding RM databases are statically defined in each module. New static database needs to be defined in the code when multiple sessions are added. Add dynamic alloc database and associate it to each session. Signed-off-by: Jay Ding Signed-off-by: Randy Schacher Signed-off-by: Venkat Duvv

[dpdk-dev] [PATCH v2 16/58] net/bnxt: update shared session functionality

2021-06-12 Thread Ajit Khaparde
From: Jay Ding - Distinguish the shared session on host side using PCI address - One session could be shared by multiple interfaces. Signed-off-by: Jay Ding Signed-off-by: Randy Schacher Signed-off-by: Venkat Duvvuru Reviewed-by: Farah Smith Reviewed-by: Ajit Khaparde --- drivers/net/bnxt/

[dpdk-dev] [PATCH v2 12/58] net/bnxt: modify TRUFLOW HWRM messages

2021-06-12 Thread Ajit Khaparde
From: Farah Smith - Move Bulk get to a direct HWRM message - Deprecate code based on HCAPI changes Signed-off-by: Farah Smith Signed-off-by: Peter Spreadborough Signed-off-by: Randy Schacher Signed-off-by: Venkat Duvvuru Reviewed-by: Ajit Khaparde --- drivers/net/bnxt/hcapi/cfa/hcapi_cfa.h

[dpdk-dev] [PATCH v2 11/58] net/bnxt: add hashing changes for Thor

2021-06-12 Thread Ajit Khaparde
From: Peter Spreadborough - Move HCAPI hashing code to common file and add Thor support. - Change DPDK EM insert for FKB to use limited size Type 3 key. - Update FKB builder to be able to tell between EM and WC keys during transform. FKB is the Flexible Key Builder in Thor used while inserting

[dpdk-dev] [PATCH v2 14/58] net/bnxt: add shared session support

2021-06-12 Thread Ajit Khaparde
From: Jay Ding There are 2 types of sessions - shared and non-shared. For non-shared all the allocated resources are owned and managed by a single session instance. No other applications have access to the resources owned by the non-shared session. For a shared session, resources are shared bet

[dpdk-dev] [PATCH v2 10/58] net/bnxt: add 64B SRAM record management with RM

2021-06-12 Thread Ajit Khaparde
From: Farah Smith HCAPI RM now manages 64Byte records instead of 8Byte. Truflow core RM will manage the same. The tf_tbl core APIs now return 8B pointer addresses. These can be used directly as SRAM pointers in Action Records. When communicating with the firmware 8 byte addresses will be used. S

[dpdk-dev] [PATCH v2 08/58] net/bnxt: add action SRAM translation

2021-06-12 Thread Ajit Khaparde
From: Farah Smith - Translate Truflow action types for Thor to HCAPI RM resource defined SRAM banks. - move module type enum definitions to tf_core API - Switch to subtype concept for RM. - alloc/free working for Thor SRAM table type for full AR. Signed-off-by: Farah Smith Signed-off-by: Rand

[dpdk-dev] [PATCH v2 07/58] net/bnxt: support L2 Context TCAM ops

2021-06-12 Thread Ajit Khaparde
From: Jay Ding - Implement TCAM get in host - Add Thor support for TCAM set/free Signed-off-by: Jay Ding Signed-off-by: Randy Schacher Signed-off-by: Venkat Duvvuru Reviewed-by: Peter Spreadborough Reviewed-by: Farah Smith Reviewed-by: Ajit Khaparde --- drivers/net/bnxt/tf_core/hwrm_tf.h

[dpdk-dev] [PATCH v2 09/58] net/bnxt: add Thor WC TCAM support

2021-06-12 Thread Ajit Khaparde
From: Jay Ding 1. Add set/get/free/alloc for WC TCAM 2. Rework the key size in slice management. 3. Add 3 FKB WC keys for WC TCAM set cli cmd 4. Add transform key function for WC TCAM FKB key 5. Add checking for key buffer length for get_tcam Signed-off-by: Jay Ding Signed-off-by: Randy Schache

[dpdk-dev] [PATCH v2 06/58] net/bnxt: add support for EM with FKB

2021-06-12 Thread Ajit Khaparde
From: Peter Spreadborough Main TF changes to support EM insert with FKB. Flexible Key builder is required to create Wild Card and Exact Match keys for TCAM lookups. Signed-off-by: Peter Spreadborough Signed-off-by: Randy Schacher Signed-off-by: Venkat Duvvuru Reviewed-by: Farah Smith Reviewe

[dpdk-dev] [PATCH v2 05/58] net/bnxt: update TRUFLOW resources

2021-06-12 Thread Ajit Khaparde
From: Farah Smith - Remove unused tables from tf_tbl_type - Encode flow type into flow handle (internal or external) - Clean up Whitney resource tables - Clean up Truflow CLI open tables and update Thor resources - Add Thor SRAM and external pool types to core API - Remove unneeded Stingray table

[dpdk-dev] [PATCH v2 02/58] net/bnxt: add base TRUFLOW support for Thor

2021-06-12 Thread Ajit Khaparde
From: Farah Smith Add infrastructure code to support TRUFLOW on Thor NICs. Also update meson.build Signed-off-by: Farah Smith Signed-off-by: Peter Spreadborough Signed-off-by: Randy Schacher Signed-off-by: Venkat Duvvuru Acked-by: Ajit Khaparde --- drivers/net/bnxt/tf_core/cfa_resource_typ

[dpdk-dev] [PATCH v2 04/58] net/bnxt: check resource reservation in TRUFLOW

2021-06-12 Thread Ajit Khaparde
From: Jay Ding - Allow tf_open to continue if no resource is allocated for some table type. - Close the session if binding fails for any table. - Close the session if no resource is allocated for all tables. Signed-off-by: Jay Ding Signed-off-by: Randy Schacher Signed-off-by: Venkat Duvvuru

[dpdk-dev] [PATCH v2 03/58] net/bnxt: add mailbox selection via dev op

2021-06-12 Thread Ajit Khaparde
From: Peter Spreadborough Add get mailbox dev op so that mailbox offset is based on device instead of a hard coded value. Signed-off-by: Peter Spreadborough Signed-off-by: Randy Schacher Signed-off-by: Venkat Duvvuru Reviewed-by: Farah Smith Acked-by: Ajit Khaparde --- drivers/net/bnxt/tf_

[dpdk-dev] [PATCH v2 00/58] enhancements to host based flow table management

2021-06-12 Thread Ajit Khaparde
This patchset adds flow table management support for Thor network adapter and introduces enhancements to the existing flow table management functionality. v1->v2: - Rebased against latest dpdk-next-net-brcm for-next-net branch. - Fixed typo errors in some of the patches - Address some issues en

[dpdk-dev] [PATCH v2 01/58] net/bnxt: add CFA folder to HCAPI directory

2021-06-12 Thread Ajit Khaparde
From: Jeffrey Huang Before introducing more HCAPI components to DPDK, the CFA code needs to be organized into a dedicated folder so it is separated from other new HCAPI components Signed-off-by: Jeffrey Huang Signed-off-by: Randy Schacher Signed-off-by: Venkat Duvvuru Reviewed-by: Farah Smith

[dpdk-dev] [PATCH 1/4] net/hns3: add query basic info support for VF

2021-06-12 Thread Min Hu (Connor)
From: Chengchang Tang There are some feautres of VF depend on PF, so it's necessary for VF to know whether current PF supports. Therefore, the final capability set of VF will be composed of the capability set of hardware and the capability set of PF. For comatibility reasons, the mailbox HNS3_MB

[dpdk-dev] [PATCH 4/4] net/hns3: supports disabling PFC by dev configure API

2021-06-12 Thread Min Hu (Connor)
From: Huisong Li If "dcb_capability_en" in "data->dev_conf" delivered from the dev_configure does not have the ETH_DCB_PFC_SUPPORT flag, the user wants to disable PFC, and only enable ETS. Therefore, this patch supports the function of disabling PFC by the field. In addition, this patch updates "

[dpdk-dev] [PATCH 2/4] net/hns3: support for VF modify VLAN filter state

2021-06-12 Thread Min Hu (Connor)
From: Chengchang Tang Since the HW limitation for VF, the VLAN filter is default enabled, and is not allowed to be closed. Now, the limitation has been removed in Kunpeng930 network engine, so this patch add support for VF to modify the VLAN filter state. A capabilities bit is added to different

[dpdk-dev] [PATCH 0/4] features for hns3 PMD

2021-06-12 Thread Min Hu (Connor)
This patch set contains 4 features for hns3 PMD: add query basic info support for VF support for VF modify VLAN filter state support multiple TC MAC pause supports disabling PFC by dev configure API Chengchang Tang (2): net/hns3: add query basic info support for VF net/hns3: support for VF mod

[dpdk-dev] [PATCH 3/4] net/hns3: support multiple TC MAC pause

2021-06-12 Thread Min Hu (Connor)
From: Huisong Li MAC PAUSE can take effect on a single TC or multiple TCs, depending on the hardware. For example, the Kunpeng 920 supports MAC pause in a single TC, and the Kunpeng 930 supports MAC pause in multiple TCs. This patch supports MAC PAUSE in multiple TC for some hardware. Signed-off

[dpdk-dev] [PATCH 3/6] net/hns3: fix incorrect fake queue rollback

2021-06-12 Thread Min Hu (Connor)
From: Huisong Li When the device supports independent Rx/Tx queues, fake queues do not need to be created in unequal Rx/Tx queues case. However, dev_configure fails to be executed on the device supported independent Rx/Tx queues, the current rollback code logic contains the fake queue. As a resul

[dpdk-dev] [PATCH 6/6] net/hns3: delete redundant blank lines

2021-06-12 Thread Min Hu (Connor)
From: HongBo Zheng Delete redundant blank lines to make: 1.Return value judgment follow the function call. 2.No blank lines at the end of a code block defined by braces. Signed-off-by: HongBo Zheng Signed-off-by: Min Hu (Connor) --- drivers/net/hns3/hns3_ethdev.c | 6 +- 1 file changed, 1

[dpdk-dev] [PATCH 2/6] net/hns3: fix delay time for waiting to stop rxtx

2021-06-12 Thread Min Hu (Connor)
From: Huisong Li When the primary process executes dev_stop or is being reset, the packet sending and receiving functions is changed. In this moment, the primary process requests secondary processes to change their Rx/Tx functions, and delays a period of time in case of crashes when queues are st

[dpdk-dev] [PATCH 0/6] bugfixes for hns3 PMD

2021-06-12 Thread Min Hu (Connor)
This patch set contains 6 bugfixes for hns3 PMD. Chengchang Tang (1): net/hns3: fix VLAN strip log HongBo Zheng (2): net/hns3: modify max reset fail retry count net/hns3: delete redundant blank lines Huisong Li (3): net/hns3: fix delay time for waiting to stop rxtx net/hns3: fix incorr

[dpdk-dev] [PATCH 5/6] net/hns3: fix config max queues to zero when fails

2021-06-12 Thread Min Hu (Connor)
From: Huisong Li The "cfg_max_queues" maintains configured max queue numbers from user, and is equal to the maximum of "nb_rx_queues" and "nb_tx_queues" in "dev->data". >From the ethdev layer framework, "nb_rx/tx_queues" in "dev->data" were set to zero in rte_eth_dev_configure() if ops.dev_confi

[dpdk-dev] [PATCH 1/6] net/hns3: modify max reset fail retry count

2021-06-12 Thread Min Hu (Connor)
From: HongBo Zheng When the device is very busy, VF reset may have to be retried many times to succeed, leading to the current max reset fail retry count not enough. This patch modify max reset fail retry count to 30 to enhance the reliability of reset function. Fixes: 2790c6464725 ("net/hns3:

[dpdk-dev] [PATCH 4/6] net/hns3: fix VLAN strip log

2021-06-12 Thread Min Hu (Connor)
From: Chengchang Tang When the current VLAN stripping is set, the log print always prompts that the enabling fails, bug if may actually be the disabling failure. Fixes: 411d23b9eafb ("net/hns3: support VLAN") Cc: sta...@dpdk.org Signed-off-by: Chengchang Tang Signed-off-by: Min Hu (Connor) --

[dpdk-dev] [PATCH] net/hns3: support Tx push quick doorbell to improve perf

2021-06-12 Thread Min Hu (Connor)
From: Chengwen Feng Kunpeng 930 support tx push mode which could improve performance, It works like below: 1. Add pcie bar45 which support driver direct write the Tx descriptor or tail reg to it. 2. Support three operations: a) direct write one Tx descriptor, b) direct write two Tx descriptors, c

[dpdk-dev] [PATCH] net/hns3: support Tx push quick doorbell to improve perf

2021-06-12 Thread Min Hu (Connor)
From: Chengwen Feng Kunpeng 930 support tx push mode which could improve performance, It works like below: 1. Add pcie bar45 which support driver direct write the Tx descriptor or tail reg to it. 2. Support three operations: a) direct write one Tx descriptor, b) direct write two Tx descriptors, c

Re: [dpdk-dev] [PATCH] eal: save error in string copy

2021-06-12 Thread Xueming(Steven) Li
> -Original Message- > From: Thomas Monjalon > Sent: Saturday, June 12, 2021 4:35 PM > To: Xueming(Steven) Li > Cc: dev@dpdk.org; Jerin Jacob ; Sunil Kumar Kori > ; Kiran Kumar K > ; Nithin Dabilpuram ; > david.march...@redhat.com > Subject: Re: [PATCH] eal: save error in string copy

[dpdk-dev] [v1] eal: save error in string copy

2021-06-12 Thread Xueming Li
From: Thomas Monjalon The string copy api rte_strscpy() doesn't set rte_errno during failures, instead it just return negative error number. Sets rte_errrno if the destination buffer is too small. Signed-off-by: Thomas Monjalon Signed-off-by: Xueming Li --- lib/eal/common/eal_common_string_f

[dpdk-dev] [PATCH 1/2] devtools: fix version pattern for fix search

2021-06-12 Thread Xueming Li
When scanning fixes from current(HEAD) branch, local tags were included and reported as version. For example: $ git tag --contains --merged 20.11_backport_202010506 // user tag v20.11 v20.11.1 This patch matches DPDK officail version pattern in search, selects the most early tag. Offici

[dpdk-dev] [PATCH 2/2] devtools: auto detect branch to search fix patches

2021-06-12 Thread Xueming Li
Current fix scan scripts scanned specified range in current(HEAD) branch. When users run it in a earlier branch, few patches were scanned. This patch auto etects branch to scan from range. Fixes: 752d8e097ec1 ("scripts: show fixes with release version of bug") Cc: Thomas Monjalon Cc: sta...@dpdk

Re: [dpdk-dev] RFC: Kunpeng DMA driver API design decision

2021-06-12 Thread Fengchengwen
OK, I will send one, thanks From:Jerin Jacob To:Thomas Monjalon Cc:Fengchengwen ;Ferruh Yigit ;dev ;Nipun Gupta ;Hemant Agrawal ;Richardson, Bruce ;Maxime Coquelin ;Honnappa Nagarahalli ;Jerin Jacob ;David Marchand Date:2021-06-12 16:41:32 Subject:Re: [dpdk-dev] RFC: Kunpeng DMA driver A

Re: [dpdk-dev] RFC: Kunpeng DMA driver API design decision

2021-06-12 Thread Jerin Jacob
On Sat, Jun 12, 2021 at 2:01 PM Thomas Monjalon wrote: > > 12/06/2021 09:01, fengchengwen: > > Hi all, > > > > We prepare support Kunpeng DMA engine under rawdev framework, and observed > > that > > there are two different implementations of the data plane API: > > 1. rte_rawdev_enqueue/dequeue_b

Re: [dpdk-dev] [PATCH] eal: save error in string copy

2021-06-12 Thread Thomas Monjalon
12/06/2021 00:49, Xueming Li: > From: Thomas Monjalon > > Sets rte_errrno if the destination buffer is too small. Would be good to mention the function name, and the reason for the change. > > Signed-off-by: Thomas Monjalon You did half of the patch, so you should add your Signed-off. > ---

Re: [dpdk-dev] RFC: Kunpeng DMA driver API design decision

2021-06-12 Thread Thomas Monjalon
12/06/2021 09:01, fengchengwen: > Hi all, > > We prepare support Kunpeng DMA engine under rawdev framework, and observed > that > there are two different implementations of the data plane API: > 1. rte_rawdev_enqueue/dequeue_buffers which was implemented by dpaa2_qdma and >octeontx2_dma drive

Re: [dpdk-dev] [PATCH v8 2/2] net/hns3: refactor SVE code compile method

2021-06-12 Thread fengchengwen
Friendly ping On 2021/5/27 15:07, Fengchengwen wrote: > > Hi, Ferruh > > Could you review this patch ? Thanks > > > From:Ruifeng Wang > To:Fengchengwen ;Thomas Monjalon > ;ferruh.yigit > Cc:dev ;Jerin Jacob ;viktorin > ;Richardson, Bruce > ;Honnappa Nagarahalli > ;jerinjacobk ;Juraj

Re: [dpdk-dev] [PATCH v3] build: fix SVE compile error with gcc8.3

2021-06-12 Thread fengchengwen
Friendly ping On 2021/5/27 15:12, Fengchengwen wrote: > Hi, Thomas > > Could you review this patch? Thanks > > > From:Ruifeng Wang > To:Fengchengwen ;Thomas Monjalon > ;ferruh.yigit > Cc:dev ;Richardson, Bruce > ;vladimir.medvedkin > ;viktorin ;Jerin Jacob > ;nd > Date:2021-05-25 14

[dpdk-dev] RFC: Kunpeng DMA driver API design decision

2021-06-12 Thread fengchengwen
Hi all, We prepare support Kunpeng DMA engine under rawdev framework, and observed that there are two different implementations of the data plane API: 1. rte_rawdev_enqueue/dequeue_buffers which was implemented by dpaa2_qdma and octeontx2_dma driver. 2. rte_ioat_enqueue_xxx/rte_ioat_completed_o