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 A22E4A046B; Sat, 9 May 2020 08:20:43 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id E62961D939; Sat, 9 May 2020 08:20:42 +0200 (CEST) Received: from mga04.intel.com (mga04.intel.com [192.55.52.120]) by dpdk.org (Postfix) with ESMTP id C55E01D937 for ; Sat, 9 May 2020 08:20:41 +0200 (CEST) IronPort-SDR: qWa3OLeQnfADUv5UaPRlqzk8UqLquZoupB50SpXzx+EKX4ie4VuPHB27hC0l5OebhRj3wkRPxV aVAKOfZzE49A== X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga003.fm.intel.com ([10.253.24.29]) by fmsmga104.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 08 May 2020 23:20:40 -0700 IronPort-SDR: g7CH/VxBDxrXRY6dl57Y1QBzaQyZWHzLaB4q8aYeHCuSK/2vt/fN8UpxKvT+sJlenpKDbC273A D8Eyyf4OpQ5A== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.73,370,1583222400"; d="scan'208";a="305692816" Received: from fmsmsx106.amr.corp.intel.com ([10.18.124.204]) by FMSMGA003.fm.intel.com with ESMTP; 08 May 2020 23:20:40 -0700 Received: from FMSMSX110.amr.corp.intel.com (10.18.116.10) by FMSMSX106.amr.corp.intel.com (10.18.124.204) with Microsoft SMTP Server (TLS) id 14.3.439.0; Fri, 8 May 2020 23:20:39 -0700 Received: from shsmsx107.ccr.corp.intel.com (10.239.4.96) by fmsmsx110.amr.corp.intel.com (10.18.116.10) with Microsoft SMTP Server (TLS) id 14.3.439.0; Fri, 8 May 2020 23:20:39 -0700 Received: from shsmsx103.ccr.corp.intel.com ([169.254.4.26]) by SHSMSX107.ccr.corp.intel.com ([169.254.9.200]) with mapi id 14.03.0439.000; Sat, 9 May 2020 14:20:37 +0800 From: "Zhang, Qi Z" To: "Guo, Jia" , "Xing, Beilei" , "Wu, Jingjing" CC: "Iremonger, Bernard" , "Ye, Xiaolong" , "dev@dpdk.org" Thread-Topic: [PATCH] net/iavf: fix input set for rss hash Thread-Index: AQHWJFIaBX4hY/SiMk+7QtKLUY/BNKifR+lQ Date: Sat, 9 May 2020 06:20:36 +0000 Message-ID: <039ED4275CED7440929022BC67E7061154812A32@SHSMSX103.ccr.corp.intel.com> References: <20200507212943.7858-1-jia.guo@intel.com> In-Reply-To: <20200507212943.7858-1-jia.guo@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-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] net/iavf: fix input set for rss hash 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" > -----Original Message----- > From: Guo, Jia > Sent: Friday, May 8, 2020 5:30 AM > To: Xing, Beilei ; Wu, Jingjing > Cc: Iremonger, Bernard ; Ye, Xiaolong > ; Zhang, Qi Z ; > dev@dpdk.org; Guo, Jia > Subject: [PATCH] net/iavf: fix input set for rss hash >=20 > Since some specific rss hash type need to combine with the protocol hash > type when configure a rss hash rule, so add the corresponding input set t= o > support these case for iavf hash. >=20 > Fixes: 7be10c3004be ("net/iavf: add RSS configuration for VF") > Signed-off-by: Jeff Guo > --- > drivers/net/iavf/iavf_hash.c | 96 > ++++++++++++++++++++++++++++++++++++ > 1 file changed, 96 insertions(+) >=20 > diff --git a/drivers/net/iavf/iavf_hash.c b/drivers/net/iavf/iavf_hash.c = index > a1d0b8364..97370aa19 100644 > --- a/drivers/net/iavf/iavf_hash.c > +++ b/drivers/net/iavf/iavf_hash.c > @@ -594,11 +594,19 @@ struct iavf_hash_match_type > iavf_hash_type_list[] =3D { > /* IPV4 */ > {IAVF_PATTERN_HINT_IPV4, ETH_RSS_L2_SRC_ONLY, > &hdrs_hint_eth_src}, > {IAVF_PATTERN_HINT_IPV4, ETH_RSS_L2_DST_ONLY, > &hdrs_hint_eth_dst}, > + {IAVF_PATTERN_HINT_IPV4, ETH_RSS_ETH | ETH_RSS_L2_SRC_ONLY, > + &hdrs_hint_eth_src}, > + {IAVF_PATTERN_HINT_IPV4, ETH_RSS_ETH | ETH_RSS_L2_DST_ONLY, > + &hdrs_hint_eth_dst}, > {IAVF_PATTERN_HINT_IPV4, ETH_RSS_ETH, &hdrs_hint_eth}, > {IAVF_PATTERN_HINT_IPV4, ETH_RSS_S_VLAN, &hdrs_hint_svlan}, > {IAVF_PATTERN_HINT_IPV4, ETH_RSS_C_VLAN, &hdrs_hint_cvlan}, > {IAVF_PATTERN_HINT_IPV4, ETH_RSS_L3_SRC_ONLY, > &hdrs_hint_ipv4_src}, > {IAVF_PATTERN_HINT_IPV4, ETH_RSS_L3_DST_ONLY, > &hdrs_hint_ipv4_dst}, > + {IAVF_PATTERN_HINT_IPV4, ETH_RSS_IPV4 | ETH_RSS_L3_SRC_ONLY, > + &hdrs_hint_ipv4_src}, > + {IAVF_PATTERN_HINT_IPV4, ETH_RSS_IPV4 | ETH_RSS_L3_DST_ONLY, > + &hdrs_hint_ipv4_dst}, > {IAVF_PATTERN_HINT_IPV4, ETH_RSS_ESP, &hdrs_hint_ipv4_esp}, > {IAVF_PATTERN_HINT_IPV4, ETH_RSS_AH, &hdrs_hint_ipv4_ah}, > {IAVF_PATTERN_HINT_IPV4, ETH_RSS_L2TPV3, > &hdrs_hint_ipv4_l2tpv3}, @@ -620,6 +628,23 @@ struct > iavf_hash_match_type iavf_hash_type_list[] =3D { > &hdrs_hint_ipv4_dst_gtpu_dwn}, > {IAVF_PATTERN_HINT_IPV4_UDP, ETH_RSS_L3_DST_ONLY, > &hdrs_hint_ipv4_dst}, > + {IAVF_PATTERN_HINT_IPV4_UDP, ETH_RSS_IPV4 | > ETH_RSS_L3_SRC_ONLY | > + ETH_RSS_L4_SRC_ONLY, &hdrs_hint_ipv4_src_udp_src_port}, > + {IAVF_PATTERN_HINT_IPV4_UDP, ETH_RSS_IPV4 | > ETH_RSS_L3_SRC_ONLY | > + ETH_RSS_L4_DST_ONLY, > &hdrs_hint_ipv4_src_udp_dst_port}, > + {IAVF_PATTERN_HINT_IPV4_UDP, ETH_RSS_IPV4 | > ETH_RSS_L3_SRC_ONLY | > + ETH_RSS_GTPU, &hdrs_hint_ipv4_src_gtpu_up}, > + {IAVF_PATTERN_HINT_IPV4_UDP, ETH_RSS_IPV4 | > ETH_RSS_L3_SRC_ONLY, > + &hdrs_hint_ipv4_src}, > + {IAVF_PATTERN_HINT_IPV4_UDP, ETH_RSS_IPV4 | > ETH_RSS_L3_DST_ONLY | > + ETH_RSS_L4_SRC_ONLY, I think the protocol type should be ETH_RSS_NONFRAG_IPV4_UDP if level 4 mod= ification is involved. And similar change also need be applied to TCP/SCTP...=20