test suite reviews and discussions
 help / color / mirror / Atom feed
* [dts] [PATCH V2] tests/malicious_driver_event_indication:code for optimizing the start mode of VF
@ 2021-03-24  2:25 Zhou Jun
  2021-03-24  3:16 ` Zhou, JunX W
  0 siblings, 1 reply; 3+ messages in thread
From: Zhou Jun @ 2021-03-24  2:25 UTC (permalink / raw)
  To: dts; +Cc: Zhou Jun

v1:
the original way:copy testpmd to another file and start VF
now:start VF with another profix file on the testpmd start PF 

v2:
modifying nonstandard variable naming

Signed-off-by: Zhou Jun <junx.w.zhou@intel.com>
---
 .../TestSuite_malicious_driver_event_indication.py | 43 ++++++++--------------
 1 file changed, 15 insertions(+), 28 deletions(-)

diff --git a/tests/TestSuite_malicious_driver_event_indication.py b/tests/TestSuite_malicious_driver_event_indication.py
index 44859ab..8b446cf 100644
--- a/tests/TestSuite_malicious_driver_event_indication.py
+++ b/tests/TestSuite_malicious_driver_event_indication.py
@@ -38,6 +38,7 @@ import re
 import time
 import traceback
 from contextlib import contextmanager
+from pmd_output import PmdOutput
 
 
 from exception import VerifyFailure
@@ -159,28 +160,12 @@ class TestSuiteMaliciousDrvEventIndication(TestCase):
     def init_vf_testpmd(self):
         self.vf_pmd_session_name = 'vf_testpmd'
         self.vf_pmd_session = self.dut.create_session(self.vf_pmd_session_name)
-        self.vf_testpmd = "{}/{}/app/testpmd_vf".format(
-            self.target_dir, self.dut.target)
-        cmd = 'rm -f {vf_pmd};cp -f {pf_pmd} {vf_pmd}'.format(
-            **{'pf_pmd': self.pf_testpmd, 'vf_pmd': self.vf_testpmd})
-        self.d_a_con(cmd)
+        self.vf_pmdout = PmdOutput(self.dut, self.vf_pmd_session)
 
     def start_vf_testpmd(self):
-        core_mask = utils.create_mask(self.vf_pmd_cores)
-        cmd = (
-            "{bin} "
-            "-v "
-            "-c {core_mask} "
-            "-n {mem_channel} "
-            "--file-prefix={prefix} "
-            "{whitelist} "
-            "-- -i ").format(**{
-                'bin': self.vf_testpmd,
-                'core_mask': core_mask,
-                'mem_channel': self.dut.get_memory_channels(),
-                'whitelist': self.vf_pmd_whitelist,
-                'prefix': 'vf_pmd', })
-        self.vf_pmd_con([cmd, "testpmd> ", 120])
+        self.vf_pmdout.start_testpmd(self.vf_pmd_cores,
+                                     eal_param="-v {}".format(self.vf_pmd_allowlst),
+                                     prefix='vf_pmd')
         self.is_vf_pmd_on = True
         cmds = [
             'set fwd txonly',
@@ -269,7 +254,7 @@ class TestSuiteMaliciousDrvEventIndication(TestCase):
         corelist = self.dut.get_core_list("1S/6C/1T", socket=socket)[2:]
         self.pf_pmd_whitelist = '-w ' + self.vf_ports_info[0].get('pf_pci')
         self.pf_pmd_cores = corelist[:2]
-        self.vf_pmd_whitelist = '-w ' + self.vf_ports_info[0].get('vfs_pci')[0]
+        self.vf_pmd_allowlst = '-w ' + self.vf_ports_info[0].get('vfs_pci')[0]
         self.vf_pmd_cores = corelist[2:]
 
     def init_params(self):
@@ -278,11 +263,6 @@ class TestSuiteMaliciousDrvEventIndication(TestCase):
 
     def preset_test_environment(self):
         self.preset_dpdk_compilation()
-        self.init_pf_testpmd()
-        self.init_vf_testpmd()
-        self.vf_create()
-        # get socket and cores
-        self.preset_pmd_res()
 
     def destroy_resource(self):
         with self.restore_dpdk_compilation():
@@ -297,7 +277,6 @@ class TestSuiteMaliciousDrvEventIndication(TestCase):
         """
         Run at the start of each test suite.
         """
-        self.init_params()
         self.dut_ports = self.dut.get_ports(self.nic)
         self.verify(len(self.dut_ports) >= 1, "Not enough ports")
         self.verify_supported_nic()
@@ -314,13 +293,21 @@ class TestSuiteMaliciousDrvEventIndication(TestCase):
         """
         Run before each test case.
         """
-        pass
+        self.init_params()
+        self.init_pf_testpmd()
+        self.init_vf_testpmd()
+        self.vf_create()
+        self.preset_pmd_res()
 
     def tear_down(self):
         """
         Run after each test case.
         """
         self.dut.kill_all()
+        self.vf_destroy()
+        if self.vf_pmd_session:
+            self.dut.close_session(self.vf_pmd_session)
+            self.vf_pmd_session = None
 
     def test_malicious_driver_event_detected(self):
         """
-- 
1.8.3.1


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [dts] [PATCH V2] tests/malicious_driver_event_indication:code for optimizing the start mode of VF
  2021-03-24  2:25 [dts] [PATCH V2] tests/malicious_driver_event_indication:code for optimizing the start mode of VF Zhou Jun
@ 2021-03-24  3:16 ` Zhou, JunX W
  2021-03-25  5:58   ` Tu, Lijuan
  0 siblings, 1 reply; 3+ messages in thread
From: Zhou, JunX W @ 2021-03-24  3:16 UTC (permalink / raw)
  To: dts

[-- Attachment #1: Type: text/plain, Size: 547 bytes --]

>-----Original Message-----
>From: Zhou, JunX W 
>Sent: Wednesday, March 24, 2021 10:26 AM
>To: dts@dpdk.org
>Cc: Zhou, JunX W <junx.w.zhou@intel.com>
>Subject: [dts][PATCH V2] tests/malicious_driver_event_indication:code for optimizing the start mode of VF
>
>v1:
>the original way:copy testpmd to another file and start VF now:start VF with another profix file on the testpmd start PF 
>
>v2:
>modifying nonstandard variable naming
>
>Signed-off-by: Zhou Jun <junx.w.zhou@intel.com>

Tested-by: Zhou, Jun <junx.w.zhou@intel.com>

[-- Attachment #2: TestSuiteMaliciousDrvEventIndication.log --]
[-- Type: application/octet-stream, Size: 253234 bytes --]

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [dts] [PATCH V2] tests/malicious_driver_event_indication:code for optimizing the start mode of VF
  2021-03-24  3:16 ` Zhou, JunX W
@ 2021-03-25  5:58   ` Tu, Lijuan
  0 siblings, 0 replies; 3+ messages in thread
From: Tu, Lijuan @ 2021-03-25  5:58 UTC (permalink / raw)
  To: Zhou, JunX W, dts



> -----Original Message-----
> From: dts <dts-bounces@dpdk.org> On Behalf Of Zhou, JunX W
> Sent: 2021年3月24日 11:17
> To: dts@dpdk.org
> Subject: Re: [dts] [PATCH V2] tests/malicious_driver_event_indication:code for
> optimizing the start mode of VF
> 
> >-----Original Message-----
> >From: Zhou, JunX W
> >Sent: Wednesday, March 24, 2021 10:26 AM
> >To: dts@dpdk.org
> >Cc: Zhou, JunX W <junx.w.zhou@intel.com>
> >Subject: [dts][PATCH V2] tests/malicious_driver_event_indication:code for
> optimizing the start mode of VF
> >
> >v1:
> >the original way:copy testpmd to another file and start VF now:start VF with
> another profix file on the testpmd start PF
> >
> >v2:
> >modifying nonstandard variable naming
> >
> >Signed-off-by: Zhou Jun <junx.w.zhou@intel.com>
> 
> Tested-by: Zhou, Jun <junx.w.zhou@intel.com>

Applied, thanks

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2021-03-25  5:58 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-03-24  2:25 [dts] [PATCH V2] tests/malicious_driver_event_indication:code for optimizing the start mode of VF Zhou Jun
2021-03-24  3:16 ` Zhou, JunX W
2021-03-25  5:58   ` Tu, Lijuan

test suite reviews and discussions

This inbox may be cloned and mirrored by anyone:

	git clone --mirror https://inbox.dpdk.org/dts/0 dts/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 dts dts/ https://inbox.dpdk.org/dts \
		dts@dpdk.org
	public-inbox-index dts

Example config snippet for mirrors.
Newsgroup available over NNTP:
	nntp://inbox.dpdk.org/inbox.dpdk.dts


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git