From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) by dpdk.org (Postfix) with ESMTP id D1BF85B15 for ; Tue, 27 Jan 2015 06:37:31 +0100 (CET) Received: from fmsmga003.fm.intel.com ([10.253.24.29]) by fmsmga101.fm.intel.com with ESMTP; 26 Jan 2015 21:37:16 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="4.97,862,1389772800"; d="scan'208";a="445595157" Received: from pgsmsx105.gar.corp.intel.com ([10.221.44.96]) by FMSMGA003.fm.intel.com with ESMTP; 26 Jan 2015 21:23:36 -0800 Received: from shsmsx103.ccr.corp.intel.com (10.239.4.69) by PGSMSX105.gar.corp.intel.com (10.221.44.96) with Microsoft SMTP Server (TLS) id 14.3.195.1; Tue, 27 Jan 2015 13:37:14 +0800 Received: from shsmsx101.ccr.corp.intel.com ([169.254.1.64]) by SHSMSX103.ccr.corp.intel.com ([169.254.4.192]) with mapi id 14.03.0195.001; Tue, 27 Jan 2015 13:37:11 +0800 From: "Xu, HuilongX" To: "Liu, Yong" , "dts@dpdk.org" Thread-Topic: [dts] [PATCH] update framework for support fortville NIC test. Thread-Index: AQHQOeKnieInWRU1L0C4bG+ekmdd25zS4eGAgACJEEA= Date: Tue, 27 Jan 2015 05:37:12 +0000 Message-ID: References: <1422329871-13274-1-git-send-email-huilongx.xu@intel.com> <86228AFD5BCD8E4EBFD2B90117B5E81E10D54317@SHSMSX103.ccr.corp.intel.com> In-Reply-To: <86228AFD5BCD8E4EBFD2B90117B5E81E10D54317@SHSMSX103.ccr.corp.intel.com> Accept-Language: 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] update framework for support fortville NIC test. 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: Tue, 27 Jan 2015 05:37:32 -0000 Hi yong, The first comments, add all FVL suites in the config file: The other suites need update code for support FVL test, I will send a= new patch for update code. The second comments, check the test NIC is FVL. I will update the code in v2 patch. -----Original Message----- From: Liu, Yong=20 Sent: Tuesday, January 27, 2015 12:58 PM To: Xu, HuilongX; dts@dpdk.org Subject: RE: [dts] [PATCH] update framework for support fortville NIC test. Hi huilong, Please see my comments in the mail. > -----Original Message----- > From: dts [mailto:dts-bounces@dpdk.org] On Behalf Of huilongx.xu > Sent: Tuesday, January 27, 2015 11:38 AM > To: dts@dpdk.org > Subject: [dts] [PATCH] update framework for support fortville NIC test. >=20 > execution_fortville.cfg is the test case list for fortville NIC >=20 > crb.py, add fortville NIC kernel driver(i40e) in dts >=20 > settings.py, add fortville NIC info in dts >=20 > testt_case.py, insmode and used i40e for fortvill NIC >=20 > Signed-off-by: huilongx.xu > --- > execution_fortville.cfg | 21 +++++++++++++++++++++ > framework/crb.py | 7 +++++++ > framework/settings.py | 6 ++++++ > framework/test_case.py | 4 ++++ > 4 files changed, 38 insertions(+), 0 deletions(-) > create mode 100644 execution_fortville.cfg >=20 > diff --git a/execution_fortville.cfg b/execution_fortville.cfg > new file mode 100644 > index 0000000..15f2ccf > --- /dev/null > +++ b/execution_fortville.cfg > @@ -0,0 +1,21 @@ > +[Execution1] > +crbs=3D > +drivername=3D > +test_suites=3D > + cmdline, > + hello_world, > + multiprocess, > + blacklist > +targets=3D > + x86_64-native-linuxapp-gcc > +parameters=3Dnic_type=3Dfortville_eagle:func=3Dtrue > + This configuration file seems not enough for FVL. Please add those suites which already supported FVL.=20 > +[Execution2] > +crbs=3D > +drivername=3D > +test_suites=3D > + l2fwd, > + l3fwd > +targets=3D > + x86_64-native-linuxapp-gcc > +parameters=3Dnic_type=3Dfortville_eagle:perf=3Dtrue > diff --git a/framework/crb.py b/framework/crb.py > index d41f51b..e2e8d92 100644 > --- a/framework/crb.py > +++ b/framework/crb.py > @@ -151,6 +151,10 @@ class Crb(object): > self.send_expect("modprobe e1000", "# ", 20) > self.send_expect("modprobe virtio_net", "# ", 20) >=20 > + self.send_expect("insmod > /usr/lib/modules/%s/kernel/drivers/net/ethernet/intel/i40e/i40e.ko"%'`una= m > e -r`', "# ", 60) > + out =3D self.send_expect("lsmod |grep i40e", "# ", 30) > + if "i40e" not in out: > + self.logger.error("please check the os install i40e driver > already."); We cannot sure everyone has Fortville validation environment. It's better t= o check when NIC is Fortville. Or you can wait for a while, we will implement new function in replace of t= his function. > try: > for (pci_bus, pci_id) in self.pci_devices_info: > """ > @@ -173,6 +177,9 @@ class Crb(object): > self.send_expect("echo -n 0000:%s > > /sys/bus/pci/drivers/e1000/bind" % pci_bus, "# ") > elif pci_id in ('1af4:1000'): > self.send_expect("echo 0000%s > > /sys/bus/pci/drivers/virtio-pci/bind" % pci_bus, "# ") > + elif pci_id in ('8086:1583','8086:1584','8086:1572'): > + if not > os.path.exists("/sys/bus/pci/drivers/i40e/"+"0000:"+pci_bus): > + self.send_expect("echo 0000:%s > > /sys/bus/pci/drivers/i40e/bind" % pci_bus, "# ") > else: > continue >=20 > diff --git a/framework/settings.py b/framework/settings.py > index 2ef8db8..40b81fb 100644 > --- a/framework/settings.py > +++ b/framework/settings.py > @@ -55,6 +55,9 @@ NICS =3D { > 'I217LM': '8086:153a', > 'I218V': '8086:1559', > 'I218LM': '8086:155a', > + 'fortville_eagle': '8086:1572', > + 'fortville_spirit': '8086:1583', > + 'fortville_spirit_single': '8086:1584', > } >=20 > DRIVERS =3D { > @@ -80,6 +83,9 @@ DRIVERS =3D { > 'I217LM': 'igb', > 'I218V': 'igb', > 'I218LM': 'igb', > + 'fortville_eagle': 'i40e', > + 'fortville_spirit': 'i40e', > + 'fortville_spirit_single':'i40e' > } >=20 > """ > diff --git a/framework/test_case.py b/framework/test_case.py > index 706003f..07fdc36 100644 > --- a/framework/test_case.py > +++ b/framework/test_case.py > @@ -76,5 +76,9 @@ class TestCase(object): > bitrate *=3D 10 # 10 Gb NICs > elif self.nic =3D=3D "avoton2c5": > bitrate *=3D 2.5 # 2.5 Gb NICs > + elif self.nic in ["fortville_spirit", "fortville_spirit_single"]= : > + bitrate *=3D 40 > + elif self.nic =3D=3D 'fortville_eagle': > + bitrate *=3D 10 >=20 > return bitrate * num_ports / 8 / (frame_size + 20) > -- > 1.7.4.4