test suite reviews and discussions
 help / color / mirror / Atom feed
* [dts] [PATCH V1] tests/dynamic_queue add more cores for testpmd
@ 2019-04-25  2:12 zhuwenhui
  2019-04-25  2:33 ` Yao, BingX Y
  2019-05-06  1:44 ` Tu, Lijuan
  0 siblings, 2 replies; 3+ messages in thread
From: zhuwenhui @ 2019-04-25  2:12 UTC (permalink / raw)
  To: dts; +Cc: zhuwenhui

When fortville spirit binds multiple ports,
starting testpmd requires more cores.

Signed-off-by: zhuwenhui <wenhuix.zhu@intel.com>
---
 tests/TestSuite_dynamic_queue.py | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/tests/TestSuite_dynamic_queue.py b/tests/TestSuite_dynamic_queue.py
index 3aad63f..e07efa8 100644
--- a/tests/TestSuite_dynamic_queue.py
+++ b/tests/TestSuite_dynamic_queue.py
@@ -27,12 +27,11 @@ class TestDynamicQueue(TestCase):
         self.dut_testpmd = PmdOutput(self.dut)
 
     def set_up(self):
-        # Because of fortville spirit limitation,can't use 2 ports for testing
+        # Fortville_spirit needs more cores to run properly
         if (self.nic in ["fortville_spirit"]):
             self.dut_testpmd.start_testpmd(
-                "Default", "--port-topology=chained --txq=%s --rxq=%s"
-                % (self.PF_QUEUE, self.PF_QUEUE), eal_param="-w %s"
-                % (self.dut.get_port_pci(self.dut_ports[0])))
+                "1S/7C/1T", "--port-topology=chained --txq=%s --rxq=%s"
+                % (self.PF_QUEUE, self.PF_QUEUE))
         elif (self.nic in ["cavium_a063"]):
             eal_opts = ""
             for port in self.dut_ports:
-- 
2.17.2


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

* Re: [dts] [PATCH V1] tests/dynamic_queue add more cores for testpmd
  2019-04-25  2:12 [dts] [PATCH V1] tests/dynamic_queue add more cores for testpmd zhuwenhui
@ 2019-04-25  2:33 ` Yao, BingX Y
  2019-05-06  1:44 ` Tu, Lijuan
  1 sibling, 0 replies; 3+ messages in thread
From: Yao, BingX Y @ 2019-04-25  2:33 UTC (permalink / raw)
  To: Zhu, WenhuiX, dts; +Cc: Zhu, WenhuiX

Tested-by: Yao, BingX Y <bingx.y.yao@intel.com>

-----Original Message-----
From: dts [mailto:dts-bounces@dpdk.org] On Behalf Of zhuwenhui
Sent: Thursday, April 25, 2019 10:13 AM
To: dts@dpdk.org
Cc: Zhu, WenhuiX <wenhuix.zhu@intel.com>
Subject: [dts] [PATCH V1] tests/dynamic_queue add more cores for testpmd

When fortville spirit binds multiple ports, starting testpmd requires more cores.

Signed-off-by: zhuwenhui <wenhuix.zhu@intel.com>
---
 tests/TestSuite_dynamic_queue.py | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/tests/TestSuite_dynamic_queue.py b/tests/TestSuite_dynamic_queue.py
index 3aad63f..e07efa8 100644
--- a/tests/TestSuite_dynamic_queue.py
+++ b/tests/TestSuite_dynamic_queue.py
@@ -27,12 +27,11 @@ class TestDynamicQueue(TestCase):
         self.dut_testpmd = PmdOutput(self.dut)
 
     def set_up(self):
-        # Because of fortville spirit limitation,can't use 2 ports for testing
+        # Fortville_spirit needs more cores to run properly
         if (self.nic in ["fortville_spirit"]):
             self.dut_testpmd.start_testpmd(
-                "Default", "--port-topology=chained --txq=%s --rxq=%s"
-                % (self.PF_QUEUE, self.PF_QUEUE), eal_param="-w %s"
-                % (self.dut.get_port_pci(self.dut_ports[0])))
+                "1S/7C/1T", "--port-topology=chained --txq=%s --rxq=%s"
+                % (self.PF_QUEUE, self.PF_QUEUE))
         elif (self.nic in ["cavium_a063"]):
             eal_opts = ""
             for port in self.dut_ports:
--
2.17.2


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

* Re: [dts] [PATCH V1] tests/dynamic_queue add more cores for testpmd
  2019-04-25  2:12 [dts] [PATCH V1] tests/dynamic_queue add more cores for testpmd zhuwenhui
  2019-04-25  2:33 ` Yao, BingX Y
@ 2019-05-06  1:44 ` Tu, Lijuan
  1 sibling, 0 replies; 3+ messages in thread
From: Tu, Lijuan @ 2019-05-06  1:44 UTC (permalink / raw)
  To: Zhu, WenhuiX, dts; +Cc: Zhu, WenhuiX

Could you please check your system has enough cores before starting tests, snice not all platform has 7 cores.

> -----Original Message-----
> From: dts [mailto:dts-bounces@dpdk.org] On Behalf Of zhuwenhui
> Sent: Thursday, April 25, 2019 10:13 AM
> To: dts@dpdk.org
> Cc: Zhu, WenhuiX <wenhuix.zhu@intel.com>
> Subject: [dts] [PATCH V1] tests/dynamic_queue add more cores for testpmd
> 
> When fortville spirit binds multiple ports, starting testpmd requires more
> cores.
> 
> Signed-off-by: zhuwenhui <wenhuix.zhu@intel.com>
> ---
>  tests/TestSuite_dynamic_queue.py | 7 +++----
>  1 file changed, 3 insertions(+), 4 deletions(-)
> 
> diff --git a/tests/TestSuite_dynamic_queue.py
> b/tests/TestSuite_dynamic_queue.py
> index 3aad63f..e07efa8 100644
> --- a/tests/TestSuite_dynamic_queue.py
> +++ b/tests/TestSuite_dynamic_queue.py
> @@ -27,12 +27,11 @@ class TestDynamicQueue(TestCase):
>          self.dut_testpmd = PmdOutput(self.dut)
> 
>      def set_up(self):
> -        # Because of fortville spirit limitation,can't use 2 ports for testing
> +        # Fortville_spirit needs more cores to run properly
>          if (self.nic in ["fortville_spirit"]):
>              self.dut_testpmd.start_testpmd(
> -                "Default", "--port-topology=chained --txq=%s --rxq=%s"
> -                % (self.PF_QUEUE, self.PF_QUEUE), eal_param="-w %s"
> -                % (self.dut.get_port_pci(self.dut_ports[0])))
> +                "1S/7C/1T", "--port-topology=chained --txq=%s --rxq=%s"
> +                % (self.PF_QUEUE, self.PF_QUEUE))
>          elif (self.nic in ["cavium_a063"]):
>              eal_opts = ""
>              for port in self.dut_ports:
> --
> 2.17.2


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

end of thread, other threads:[~2019-05-06  1:44 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-04-25  2:12 [dts] [PATCH V1] tests/dynamic_queue add more cores for testpmd zhuwenhui
2019-04-25  2:33 ` Yao, BingX Y
2019-05-06  1:44 ` 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).