From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id A4FE1A0C4B; Mon, 12 Jul 2021 05:23:02 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 79DD040688; Mon, 12 Jul 2021 05:23:02 +0200 (CEST) Received: from mga02.intel.com (mga02.intel.com [134.134.136.20]) by mails.dpdk.org (Postfix) with ESMTP id 2145340685 for ; Mon, 12 Jul 2021 05:22:59 +0200 (CEST) X-IronPort-AV: E=McAfee;i="6200,9189,10042"; a="197101429" X-IronPort-AV: E=Sophos;i="5.84,232,1620716400"; d="scan'208";a="197101429" Received: from fmsmga008.fm.intel.com ([10.253.24.58]) by orsmga101.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 11 Jul 2021 20:22:57 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.84,232,1620716400"; d="scan'208";a="464120779" Received: from fmsmsx602.amr.corp.intel.com ([10.18.126.82]) by fmsmga008.fm.intel.com with ESMTP; 11 Jul 2021 20:22:56 -0700 Received: from shsmsx606.ccr.corp.intel.com (10.109.6.216) by fmsmsx602.amr.corp.intel.com (10.18.126.82) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2242.10; Sun, 11 Jul 2021 20:22:56 -0700 Received: from shsmsx601.ccr.corp.intel.com (10.109.6.141) by SHSMSX606.ccr.corp.intel.com (10.109.6.216) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2242.10; Mon, 12 Jul 2021 11:22:54 +0800 Received: from shsmsx601.ccr.corp.intel.com ([10.109.6.141]) by SHSMSX601.ccr.corp.intel.com ([10.109.6.141]) with mapi id 15.01.2242.010; Mon, 12 Jul 2021 11:22:54 +0800 From: "Tu, Lijuan" To: "Sun, QinX" , "dts@dpdk.org" , "Fu, Qi" CC: "Sun, QinX" Thread-Topic: [dts] [PATCH V1] tests/cvl_dcf_flow_priority: modify PPP proto field to byte type Thread-Index: AQHXcimbtXsMU6u4b0yosrMD9/L+B6s+tTQg Date: Mon, 12 Jul 2021 03:22:54 +0000 Message-ID: <543fa3b64eda4733ab1db79278f8b374@intel.com> References: <20210706053843.6254-1-qinx.sun@intel.com> In-Reply-To: <20210706053843.6254-1-qinx.sun@intel.com> Accept-Language: zh-CN, en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: dlp-reaction: no-action dlp-version: 11.5.1.3 dlp-product: dlpe-windows x-originating-ip: [10.239.127.36] Content-Type: text/plain; charset="iso-2022-jp" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Subject: Re: [dts] [PATCH V1] tests/cvl_dcf_flow_priority: modify PPP proto field to byte type X-BeenThere: dts@dpdk.org X-Mailman-Version: 2.1.29 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" > -----Original Message----- > From: dts On Behalf Of Qin Sun > Sent: 2021=1B$BG/=1B(B7=1B$B7n=1B(B6=1B$BF|=1B(B 13:39 > To: dts@dpdk.org > Cc: Sun, QinX > Subject: [dts] [PATCH V1] tests/cvl_dcf_flow_priority: modify PPP proto f= ield to > byte type >=20 > modify PPP proto field to byte type to adapt scapy2.4.4, replace 'insmod = xxx' > with 'modprobe ice' to reduce configure file >=20 > Signed-off-by: Qin Sun > --- > tests/TestSuite_cvl_dcf_flow_priority.py | 7 ++----- > 1 file changed, 2 insertions(+), 5 deletions(-) mode change 100644 =3D>= 100755 > tests/TestSuite_cvl_dcf_flow_priority.py >=20 > diff --git a/tests/TestSuite_cvl_dcf_flow_priority.py > b/tests/TestSuite_cvl_dcf_flow_priority.py > old mode 100644 > new mode 100755 > index 532d993..1e9c1f5 > --- a/tests/TestSuite_cvl_dcf_flow_priority.py > +++ b/tests/TestSuite_cvl_dcf_flow_priority.py > @@ -1155,11 +1155,8 @@ class CVLDCFFlowPriorityTest(TestCase): > """ > Run before each test case. > """ > - self.suite_config =3D rfc.get_suite_config(self) > - ice_driver_file_location =3D self.suite_config["ice_driver_file_= location"] > self.dut.send_expect("rmmod ice","#",30) > - self.dut.send_expect("insmod %s" % ice_driver_file_location, "# = ", 60) > - time.sleep(10) > + self.dut.send_expect("modprobe ice","#",30) @Qi: why reload ice drive in set_up? I think there must be some limitation = here. @Qin, I think ice have been load before , so it is redundant . >=20 > def create_testpmd_command(self): > """ > @@ -1733,7 +1730,7 @@ class CVLDCFFlowPriorityTest(TestCase): > self.dut.send_expect("flow destroy 0 rule 1", "testpmd> ", 15) > matched_dic["expect_results"]["expect_pkts"]=3D0 > self.send_and_check_packets(matched_dic) > - matched_dic =3D > {"scapy_str":['Ether(dst=3D"00:11:22:33:44:55",type=3D0x8100)/Dot1Q(vlan= =3D1,type=3D > 0x8100)/Dot1Q(vlan=3D0x2,type=3D0x8864)/PPPoE(sessionid=3D0x1)/PPP(proto= =3D0x21)/ > IP()/UDP(dport=3D23)/("X"*480)'], > + matched_dic =3D > + {"scapy_str":['Ether(dst=3D"00:11:22:33:44:55",type=3D0x8100)/Dot1Q(vla= n=3D1 > + ,type=3D0x8100)/Dot1Q(vlan=3D0x2,type=3D0x8864)/PPPoE(sessionid=3D1)/PP= P(b\'\\ > + x00\\x21\')/IP()/UDP(dport=3D23)/("X"*480)'], > "check_func":{"func":rfc.check_vf_rx_packets_numb= er, > "param":{"expect_port":2, "expect_q= ueue":"null"}}, > "expect_results":{"expect_pkts":1}} > -- > 2.25.1