Re: [dts] [PATCH V1] tests/hotpiug_mp:add parameters when igb_uio

["Ma, LihongX" <lihongx.ma@intel.com>]

Hi, wenhui
From the patch, I think the only different of the command between igb_uio and other driver is the "--iova-mode=pa"
Driver igb_uio has the param of "--iova-mode=pa" and other did not.
So I think maybe you can define a params of it.
Eg : 
	self.iova_param = ""
	if self.drivername in ['igb_uio']:
		self.iova_param = "--iova-mode=pa"

	.........
	out = self.session_pri.send_expect("./examples/multi_process/hotplug_mp/%s/hotplug_mp %s --proc-type=auto"
				% (self.target, self.iova_param), "example>")

-----Original Message-----
From: dts [mailto:dts-bounces@dpdk.org] On Behalf Of zhuwenhui
Sent: Monday, July 29, 2019 1:22 PM
To: dts@dpdk.org
Cc: Zhu, WenhuiX <wenhuix.zhu@intel.com>
Subject: [dts] [PATCH V1] tests/hotpiug_mp:add parameters when igb_uio

change test cases based on dpdk bbe29a9bd7ab6feab.

Signed-off-by: zhuwenhui <wenhuix.zhu@intel.com>
---
 tests/TestSuite_hotplug_mp.py | 42 ++++++++++++++++++++++++-----------
 1 file changed, 29 insertions(+), 13 deletions(-)

diff --git a/tests/TestSuite_hotplug_mp.py b/tests/TestSuite_hotplug_mp.py index ff89164..62f1982 100644
--- a/tests/TestSuite_hotplug_mp.py
+++ b/tests/TestSuite_hotplug_mp.py
@@ -42,19 +42,35 @@ class TestHotplugMp(TestCase):
         """
         Setup primary process and two secondary processes.
         """
-        out = self.session_pri.send_expect(
-            "./examples/multi_process/hotplug_mp/%s/hotplug_mp --proc-type=auto"
-            % self.target, "example>")
-        self.verify("Auto-detected process type: PRIMARY" in out,
-                    "Failed to setup primary process!")
-        for out in [self.session_sec_1.send_expect(
-                        "./examples/multi_process/hotplug_mp/%s/hotplug_mp --proc-type=auto"
-                        % self.target, "example>"),
-                    self.session_sec_2.send_expect(
-                        "./examples/multi_process/hotplug_mp/%s/hotplug_mp --proc-type=auto"
-                        % self.target, "example>")]:
-            self.verify("Auto-detected process type: SECONDARY" in out,
-                        "Failed to setup secondary process!")
+        if self.drivername == "igb_uio":
+            out = self.session_pri.send_expect(
+                "./examples/multi_process/hotplug_mp/%s/hotplug_mp --iova-mode=pa --proc-type=auto"
+                % self.target, "example>")
+            self.verify("Auto-detected process type: PRIMARY" in out,
+                        "Failed to setup primary process!")
+            for out in [self.session_sec_1.send_expect(
+                            "./examples/multi_process/hotplug_mp/%s/hotplug_mp --iova-mode=pa --proc-type=auto"
+                            % self.target, "example>"),
+                        self.session_sec_2.send_expect(
+                            "./examples/multi_process/hotplug_mp/%s/hotplug_mp --iova-mode=pa --proc-type=auto"
+                            % self.target, "example>")]:
+                self.verify("Auto-detected process type: SECONDARY" in out,
+                            "Failed to setup secondary process!")
+
+        else:
+            out = self.session_pri.send_expect(
+                "./examples/multi_process/hotplug_mp/%s/hotplug_mp --proc-type=auto"
+                % self.target, "example>")
+            self.verify("Auto-detected process type: PRIMARY" in out,
+                        "Failed to setup primary process!")
+            for out in [self.session_sec_1.send_expect(
+                            "./examples/multi_process/hotplug_mp/%s/hotplug_mp --proc-type=auto"
+                            % self.target, "example>"),
+                        self.session_sec_2.send_expect(
+                            "./examples/multi_process/hotplug_mp/%s/hotplug_mp --proc-type=auto"
+                            % self.target, "example>")]:
+                self.verify("Auto-detected process type: SECONDARY" in out,
+                            "Failed to setup secondary process!")
 
     def multi_process_quit(self):
         """
--
2.17.2