From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga14.intel.com (mga14.intel.com [192.55.52.115]) by dpdk.org (Postfix) with ESMTP id E12072C6A for ; Mon, 28 Aug 2017 03:16:19 +0200 (CEST) Received: from fmsmga004.fm.intel.com ([10.253.24.48]) by fmsmga103.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 27 Aug 2017 18:16:18 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.41,439,1498546800"; d="scan'208";a="304991392" Received: from fmsmsx106.amr.corp.intel.com ([10.18.124.204]) by fmsmga004.fm.intel.com with ESMTP; 27 Aug 2017 18:16:18 -0700 Received: from shsmsx104.ccr.corp.intel.com (10.239.4.70) by FMSMSX106.amr.corp.intel.com (10.18.124.204) with Microsoft SMTP Server (TLS) id 14.3.319.2; Sun, 27 Aug 2017 18:16:18 -0700 Received: from shsmsx103.ccr.corp.intel.com ([169.254.4.219]) by SHSMSX104.ccr.corp.intel.com ([169.254.5.117]) with mapi id 14.03.0319.002; Mon, 28 Aug 2017 09:16:16 +0800 From: "Liu, Yong" To: "Lu, PeipeiX" , "dts@dpdk.org" CC: "Lu, PeipeiX" Thread-Topic: [dts] [PATCH V1] tests/generic_filter: fix testpmd not support 128 queues Thread-Index: AQHTHX0JaGvd7plQx0iNzuRIKBAUpaKY+VUg Date: Mon, 28 Aug 2017 01:16:16 +0000 Message-ID: <86228AFD5BCD8E4EBFD2B90117B5E81E62E77AFB@SHSMSX103.ccr.corp.intel.com> References: <1503650148-92405-1-git-send-email-peipeix.lu@intel.com> In-Reply-To: <1503650148-92405-1-git-send-email-peipeix.lu@intel.com> Accept-Language: zh-CN, en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: dlp-product: dlpe-windows dlp-version: 11.0.0.116 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: [dts] [PATCH V1] tests/generic_filter: fix testpmd not support 128 queues 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: , X-List-Received-Date: Mon, 28 Aug 2017 01:16:20 -0000 Peipei, Look like your patch only aimed for 8259x nics. If that please add judgemen= t that check self.kdriver is ixgbe first and then do these changes. It will waste some time if we are running test over other types of Nics. Thanks, Marvin > -----Original Message----- > From: dts [mailto:dts-bounces@dpdk.org] On Behalf Of lu,peipei > Sent: Friday, August 25, 2017 4:36 PM > To: dts@dpdk.org > Cc: Lu, PeipeiX > Subject: [dts] [PATCH V1] tests/generic_filter: fix testpmd not support > 128 queues >=20 > due to testpmd support 64 queues by default,so test the case need to > modify ixgbe_ethdev.h. >=20 > Signed-off-by: lu,peipei > --- > tests/TestSuite_generic_filter.py | 5 ++++- > 1 file changed, 4 insertions(+), 1 deletion(-) >=20 > diff --git a/tests/TestSuite_generic_filter.py > b/tests/TestSuite_generic_filter.py > index c42ec5a..a8eb2d9 100644 > --- a/tests/TestSuite_generic_filter.py > +++ b/tests/TestSuite_generic_filter.py > @@ -55,6 +55,8 @@ class TestGeneric_filter(TestCase): >=20 > Generic filter Prerequistites > """ > + self.dut.send_expect("sed -i -e 's/#define > IXGBE_NONE_MODE_TX_NB_QUEUES 64$/#define IXGBE_NONE_MODE_TX_NB_QUEUES > 128/' drivers/net/ixgbe/ixgbe_ethdev.h", "# ",30) > + self.dut.build_install_dpdk(self.target) >=20 > # Based on h/w type, choose how many ports to use > ports =3D self.dut.get_ports(self.nic) > @@ -690,7 +692,7 @@ class TestGeneric_filter(TestCase): >=20 > def test_128_queues(self): > # testpmd can't support assign queue to received package, so > can't test > - self.verify(False, "testpmd not support assign queue 127 receive= d > package") > + #self.verify(False, "testpmd not support assign queue 127 > received package") > if self.nic =3D=3D "niantic": > global valports > total_mbufs =3D self.request_mbufs(128) * len(valports) > @@ -825,3 +827,4 @@ class TestGeneric_filter(TestCase): > Run after each test case. > """ > self.dut.kill_all() > + self.dut.send_expect("sed -i -e 's/#define > IXGBE_NONE_MODE_TX_NB_QUEUES 128$/#define IXGBE_NONE_MODE_TX_NB_QUEUES > 64/' drivers/net/ixgbe/ixgbe_ethdev.h", "# ",30) > -- > 1.9.3