From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id 94ABBA2EF9 for ; Mon, 16 Sep 2019 03:47:08 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 96EF11C12E; Mon, 16 Sep 2019 03:47:07 +0200 (CEST) Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) by dpdk.org (Postfix) with ESMTP id 364071C12D for ; Mon, 16 Sep 2019 03:47:04 +0200 (CEST) X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga003.fm.intel.com ([10.253.24.29]) by fmsmga101.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 15 Sep 2019 18:47:04 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.64,510,1559545200"; d="scan'208";a="193308563" Received: from fmsmsx106.amr.corp.intel.com ([10.18.124.204]) by FMSMGA003.fm.intel.com with ESMTP; 15 Sep 2019 18:47:04 -0700 Received: from fmsmsx120.amr.corp.intel.com (10.18.124.208) by FMSMSX106.amr.corp.intel.com (10.18.124.204) with Microsoft SMTP Server (TLS) id 14.3.439.0; Sun, 15 Sep 2019 18:47:04 -0700 Received: from shsmsx107.ccr.corp.intel.com (10.239.4.96) by fmsmsx120.amr.corp.intel.com (10.18.124.208) with Microsoft SMTP Server (TLS) id 14.3.439.0; Sun, 15 Sep 2019 18:47:03 -0700 Received: from shsmsx104.ccr.corp.intel.com ([169.254.5.32]) by SHSMSX107.ccr.corp.intel.com ([169.254.9.89]) with mapi id 14.03.0439.000; Mon, 16 Sep 2019 09:47:01 +0800 From: "Su, Simei" To: "Ye, Xiaolong" CC: "Zhang, Qi Z" , "Yang, Qiming" , "Xing, Beilei" , "dev@dpdk.org" , "Yigit, Ferruh" Thread-Topic: [PATCH v2] net/ice: enable advanced RSS Thread-Index: AQHVZx7pOHumTZOf706tEXs3vIDDp6cnt0MAgAXWWHA= Date: Mon, 16 Sep 2019 01:47:01 +0000 Message-ID: <65F28F834D25B54B9EC1999B623071B30C448168@SHSMSX104.ccr.corp.intel.com> References: <1567866291-268287-1-git-send-email-simei.su@intel.com> <1568041014-341418-1-git-send-email-simei.su@intel.com> <20190912161910.GB71525@intel.com> In-Reply-To: <20190912161910.GB71525@intel.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: dlp-product: dlpe-windows dlp-version: 11.2.0.6 dlp-reaction: no-action x-titus-metadata-40: eyJDYXRlZ29yeUxhYmVscyI6IiIsIk1ldGFkYXRhIjp7Im5zIjoiaHR0cDpcL1wvd3d3LnRpdHVzLmNvbVwvbnNcL0ludGVsMyIsImlkIjoiNzNiM2FkNTItMDhmNi00YTJmLTk4ZGUtZmJlZGY2MGZmZDdhIiwicHJvcHMiOlt7Im4iOiJDVFBDbGFzc2lmaWNhdGlvbiIsInZhbHMiOlt7InZhbHVlIjoiQ1RQX05UIn1dfV19LCJTdWJqZWN0TGFiZWxzIjpbXSwiVE1DVmVyc2lvbiI6IjE3LjEwLjE4MDQuNDkiLCJUcnVzdGVkTGFiZWxIYXNoIjoiWUN6cHpcL25WWU1IckZ1QUQyVGJCeFdyOGVnR3JlTUE0Qms3U2thY0JuRWlcLytzQ3BrSU15RGo0enZzejdORWZoIn0= x-ctpclassification: CTP_NT x-originating-ip: [10.239.127.40] Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Subject: Re: [dpdk-dev] [PATCH v2] net/ice: enable advanced RSS X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" Hi, xiaolong > -----Original Message----- > From: Ye, Xiaolong > Sent: Friday, September 13, 2019 12:19 AM > To: Su, Simei > Cc: Zhang, Qi Z ; Yang, Qiming ; > Xing, Beilei ; dev@dpdk.org; Yigit, Ferruh > > Subject: Re: [PATCH v2] net/ice: enable advanced RSS >=20 > On 09/09, Simei Su wrote: > >This patch supports the following features: > > (1)inner header hash for tunnel packets, including comms package. > > (2)symmetric hash by rte_flow RSS action. > > (3)input set change by rte_flow RSS action. > > > >Signed-off-by: Simei Su > >--- > >This patch depends on the following patches on patchwork: > > (1)https://patchwork.dpdk.org/patch/58546/ > > [2/4] net/ice: rework for generic flow enabling > > (2)https://patchwork.dpdk.org/patch/57137/ > > [v2,1/2] ethdev: add symmetric toeplitz hash support > > (3)https://patchwork.dpdk.org/patch/57138/ > > [v2,2/2] app/testpmd: add symmetric toeplitz hash support > > (4)https://patchwork.dpdk.org/patch/57601/ > > [1/2] ethdev: extend RSS offload types > > (5)https://patchwork.dpdk.org/patch/57602/ > > [2/2] app/testpmd: add RSS offload types extending support > > > > drivers/net/ice/ice_ethdev.c | 7 + > > drivers/net/ice/ice_hash.c | 548 > +++++++++++++++++++++++++++++++++++++++++++ > > 2 files changed, 555 insertions(+) > > create mode 100644 drivers/net/ice/ice_hash.c > > > >diff --git a/drivers/net/ice/ice_ethdev.c > >b/drivers/net/ice/ice_ethdev.c index f5cc647..3766a32 100644 > >--- a/drivers/net/ice/ice_ethdev.c > >+++ b/drivers/net/ice/ice_ethdev.c > >@@ -1874,6 +1874,7 @@ static int ice_init_rss(struct ice_pf *pf) > > uint16_t i, nb_q; > > int ret =3D 0; > > bool is_safe_mode =3D pf->adapter->is_safe_mode; > >+ uint32_t reg; > > > > rss_conf =3D &dev->data->dev_conf.rx_adv_conf.rss_conf; > > nb_q =3D dev->data->nb_rx_queues; > >@@ -1917,6 +1918,12 @@ static int ice_init_rss(struct ice_pf *pf) > > if (ret) > > return -EINVAL; > > > >+ /* Enable registers for symmetric_toeplitz function. */ > >+ reg =3D ICE_READ_REG(hw, VSIQF_HASH_CTL(vsi->vsi_id)); > >+ reg =3D (reg & (~VSIQF_HASH_CTL_HASH_SCHEME_M)) | > >+ (1 << VSIQF_HASH_CTL_HASH_SCHEME_S); > >+ ICE_WRITE_REG(hw, VSIQF_HASH_CTL(vsi->vsi_id), reg); > >+ > > /* configure RSS for IPv4 with input set IPv4 src/dst */ > > ret =3D ice_add_rss_cfg(hw, vsi->idx, ICE_FLOW_HASH_IPV4, > > ICE_FLOW_SEG_HDR_IPV4, 0); > >diff --git a/drivers/net/ice/ice_hash.c b/drivers/net/ice/ice_hash.c > >new file mode 100644 index 0000000..fbd7cd7 > >--- /dev/null > >+++ b/drivers/net/ice/ice_hash.c > >@@ -0,0 +1,548 @@ > >+/* SPDX-License-Identifier: BSD-3-Clause > >+ * Copyright(c) 2019 Intel Corporation */ > >+ > >+#include > >+#include > >+#include > >+#include > >+#include > >+#include > >+#include > >+ > >+#include > >+#include > >+#include > >+#include > >+#include > >+#include > >+#include > >+#include > >+ > >+#include "ice_logs.h" > >+#include "base/ice_type.h" > >+#include "base/ice_flow.h" > >+#include "ice_ethdev.h" > >+#include "ice_generic_flow.h" > >+ > >+#define ICE_ACTION_RSS_MAX_QUEUE_NUM 32 > >+ > >+static struct ice_pattern_match_item ice_hash_pattern_list_os[]; > >+static struct ice_pattern_match_item ice_hash_pattern_list_comms[]; >=20 > Above two forward declaration seems unnecessary. >=20 Yes, the two declaration is unnecessary, thanks for your reminder. > >+ > >+struct rss_type_match_hdr { > >+ uint32_t hdr_mask; > >+ uint64_t eth_rss_hint; > >+}; > >+ > >+struct ice_hash_match_type { > >+ uint64_t hash_type; > >+ uint64_t hash_flds; > >+}; > >+ > >+struct rss_meta { > >+ uint32_t pkt_hdr; > >+ uint64_t hash_flds; > >+ uint8_t hash_function; > >+}; > >+ > >+static int > >+ice_hash_init(struct ice_adapter *ad); > >+ > >+static int > >+ice_hash_create(struct ice_adapter *ad, > >+ struct rte_flow *flow, > >+ void *meta, > >+ struct rte_flow_error *error); > >+ > >+static int > >+ice_hash_destroy(struct ice_adapter *ad, > >+ struct rte_flow *flow, > >+ struct rte_flow_error *error); > >+ > >+static void > >+ice_hash_uninit(struct ice_adapter *ad); > >+ > >+static void > >+ice_hash_free(struct rte_flow *flow); > >+ > >+static int > >+ice_hash_parse_pattern_action(struct ice_adapter *ad, > >+ struct ice_pattern_match_item *array, > >+ uint32_t array_len, > >+ const struct rte_flow_item pattern[], > >+ const struct rte_flow_action actions[], > >+ void **meta, > >+ struct rte_flow_error *error); > >+ > >+/* The first member is protocol header, the second member is > >+ETH_RSS_*. */ const struct rss_type_match_hdr hint_0 =3D { > >+ ICE_FLOW_SEG_HDR_NONE, 0}; > >+const struct rss_type_match_hdr hint_1 =3D { > >+ ICE_FLOW_SEG_HDR_IPV4, ETH_RSS_IPV4}; > >+const struct rss_type_match_hdr hint_2 =3D { > >+ ICE_FLOW_SEG_HDR_IPV4|ICE_FLOW_SEG_HDR_UDP, > >+ETH_RSS_NONFRAG_IPV4_UDP}; const struct rss_type_match_hdr hint_3 =3D { > >+ ICE_FLOW_SEG_HDR_IPV4|ICE_FLOW_SEG_HDR_TCP, > >+ETH_RSS_NONFRAG_IPV4_TCP}; const struct rss_type_match_hdr hint_4 =3D { > >+ ICE_FLOW_SEG_HDR_IPV4|ICE_FLOW_SEG_HDR_SCTP, > >+ETH_RSS_NONFRAG_IPV4_SCTP}; const struct rss_type_match_hdr hint_5 =3D = { > >+ ICE_FLOW_SEG_HDR_IPV6, ETH_RSS_IPV6}; > >+const struct rss_type_match_hdr hint_6 =3D { > >+ ICE_FLOW_SEG_HDR_IPV6|ICE_FLOW_SEG_HDR_UDP, > >+ETH_RSS_NONFRAG_IPV6_UDP}; const struct rss_type_match_hdr hint_7 =3D { > >+ ICE_FLOW_SEG_HDR_IPV6|ICE_FLOW_SEG_HDR_TCP, > >+ETH_RSS_NONFRAG_IPV6_TCP}; const struct rss_type_match_hdr hint_8 =3D { > >+ ICE_FLOW_SEG_HDR_IPV6|ICE_FLOW_SEG_HDR_SCTP, > >+ETH_RSS_NONFRAG_IPV6_SCTP}; const struct rss_type_match_hdr hint_9 =3D = { > >+ ICE_FLOW_SEG_HDR_GTPU_IP, ETH_RSS_IPV4}; > >+const struct rss_type_match_hdr hint_10 =3D { > >+ ICE_FLOW_SEG_HDR_PPPOE, ETH_RSS_IPV4}; > >+const struct rss_type_match_hdr hint_11 =3D { > >+ ICE_FLOW_SEG_HDR_PPPOE, ETH_RSS_NONFRAG_IPV4_UDP}; > >+const struct rss_type_match_hdr hint_12 =3D { > >+ ICE_FLOW_SEG_HDR_PPPOE, ETH_RSS_NONFRAG_IPV4_TCP}; > >+const struct rss_type_match_hdr hint_13 =3D { > >+ ICE_FLOW_SEG_HDR_PPPOE, ETH_RSS_NONFRAG_IPV4_SCTP}; > >+ > >+/* Supported pattern for os default package. */ static struct > >+ice_pattern_match_item ice_hash_pattern_list_os[] =3D { > >+ {pattern_eth_ipv4, ICE_INSET_NONE, (uint64_t)(&hint_1)}, > >+ {pattern_eth_ipv4_udp, ICE_INSET_NONE, (uint64_t)(&hint_2)}, > >+ {pattern_eth_ipv4_tcp, ICE_INSET_NONE, (uint64_t)(&hint_3)}, > >+ {pattern_eth_ipv4_sctp, ICE_INSET_NONE, (uint64_t)(&hint_4)}, > >+ {pattern_eth_ipv6, ICE_INSET_NONE, (uint64_t)(&hint_5)}, > >+ {pattern_eth_ipv6_udp, ICE_INSET_NONE, (uint64_t)(&hint_6)}, > >+ {pattern_eth_ipv6_tcp, ICE_INSET_NONE, (uint64_t)(&hint_7)}, > >+ {pattern_eth_ipv6_sctp, ICE_INSET_NONE, (uint64_t)(&hint_8)}, > >+ {pattern_empty, ICE_INSET_NONE, (uint64_t)(&hint_0)}, > >+}; > >+ > >+/* Supported pattern for comms package. */ static struct > >+ice_pattern_match_item ice_hash_pattern_list_comms[] =3D { > >+ {pattern_eth_ipv4, ICE_INSET_NONE, (uint64_t)(&hint_1)}, > >+ {pattern_eth_ipv4_udp, ICE_INSET_NONE, (uint64_t)(&hint_2)}, > >+ {pattern_eth_ipv4_tcp, ICE_INSET_NONE, (uint64_t)(&hint_3)}, > >+ {pattern_eth_ipv4_sctp, ICE_INSET_NONE, (uint64_t)(&hint_4)}, > >+ {pattern_eth_ipv6, ICE_INSET_NONE, (uint64_t)(&hint_5)}, > >+ {pattern_eth_ipv6_udp, ICE_INSET_NONE, (uint64_t)(&hint_6)}, > >+ {pattern_eth_ipv6_tcp, ICE_INSET_NONE, (uint64_t)(&hint_7)}, > >+ {pattern_eth_ipv6_sctp, ICE_INSET_NONE, (uint64_t)(&hint_8)}, > >+ {pattern_empty, ICE_INSET_NONE, (uint64_t)(&hint_0)}, > >+ {pattern_eth_ipv4_gtpu_ipv4, ICE_INSET_NONE, > (uint64_t)(&hint_9)}, > >+ {pattern_eth_ipv4_gtpu_ipv4_udp, ICE_INSET_NONE, > (uint64_t)(&hint_9)}, > >+ {pattern_eth_ipv4_gtpu_ipv4_tcp, ICE_INSET_NONE, > (uint64_t)(&hint_9)}, > >+ {pattern_eth_pppoes_ipv4, ICE_INSET_NONE, (uint64_t)(&hint_10)}, > >+ {pattern_eth_pppoes_ipv4_udp, ICE_INSET_NONE, > (uint64_t)(&hint_11)}, > >+ {pattern_eth_pppoes_ipv4_tcp, ICE_INSET_NONE, > (uint64_t)(&hint_12)}, > >+ {pattern_eth_pppoes_ipv4_sctp, ICE_INSET_NONE, > (uint64_t)(&hint_13)}, > >+}; > >+ > >+/** > >+ * The first member is input set combination, > >+ * the second member is hash fields. > >+ */ > >+struct ice_hash_match_type ice_hash_type_list[] =3D { > >+ {ETH_RSS_IPV4|ETH_RSS_L3_SRC_ONLY, > BIT_ULL(ICE_FLOW_FIELD_IDX_IPV4_SA)}, > >+ {ETH_RSS_IPV4|ETH_RSS_L3_DST_ONLY, > BIT_ULL(ICE_FLOW_FIELD_IDX_IPV4_DA)}, > >+ {ETH_RSS_IPV4, > ICE_FLOW_HASH_IPV4}, > >+ > {ETH_RSS_NONFRAG_IPV4_UDP|ETH_RSS_L3_SRC_ONLY|ETH_RSS_L4_S > RC_ONLY, > BIT_ULL(ICE_FLOW_FIELD_IDX_IPV4_SA)|BIT_ULL(ICE_FLOW_FIELD_IDX > _UDP_SRC_PORT)}, > >+ > {ETH_RSS_NONFRAG_IPV4_UDP|ETH_RSS_L3_SRC_ONLY|ETH_RSS_L4_D > ST_ONLY, > BIT_ULL(ICE_FLOW_FIELD_IDX_IPV4_SA)|BIT_ULL(ICE_FLOW_FIELD_IDX > _UDP_DST_PORT)}, > >+ {ETH_RSS_NONFRAG_IPV4_UDP|ETH_RSS_L3_SRC_ONLY, > BIT_ULL(ICE_FLOW_FIELD_IDX_IPV4_SA)|ICE_FLOW_HASH_UDP_PORT}, > >+ > {ETH_RSS_NONFRAG_IPV4_UDP|ETH_RSS_L3_DST_ONLY|ETH_RSS_L4_S > RC_ONLY, > BIT_ULL(ICE_FLOW_FIELD_IDX_IPV4_DA)|BIT_ULL(ICE_FLOW_FIELD_IDX > _UDP_SRC_PORT)}, > >+ > {ETH_RSS_NONFRAG_IPV4_UDP|ETH_RSS_L3_DST_ONLY|ETH_RSS_L4_D > ST_ONLY, > BIT_ULL(ICE_FLOW_FIELD_IDX_IPV4_DA)|BIT_ULL(ICE_FLOW_FIELD_IDX > _UDP_DST_PORT)}, > >+ {ETH_RSS_NONFRAG_IPV4_UDP|ETH_RSS_L3_DST_ONLY, > BIT_ULL(ICE_FLOW_FIELD_IDX_IPV4_DA)|ICE_FLOW_HASH_UDP_PORT}, > >+ {ETH_RSS_NONFRAG_IPV4_UDP|ETH_RSS_L4_SRC_ONLY, > ICE_FLOW_HASH_IPV4|BIT_ULL(ICE_FLOW_FIELD_IDX_UDP_SRC_PORT)} > , > >+ {ETH_RSS_NONFRAG_IPV4_UDP|ETH_RSS_L4_DST_ONLY, > ICE_FLOW_HASH_IPV4|BIT_ULL(ICE_FLOW_FIELD_IDX_UDP_DST_PORT)} > , > >+ {ETH_RSS_NONFRAG_IPV4_UDP, > ICE_HASH_UDP_IPV4}, > >+ > {ETH_RSS_NONFRAG_IPV4_TCP|ETH_RSS_L3_SRC_ONLY|ETH_RSS_L4_SR > C_ONLY, > BIT_ULL(ICE_FLOW_FIELD_IDX_IPV4_SA)|BIT_ULL(ICE_FLOW_FIELD_IDX > _TCP_SRC_PORT)}, > >+ > {ETH_RSS_NONFRAG_IPV4_TCP|ETH_RSS_L3_SRC_ONLY|ETH_RSS_L4_DS > T_ONLY, > BIT_ULL(ICE_FLOW_FIELD_IDX_IPV4_SA)|BIT_ULL(ICE_FLOW_FIELD_IDX > _TCP_DST_PORT)}, > >+ {ETH_RSS_NONFRAG_IPV4_TCP|ETH_RSS_L3_SRC_ONLY, > BIT_ULL(ICE_FLOW_FIELD_IDX_IPV4_SA)|ICE_FLOW_HASH_TCP_PORT}, > >+ > {ETH_RSS_NONFRAG_IPV4_TCP|ETH_RSS_L3_DST_ONLY|ETH_RSS_L4_SR > C_ONLY, > BIT_ULL(ICE_FLOW_FIELD_IDX_IPV4_DA)|BIT_ULL(ICE_FLOW_FIELD_IDX > _TCP_SRC_PORT)}, > >+ > {ETH_RSS_NONFRAG_IPV4_TCP|ETH_RSS_L3_DST_ONLY|ETH_RSS_L4_D > ST_ONLY, > BIT_ULL(ICE_FLOW_FIELD_IDX_IPV4_DA)|BIT_ULL(ICE_FLOW_FIELD_IDX > _TCP_DST_PORT)}, > >+ {ETH_RSS_NONFRAG_IPV4_TCP|ETH_RSS_L3_DST_ONLY, > BIT_ULL(ICE_FLOW_FIELD_IDX_IPV4_DA)|ICE_FLOW_HASH_TCP_PORT}, > >+ {ETH_RSS_NONFRAG_IPV4_TCP|ETH_RSS_L4_SRC_ONLY, > ICE_FLOW_HASH_IPV4|BIT_ULL(ICE_FLOW_FIELD_IDX_TCP_SRC_PORT)}, > >+ {ETH_RSS_NONFRAG_IPV4_TCP|ETH_RSS_L4_DST_ONLY, > ICE_FLOW_HASH_IPV4|BIT_ULL(ICE_FLOW_FIELD_IDX_TCP_DST_PORT)}, > >+ {ETH_RSS_NONFRAG_IPV4_TCP, > ICE_HASH_TCP_IPV4}, > >+ > {ETH_RSS_NONFRAG_IPV4_SCTP|ETH_RSS_L3_SRC_ONLY|ETH_RSS_L4_S > RC_ONLY, > BIT_ULL(ICE_FLOW_FIELD_IDX_IPV4_SA)|BIT_ULL(ICE_FLOW_FIELD_IDX > _SCTP_SRC_PORT)}, > >+ > {ETH_RSS_NONFRAG_IPV4_SCTP|ETH_RSS_L3_SRC_ONLY|ETH_RSS_L4_D > ST_ONLY, > BIT_ULL(ICE_FLOW_FIELD_IDX_IPV4_SA)|BIT_ULL(ICE_FLOW_FIELD_IDX > _SCTP_DST_PORT)}, > >+ {ETH_RSS_NONFRAG_IPV4_SCTP|ETH_RSS_L3_SRC_ONLY, > BIT_ULL(ICE_FLOW_FIELD_IDX_IPV4_SA)|ICE_FLOW_HASH_SCTP_PORT}, > >+ > {ETH_RSS_NONFRAG_IPV4_SCTP|ETH_RSS_L3_DST_ONLY|ETH_RSS_L4_S > RC_ONLY, > BIT_ULL(ICE_FLOW_FIELD_IDX_IPV4_DA)|BIT_ULL(ICE_FLOW_FIELD_IDX > _SCTP_SRC_PORT)}, > >+ > {ETH_RSS_NONFRAG_IPV4_SCTP|ETH_RSS_L3_DST_ONLY|ETH_RSS_L4_ > DST_ONLY, > BIT_ULL(ICE_FLOW_FIELD_IDX_IPV4_DA)|BIT_ULL(ICE_FLOW_FIELD_IDX > _SCTP_DST_PORT)}, > >+ {ETH_RSS_NONFRAG_IPV4_SCTP|ETH_RSS_L3_DST_ONLY, > BIT_ULL(ICE_FLOW_FIELD_IDX_IPV4_DA)|ICE_FLOW_HASH_SCTP_PORT} > , > >+ {ETH_RSS_NONFRAG_IPV4_SCTP|ETH_RSS_L4_SRC_ONLY, > ICE_FLOW_HASH_IPV4|BIT_ULL(ICE_FLOW_FIELD_IDX_SCTP_SRC_PORT) > }, > >+ {ETH_RSS_NONFRAG_IPV4_SCTP|ETH_RSS_L4_DST_ONLY, > ICE_FLOW_HASH_IPV4|BIT_ULL(ICE_FLOW_FIELD_IDX_SCTP_DST_PORT) > }, > >+ {ETH_RSS_NONFRAG_IPV4_SCTP, > ICE_HASH_SCTP_IPV4}, > >+ {ETH_RSS_IPV6|ETH_RSS_L3_SRC_ONLY, > BIT_ULL(ICE_FLOW_FIELD_IDX_IPV6_SA)}, > >+ {ETH_RSS_IPV6|ETH_RSS_L3_DST_ONLY, > BIT_ULL(ICE_FLOW_FIELD_IDX_IPV6_DA)}, > >+ {ETH_RSS_IPV6, > ICE_FLOW_HASH_IPV6}, > >+ > {ETH_RSS_NONFRAG_IPV6_UDP|ETH_RSS_L3_SRC_ONLY|ETH_RSS_L4_S > RC_ONLY, > BIT_ULL(ICE_FLOW_FIELD_IDX_IPV6_SA)|BIT_ULL(ICE_FLOW_FIELD_IDX > _UDP_SRC_PORT)}, > >+ > {ETH_RSS_NONFRAG_IPV6_UDP|ETH_RSS_L3_SRC_ONLY|ETH_RSS_L4_D > ST_ONLY, > BIT_ULL(ICE_FLOW_FIELD_IDX_IPV6_SA)|BIT_ULL(ICE_FLOW_FIELD_IDX > _UDP_DST_PORT)}, > >+ {ETH_RSS_NONFRAG_IPV6_UDP|ETH_RSS_L3_SRC_ONLY, > BIT_ULL(ICE_FLOW_FIELD_IDX_IPV6_SA)|ICE_FLOW_HASH_UDP_PORT}, > >+ > {ETH_RSS_NONFRAG_IPV6_UDP|ETH_RSS_L3_DST_ONLY|ETH_RSS_L4_S > RC_ONLY, > BIT_ULL(ICE_FLOW_FIELD_IDX_IPV6_DA)|BIT_ULL(ICE_FLOW_FIELD_IDX > _UDP_SRC_PORT)}, > >+ > {ETH_RSS_NONFRAG_IPV6_UDP|ETH_RSS_L3_DST_ONLY|ETH_RSS_L4_D > ST_ONLY, > BIT_ULL(ICE_FLOW_FIELD_IDX_IPV6_DA)|BIT_ULL(ICE_FLOW_FIELD_IDX > _UDP_DST_PORT)}, > >+ {ETH_RSS_NONFRAG_IPV6_UDP|ETH_RSS_L3_DST_ONLY, > BIT_ULL(ICE_FLOW_FIELD_IDX_IPV6_DA)|ICE_FLOW_HASH_UDP_PORT}, > >+ {ETH_RSS_NONFRAG_IPV6_UDP|ETH_RSS_L4_SRC_ONLY, > ICE_FLOW_HASH_IPV6|BIT_ULL(ICE_FLOW_FIELD_IDX_UDP_SRC_PORT)} > , > >+ {ETH_RSS_NONFRAG_IPV6_UDP|ETH_RSS_L4_DST_ONLY, > ICE_FLOW_HASH_IPV6|BIT_ULL(ICE_FLOW_FIELD_IDX_UDP_DST_PORT)} > , > >+ {ETH_RSS_NONFRAG_IPV6_UDP, > ICE_HASH_UDP_IPV6}, > >+ > {ETH_RSS_NONFRAG_IPV6_TCP|ETH_RSS_L3_SRC_ONLY|ETH_RSS_L4_SR > C_ONLY, > BIT_ULL(ICE_FLOW_FIELD_IDX_IPV6_SA)|BIT_ULL(ICE_FLOW_FIELD_IDX > _TCP_SRC_PORT)}, > >+ > {ETH_RSS_NONFRAG_IPV6_TCP|ETH_RSS_L3_SRC_ONLY|ETH_RSS_L4_DS > T_ONLY, > BIT_ULL(ICE_FLOW_FIELD_IDX_IPV6_SA)|BIT_ULL(ICE_FLOW_FIELD_IDX > _TCP_DST_PORT)}, > >+ {ETH_RSS_NONFRAG_IPV6_TCP|ETH_RSS_L3_SRC_ONLY, > BIT_ULL(ICE_FLOW_FIELD_IDX_IPV6_SA)|ICE_FLOW_HASH_TCP_PORT}, > >+ > {ETH_RSS_NONFRAG_IPV6_TCP|ETH_RSS_L3_DST_ONLY|ETH_RSS_L4_SR > C_ONLY, > BIT_ULL(ICE_FLOW_FIELD_IDX_IPV6_DA)|BIT_ULL(ICE_FLOW_FIELD_IDX > _TCP_SRC_PORT)}, > >+ > {ETH_RSS_NONFRAG_IPV6_TCP|ETH_RSS_L3_DST_ONLY|ETH_RSS_L4_D > ST_ONLY, > BIT_ULL(ICE_FLOW_FIELD_IDX_IPV6_DA)|BIT_ULL(ICE_FLOW_FIELD_IDX > _TCP_DST_PORT)}, > >+ {ETH_RSS_NONFRAG_IPV6_TCP|ETH_RSS_L3_DST_ONLY, > BIT_ULL(ICE_FLOW_FIELD_IDX_IPV6_DA)|ICE_FLOW_HASH_TCP_PORT}, > >+ {ETH_RSS_NONFRAG_IPV6_TCP|ETH_RSS_L4_SRC_ONLY, > ICE_FLOW_HASH_IPV6|BIT_ULL(ICE_FLOW_FIELD_IDX_TCP_SRC_PORT)}, > >+ {ETH_RSS_NONFRAG_IPV6_TCP|ETH_RSS_L4_DST_ONLY, > ICE_FLOW_HASH_IPV6|BIT_ULL(ICE_FLOW_FIELD_IDX_TCP_DST_PORT)}, > >+ {ETH_RSS_NONFRAG_IPV6_TCP, > ICE_HASH_TCP_IPV6}, > >+ > {ETH_RSS_NONFRAG_IPV6_SCTP|ETH_RSS_L3_SRC_ONLY|ETH_RSS_L4_S > RC_ONLY, > BIT_ULL(ICE_FLOW_FIELD_IDX_IPV6_SA)|BIT_ULL(ICE_FLOW_FIELD_IDX > _SCTP_SRC_PORT)}, > >+ > {ETH_RSS_NONFRAG_IPV6_SCTP|ETH_RSS_L3_SRC_ONLY|ETH_RSS_L4_D > ST_ONLY, > BIT_ULL(ICE_FLOW_FIELD_IDX_IPV6_SA)|BIT_ULL(ICE_FLOW_FIELD_IDX > _SCTP_DST_PORT)}, > >+ {ETH_RSS_NONFRAG_IPV6_SCTP|ETH_RSS_L3_SRC_ONLY, > BIT_ULL(ICE_FLOW_FIELD_IDX_IPV6_SA)|ICE_FLOW_HASH_SCTP_PORT}, > >+ > {ETH_RSS_NONFRAG_IPV6_SCTP|ETH_RSS_L3_DST_ONLY|ETH_RSS_L4_S > RC_ONLY, > BIT_ULL(ICE_FLOW_FIELD_IDX_IPV6_DA)|BIT_ULL(ICE_FLOW_FIELD_IDX > _SCTP_SRC_PORT)}, > >+ > {ETH_RSS_NONFRAG_IPV6_SCTP|ETH_RSS_L3_DST_ONLY|ETH_RSS_L4_ > DST_ONLY, > BIT_ULL(ICE_FLOW_FIELD_IDX_IPV6_DA)|BIT_ULL(ICE_FLOW_FIELD_IDX > _SCTP_DST_PORT)}, > >+ {ETH_RSS_NONFRAG_IPV6_SCTP|ETH_RSS_L3_DST_ONLY, > BIT_ULL(ICE_FLOW_FIELD_IDX_IPV6_DA)|ICE_FLOW_HASH_SCTP_PORT} > , > >+ {ETH_RSS_NONFRAG_IPV6_SCTP|ETH_RSS_L4_SRC_ONLY, > ICE_FLOW_HASH_IPV6|BIT_ULL(ICE_FLOW_FIELD_IDX_SCTP_SRC_PORT) > }, > >+ {ETH_RSS_NONFRAG_IPV6_SCTP|ETH_RSS_L4_DST_ONLY, > ICE_FLOW_HASH_IPV6|BIT_ULL(ICE_FLOW_FIELD_IDX_SCTP_DST_PORT) > }, > >+ {ETH_RSS_NONFRAG_IPV6_SCTP, > ICE_HASH_SCTP_IPV6}, > >+}; > >+ > >+static struct ice_flow_engine ice_hash_engine =3D { > >+ .init =3D ice_hash_init, > >+ .create =3D ice_hash_create, > >+ .destroy =3D ice_hash_destroy, > >+ .uninit =3D ice_hash_uninit, > >+ .free =3D ice_hash_free, > >+ .type =3D ICE_FLOW_ENGINE_HASH, > >+}; > >+ > >+/* Register parser for os package. */ > >+static struct ice_flow_parser ice_hash_parser_os =3D { > >+ .engine =3D &ice_hash_engine, > >+ .array =3D ice_hash_pattern_list_os, > >+ .array_len =3D RTE_DIM(ice_hash_pattern_list_os), > >+ .parse_pattern_action =3D ice_hash_parse_pattern_action, > >+ .stage =3D ICE_FLOW_STAGE_RSS, > >+}; > >+ > >+/* Register parser for comms package. */ static struct ice_flow_parser > >+ice_hash_parser_comms =3D { > >+ .engine =3D &ice_hash_engine, > >+ .array =3D ice_hash_pattern_list_comms, > >+ .array_len =3D RTE_DIM(ice_hash_pattern_list_comms), > >+ .parse_pattern_action =3D ice_hash_parse_pattern_action, > >+ .stage =3D ICE_FLOW_STAGE_RSS, > >+}; > >+ > >+RTE_INIT(ice_hash_init_log) >=20 > I don't think this function has something to do with 'log', better to use= name like > ice_hash_engine_init. >=20 =20 I write "ice_hash_init_log" referring to other module. I also think it ha= s nothing to do with 'log'. Ok, I will modify it. > >+{ > >+ struct ice_flow_engine *engine =3D &ice_hash_engine; > >+ ice_register_flow_engine(engine); > >+} > >+ > >+static int > >+ice_hash_init(struct ice_adapter *ad) > >+{ > >+ int ret =3D 0; >=20 > Unnecessary initialization for the ret. >=20 Ok, will change it. > >+ > >+ if (ad->active_pkg_type =3D=3D ICE_PKG_TYPE_OS_DEFAULT) { > >+ struct ice_flow_parser *parser =3D &ice_hash_parser_os; > >+ ret =3D ice_register_parser(parser, ad); > >+ } else if (ad->active_pkg_type =3D=3D ICE_PKG_TYPE_COMMS) { > >+ struct ice_flow_parser *parser =3D &ice_hash_parser_comms; > >+ ret =3D ice_register_parser(parser, ad); > >+ } > >+ >=20 > What about: >=20 > struct ice_flow_parser *parser; >=20 > if (ad->active_pkg_type =3D=3D ICE_PKG_TYPE_OS_DEFAULT) { > parser =3D &ice_hash_parser_os; > } else if (ad->active_pkg_type =3D=3D ICE_PKG_TYPE_COMMS) { > parser =3D &ice_hash_parser_comms; > } >=20 > return ice_register_parser(parser, ad); >=20 It looks a good advice, can avoid unnecessary redundancy. > >+ return ret; > >+} > >+ > >+static int > >+ice_hash_check_inset(const struct rte_flow_item pattern[], > >+ struct rte_flow_error *error) > >+{ > >+ const struct rte_flow_item *item =3D pattern; > >+ > >+ for (item =3D pattern; item->type !=3D RTE_FLOW_ITEM_TYPE_END; item++)= { > >+ if (item->last) { > >+ rte_flow_error_set(error, EINVAL, > >+ RTE_FLOW_ERROR_TYPE_ITEM, item, > >+ "Not support range"); > >+ return -rte_errno; > >+ } > >+ > >+ /* Ignore spec and mask. */ > >+ if (item->spec || item->mask) { > >+ rte_flow_error_set(error, EINVAL, > >+ RTE_FLOW_ERROR_TYPE_ITEM, item, > >+ "Invalid mask."); >=20 > "Invalid spec/mask." >=20 Yes, I ignore it, will add it. > >+ return -rte_errno; > >+ } > >+ } > >+ > >+ return 0; > >+} > >+ > >+static int > >+ice_hash_parse_action(struct ice_pattern_match_item > *pattern_match_item, > >+ const struct rte_flow_action actions[], > >+ void **meta, > >+ struct rte_flow_error *error) > >+{ > >+ const struct rte_flow_action *action; > >+ enum rte_flow_action_type action_type; > >+ const struct rte_flow_action_rss *rss; > >+ struct rss_type_match_hdr *m =3D (struct rss_type_match_hdr *) > >+ (pattern_match_item->meta); > >+ uint32_t type_list_len =3D RTE_DIM(ice_hash_type_list); > >+ struct ice_hash_match_type *type_match_item; > >+ > >+ /* Supported action is RSS. */ > >+ for (action =3D actions; action->type !=3D > >+ RTE_FLOW_ACTION_TYPE_END; action++) { > >+ action_type =3D action->type; > >+ switch (action_type) { > >+ case RTE_FLOW_ACTION_TYPE_RSS: > >+ rss =3D action->conf; > >+ uint16_t i; > >+ > >+ /* Check if pattern is empty. */ > >+ if (((pattern_match_item->pattern_list) !=3D > >+ pattern_empty) && (rss->func =3D=3D > >+ RTE_ETH_HASH_FUNCTION_SIMPLE_XOR)) > >+ return rte_flow_error_set(error, ENOTSUP, > >+ RTE_FLOW_ERROR_TYPE_ACTION, action, > >+ "Not supported flow"); > >+ > >+ /* Check if rss types match pattern. */ > >+ if (rss->func !=3D RTE_ETH_HASH_FUNCTION_SIMPLE_XOR) { > >+ if (((rss->types & ETH_RSS_IPV4) !=3D m->eth_rss_hint) || > >+ ((rss->types & ETH_RSS_NONFRAG_IPV4_UDP) !=3D > m->eth_rss_hint) || > >+ ((rss->types & ETH_RSS_NONFRAG_IPV4_TCP) !=3D > m->eth_rss_hint) || > >+ ((rss->types & ETH_RSS_NONFRAG_IPV4_SCTP) !=3D > m->eth_rss_hint) || > >+ ((rss->types & ETH_RSS_IPV6) !=3D m->eth_rss_hint) || > >+ ((rss->types & ETH_RSS_NONFRAG_IPV6_UDP) !=3D > m->eth_rss_hint) || > >+ ((rss->types & ETH_RSS_NONFRAG_IPV6_TCP) !=3D > m->eth_rss_hint) || > >+ ((rss->types & ETH_RSS_NONFRAG_IPV6_SCTP) !=3D > m->eth_rss_hint)) > >+ return rte_flow_error_set(error, > >+ ENOTSUP, RTE_FLOW_ERROR_TYPE_ACTION, > >+ action, "Not supported RSS types"); > >+ } > >+ > >+ if (rss->level) > >+ return rte_flow_error_set(error, ENOTSUP, > >+ RTE_FLOW_ERROR_TYPE_ACTION, action, > >+ "a nonzero RSS encapsulation level is not supported"); > >+ > >+ if (rss->key_len =3D=3D 0) > >+ return rte_flow_error_set(error, ENOTSUP, > >+ RTE_FLOW_ERROR_TYPE_ACTION, action, > >+ "RSS hash key_len mustn't be 0"); > >+ > >+ if ((rss->queue_num) > ICE_ACTION_RSS_MAX_QUEUE_NUM) > >+ return rte_flow_error_set(error, ENOTSUP, > >+ RTE_FLOW_ERROR_TYPE_ACTION, action, > >+ "too many queues for RSS context"); > >+ > >+ /* Check hash function and save it to rss_meta. */ > >+ if (rss->func =3D=3D > >+ RTE_ETH_HASH_FUNCTION_SIMPLE_XOR) > >+ ((struct rss_meta *)*meta)->hash_function =3D > >+ RTE_ETH_HASH_FUNCTION_SIMPLE_XOR; > >+ > >+ if (rss->func =3D=3D > >+ RTE_ETH_HASH_FUNCTION_SYMMETRIC_TOEPLITZ) > >+ ((struct rss_meta *)*meta)->hash_function =3D > >+ RTE_ETH_HASH_FUNCTION_SYMMETRIC_TOEPLITZ; > >+ > >+ type_match_item =3D rte_zmalloc("ice_type_match_item", > >+ sizeof(struct ice_hash_match_type), 0); > >+ if (!type_match_item) { > >+ rte_flow_error_set(error, EINVAL, > >+ RTE_FLOW_ERROR_TYPE_HANDLE, NULL, > >+ "No memory for type_match_item"); > >+ return -ENOMEM; > >+ } > >+ > >+ /* Find matched hash fields according to hash type. */ > >+ for (i =3D 0; i < type_list_len; i++) { > >+ if (rss->types =3D=3D > >+ ice_hash_type_list[i].hash_type) { > >+ type_match_item->hash_type =3D > >+ ice_hash_type_list[i].hash_type; > >+ type_match_item->hash_flds =3D > >+ ice_hash_type_list[i].hash_flds; > >+ } > >+ } > >+ > >+ /* Save hash fileds to rss_meta. */ > >+ ((struct rss_meta *)*meta)->hash_flds =3D > >+ type_match_item->hash_flds; > >+ > >+ rte_free(type_match_item); > >+ break; > >+ > >+ case RTE_FLOW_ACTION_TYPE_END: > >+ return 0; > >+ break; > >+ > >+ default: > >+ rte_flow_error_set(error, EINVAL, > >+ RTE_FLOW_ERROR_TYPE_ACTION, action, > >+ "Invalid action."); > >+ return -rte_errno; > >+ } > >+ } > >+ > >+ return 0; > >+} > >+ > >+static int > >+ice_hash_parse_pattern_action(__rte_unused struct ice_adapter *ad, > >+ struct ice_pattern_match_item *array, > >+ uint32_t array_len, > >+ const struct rte_flow_item pattern[], > >+ const struct rte_flow_action actions[], > >+ void **meta, > >+ struct rte_flow_error *error) > >+{ > >+ int ret =3D 0; > >+ struct ice_pattern_match_item *pattern_match_item =3D NULL; > >+ struct rss_meta *rss_meta_ptr =3D NULL; >=20 > Unnecessary initializations. >=20 Ok, will change it. > >+ > >+ rss_meta_ptr =3D rte_zmalloc(NULL, sizeof(*rss_meta_ptr), 0); > >+ if (!rss_meta_ptr) { > >+ rte_flow_error_set(error, EINVAL, > >+ RTE_FLOW_ERROR_TYPE_HANDLE, NULL, > >+ "No memory for rss_meta_ptr"); > >+ return -ENOMEM; > >+ } > >+ > >+ /* Check rss supported pattern and find matched pattern. */ > >+ pattern_match_item =3D ice_search_pattern_match_item(pattern, > >+ array, array_len, error); > >+ if (!pattern_match_item) > >+ return -rte_errno; > >+ > >+ ret =3D ice_hash_check_inset(pattern, error); > >+ if (ret) > >+ return -rte_errno; > >+ > >+ /* Save protocol header to rss_meta. */ > >+ *meta =3D rss_meta_ptr; > >+ ((struct rss_meta *)*meta)->pkt_hdr =3D ((struct rss_type_match_hdr *) > >+ (pattern_match_item->meta))->hdr_mask; > >+ > >+ /* Check rss action. */ > >+ ret =3D ice_hash_parse_action(pattern_match_item, actions, meta, error= ); > >+ if (ret) > >+ return -rte_errno; > >+ > >+ rte_free(pattern_match_item); > >+ > >+ return 0; > >+} > >+ > >+static int > >+ice_hash_create(struct ice_adapter *ad, > >+ struct rte_flow *flow, > >+ void *meta, > >+ __rte_unused struct rte_flow_error *error) { > >+ struct ice_pf *pf =3D &ad->pf; > >+ struct ice_hw *hw =3D ICE_PF_TO_HW(pf); > >+ struct ice_vsi *vsi =3D pf->main_vsi; > >+ int ret =3D 0; > >+ uint32_t reg; > >+ struct ice_rss_cfg *filter_ptr; > >+ > >+ uint32_t headermask =3D ((struct rss_meta *)meta)->pkt_hdr; > >+ uint64_t hash_field =3D ((struct rss_meta *)meta)->hash_flds; > >+ uint8_t hash_function =3D ((struct rss_meta *)meta)->hash_function; > >+ > >+ filter_ptr =3D rte_zmalloc("ice_rss_filter", > >+ sizeof(struct ice_rss_cfg), 0); > >+ if (!filter_ptr) { > >+ rte_flow_error_set(error, EINVAL, > >+ RTE_FLOW_ERROR_TYPE_HANDLE, NULL, > >+ "No memory for filter_ptr"); > >+ return -ENOMEM; > >+ } > >+ > >+ if (hash_function =3D=3D RTE_ETH_HASH_FUNCTION_SIMPLE_XOR) { > >+ /* Enable registers for simple_xor hash function. */ > >+ reg =3D ICE_READ_REG(hw, VSIQF_HASH_CTL(vsi->vsi_id)); > >+ reg =3D (reg & (~VSIQF_HASH_CTL_HASH_SCHEME_M)) | > >+ (2 << VSIQF_HASH_CTL_HASH_SCHEME_S); > >+ ICE_WRITE_REG(hw, VSIQF_HASH_CTL(vsi->vsi_id), reg); > >+ > >+ filter_ptr->symm =3D hash_function; > >+ > >+ goto out; > >+ } else if (hash_function =3D=3D > RTE_ETH_HASH_FUNCTION_SYMMETRIC_TOEPLITZ) { > >+ ret =3D ice_add_rss_cfg(hw, vsi->idx, hash_field, headermask, 1); > >+ if (ret) > >+ rte_flow_error_set(error, EINVAL, > >+ RTE_FLOW_ERROR_TYPE_HANDLE, NULL, > >+ "rss flow create fail"); >=20 > Do we need to error out here? >=20 Yes, it's my negligence that other than return error, also need to error = out here. I will add it in v3 patch. > >+ } else { > >+ ret =3D ice_add_rss_cfg(hw, vsi->idx, hash_field, headermask, 0); > >+ if (ret) > >+ rte_flow_error_set(error, EINVAL, > >+ RTE_FLOW_ERROR_TYPE_HANDLE, NULL, > >+ "rss flow create fail"); >=20 > Ditto. >=20 > >+ } > >+ > >+ filter_ptr->packet_hdr =3D headermask; > >+ filter_ptr->hashed_flds =3D hash_field; > >+ > >+out: > >+ flow->rule =3D filter_ptr; > >+ rte_free(meta); > >+ return 0; > >+} > >+ > >+static int > >+ice_hash_destroy(struct ice_adapter *ad, > >+ struct rte_flow *flow, > >+ __rte_unused struct rte_flow_error *error) >=20 > It's actually used in this function, so just remove the __rte_unused tag. >=20 Yes, I forgot to remove it, thanks! > >+{ > >+ struct ice_pf *pf =3D ICE_DEV_PRIVATE_TO_PF(ad); > >+ struct ice_hw *hw =3D ICE_PF_TO_HW(pf); > >+ struct ice_vsi *vsi =3D pf->main_vsi; > >+ int ret =3D 0; >=20 > Unnecessary initialization. >=20 Ok, will change it. > >+ uint32_t reg; > >+ struct ice_rss_cfg *filter_ptr; > >+ > >+ filter_ptr =3D (struct ice_rss_cfg *)flow->rule; > >+ > >+ if (filter_ptr->symm =3D=3D RTE_ETH_HASH_FUNCTION_SIMPLE_XOR) { > >+ /* Return to symmetric_toeplitz state. */ > >+ reg =3D ICE_READ_REG(hw, VSIQF_HASH_CTL(vsi->vsi_id)); > >+ reg =3D (reg & (~VSIQF_HASH_CTL_HASH_SCHEME_M)) | > >+ (1 << VSIQF_HASH_CTL_HASH_SCHEME_S); > >+ ICE_WRITE_REG(hw, VSIQF_HASH_CTL(vsi->vsi_id), reg); > >+ } else { > >+ ret =3D ice_rem_vsi_rss_cfg(hw, vsi->idx); > >+ if (ret) > >+ rte_flow_error_set(error, EINVAL, > >+ RTE_FLOW_ERROR_TYPE_HANDLE, NULL, > >+ "rss flow destroy fail"); >=20 > If ice_rem_vsi_rss_cfg fails here, do we still need to proceed to free th= e flow > other than return error here? >=20 Yes, as to this, I will add it. > >+ } > >+ > >+ rte_free(filter_ptr); > >+ > >+ return 0; > >+} > >+ > >+static void > >+ice_hash_uninit(struct ice_adapter *ad) { > >+ if (ad->active_pkg_type =3D=3D ICE_PKG_TYPE_OS_DEFAULT) > >+ ice_unregister_parser(&ice_hash_parser_os, ad); > >+ else if (ad->active_pkg_type =3D=3D ICE_PKG_TYPE_COMMS) > >+ ice_unregister_parser(&ice_hash_parser_comms, ad); } > >+ > >+static void > >+ice_hash_free(struct rte_flow *flow) > >+{ > >+ rte_free(flow->rule); > >+} > >-- > >1.8.3.1 > > Br Simei