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 ADEDAA046B for ; Tue, 20 Aug 2019 05:07:21 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 681AC1BE9C; Tue, 20 Aug 2019 05:07:21 +0200 (CEST) Received: from mga14.intel.com (mga14.intel.com [192.55.52.115]) by dpdk.org (Postfix) with ESMTP id 66E271BE95 for ; Tue, 20 Aug 2019 05:07:19 +0200 (CEST) X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga004.fm.intel.com ([10.253.24.48]) by fmsmga103.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 19 Aug 2019 20:07:18 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.64,407,1559545200"; d="scan'208";a="202533045" Received: from fmsmsx104.amr.corp.intel.com ([10.18.124.202]) by fmsmga004.fm.intel.com with ESMTP; 19 Aug 2019 20:07:18 -0700 Received: from shsmsx103.ccr.corp.intel.com (10.239.4.69) by fmsmsx104.amr.corp.intel.com (10.18.124.202) with Microsoft SMTP Server (TLS) id 14.3.439.0; Mon, 19 Aug 2019 20:07:18 -0700 Received: from shsmsx104.ccr.corp.intel.com ([169.254.5.112]) by SHSMSX103.ccr.corp.intel.com ([169.254.4.139]) with mapi id 14.03.0439.000; Tue, 20 Aug 2019 11:07:16 +0800 From: "Lin, Xueqin" To: "He, Zhiwei" , "dts@dpdk.org" CC: "He, Zhiwei" Thread-Topic: [dts] [PATCH] /tests/TestSuite_nvgre : add ipv6 detect and ipv6checksum for CVL25gand100g nic Add ipv6 detect case and ipv6 checksum offload case for the CVL 25g and 100g nic test Add CVL25G and 100g nic support default none vector Thread-Index: AQHVVvvTI1hqAR6k2UK3vJKRqJL7Y6cDVwTQ Date: Tue, 20 Aug 2019 03:07:16 +0000 Message-ID: <0D300480287911409D9FF92C1FA2A3355B555415@SHSMSX104.ccr.corp.intel.com> References: <1566266743-107013-1-git-send-email-zhiwei.he@intel.com> In-Reply-To: <1566266743-107013-1-git-send-email-zhiwei.he@intel.com> Accept-Language: zh-CN, en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: 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: [dts] [PATCH] /tests/TestSuite_nvgre : add ipv6 detect and ipv6checksum for CVL25gand100g nic Add ipv6 detect case and ipv6 checksum offload case for the CVL 25g and 100g nic test Add CVL25G and 100g nic support default none vector X-BeenThere: dts@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: test suite reviews and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dts-bounces@dpdk.org Sender: "dts" Patch title is too long, you could put some explanation in the comment.=20 Some comment as below.=20 Best regards, Xueqin > -----Original Message----- > From: dts [mailto:dts-bounces@dpdk.org] On Behalf Of zhiwei.he > Sent: Tuesday, August 20, 2019 10:06 AM > To: dts@dpdk.org > Cc: He, Zhiwei > Subject: [dts] [PATCH] /tests/TestSuite_nvgre : add ipv6 detect and > ipv6checksum for CVL25gand100g nic Add ipv6 detect case and ipv6 > checksum offload case for the CVL 25g and 100g nic test Add CVL25G and > 100g nic support default none vector >=20 > From: "zhiwei.he" >=20 > Signed-off-by: zhiwei.he > --- > tests/TestSuite_nvgre.py | 57 > +++++++++++++++++++++++++++++++++++++++++++----- > 1 file changed, 52 insertions(+), 5 deletions(-) >=20 > diff --git a/tests/TestSuite_nvgre.py b/tests/TestSuite_nvgre.py index > 62aac3a..2fd304a 100644 > --- a/tests/TestSuite_nvgre.py > +++ b/tests/TestSuite_nvgre.py > @@ -93,8 +93,8 @@ class NvgreTestConfig(object): > self.outer_ip_dst =3D '192.168.1.2' > self.outer_ip_invalid =3D 0 >=20 > - self.outer_ip6_src =3D 'N/A' > - self.outer_ip6_dst =3D 'N/A' > + self.outer_ip6_src =3D 'FE80:0:0:0:0:0:0:0' > + self.outer_ip6_dst =3D 'FE80:0:0:0:0:0:0:1' > self.outer_ip6_invalid =3D 0 > """ > gre info > @@ -150,8 +150,8 @@ class NvgreTestConfig(object): > self.inner_ip_dst =3D '192.168.2.2' > self.inner_ip_invalid =3D 0 >=20 > - self.inner_ip6_src =3D 'N/A' > - self.inner_ip6_dst =3D 'N/A' > + self.inner_ip6_src =3D 'FE80:0:0:0:0:0:0:0' > + self.inner_ip6_dst =3D 'FE80:0:0:0:0:0:0:1' > self.inner_ip6_invalid =3D 0 >=20 > self.inner_l4_type =3D 'UDP' > @@ -361,10 +361,12 @@ class TestNvgre(TestCase): > nvgre Prerequisites > """ > # this feature only enable in FVL now > - if self.nic in ["fortville_eagle", "fortville_spirit", "fortvill= e_spirit_single", > "fortville_25g", "fortpark_TLV", "cavium_a063", "cavium_a064"]: > + if self.nic in [ "fortville_eagle", "fortville_spirit", "fortvil= le_spirit_single", > "fortville_25g", "fortpark_TLV", "cavium_a063", "cavium_a064"]: > self.compile_switch =3D 'CONFIG_RTE_LIBRTE_I40E_INC_VECTOR' > elif self.nic in ["sageville", "sagepond"]: > self.compile_switch =3D 'CONFIG_RTE_IXGBE_INC_VECTOR' > + elif self.nic in ["columbiaville_25g","columbiaville_100g"]: > + print "CVL support default none VECTOR" > else: > self.verify(False, "%s not support NVGRE case" % self.nic) > # Based on h/w type, choose how many ports to use @@ -616,7 +618= ,52 > @@ class TestNvgre(TestCase): > # verify saved pcap checksum same to expected checksum > for key in chksums_default: > self.verify(chksums[key] =3D=3D chksums_default[key], "%s no= t matched > to %s" % (key, chksums_default[key])) > + def test_nvgre_ipv6(self): > + """ > + verify nvgre packet with ipv6 > + """ > + outer_l3_type =3D "IPv6" > + inner_l3_type =3D "IPv6" > + outer_ip6_src =3D 'FE80:0:0:0:0:0:0:0' > + outer_ip6_dst =3D 'FE80:0:0:0:0:0:0:1' > + inner_ip6_src =3D 'FE80:0:0:0:0:0:0:0' > + inner_ip6_dst =3D 'FE80:0:0:0:0:0:0:1' >=20 [xueqin] have defined self.inner_ip6_src, self.inner_ipv6_dst, self.outer_i= pv6_src, self.outer_ipv6_dst above, why repeat to add ipv6 src, dst... simi= lar code here?=20 Are you sure code could be called? Pls check create_pcap() code. > + # check no nvgre packet > + self.nvgre_detect(outer_l3_type =3D "IPv6", inner_l3_type=3D"IPv= 6", > outer_ip_proto=3D0xFF) [xueqin] if run no nvgre packet, should only set outer_ip_proto=3D0xFF. outer_l3_type =3D "IPv6", inner_l3_type=3D"IPv6 are for nvgre packet. > + # check nvgre + IP inner packet [xueqin] below is ipv6 inner packet. > + self.nvgre_detect(outer_l3_type =3D "IPv6", inner_l3_type=3D"IPv= 6", > inner_l4_type=3D'None') > + # check nvgre + udp inner packet [xueqin] below is tcp inner packet > + self.nvgre_detect(outer_l3_type =3D "IPv6", inner_l3_type=3D"IPv= 6", > inner_l4_type=3D'TCP') > + # check nvgre + SCTP inner packet > + self.nvgre_detect(outer_l3_type =3D "IPv6", inner_l3_type=3D"IPv= 6", > inner_l4_type=3D'SCTP') > + # check nvgre + UDP inner packet > + self.nvgre_detect(outer_l3_type =3D "IPv6", inner_l3_type=3D"IPv= 6", > inner_l4_type=3D'UDP') > + # check nvgre + vlan inner packet > + self.nvgre_detect(outer_l3_type =3D "IPv6", inner_l3_type=3D"IPv= 6", > outer_vlan=3D1) > + # check vlan nvgre + vlan inner packet > + self.nvgre_detect(outer_l3_type =3D "IPv6", inner_l3_type=3D"IPv= 6", > + outer_vlan=3D1, inner_vlan=3D1) > + > + def test_nvgre_ipv6_checksum_offload(self): > + outer_l3_type =3D "IPv6" > + inner_l3_type =3D "IPv6" > + outer_ip6_src =3D 'FE80:0:0:0:0:0:0:0' > + outer_ip6_dst =3D 'FE80:0:0:0:0:0:0:1' > + inner_ip6_src =3D 'FE80:0:0:0:0:0:0:0' > + inner_ip6_dst =3D 'FE80:0:0:0:0:0:0:1' [xueqin] same as above. > + # check nvgre packet + inner ip checksum invalid [xueqin] inner ipv6.=20 > + self.nvgre_checksum(outer_l3_type =3D "IPv6", inner_l3_type=3D"I= Pv6", > inner_l4_invalid=3D1) > + # check nvgre packet + inner tcp checksum invalid > + self.nvgre_checksum(outer_l3_type =3D "IPv6", inner_l3_type=3D"I= Pv6", > inner_l4_invalid=3D1, inner_l4_type=3D'TCP') > + #check nvgre packet + inner sctp checksum invalid > + self.nvgre_checksum(outer_l3_type =3D "IPv6", inner_l3_type=3D"I= Pv6", > inner_l4_invalid=3D1, inner_l4_type=3D'SCTP') > + # check vlan nvgre packet + inner vlan + inner udp checksum inva= lid > + self.nvgre_checksum(outer_l3_type =3D "IPv6", inner_l3_type=3D"I= Pv6", > outer_vlan=3D1, inner_l4_invalid=3D1, inner_l4_type=3D'UDP') > + # check vlan nvgre packet + inner vlan + inner tcp checksum inva= lid > + self.nvgre_checksum(outer_l3_type =3D "IPv6", inner_l3_type=3D"I= Pv6", > outer_vlan=3D1, inner_l4_invalid=3D1, inner_l4_type=3D'TCP') > + # check vlan nvgre packet + inner vlan + inner sctp checksum inv= alid > + self.nvgre_checksum(outer_l3_type =3D "IPv6", > + inner_l3_type=3D"IPv6", outer_vlan=3D1, inner_l4_invalid=3D1, > + inner_l4_type=3D'SCTP') > + > def test_nvgre_ipv4(self): > """ > verify nvgre packet with ipv4 > -- > 1.8.3.1