* [dts] [PATCH V1] tests/hotpiug_mp:add parameters when igb_uio @ 2019-07-29 5:22 zhuwenhui 2019-07-29 5:56 ` Zhu, ShuaiX 2019-07-29 6:36 ` Ma, LihongX 0 siblings, 2 replies; 4+ messages in thread From: zhuwenhui @ 2019-07-29 5:22 UTC (permalink / raw) To: dts; +Cc: zhuwenhui 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 ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [dts] [PATCH V1] tests/hotpiug_mp:add parameters when igb_uio 2019-07-29 5:22 [dts] [PATCH V1] tests/hotpiug_mp:add parameters when igb_uio zhuwenhui @ 2019-07-29 5:56 ` Zhu, ShuaiX 2019-07-29 6:36 ` Ma, LihongX 1 sibling, 0 replies; 4+ messages in thread From: Zhu, ShuaiX @ 2019-07-29 5:56 UTC (permalink / raw) To: Zhu, WenhuiX, dts; +Cc: Zhu, WenhuiX, Zhu, ShuaiX Tested-by: Zhu, ShuaiX <shuaix.zhu@intel.com> > -----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 ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [dts] [PATCH V1] tests/hotpiug_mp:add parameters when igb_uio 2019-07-29 5:22 [dts] [PATCH V1] tests/hotpiug_mp:add parameters when igb_uio zhuwenhui 2019-07-29 5:56 ` Zhu, ShuaiX @ 2019-07-29 6:36 ` Ma, LihongX 2019-07-29 6:59 ` Zhu, WenhuiX 1 sibling, 1 reply; 4+ messages in thread From: Ma, LihongX @ 2019-07-29 6:36 UTC (permalink / raw) To: Zhu, WenhuiX, dts; +Cc: Zhu, WenhuiX 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 ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [dts] [PATCH V1] tests/hotpiug_mp:add parameters when igb_uio 2019-07-29 6:36 ` Ma, LihongX @ 2019-07-29 6:59 ` Zhu, WenhuiX 0 siblings, 0 replies; 4+ messages in thread From: Zhu, WenhuiX @ 2019-07-29 6:59 UTC (permalink / raw) To: Ma, LihongX, dts OK, Thanks for your advice. Thanks, Wenhui. -----Original Message----- From: Ma, LihongX Sent: Monday, July 29, 2019 2:36 PM To: Zhu, WenhuiX <wenhuix.zhu@intel.com>; dts@dpdk.org Cc: Zhu, WenhuiX <wenhuix.zhu@intel.com> Subject: RE: [dts] [PATCH V1] tests/hotpiug_mp:add parameters when igb_uio 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 ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2019-07-29 6:59 UTC | newest] Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2019-07-29 5:22 [dts] [PATCH V1] tests/hotpiug_mp:add parameters when igb_uio zhuwenhui 2019-07-29 5:56 ` Zhu, ShuaiX 2019-07-29 6:36 ` Ma, LihongX 2019-07-29 6:59 ` Zhu, WenhuiX
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).