Re: [dpdk-dev] [EXT] [PATCH v3 3/4] regexdev: add regexdev core functions

2020-06-02 Thread Guy Kaneti
> -Original Message- > From: Ori Kam > Sent: Thursday, May 07, 2020 12:46 PM > To: Jerin Jacob Kollanukkaran ; > xiang.w.w...@intel.com > Cc: Guy Kaneti ; dev@dpdk.org; Pavan Nikhilesh > Bhagavatula ; shah...@mellanox.com; > hemant.agra...@nxp.com; op...@mellanox.com; al...@mellanox.com

Re: [dpdk-dev] [EXT] [PATCH v3 2/4] regexdev: add regex core h file

2020-06-02 Thread Guy Kaneti
Acked-by: Guy Kaneti

Re: [dpdk-dev] [PATCH 03/52] net/ice/base: disable profile merge for FDIR

2020-06-02 Thread Yang, Qiming
> -Original Message- > From: Zhang, Qi Z > Sent: Wednesday, June 3, 2020 10:39 > To: Yang, Qiming > Cc: dev@dpdk.org; Ye, Xiaolong ; Zhang, Qi Z > ; Cao, Yahui ; Stillwell Jr, Paul > M > Subject: [PATCH 03/52] net/ice/base: disable profile merge for FDIR > > For Flow Director, we don'

Re: [dpdk-dev] [RFC 1/6] eal: add power management intrinsics

2020-06-02 Thread Jerin Jacob
On Wed, Jun 3, 2020 at 11:53 AM Honnappa Nagarahalli wrote: > > > > > > > On Thu, May 28, 2020 at 9:08 PM Ananyev, Konstantin > > wrote: > > > > > > > > > > > Hi Anatoly, > > > > > > > > > >> > > > > >> Add two new power management intrinsics, and provide an > > > > >> implementation in eal/x86

Re: [dpdk-dev] [RFC 1/6] eal: add power management intrinsics

2020-06-02 Thread Honnappa Nagarahalli
> > On Thu, May 28, 2020 at 9:08 PM Ananyev, Konstantin > wrote: > > > > > > > > Hi Anatoly, > > > > > > > >> > > > >> Add two new power management intrinsics, and provide an > > > >> implementation in eal/x86 based on UMONITOR/UMWAIT instructions. > > > >> The instructions are implemented as r

Re: [dpdk-dev] [PATCH v6 06/11] trace: add size_t field emitter

2020-06-02 Thread Jerin Jacob
On Wed, Jun 3, 2020 at 4:35 AM Dmitry Kozlyuk wrote: > > It is not guaranteed that sizeof(long) == sizeof(size_t). On Windows, > sizeof(long) == 4 and sizeof(size_t) == 8 for 64-bit programs. > Tracepoints using "long" field emitter are therefore invalid there. > Add dedicated field emitter for si

Re: [dpdk-dev] Marvell DPDK v20.08 Roadmap

2020-06-02 Thread Jerin Jacob
On Wed, Jun 3, 2020 at 3:29 AM Thomas Monjalon wrote: > > 02/06/2020 19:36, Jerin Jacob: > > On Tue, Jun 2, 2020 at 10:16 PM Thomas Monjalon wrote: > > > 29/05/2020 14:50, Jerin Kollanukkaran: > > > > Regexdev: > > > > ~ > > > > 11) Review regexdev: introduce regexdev subsystem > > > > >

Re: [dpdk-dev] Marvell DPDK v20.08 Roadmap

2020-06-02 Thread Jerin Jacob
On Wed, Jun 3, 2020 at 12:04 AM Ori Kam wrote: > > > > > -Original Message- > > From: Jerin Jacob > > Sent: Tuesday, June 2, 2020 8:37 PM > > Subject: Re: [dpdk-dev] Marvell DPDK v20.08 Roadmap > > > > On Tue, Jun 2, 2020 at 10:16 PM Thomas Monjalon > > wrote: > > > > > > 29/05/2020 14:5

[dpdk-dev] [PATCH 49/52] net/ice/base: fix uninitialized flag

2020-06-02 Thread Qi Zhang
This patch add initialization for prof_res_bm_init flag to zero in order that the possible resource for fv in the files can be initialized. Fixes: 453d087ccaff ("net/ice/base: add common functions") Cc: sta...@dpdk.org Signed-off-by: Wei Zhao Signed-off-by: Paul M. Stillwell Jr Signed-off-by: Q

[dpdk-dev] [PATCH 51/52] net/ice/base: add 1G SGMII PHY type

2020-06-02 Thread Qi Zhang
There isn't a case for 1G SGMII in ice_get_media_type() so add the handling for it. Also handle the special case where some direct attach cables may report that they support 1G SGMII, but that is erroneous since SGMII is supposed to be a backplane media type (between a MAC and a PHY). If the drive

[dpdk-dev] [PATCH 52/52] net/ice/base: update IPV4 and IPV6 flow ptype masks

2020-06-02 Thread Qi Zhang
In the flow API, add ability to add IPV4/IPV6 rules that match on packets with or without inner L4 protocols. Also, remove PPPOD packet from PPPOE bitmap. Signed-off-by: Dan Nowlin Signed-off-by: Paul M. Stillwell Jr Signed-off-by: Qi Zhang --- drivers/net/ice/base/ice_flow.c | 166

[dpdk-dev] [PATCH 50/52] net/ice/base: add more device ID support

2020-06-02 Thread Qi Zhang
Add support for E823L devices. Signed-off-by: Qi Zhang --- drivers/net/ice/base/ice_common.c | 5 + drivers/net/ice/base/ice_devids.h | 10 ++ drivers/net/ice/base/ice_nvm.c| 5 + drivers/net/ice/ice_ethdev.c | 5 + 4 files changed, 25 insertions(+) diff --git a/

[dpdk-dev] [PATCH 45/52] net/ice/base: add a new command to LLDP commands

2020-06-02 Thread Qi Zhang
Add support for LLDP forwarding to SW programming in FW LLDP Filter Control is 0x0A0A. Signed-off-by: Sharon Haroni Signed-off-by: Paul M. Stillwell Jr Signed-off-by: Qi Zhang --- drivers/net/ice/base/ice_adminq_cmd.h | 14 ++ 1 file changed, 14 insertions(+) diff --git a/drivers/

[dpdk-dev] [PATCH 44/52] net/ice/base: distribute Tx queues evenly

2020-06-02 Thread Qi Zhang
Distribute the tx queues evenly across all queue groups. This will help the queues to get more equal sharing among the queues when all are in use. In the previous algorithm, the next queue group node will be picked up only after the previous one filled with max children. For example: if VSI is con

[dpdk-dev] [PATCH 48/52] net/ice/base: add more tunnel type for IPv4 and IPv6

2020-06-02 Thread Qi Zhang
This patch add more tunnel type defination 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 switch filter rule. For example: we can download a switch rule to direct ipv4 packet with specific source and destination ip address t

[dpdk-dev] [PATCH 47/52] net/ice/base: fix reference count when update VSI list

2020-06-02 Thread Qi Zhang
The parameter ref_cnt is used for tracking how many rules are reusing this VSI list, so it can only be updated when a rule which using this list be deleted. Fixes: f89aa3affa9e ("net/ice/base: support removing advanced rule") Cc: sta...@dpdk.org Signed-off-by: Wei Zhao Signed-off-by: Paul M. Sti

[dpdk-dev] [PATCH 46/52] net/ice/base: remove unused code for VSI list free

2020-06-02 Thread Qi Zhang
When free vsi list resource after vsi list update to empty, some useless code in function ice_remove_vsi_list_rule() should be deleted. Signed-off-by: Wei Zhao Signed-off-by: Paul M. Stillwell Jr Signed-off-by: Qi Zhang --- drivers/net/ice/base/ice_switch.c | 17 + 1 file chang

[dpdk-dev] [PATCH 43/52] net/ice/base: adjust scheduler default BW weight

2020-06-02 Thread Qi Zhang
By default the queues are configured in legacy mode. The default BW settings for legacy/advanced modes are different. The existing code was using the advanced mode default value of 1 which was incorrect. This caused the unbalanced BW sharing among siblings. The recommneded default value is applied.

[dpdk-dev] [PATCH 39/52] net/ice/base: remove unimplemented function prototypes

2020-06-02 Thread Qi Zhang
There are no implementations for these two functions so remove the prototypes. Signed-off-by: Tony Nguyen Signed-off-by: Paul M. Stillwell Jr Signed-off-by: Qi Zhang --- drivers/net/ice/base/ice_fdir.h | 5 - 1 file changed, 5 deletions(-) diff --git a/drivers/net/ice/base/ice_fdir.h b/dr

[dpdk-dev] [PATCH 42/52] net/ice/base: initialize AQ failure variable when set fc

2020-06-02 Thread Qi Zhang
ice_set_fc takes a u8 pointer 'aq_failures' as an input parameter. If this function encounters an error, in addition to returning an appropriate ice_status enum code, it also populates aq_failures with a link specific error value. If the caller does not initialize this variable to 0 before calling

[dpdk-dev] [PATCH 37/52] net/ice/base: fix for memory leak

2020-06-02 Thread Qi Zhang
Handles memory leaks during control queue initialization and buffer allocation failures. The MACRO - ICE_FREE_CQ_BUFS is modified to re-use for this fix. Fixes: 6c1f26be50a2 ("net/ice/base: add control queue information") Cc: sta...@dpdk.org Signed-off-by: Surabhi Boob Signed-off-by: Paul M. Sti

[dpdk-dev] [PATCH 41/52] net/ice/base: avoid PPPoE ipv4 overlap

2020-06-02 Thread Qi Zhang
When PPPOE header is not selected, pppoe should not be included in ipv4 ptype bitmaps. Signed-off-by: Yahui Cao Signed-off-by: Paul M. Stillwell Jr Signed-off-by: Qi Zhang --- drivers/net/ice/base/ice_flow.c | 8 ++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/drivers/ne

[dpdk-dev] [PATCH 40/52] net/ice/base: add new API to check all autoneg enable bits

2020-06-02 Thread Qi Zhang
struct ice_aqc_get_phy_caps_data has multiple autoneg enable bits. ice_is_phy_caps_an_enabled checks all bits and returns true if any autoneg enable bits are set. Signed-off-by: Paul Greenwalt Signed-off-by: Paul M. Stillwell Jr Signed-off-by: Qi Zhang --- drivers/net/ice/base/ice_common.c | 1

[dpdk-dev] [PATCH 38/52] net/ice/base: add entries in Profile TCAM with priority

2020-06-02 Thread Qi Zhang
The profile TCAM tables are implemented such that entries with a smaller index in the table have a higher priority. When records to be added to the table have flags to differentiate between standard PTG and VSIG records, then these entries need to have higher priority in order to be found and proce

[dpdk-dev] [PATCH 32/52] net/ice/base: reset capabilities before parsing

2020-06-02 Thread Qi Zhang
The capability flags used to report whether an NVM component has a pending update are stored as simple booleans. If ice_parse_caps finds the relevant capability then the boolean is set to true. If the capability is not provided by firmware, then the boolean value will be left alone. This works dur

[dpdk-dev] [PATCH 35/52] net/ice/base: correct return value

2020-06-02 Thread Qi Zhang
Function ice_rem_adv_rule_id return incorrect error code (ICE_ERR_PARAM) whereas it should have returned ICE_ERR_DOES_NOT_EXIST return code if filter list is empty or unable to find "rule" in list Signed-off-by: Kiran Patil Signed-off-by: Paul M. Stillwell Jr Signed-off-by: Qi Zhang --- driver

[dpdk-dev] [PATCH 33/52] net/ice/base: add RL profile bit mask check

2020-06-02 Thread Qi Zhang
Mask bits before accessing the profile type field. Signed-off-by: Tarun Singh Signed-off-by: Paul M. Stillwell Jr Signed-off-by: Qi Zhang --- drivers/net/ice/base/ice_sched.c | 11 ++- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/drivers/net/ice/base/ice_sched.c b/driv

[dpdk-dev] [PATCH 31/52] net/ice/base: reset flags when all rules are deleted

2020-06-02 Thread Qi Zhang
To avoid having stale information about "adv_rule" per recipe, when all rules associated to a given recipe are deleted, reset the "adv_rule" flag otherwise it causes problem later on when decisions about filter rules being present or not are made based on "adv_rule" flag. Removed setting "adv_rule

[dpdk-dev] [PATCH 36/52] net/ice/base: remove unneeded variable

2020-06-02 Thread Qi Zhang
The variable 'adj' is initialized to 0 and later used in an addition, but the value is never changed from 0. Remove the variable since it's doing nothing. Signed-off-by: Tony Nguyen Signed-off-by: Paul M. Stillwell Jr Signed-off-by: Qi Zhang --- drivers/net/ice/base/ice_flow.c | 3 +-- 1 file

[dpdk-dev] [PATCH 34/52] net/ice/base: update the vsi handle to remaining VSI

2020-06-02 Thread Qi Zhang
Needs to update the VSI handle to the last remaining VSI using the rule for ICE_FWD_TO_VSI. Otherwise it may have error for deleting the rule. Signed-off-by: Haiyue Wang Signed-off-by: Paul M. Stillwell Jr Signed-off-by: Qi Zhang --- drivers/net/ice/base/ice_switch.c | 1 + 1 file changed, 1 i

[dpdk-dev] [PATCH 30/52] net/ice/base: add support for more PPPoE packet type

2020-06-02 Thread Qi Zhang
This patch add more dummy packet type for pppoe packet, it enable tcp/udp layer of ipv4/ipv6 for pppoe payload, so we can use L4 dst/src port as input set for switch filter. Signed-off-by: Wei Zhao Signed-off-by: Paul M. Stillwell Jr Signed-off-by: Qi Zhang --- drivers/net/ice/base/ice_protoco

[dpdk-dev] [PATCH 28/52] net/ice/base: return correct error code

2020-06-02 Thread Qi Zhang
Return ICE_ERR_DOES_NOT_EXIST return code if admin command error code is ICE_AQ_RC_ENOENT (not exist). ice_aq_sw_rules is used when switch rule is getting added/deleted/updated. In case of delete/update switch rule, admin command can return ICE_AQ_RC_ENOENT error code if such rule does not exist, h

[dpdk-dev] [PATCH 26/52] net/ice/base: use macro for sizeof

2020-06-02 Thread Qi Zhang
The definition of ICE_SW_RULE_RX_TX_ETH_HDR_SIZE open codes the size of a structure field. Replace this with the use of FIELD_SIZEOF. Signed-off-by: Jacob Keller Signed-off-by: Paul M. Stillwell Jr Signed-off-by: Qi Zhang --- drivers/net/ice/base/ice_switch.h | 17 - 1 file cha

[dpdk-dev] [PATCH 27/52] net/ice/base: add debug logs

2020-06-02 Thread Qi Zhang
Add debug logs for ice_aq_get_phy_caps(), and format ice_aq_set_phy_cfg() and ice_aq_get_link_info() debug logs to make them more readable. Signed-off-by: Paul Greenwalt Signed-off-by: Paul M. Stillwell Jr Signed-off-by: Qi Zhang --- drivers/net/ice/base/ice_common.c | 69 +

[dpdk-dev] [PATCH 29/52] net/ice/base: remove unnecessary code

2020-06-02 Thread Qi Zhang
Remove unncessary case branch. Signed-off-by: Qi Zhang --- drivers/net/ice/base/ice_flow.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/drivers/net/ice/base/ice_flow.c b/drivers/net/ice/base/ice_flow.c index 030a55ba7..53ff5f4be 100644 --- a/drivers/net/ice/base/ice_flow.c +++ b/drivers

[dpdk-dev] [PATCH 20/52] net/ice/baes: add NVM help functions

2020-06-02 Thread Qi Zhang
Add couple functions that DPDK would like to use for accessing the NVM. Signed-off-by: Paul M Stillwell Jr Signed-off-by: Qi Zhang --- drivers/net/ice/base/ice_common.h | 1 - drivers/net/ice/base/ice_nvm.c| 67 +-- drivers/net/ice/base/ice_nvm.h| 10

[dpdk-dev] [PATCH 22/52] net/ice/base: increase timeout after PFR

2020-06-02 Thread Qi Zhang
To allow for resets during package download, increase the timeout period after performing a PFR. The time waited is the global config lock timeout plus the normal PFSWR timeout. Signed-off-by: Dan Nowlin Signed-off-by: Paul M. Stillwell Jr Signed-off-by: Qi Zhang --- drivers/net/ice/base/ice_c

[dpdk-dev] [PATCH 24/52] net/ice/base: adjust function signature style format

2020-06-02 Thread Qi Zhang
Where possible, cuddle multiple lines of function signatures to be consistent throughout the code. Signed-off-by: Bruce Allan Signed-off-by: Paul M. Stillwell Jr Signed-off-by: Qi Zhang --- drivers/net/ice/base/ice_common.c | 3 +-- drivers/net/ice/base/ice_controlq.c | 9 +++-- driver

[dpdk-dev] [PATCH 25/52] net/ice/base: add RSS support for IPv6 prefix

2020-06-02 Thread Qi Zhang
Some IPv6 prefix related fields are defined in this patch, so that we can use prefixes instead of full IPv6 address for RSS. These prefixes include the first 32, 48, 64 bits of both SRC and DST IPV6 address. Signed-off-by: Junfeng Guo Signed-off-by: Paul M. Stillwell Jr Signed-off-by: Qi Zhang

[dpdk-dev] [PATCH 21/52] net/ice/base: allow GENEVE and VXLAN rules with VLAN

2020-06-02 Thread Qi Zhang
When programming GENEVE and VXLAN switch rules, there are some instances where both VLAN tagged packets plus non-VLAN tagged packets are needed to match the rule. In order to perform this action in one rule, the switch code needs to setup the packet flag mask to ignore the VLAN packet flag. This w

[dpdk-dev] [PATCH 23/52] net/ice/base: remove unnecessary braces

2020-06-02 Thread Qi Zhang
This patch mainly does cleanups related to unnecessary braces. A few other minor fixes are included which include fixing a typo, pulling up some lines, and fixing RCT. Signed-off-by: Tony Nguyen Signed-off-by: Paul M. Stillwell Jr Signed-off-by: Qi Zhang --- drivers/net/ice/base/ice_flex_pipe

[dpdk-dev] [PATCH 19/52] net/ice/base: initialize Set PHY Configuration FEC fields

2020-06-02 Thread Qi Zhang
Currently the caller needs to initialize the ice_cfg_phy_fec() parameter ice_aqc_set_phy_cfg_data FEC fields before calling. However, this is not necessary since ice_cfg_phy_fec() calls Get PHY Capabilities. Initialize ice_aqc_set_phy_cfg_data FEC capabilities and FEC option fields from Get PHY Ca

[dpdk-dev] [PATCH 15/52] net/ice/base: group function protoypes together

2020-06-02 Thread Qi Zhang
There are some function prototypes at the beginning of the file and some at the end, group them all together so that they are in one consistent location. Signed-off-by: Tony Nguyen Signed-off-by: Paul M. Stillwell Jr Signed-off-by: Qi Zhang --- drivers/net/ice/base/ice_fdir.h | 83

[dpdk-dev] [PATCH 16/52] net/ice/base: cleanup comment formatting

2020-06-02 Thread Qi Zhang
Add missing space between end of comment text and comment terminator, and remove unnecessary punctuation. Signed-off-by: Bruce Allan Signed-off-by: Paul M. Stillwell Jr Signed-off-by: Qi Zhang --- drivers/net/ice/base/ice_acl.c| 25 +- drivers/net/ice/base/ice_acl_ctrl.

[dpdk-dev] [PATCH 14/52] net/ice/base: change IPV6 training packet

2020-06-02 Thread Qi Zhang
Add additional UDP payload to allow for additional headers such as ESP. Signed-off-by: Dan Nowlin Signed-off-by: Paul M. Stillwell Jr Signed-off-by: Qi Zhang --- drivers/net/ice/base/ice_switch.c | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/drivers/net/ice/base/ic

[dpdk-dev] [PATCH 17/52] net/ice/base: add FDIR support for L2TPV3 ESP AH and PFCP

2020-06-02 Thread Qi Zhang
FDIR can forward: - L2TPV3 packets with session id. - IPSEC ESP packets with security parameter index. - IPSEC AH packets with security parameter index. - NAT_T ESP packets with security parameter index. - PFCP packets with s field value. Signed-off-by: Yahui Cao Signed-off-by: Paul M. Stillwell

[dpdk-dev] [PATCH 18/52] net/ice/base: add FD completion report option

2020-06-02 Thread Qi Zhang
This patch introduces comp_report variable so that the called can determine whether to report completion on error or on all cases. Signed-off-by: Yahui Cao Signed-off-by: Paul M. Stillwell Jr Signed-off-by: Qi Zhang --- drivers/net/ice/base/ice_fdir.c | 2 +- drivers/net/ice/base/ice_fdir.h

[dpdk-dev] [PATCH 09/52] net/ice/base: add AUI media type

2020-06-02 Thread Qi Zhang
Add and report AUI PHY types as an AUI media type Signed-off-by: Doug Dziggel Signed-off-by: Paul M. Stillwell Jr Signed-off-by: Qi Zhang --- drivers/net/ice/base/ice_common.c | 5 +++-- drivers/net/ice/base/ice_type.h | 1 + 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/dri

[dpdk-dev] [PATCH 13/52] net/ice/base: refactor flow director filter swap

2020-06-02 Thread Qi Zhang
Move the swap of flow director addresses and ports into training packet generation. This reduces the code written for ACL. Signed-off-by: Henry Tieman Signed-off-by: Paul M. Stillwell Jr Signed-off-by: Qi Zhang --- drivers/net/ice/base/ice_fdir.c | 59 ---

[dpdk-dev] [PATCH 11/52] net/ice/base: update PHY type high max index value

2020-06-02 Thread Qi Zhang
As currently, we are supporting only 5 PHY_SPEEDs for phy_type_high. Thus, we should adjust the value of ICE_PHY_TYPE_HIGH_MAX_INDEX to 5. Signed-off-by: Chinh T Cao Signed-off-by: Paul M. Stillwell Jr Signed-off-by: Qi Zhang --- drivers/net/ice/base/ice_adminq_cmd.h | 2 +- 1 file changed, 1

[dpdk-dev] [PATCH 10/52] net/ice/base: fix variable type for ACL

2020-06-02 Thread Qi Zhang
The commit ef92cee94cdb ("ice-shared: Fix remaining minor casting issues") changed the idx variable within ice_acl_add_entry() from a u16 to a u8. This causes the code to truncate the values greater than 255 to 255 or less when calling ice_aq_program_acl_entry() resulting in the wrong TCAM index be

[dpdk-dev] [PATCH 08/52] net/ice/base: improve VSI filters rebuild

2020-06-02 Thread Qi Zhang
This change improve VSI filter configuration rebuild for multiport configuration, ie. where 1 PF includes more than one logical port. For some functions, association between port and corresponding switch_info or port_info structure has been lost because by default the pointer to the first element o

[dpdk-dev] [PATCH 12/52] net/ice/base: consolidate VF Promiscuous mode

2020-06-02 Thread Qi Zhang
Consolidate the Promiscuous rule for SMBM on the chosen logical port. Signed-off-by: Shibin Koikkara Reeny Signed-off-by: Paul M. Stillwell Jr Signed-off-by: Qi Zhang --- drivers/net/ice/base/ice_switch.c | 145 ++ 1 file changed, 115 insertions(+), 30 delet

[dpdk-dev] [PATCH 06/52] net/ice/base: report AOC PHY Types as Fiber

2020-06-02 Thread Qi Zhang
Report AOC types as fiber instead of unknown Signed-off-by: Doug Dziggel Signed-off-by: Paul M. Stillwell Jr Signed-off-by: Qi Zhang --- drivers/net/ice/base/ice_common.c | 12 1 file changed, 12 insertions(+) diff --git a/drivers/net/ice/base/ice_common.c b/drivers/net/ice/base

[dpdk-dev] [PATCH 03/52] net/ice/base: disable profile merge for FDIR

2020-06-02 Thread Qi Zhang
For Flow Director, we don't want to re-use a existed profile with the same field vector and mask. Merging two different flow_type’s field vector will also make them sharing trained rule and cause rule interference. For example: issue rule A: IPV4_TCP matching tcp src&dst port 80 to queue 8 issue

[dpdk-dev] [PATCH 04/52] net/ice/base: avoid undefined behavior

2020-06-02 Thread Qi Zhang
When writing the driver's struct ice_tlan_ctx structure, do not write the 8-bit element int_q_state with the associated internal-to-hardware field which is 122-bits, otherwise the helper function ice_write_byte() will use undefined behavior when setting the mask used for that write. This should no

[dpdk-dev] [PATCH 07/52] net/ice/base: gate devices from FW link override

2020-06-02 Thread Qi Zhang
Currently, the FW link override feature is only permitted for E810 devices. However, the ice_fw_supports_link_override() guards against FW versions irrespective of the device. This assumes FW versions between the families are aligned, which is not the case. Signed-off-by: Jeb Cramer Signed-off-b

[dpdk-dev] [PATCH 02/52] net/ice/base: add FDIR program status WB macro

2020-06-02 Thread Qi Zhang
Add descriptor field offset and mask definition. It is used to parse FDIR rx descriptor field value. Signed-off-by: Yahui Cao Signed-off-by: Paul M. Stillwell Jr Signed-off-by: Qi Zhang --- drivers/net/ice/base/ice_lan_tx_rx.h | 44 1 file changed, 44 inser

[dpdk-dev] [PATCH 05/52] net/ice/base: consolidate implementation of MAC config set

2020-06-02 Thread Qi Zhang
Consolidate implementation of ice_aq_set_mac_cfg for switch mode and NIC mode. As per the specification, the driver needs to call set_mac_cfg (opcode 0x0603) to be able to exercise jumbo frames. Signed-off-by: Anirudh Venkataramanan Signed-off-by: Jeb Cramer Signed-off-by: Paul M. Stillwell Jr

[dpdk-dev] [PATCH 00/52] net/ice: base code update

2020-06-02 Thread Qi Zhang
Upgrade base with below main changes 1. add more protocol support - L2TPv3/ESP/AH/PFCP. 2. Add IPv6 prefix RSS support. 3. support GENEVE and VXLAN rules with VLAN. 4. Add E823L device IDs 5. couple bug fixes and code clean. Qi Zhang (52): net/ice/base: add support for non-IP Layer2 protocol

[dpdk-dev] [PATCH 01/52] net/ice/base: add support for non-IP Layer2 protocol

2020-06-02 Thread Qi Zhang
FDIR can forward Ethernet packets with non-IP ethertype. Signed-off-by: Yahui Cao Signed-off-by: Paul M. Stillwell Jr Signed-off-by: Qi Zhang --- drivers/net/ice/base/ice_fdir.c | 15 +++ drivers/net/ice/base/ice_fdir.h | 2 ++ drivers/net/ice/base/ice_flow.c | 29

Re: [dpdk-dev] [PATCH v6 01/11] eal: replace rte_page_sizes with a set of constants

2020-06-02 Thread Stephen Hemminger
On Wed, 3 Jun 2020 02:03:19 +0300 Dmitry Kozlyuk wrote: > Clang on Windows follows MS ABI where enum values are limited to 2^31-1. > Enum rte_page_sizes has members valued above this limit, which get > wrapped to zero, resulting in compilation error (duplicate values in > enum). Using MS ABI is

Re: [dpdk-dev] [PATCH v3 0/4] Memory corruption due to HW rings allocation

2020-06-02 Thread Zhao1, Wei
I am sure i40e FDIR filter also allocate ring for filter programing, it also has the same issue. Ice NIC the same. Although we can avoid this issue by change ring name to "igb_tx_ring" and so on, but the issue of memory for ring not freed after close is still exsit. Support this comment. If no on

[dpdk-dev] [PATCH v6 11/11] eal/windows: implement basic memory management

2020-06-02 Thread Dmitry Kozlyuk
Basic memory management supports core libraries and PMDs operating in IOVA as PA mode. It uses a kernel-mode driver, virt2phys, to obtain IOVAs of hugepages allocated from user-mode. Multi-process mode is not implemented and is forcefully disabled at startup. Assign myself as a maintainer for Windo

[dpdk-dev] [PATCH v6 10/11] eal/windows: initialize hugepage info

2020-06-02 Thread Dmitry Kozlyuk
Add hugepages discovery ("large pages" in Windows terminology) and update documentation for required privilege setup. Only 2MB hugepages are supported and their number is estimated roughly due to the lack or unstable status of suitable OS APIs. Assign myself as maintainer for the implementation fil

[dpdk-dev] [PATCH v6 09/11] eal/windows: improve CPU and NUMA node detection

2020-06-02 Thread Dmitry Kozlyuk
1. Map CPU cores to their respective NUMA nodes as reported by system. 2. Support systems with more than 64 cores (multiple processor groups). 3. Fix magic constants, styling issues, and compiler warnings. 4. Add EAL private function to map DPDK socket ID to NUMA node number. Fixes: 53ffd9f080fc (

[dpdk-dev] [PATCH v6 05/11] eal/mem: extract common code for dynamic memory allocation

2020-06-02 Thread Dmitry Kozlyuk
Code in Linux EAL that supports dynamic memory allocation (as opposed to static allocation used by FreeBSD) is not OS-dependent and can be reused by Windows EAL. Move such code to a file compiled only for the OS that require it. Keep Anatoly Burakov maintainer of extracted code. Signed-off-by: Dmi

[dpdk-dev] [PATCH v6 06/11] trace: add size_t field emitter

2020-06-02 Thread Dmitry Kozlyuk
It is not guaranteed that sizeof(long) == sizeof(size_t). On Windows, sizeof(long) == 4 and sizeof(size_t) == 8 for 64-bit programs. Tracepoints using "long" field emitter are therefore invalid there. Add dedicated field emitter for size_t and use it to store size_t values in all existing tracepoin

[dpdk-dev] [PATCH v6 08/11] eal/windows: replace sys/queue.h with a complete one from FreeBSD

2020-06-02 Thread Dmitry Kozlyuk
Limited version imported previously lacks at least SLIST macros. Import a complete file from FreeBSD, since its license exception is already approved by Technical Board. Signed-off-by: Dmitry Kozlyuk --- lib/librte_eal/windows/include/sys/queue.h | 663 +++-- 1 file changed, 601

[dpdk-dev] [PATCH v6 07/11] eal/windows: add tracing support stubs

2020-06-02 Thread Dmitry Kozlyuk
EAL common code depends on tracepoint calls, but generic implementation cannot be enabled on Windows due to missing standard library facilities. Add stub functions to support tracepoint compilation, so that common code does not have to conditionally include tracepoints until proper support is added

[dpdk-dev] [PATCH v6 03/11] eal: introduce memory management wrappers

2020-06-02 Thread Dmitry Kozlyuk
Introduce OS-independent wrappers for memory management operations used across DPDK and specifically in common code of EAL: * rte_mem_map() * rte_mem_unmap() * rte_mem_page_size() * rte_mem_lock() Windows uses different APIs for memory mapping and reservation, while Unices reserve memory by mappi

[dpdk-dev] [PATCH v6 04/11] eal/mem: extract common code for memseg list initialization

2020-06-02 Thread Dmitry Kozlyuk
All supported OS create memory segment lists (MSL) and reserve VA space for them in a nearly identical way. Move common code into EAL private functions to reduce duplication. Signed-off-by: Dmitry Kozlyuk --- lib/librte_eal/common/eal_common_memory.c | 97 ++ lib/librte_eal/comm

[dpdk-dev] [PATCH v6 00/11] Windows basic memory management

2020-06-02 Thread Dmitry Kozlyuk
This patchset implements basic MM with the following features: * Hugepages are dynamically allocated in user-mode. * Only 2MB hugepages are supported. * IOVA is always PA, obtained through kernel-mode driver. * No 32-bit support (presumably not demanded). * Ni multi-process support (it is forceful

[dpdk-dev] [PATCH v6 01/11] eal: replace rte_page_sizes with a set of constants

2020-06-02 Thread Dmitry Kozlyuk
Clang on Windows follows MS ABI where enum values are limited to 2^31-1. Enum rte_page_sizes has members valued above this limit, which get wrapped to zero, resulting in compilation error (duplicate values in enum). Using MS ABI is mandatory for Windows EAL to call Win32 APIs. Remove rte_page_size

[dpdk-dev] [PATCH v6 02/11] eal: introduce internal wrappers for file operations

2020-06-02 Thread Dmitry Kozlyuk
Introduce OS-independent wrappers in order to support common EAL code on Unix and Windows: * eal_file_create: open an existing file. * eal_file_open: create a file or open it if exists. * eal_file_lock: lock or unlock an open file. * eal_file_truncate: enforce a given size for an open file. Imple

Re: [dpdk-dev] Marvell DPDK v20.08 Roadmap

2020-06-02 Thread Thomas Monjalon
02/06/2020 19:36, Jerin Jacob: > On Tue, Jun 2, 2020 at 10:16 PM Thomas Monjalon wrote: > > 29/05/2020 14:50, Jerin Kollanukkaran: > > > Regexdev: > > > ~ > > > 11) Review regexdev: introduce regexdev subsystem > > > > If I remember well, you were targetting DPDK 20.08 > > to introduce a n

Re: [dpdk-dev] mlx5 & pdump: convert HW timestamps to nanoseconds

2020-06-02 Thread PATRICK KEROULAS
On Tue, May 26, 2020 at 12:00 PM Slava Ovsiienko wrote: > ConnectX HW timestamp is the captured value of internal 64-bit counter > running at the frequency, > reported in the device_frequency_khz field of struct > mlx5_ifc_cmd_hca_cap_bits{}. > This structure is queried in mlx5_devx_cmd_query_hc

Re: [dpdk-dev] [RFC] ethdev: add fragment attribute to IPv6 item

2020-06-02 Thread Adrien Mazarguil
Hi Ori, Andrew, Delek, (been a while eh?) On Tue, Jun 02, 2020 at 06:28:41PM +, Ori Kam wrote: > Hi Andrew, > > PSB, [...] > > > diff --git a/lib/librte_ethdev/rte_flow.h b/lib/librte_ethdev/rte_flow.h > > > index b0e4199..3bc8ce1 100644 > > > --- a/lib/librte_ethdev/rte_flow.h > > > +++ b/l

Re: [dpdk-dev] Marvell DPDK v20.08 Roadmap

2020-06-02 Thread Ori Kam
> -Original Message- > From: Jerin Jacob > Sent: Tuesday, June 2, 2020 8:37 PM > Subject: Re: [dpdk-dev] Marvell DPDK v20.08 Roadmap > > On Tue, Jun 2, 2020 at 10:16 PM Thomas Monjalon > wrote: > > > > 29/05/2020 14:50, Jerin Kollanukkaran: > > > Regexdev: > > > ~ > > > 11) Rev

Re: [dpdk-dev] [RFC] ethdev: add fragment attribute to IPv6 item

2020-06-02 Thread Ori Kam
Hi Andrew, PSB, Best, Ori > -Original Message- > From: Andrew Rybchenko > Sent: Tuesday, June 2, 2020 5:33 PM > Subject: Re: [RFC] ethdev: add fragment attribute to IPv6 item > > On 5/31/20 5:43 PM, Dekel Peled wrote: > > Using the current implementation of DPDK, an application cannot

Re: [dpdk-dev] [PATCH 0/4] Extend --lcores to run on cores > RTE_MAX_LCORE

2020-06-02 Thread Song, Keesang
[AMD Public Use] Thanks Thomas for the response. For a correction, the patchwork has not been submitted for the current LTS release, 19.11.2, but was merged into 20.02 and onward. The reason I brought this again was to address LTS users and many other application based on the LTS releases(18.11

Re: [dpdk-dev] [PATCH 0/4] Extend --lcores to run on cores > RTE_MAX_LCORE

2020-06-02 Thread Song, Keesang
[AMD Public Use] Hi Thomas & David, We haven't got the final status on this patch, and I don't see this change even from the latest LTS 20.04 repo. So I'd like to confirm whether this patch has been safely submitted to the main upstream. Can you check the status of that commit? https://patchwo

Re: [dpdk-dev] [PATCH 0/4] Extend --lcores to run on cores > RTE_MAX_LCORE

2020-06-02 Thread Song, Keesang
[AMD Official Use Only - Internal Distribution Only] Hi Thomas & David, We haven't got the final status on this patch, and I don't see this change even from the latest LTS 20.04 repo. So I'd like to confirm whether this patch has been safely submitted to the main upstream. Can you check the sta

Re: [dpdk-dev] [dpdk-stable] [PATCH] net/failsafe: report valid RSS RETA size

2020-06-02 Thread Ferruh Yigit
On 5/27/2020 11:02 PM, Stephen Hemminger wrote: > On Wed, 27 May 2020 23:35:25 +0200 > Gaëtan Rivet wrote: > >> On 27/05/20 18:30 +0300, Andrew Rybchenko wrote: >>> On 5/27/20 6:07 PM, Gaëtan Rivet wrote: On 27/05/20 15:34 +0100, Andrew Rybchenko wrote: > From: Ian Dolzhansky >

Re: [dpdk-dev] [PATCH] doc: removed typing mistake

2020-06-02 Thread Ferruh Yigit
On 5/29/2020 3:47 PM, Muhammad Bilal wrote: > There was a duplicate command instruction in the documentation of memif > so I have removed the 1 command from it. > > Fixes: cd3365d2 ("net/memif: enable loopback") > Cc: jmilan@gmail.com > > Signed-off-by: Muhammad Bilal > --- > doc/guides

Re: [dpdk-dev] Marvell DPDK v20.08 Roadmap

2020-06-02 Thread Jerin Jacob
On Tue, Jun 2, 2020 at 10:16 PM Thomas Monjalon wrote: > > 29/05/2020 14:50, Jerin Kollanukkaran: > > Regexdev: > > ~ > > 11) Review regexdev: introduce regexdev subsystem > > If I remember well, you were targetting DPDK 20.08 > to introduce a new regex PMD. > I am surprised this roadmap m

Re: [dpdk-dev] Questions Questions about hash function enum

2020-06-02 Thread Ferruh Yigit
On 6/1/2020 2:31 AM, oulijun wrote: > Hi Ferruh Yigit and others > > I am learning data structure defined by dpdk framework and I noticed > a data structure definition below: > > enum rte_eth_hash_function { > RTE_ETH_HASH_FUNCTION_DEFAULT = 0, > RTE_ETH_HASH_FUNCTION_TOEPLITZ, /**

Re: [dpdk-dev] [PATCH] test/autotest: checking for skipped or unsupported tests

2020-06-02 Thread David Christensen
]> - When running the dedault configuration of autotest of the make test it'd take 900 seconds (15 minutes) for the script TIMEOUT and marks Failed for a specific device test that is not supported on the system under test. - Adding the checking for those tests, print out as "Skipped [Not

Re: [dpdk-dev] Marvell DPDK v20.08 Roadmap

2020-06-02 Thread Thomas Monjalon
29/05/2020 14:50, Jerin Kollanukkaran: > Regexdev: > ~ > 11) Review regexdev: introduce regexdev subsystem If I remember well, you were targetting DPDK 20.08 to introduce a new regex PMD. I am surprised this roadmap mentions only reviewing the API.

Re: [dpdk-dev] Suggestion to improve the code review

2020-06-02 Thread Jerin Jacob
On Tue, Jun 2, 2020 at 8:27 PM Ferruh Yigit wrote: > > On 6/2/2020 1:27 PM, Jerin Jacob wrote: > > On Wed, May 27, 2020 at 3:38 PM Gaëtan Rivet wrote: > >> > >> On 27/05/20 09:28 +, Jerin Kollanukkaran wrote: > >>> I think, original discussion[1] on this topic got lost in GitHub vs > >>> cur

[dpdk-dev] [PATCH v2] test: test patch for checking patchwork auto delegation feature

2020-06-02 Thread jerinj
From: Jerin Jacob This is a test patch to test patchwork auto delegation feature from git send email. Signed-off-by: Jerin Jacob --- README | 3 +++ 1 file changed, 3 insertions(+) diff --git a/README b/README index 55df15864..adf7a7c31 100644 --- a/README +++ b/README @@ -9,3 +9,6 @@ API doc

[dpdk-dev] [PATCH 1/2] common/mlx5: share kernel interface name getter

2020-06-02 Thread Matan Azrad
Some configuration of the mlx5 port are done by the kernel net device associated to the IB device represents the PCI device. The DPDK mlx5 driver uses linux system calls, for example ioctl, in order to configure per port configurations requested by the DPDK user. One of the basic knowledges requi

[dpdk-dev] [PATCH 2/2] vdpa/mlx5: support MTU feature

2020-06-02 Thread Matan Azrad
The guest virtio device may request MTU updating when the vhost backend device exposes a capability to support it. Exspose the MTU feature capability. In configuration time, check the requested MTU and update it in the HW device. Signed-off-by: Matan Azrad --- doc/guides/rel_notes/release_20_0

[dpdk-dev] [PATCH 2/2] vdpa/mlx5: adjust virtio queue protection domain

2020-06-02 Thread Matan Azrad
In other to fill the new requirement for virtq configuration, set the single PD managed by the driver for all the virtqs. Cc: sta...@dpdk.org Signed-off-by: Matan Azrad Signed-off-by: Xueming Li --- drivers/vdpa/mlx5/mlx5_vdpa.c | 38 ++- drivers/vdpa/mlx5

[dpdk-dev] [PATCH 1/2] common/mlx5: add virtio queue protection domain

2020-06-02 Thread Matan Azrad
Starting from FW version 22.27.4002, it is required to configure protection domain (PD) for each virtq created by DevX. Add PD requirement in virtq DevX APIs. Cc: sta...@dpdk.org Signed-off-by: Matan Azrad Signed-off-by: Xueming Li --- drivers/common/mlx5/mlx5_devx_cmds.c | 1 + drivers/commo

[dpdk-dev] [PATCH v3 1/4] vhost: inroduce operation to get vDPA queue stats

2020-06-02 Thread Matan Azrad
The vDPA device offloads all the datapath of the vhost device to the HW device. In order to expose to the user traffic information this patch introduces new 3 APIs to get traffic statistics, the device statistics name and to reset the statistics per virtio queue. The statistics are taken directly

[dpdk-dev] [PATCH] test: test patch for checking patchwork auto delegation feature

2020-06-02 Thread jerinj
From: Jerin Jacob This is a test patch to test patchwork auto delegation feature from git send email. Signed-off-by: Jerin Jacob --- README | 3 +++ 1 file changed, 3 insertions(+) diff --git a/README b/README index 55df15864..adf7a7c31 100644 --- a/README +++ b/README @@ -9,3 +9,6 @@ API doc

[dpdk-dev] [PATCH v3 2/4] common/mlx5: support DevX virtq stats operations

2020-06-02 Thread Matan Azrad
Add DevX API to create and query virtio queue statistics from the HW. The next counters are supported by the HW per virtio queue: received_desc. completed_desc. error_cqes. bad_desc_errors. exceed_max_chain. invalid_buffer. Signed-off-by: Matan Azrad

[dpdk-dev] [PATCH v3 4/4] examples/vdpa: add statistics show command

2020-06-02 Thread Matan Azrad
A new vDPA driver feature was added to query the virtq statistics from the HW. Use this feature to show the HW queues statistics for the virtqs. Command description: stats X Y. X is the device ID. Y is the queue ID, Y=0x to show all the virtio queues statistics of the device X. Signed-off-by

[dpdk-dev] [PATCH v3 3/4] vdpa/mlx5: support virtio queue statistics get

2020-06-02 Thread Matan Azrad
Add support for statistics operations. A DevX counter object is allocated per virtq in order to manage the virtq statistics. The counter object is allocated before the virtq creation and destroyed after it, so the statistics are valid only in the life time of the virtq. Signed-off-by: Matan Azra

  1   2   >