* Re: [dts] [PATCH V1] tests/package_download:add new case
2019-12-27 12:04 [dts] [PATCH V1] tests/package_download:add new case Wenjie Li
@ 2019-12-27 3:23 ` Li, WenjieX A
2020-01-02 7:13 ` Chen, Zhaoyan
2020-01-03 2:56 ` Tu, Lijuan
1 sibling, 1 reply; 4+ messages in thread
From: Li, WenjieX A @ 2019-12-27 3:23 UTC (permalink / raw)
To: dts; +Cc: Li, WenjieX A
[-- Attachment #1: Type: text/plain, Size: 4204 bytes --]
Tested-by: Li, WenjieX A <wenjiex.a.li@intel.com>
> -----Original Message-----
> From: Li, WenjieX A
> Sent: Friday, December 27, 2019 8:04 PM
> To: dts@dpdk.org
> Cc: Li, WenjieX A <wenjiex.a.li@intel.com>
> Subject: [dts][PATCH V1] tests/package_download:add new case
>
> add new case check_specific_package_loading
>
> Signed-off-by: Wenjie Li <wenjiex.a.li@intel.com>
> ---
> ...e_enable_package_download_in_ice_driver.py | 57 +++++++++++++++++++
> 1 file changed, 57 insertions(+)
>
> diff --git a/tests/TestSuite_enable_package_download_in_ice_driver.py
> b/tests/TestSuite_enable_package_download_in_ice_driver.py
> index fd6a27e..f28f0e6 100644
> --- a/tests/TestSuite_enable_package_download_in_ice_driver.py
> +++ b/tests/TestSuite_enable_package_download_in_ice_driver.py
> @@ -295,6 +295,63 @@ class
> TestEnable_Package_Download_In_Ice_Driver(TestCase):
> for error_message in error_messages:
> self.verify(error_message in out, "There should be '%s' in out: %s" %
> (error_message, out))
>
> + def get_sn(self, nic_pci):
> + cmd = "lspci -vs %s | grep 'Device Serial Number'" % nic_pci
> + out = self.dut.send_expect(cmd, "#")
> + sn_temp = re.findall(r'Device Serial Number (.*)', out)
> + sn = re.sub("-", "", sn_temp[0])
> + return sn
> +
> + def check_env(self):
> + """
> + Check the DUT has two or more CVL NICs. If not, return "the ENV needs at
> least two CVL NICs"
> + """
> + self.nic_pci = [self.dut.ports_info[0]['pci'], self.dut.ports_info[-1]['pci']]
> + self.nic_sn = [self.get_sn(self.nic_pci[0]), self.get_sn(self.nic_pci[1])]
> + self.verify(self.nic_sn[0] != self.nic_sn[1], "the case needs >=2 CVL NICs
> with different Serial Numbers")
> +
> + def copy_specify_ice_pkg(self, pkg_ver):
> + """
> + Copy 2 different ``ice-xxx.pkg`` from dts/dep to dut /tmp/
> + pkg_files = ['ice-1.3.4.0.pkg', 'ice-1.3.10.0.pkg']
> + """
> + dst = "/tmp"
> + pkg_file = "ice-%s.pkg" % pkg_ver
> + src_file = r'./dep/%s' % pkg_file
> + self.dut.session.copy_file_to(src_file, dst)
> +
> + def generate_delete_specify_pkg(self, pkg_ver, sn, key="true"):
> + self.dut.send_expect("rm -rf /lib/firmware/intel/ice/ddp/ice-%s.pkg" % sn,
> "#")
> + if key == "true":
> + self.dut.send_expect("\cp /tmp/ice-%s.pkg
> /lib/firmware/intel/ice/ddp/ice-%s.pkg" % (pkg_ver, sn), "#")
> +
> + def test_check_specific_package_loading(self):
> + """
> + Copy 2 different ``ice.pkg`` into ``/lib/firmware/intel/ice/ddp/``,
> + and rename 1 ice.pkg to ice-<interface serial number>.pkg, e.g. ice-
> 8ce1abfffffefd3c.pkg
> + Launch testpmd with 1 default interface and 1 specific interface,
> + check the ice-<interface serial number>.pkg is loadded by the specific
> interface.
> + """
> + self.check_env()
> + self.use_correct_ice_pkg(flag="true")
> + self.new_pkgs = ["1.3.10.0", "1.3.4.0"]
> + for i in range(len(self.new_pkgs)):
> + self.copy_specify_ice_pkg(self.new_pkgs[i])
> + self.generate_delete_specify_pkg(pkg_ver=self.new_pkgs[i],
> sn=self.nic_sn[i], key="true")
> +
> + eal_param = "-w %s " % self.nic_pci[0] + "-w %s " % self.nic_pci[1] + "--log-
> level=8"
> + out = self.dut_testpmd.execute_cmd("./%s/app/testpmd %s -- -i " %
> (self.target, eal_param))
> + self.dut_testpmd.quit()
> +
> + # Delete ice-<interface serial number>.pkg to recover the ENV
> + for i in range(len(self.new_pkgs)):
> + self.generate_delete_specify_pkg(pkg_ver=self.new_pkgs[i],
> sn=self.nic_sn[i], key="false")
> +
> + self.actual_pcis = re.findall(r"device (.*) on", out)
> + self.pkgs = re.findall(r"is: (.*),", out)
> + self.verify(self.actual_pcis == self.nic_pci and self.pkgs == self.new_pkgs,
> + "The nics pci info/pkg are not matched")
> +
> def tear_down(self):
> self.dut_testpmd.quit()
>
> --
> 2.17.1
[-- Attachment #2: TestEnable_Package_Download_In_Ice_Driver.log --]
[-- Type: application/octet-stream, Size: 219922 bytes --]
^ permalink raw reply [flat|nested] 4+ messages in thread
* [dts] [PATCH V1] tests/package_download:add new case
@ 2019-12-27 12:04 Wenjie Li
2019-12-27 3:23 ` Li, WenjieX A
2020-01-03 2:56 ` Tu, Lijuan
0 siblings, 2 replies; 4+ messages in thread
From: Wenjie Li @ 2019-12-27 12:04 UTC (permalink / raw)
To: dts; +Cc: Wenjie Li
add new case check_specific_package_loading
Signed-off-by: Wenjie Li <wenjiex.a.li@intel.com>
---
...e_enable_package_download_in_ice_driver.py | 57 +++++++++++++++++++
1 file changed, 57 insertions(+)
diff --git a/tests/TestSuite_enable_package_download_in_ice_driver.py b/tests/TestSuite_enable_package_download_in_ice_driver.py
index fd6a27e..f28f0e6 100644
--- a/tests/TestSuite_enable_package_download_in_ice_driver.py
+++ b/tests/TestSuite_enable_package_download_in_ice_driver.py
@@ -295,6 +295,63 @@ class TestEnable_Package_Download_In_Ice_Driver(TestCase):
for error_message in error_messages:
self.verify(error_message in out, "There should be '%s' in out: %s" % (error_message, out))
+ def get_sn(self, nic_pci):
+ cmd = "lspci -vs %s | grep 'Device Serial Number'" % nic_pci
+ out = self.dut.send_expect(cmd, "#")
+ sn_temp = re.findall(r'Device Serial Number (.*)', out)
+ sn = re.sub("-", "", sn_temp[0])
+ return sn
+
+ def check_env(self):
+ """
+ Check the DUT has two or more CVL NICs. If not, return "the ENV needs at least two CVL NICs"
+ """
+ self.nic_pci = [self.dut.ports_info[0]['pci'], self.dut.ports_info[-1]['pci']]
+ self.nic_sn = [self.get_sn(self.nic_pci[0]), self.get_sn(self.nic_pci[1])]
+ self.verify(self.nic_sn[0] != self.nic_sn[1], "the case needs >=2 CVL NICs with different Serial Numbers")
+
+ def copy_specify_ice_pkg(self, pkg_ver):
+ """
+ Copy 2 different ``ice-xxx.pkg`` from dts/dep to dut /tmp/
+ pkg_files = ['ice-1.3.4.0.pkg', 'ice-1.3.10.0.pkg']
+ """
+ dst = "/tmp"
+ pkg_file = "ice-%s.pkg" % pkg_ver
+ src_file = r'./dep/%s' % pkg_file
+ self.dut.session.copy_file_to(src_file, dst)
+
+ def generate_delete_specify_pkg(self, pkg_ver, sn, key="true"):
+ self.dut.send_expect("rm -rf /lib/firmware/intel/ice/ddp/ice-%s.pkg" % sn, "#")
+ if key == "true":
+ self.dut.send_expect("\cp /tmp/ice-%s.pkg /lib/firmware/intel/ice/ddp/ice-%s.pkg" % (pkg_ver, sn), "#")
+
+ def test_check_specific_package_loading(self):
+ """
+ Copy 2 different ``ice.pkg`` into ``/lib/firmware/intel/ice/ddp/``,
+ and rename 1 ice.pkg to ice-<interface serial number>.pkg, e.g. ice-8ce1abfffffefd3c.pkg
+ Launch testpmd with 1 default interface and 1 specific interface,
+ check the ice-<interface serial number>.pkg is loadded by the specific interface.
+ """
+ self.check_env()
+ self.use_correct_ice_pkg(flag="true")
+ self.new_pkgs = ["1.3.10.0", "1.3.4.0"]
+ for i in range(len(self.new_pkgs)):
+ self.copy_specify_ice_pkg(self.new_pkgs[i])
+ self.generate_delete_specify_pkg(pkg_ver=self.new_pkgs[i], sn=self.nic_sn[i], key="true")
+
+ eal_param = "-w %s " % self.nic_pci[0] + "-w %s " % self.nic_pci[1] + "--log-level=8"
+ out = self.dut_testpmd.execute_cmd("./%s/app/testpmd %s -- -i " % (self.target, eal_param))
+ self.dut_testpmd.quit()
+
+ # Delete ice-<interface serial number>.pkg to recover the ENV
+ for i in range(len(self.new_pkgs)):
+ self.generate_delete_specify_pkg(pkg_ver=self.new_pkgs[i], sn=self.nic_sn[i], key="false")
+
+ self.actual_pcis = re.findall(r"device (.*) on", out)
+ self.pkgs = re.findall(r"is: (.*),", out)
+ self.verify(self.actual_pcis == self.nic_pci and self.pkgs == self.new_pkgs,
+ "The nics pci info/pkg are not matched")
+
def tear_down(self):
self.dut_testpmd.quit()
--
2.17.1
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [dts] [PATCH V1] tests/package_download:add new case
2019-12-27 3:23 ` Li, WenjieX A
@ 2020-01-02 7:13 ` Chen, Zhaoyan
0 siblings, 0 replies; 4+ messages in thread
From: Chen, Zhaoyan @ 2020-01-02 7:13 UTC (permalink / raw)
To: Li, WenjieX A, dts; +Cc: Li, WenjieX A, Chen, Zhaoyan
Acked-by: Zhaoyan Chen <zhaoyan.chen@intel.com>
Regards,
Zhaoyan Chen
> -----Original Message-----
> From: dts <dts-bounces@dpdk.org> On Behalf Of Li, WenjieX A
> Sent: Friday, December 27, 2019 11:24 AM
> To: dts@dpdk.org
> Cc: Li, WenjieX A <wenjiex.a.li@intel.com>
> Subject: Re: [dts] [PATCH V1] tests/package_download:add new case
>
> Tested-by: Li, WenjieX A <wenjiex.a.li@intel.com>
>
> > -----Original Message-----
> > From: Li, WenjieX A
> > Sent: Friday, December 27, 2019 8:04 PM
> > To: dts@dpdk.org
> > Cc: Li, WenjieX A <wenjiex.a.li@intel.com>
> > Subject: [dts][PATCH V1] tests/package_download:add new case
> >
> > add new case check_specific_package_loading
> >
> > Signed-off-by: Wenjie Li <wenjiex.a.li@intel.com>
> > ---
> > ...e_enable_package_download_in_ice_driver.py | 57
> > +++++++++++++++++++
> > 1 file changed, 57 insertions(+)
> >
> > diff --git a/tests/TestSuite_enable_package_download_in_ice_driver.py
> > b/tests/TestSuite_enable_package_download_in_ice_driver.py
> > index fd6a27e..f28f0e6 100644
> > --- a/tests/TestSuite_enable_package_download_in_ice_driver.py
> > +++ b/tests/TestSuite_enable_package_download_in_ice_driver.py
> > @@ -295,6 +295,63 @@ class
> > TestEnable_Package_Download_In_Ice_Driver(TestCase):
> > for error_message in error_messages:
> > self.verify(error_message in out, "There should be '%s'
> > in out: %s" % (error_message, out))
> >
> > + def get_sn(self, nic_pci):
> > + cmd = "lspci -vs %s | grep 'Device Serial Number'" % nic_pci
> > + out = self.dut.send_expect(cmd, "#")
> > + sn_temp = re.findall(r'Device Serial Number (.*)', out)
> > + sn = re.sub("-", "", sn_temp[0])
> > + return sn
> > +
> > + def check_env(self):
> > + """
> > + Check the DUT has two or more CVL NICs. If not, return "the
> > + ENV needs at
> > least two CVL NICs"
> > + """
> > + self.nic_pci = [self.dut.ports_info[0]['pci'], self.dut.ports_info[-1]['pci']]
> > + self.nic_sn = [self.get_sn(self.nic_pci[0]), self.get_sn(self.nic_pci[1])]
> > + self.verify(self.nic_sn[0] != self.nic_sn[1], "the case needs
> > + >=2 CVL NICs
> > with different Serial Numbers")
> > +
> > + def copy_specify_ice_pkg(self, pkg_ver):
> > + """
> > + Copy 2 different ``ice-xxx.pkg`` from dts/dep to dut /tmp/
> > + pkg_files = ['ice-1.3.4.0.pkg', 'ice-1.3.10.0.pkg']
> > + """
> > + dst = "/tmp"
> > + pkg_file = "ice-%s.pkg" % pkg_ver
> > + src_file = r'./dep/%s' % pkg_file
> > + self.dut.session.copy_file_to(src_file, dst)
> > +
> > + def generate_delete_specify_pkg(self, pkg_ver, sn, key="true"):
> > + self.dut.send_expect("rm -rf
> > + /lib/firmware/intel/ice/ddp/ice-%s.pkg" % sn,
> > "#")
> > + if key == "true":
> > + self.dut.send_expect("\cp /tmp/ice-%s.pkg
> > /lib/firmware/intel/ice/ddp/ice-%s.pkg" % (pkg_ver, sn), "#")
> > +
> > + def test_check_specific_package_loading(self):
> > + """
> > + Copy 2 different ``ice.pkg`` into ``/lib/firmware/intel/ice/ddp/``,
> > + and rename 1 ice.pkg to ice-<interface serial number>.pkg,
> > + e.g. ice-
> > 8ce1abfffffefd3c.pkg
> > + Launch testpmd with 1 default interface and 1 specific interface,
> > + check the ice-<interface serial number>.pkg is loadded by the
> > + specific
> > interface.
> > + """
> > + self.check_env()
> > + self.use_correct_ice_pkg(flag="true")
> > + self.new_pkgs = ["1.3.10.0", "1.3.4.0"]
> > + for i in range(len(self.new_pkgs)):
> > + self.copy_specify_ice_pkg(self.new_pkgs[i])
> > +
> > + self.generate_delete_specify_pkg(pkg_ver=self.new_pkgs[i],
> > sn=self.nic_sn[i], key="true")
> > +
> > + eal_param = "-w %s " % self.nic_pci[0] + "-w %s " %
> > + self.nic_pci[1] + "--log-
> > level=8"
> > + out = self.dut_testpmd.execute_cmd("./%s/app/testpmd %s -- -i
> > + " %
> > (self.target, eal_param))
> > + self.dut_testpmd.quit()
> > +
> > + # Delete ice-<interface serial number>.pkg to recover the ENV
> > + for i in range(len(self.new_pkgs)):
> > +
> > + self.generate_delete_specify_pkg(pkg_ver=self.new_pkgs[i],
> > sn=self.nic_sn[i], key="false")
> > +
> > + self.actual_pcis = re.findall(r"device (.*) on", out)
> > + self.pkgs = re.findall(r"is: (.*),", out)
> > + self.verify(self.actual_pcis == self.nic_pci and self.pkgs == self.new_pkgs,
> > + "The nics pci info/pkg are not matched")
> > +
> > def tear_down(self):
> > self.dut_testpmd.quit()
> >
> > --
> > 2.17.1
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [dts] [PATCH V1] tests/package_download:add new case
2019-12-27 12:04 [dts] [PATCH V1] tests/package_download:add new case Wenjie Li
2019-12-27 3:23 ` Li, WenjieX A
@ 2020-01-03 2:56 ` Tu, Lijuan
1 sibling, 0 replies; 4+ messages in thread
From: Tu, Lijuan @ 2020-01-03 2:56 UTC (permalink / raw)
To: Li, WenjieX A, dts; +Cc: Li, WenjieX A
applied
> -----Original Message-----
> From: dts [mailto:dts-bounces@dpdk.org] On Behalf Of Wenjie Li
> Sent: Friday, December 27, 2019 8:04 PM
> To: dts@dpdk.org
> Cc: Li, WenjieX A <wenjiex.a.li@intel.com>
> Subject: [dts] [PATCH V1] tests/package_download:add new case
>
> add new case check_specific_package_loading
>
> Signed-off-by: Wenjie Li <wenjiex.a.li@intel.com>
> ---
> ...e_enable_package_download_in_ice_driver.py | 57
> +++++++++++++++++++
> 1 file changed, 57 insertions(+)
>
> diff --git a/tests/TestSuite_enable_package_download_in_ice_driver.py
> b/tests/TestSuite_enable_package_download_in_ice_driver.py
> index fd6a27e..f28f0e6 100644
> --- a/tests/TestSuite_enable_package_download_in_ice_driver.py
> +++ b/tests/TestSuite_enable_package_download_in_ice_driver.py
> @@ -295,6 +295,63 @@ class
> TestEnable_Package_Download_In_Ice_Driver(TestCase):
> for error_message in error_messages:
> self.verify(error_message in out, "There should be '%s' in out: %s" %
> (error_message, out))
>
> + def get_sn(self, nic_pci):
> + cmd = "lspci -vs %s | grep 'Device Serial Number'" % nic_pci
> + out = self.dut.send_expect(cmd, "#")
> + sn_temp = re.findall(r'Device Serial Number (.*)', out)
> + sn = re.sub("-", "", sn_temp[0])
> + return sn
> +
> + def check_env(self):
> + """
> + Check the DUT has two or more CVL NICs. If not, return "the ENV needs
> at least two CVL NICs"
> + """
> + self.nic_pci = [self.dut.ports_info[0]['pci'], self.dut.ports_info[-1]['pci']]
> + self.nic_sn = [self.get_sn(self.nic_pci[0]), self.get_sn(self.nic_pci[1])]
> + self.verify(self.nic_sn[0] != self.nic_sn[1], "the case needs >=2 CVL NICs
> with different Serial Numbers")
> +
> + def copy_specify_ice_pkg(self, pkg_ver):
> + """
> + Copy 2 different ``ice-xxx.pkg`` from dts/dep to dut /tmp/
> + pkg_files = ['ice-1.3.4.0.pkg', 'ice-1.3.10.0.pkg']
> + """
> + dst = "/tmp"
> + pkg_file = "ice-%s.pkg" % pkg_ver
> + src_file = r'./dep/%s' % pkg_file
> + self.dut.session.copy_file_to(src_file, dst)
> +
> + def generate_delete_specify_pkg(self, pkg_ver, sn, key="true"):
> + self.dut.send_expect("rm -rf /lib/firmware/intel/ice/ddp/ice-%s.pkg" %
> sn, "#")
> + if key == "true":
> + self.dut.send_expect("\cp /tmp/ice-%s.pkg
> /lib/firmware/intel/ice/ddp/ice-%s.pkg" % (pkg_ver, sn), "#")
> +
> + def test_check_specific_package_loading(self):
> + """
> + Copy 2 different ``ice.pkg`` into ``/lib/firmware/intel/ice/ddp/``,
> + and rename 1 ice.pkg to ice-<interface serial number>.pkg, e.g. ice-
> 8ce1abfffffefd3c.pkg
> + Launch testpmd with 1 default interface and 1 specific interface,
> + check the ice-<interface serial number>.pkg is loadded by the specific
> interface.
> + """
> + self.check_env()
> + self.use_correct_ice_pkg(flag="true")
> + self.new_pkgs = ["1.3.10.0", "1.3.4.0"]
> + for i in range(len(self.new_pkgs)):
> + self.copy_specify_ice_pkg(self.new_pkgs[i])
> + self.generate_delete_specify_pkg(pkg_ver=self.new_pkgs[i],
> sn=self.nic_sn[i], key="true")
> +
> + eal_param = "-w %s " % self.nic_pci[0] + "-w %s " % self.nic_pci[1] + "--
> log-level=8"
> + out = self.dut_testpmd.execute_cmd("./%s/app/testpmd %s -- -i " %
> (self.target, eal_param))
> + self.dut_testpmd.quit()
> +
> + # Delete ice-<interface serial number>.pkg to recover the ENV
> + for i in range(len(self.new_pkgs)):
> + self.generate_delete_specify_pkg(pkg_ver=self.new_pkgs[i],
> sn=self.nic_sn[i], key="false")
> +
> + self.actual_pcis = re.findall(r"device (.*) on", out)
> + self.pkgs = re.findall(r"is: (.*),", out)
> + self.verify(self.actual_pcis == self.nic_pci and self.pkgs == self.new_pkgs,
> + "The nics pci info/pkg are not matched")
> +
> def tear_down(self):
> self.dut_testpmd.quit()
>
> --
> 2.17.1
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2020-01-03 2:56 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-12-27 12:04 [dts] [PATCH V1] tests/package_download:add new case Wenjie Li
2019-12-27 3:23 ` Li, WenjieX A
2020-01-02 7:13 ` Chen, Zhaoyan
2020-01-03 2:56 ` Tu, Lijuan
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).