test suite reviews and discussions
 help / color / mirror / Atom feed
* [dts] [PATCH] tests: Add different queue number
@ 2019-01-15 11:09 Peng Yuan
  2019-01-17  7:56 ` Zhu, WenhuiX
  2019-01-21  6:43 ` Tu, Lijuan
  0 siblings, 2 replies; 3+ messages in thread
From: Peng Yuan @ 2019-01-15 11:09 UTC (permalink / raw)
  To: dts; +Cc: Peng Yuan

Add different queue number and iptypes to TestSuite_pmdrssreta.py

Signed-off-by: Peng Yuan <yuan.peng@intel.com>

diff --git a/tests/TestSuite_pmdrssreta.py b/tests/TestSuite_pmdrssreta.py
index c3bbba2..1fc4739 100644
--- a/tests/TestSuite_pmdrssreta.py
+++ b/tests/TestSuite_pmdrssreta.py
@@ -37,7 +37,7 @@ import time
 import random
 import re
 import utils
-testQueues = [16]
+testQueues = [2, 9, 16]
 reta_entries = []
 reta_lines = []
 
@@ -173,9 +173,6 @@ class TestPmdrssreta(TestCase):
         Run at the start of each test suite.
         """
 
-        #self.verify(
-        #    self.nic in ["niantic", "fortville_eagle", "fortville_spirit", "fortville_spirit_single"],
-        #    "NIC Unsupported: " + str(self.nic))
         ports = self.dut.get_ports(self.nic)
         self.ports_socket = self.dut.get_numa_id(ports[0])
         self.verify(len(ports) >= 1, "Not enough ports available")
@@ -192,7 +189,13 @@ class TestPmdrssreta(TestCase):
         dutPorts = self.dut.get_ports(self.nic)
         localPort = self.tester.get_local_port(dutPorts[0])
         itf = self.tester.get_interface(localPort)
-        iptypes = ['IPV4']
+        iptypes = {'IPV4': 'ip',
+                   'IPV4&UDP': 'udp',
+                   'IPV4&TCP': 'tcp',
+                   'IPV6': 'ip',
+                   'IPV6&UDP': 'udp',
+                   'IPV6&TCP': 'tcp'
+                   }
 
         self.dut.kill_all()
 
@@ -205,12 +208,17 @@ class TestPmdrssreta(TestCase):
                 self.pmdout.start_testpmd(
                     "all", "--mbcache=128 --rxq=%d --txq=%d" % (queue, queue), socket=self.ports_socket)
 
-            for iptype in iptypes:
+            for iptype, rsstype in iptypes.items():
+            #for iptype in iptypes:
                 self.dut.send_expect("set verbose 8", "testpmd> ")
                 self.dut.send_expect("set fwd rxonly", "testpmd> ")
                 self.dut.send_expect(
                     "set nbcore %d" % (queue + 1), "testpmd> ")
 
+                out = self.dut.send_expect(
+                    "port config all rss %s" % rsstype, "testpmd> ")
+                self.verify("error" not in out, "Configuration of RSS hash failed: Invalid argument")
+
                 # configure the reta with specific mappings.
                 if(self.nic in ["niantic", "redrockcanyou", "atwood", "boulderrapid"]):
                     for i in range(128):
-- 
2.5.0

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

* Re: [dts] [PATCH] tests: Add different queue number
  2019-01-15 11:09 [dts] [PATCH] tests: Add different queue number Peng Yuan
@ 2019-01-17  7:56 ` Zhu, WenhuiX
  2019-01-21  6:43 ` Tu, Lijuan
  1 sibling, 0 replies; 3+ messages in thread
From: Zhu, WenhuiX @ 2019-01-17  7:56 UTC (permalink / raw)
  To: Peng, Yuan, dts; +Cc: Peng, Yuan


Tested-by: Zhu, WenhuiX <wenhuix.zhu@intel.com>

-----Original Message-----
From: dts [mailto:dts-bounces@dpdk.org] On Behalf Of Peng Yuan
Sent: Tuesday, January 15, 2019 7:10 PM
To: dts@dpdk.org
Cc: Peng, Yuan <yuan.peng@intel.com>
Subject: [dts] [PATCH] tests: Add different queue number

Add different queue number and iptypes to TestSuite_pmdrssreta.py

Signed-off-by: Peng Yuan <yuan.peng@intel.com>

diff --git a/tests/TestSuite_pmdrssreta.py b/tests/TestSuite_pmdrssreta.py index c3bbba2..1fc4739 100644
--- a/tests/TestSuite_pmdrssreta.py
+++ b/tests/TestSuite_pmdrssreta.py
@@ -37,7 +37,7 @@ import time
 import random
 import re
 import utils
-testQueues = [16]
+testQueues = [2, 9, 16]
 reta_entries = []
 reta_lines = []
 
@@ -173,9 +173,6 @@ class TestPmdrssreta(TestCase):
         Run at the start of each test suite.
         """
 
-        #self.verify(
-        #    self.nic in ["niantic", "fortville_eagle", "fortville_spirit", "fortville_spirit_single"],
-        #    "NIC Unsupported: " + str(self.nic))
         ports = self.dut.get_ports(self.nic)
         self.ports_socket = self.dut.get_numa_id(ports[0])
         self.verify(len(ports) >= 1, "Not enough ports available") @@ -192,7 +189,13 @@ class TestPmdrssreta(TestCase):
         dutPorts = self.dut.get_ports(self.nic)
         localPort = self.tester.get_local_port(dutPorts[0])
         itf = self.tester.get_interface(localPort)
-        iptypes = ['IPV4']
+        iptypes = {'IPV4': 'ip',
+                   'IPV4&UDP': 'udp',
+                   'IPV4&TCP': 'tcp',
+                   'IPV6': 'ip',
+                   'IPV6&UDP': 'udp',
+                   'IPV6&TCP': 'tcp'
+                   }
 
         self.dut.kill_all()
 
@@ -205,12 +208,17 @@ class TestPmdrssreta(TestCase):
                 self.pmdout.start_testpmd(
                     "all", "--mbcache=128 --rxq=%d --txq=%d" % (queue, queue), socket=self.ports_socket)
 
-            for iptype in iptypes:
+            for iptype, rsstype in iptypes.items():
+            #for iptype in iptypes:
                 self.dut.send_expect("set verbose 8", "testpmd> ")
                 self.dut.send_expect("set fwd rxonly", "testpmd> ")
                 self.dut.send_expect(
                     "set nbcore %d" % (queue + 1), "testpmd> ")
 
+                out = self.dut.send_expect(
+                    "port config all rss %s" % rsstype, "testpmd> ")
+                self.verify("error" not in out, "Configuration of RSS 
+ hash failed: Invalid argument")
+
                 # configure the reta with specific mappings.
                 if(self.nic in ["niantic", "redrockcanyou", "atwood", "boulderrapid"]):
                     for i in range(128):
--
2.5.0

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

* Re: [dts] [PATCH] tests: Add different queue number
  2019-01-15 11:09 [dts] [PATCH] tests: Add different queue number Peng Yuan
  2019-01-17  7:56 ` Zhu, WenhuiX
@ 2019-01-21  6:43 ` Tu, Lijuan
  1 sibling, 0 replies; 3+ messages in thread
From: Tu, Lijuan @ 2019-01-21  6:43 UTC (permalink / raw)
  To: Peng, Yuan, dts; +Cc: Peng, Yuan

Applied, thanks

> -----Original Message-----
> From: dts [mailto:dts-bounces@dpdk.org] On Behalf Of Peng Yuan
> Sent: Tuesday, January 15, 2019 7:10 PM
> To: dts@dpdk.org
> Cc: Peng, Yuan <yuan.peng@intel.com>
> Subject: [dts] [PATCH] tests: Add different queue number
> 
> Add different queue number and iptypes to TestSuite_pmdrssreta.py
> 
> Signed-off-by: Peng Yuan <yuan.peng@intel.com>
> 
> diff --git a/tests/TestSuite_pmdrssreta.py b/tests/TestSuite_pmdrssreta.py index
> c3bbba2..1fc4739 100644
> --- a/tests/TestSuite_pmdrssreta.py
> +++ b/tests/TestSuite_pmdrssreta.py
> @@ -37,7 +37,7 @@ import time
>  import random
>  import re
>  import utils
> -testQueues = [16]
> +testQueues = [2, 9, 16]
>  reta_entries = []
>  reta_lines = []
> 
> @@ -173,9 +173,6 @@ class TestPmdrssreta(TestCase):
>          Run at the start of each test suite.
>          """
> 
> -        #self.verify(
> -        #    self.nic in ["niantic", "fortville_eagle", "fortville_spirit",
> "fortville_spirit_single"],
> -        #    "NIC Unsupported: " + str(self.nic))
>          ports = self.dut.get_ports(self.nic)
>          self.ports_socket = self.dut.get_numa_id(ports[0])
>          self.verify(len(ports) >= 1, "Not enough ports available") @@ -192,7
> +189,13 @@ class TestPmdrssreta(TestCase):
>          dutPorts = self.dut.get_ports(self.nic)
>          localPort = self.tester.get_local_port(dutPorts[0])
>          itf = self.tester.get_interface(localPort)
> -        iptypes = ['IPV4']
> +        iptypes = {'IPV4': 'ip',
> +                   'IPV4&UDP': 'udp',
> +                   'IPV4&TCP': 'tcp',
> +                   'IPV6': 'ip',
> +                   'IPV6&UDP': 'udp',
> +                   'IPV6&TCP': 'tcp'
> +                   }
> 
>          self.dut.kill_all()
> 
> @@ -205,12 +208,17 @@ class TestPmdrssreta(TestCase):
>                  self.pmdout.start_testpmd(
>                      "all", "--mbcache=128 --rxq=%d --txq=%d" % (queue, queue),
> socket=self.ports_socket)
> 
> -            for iptype in iptypes:
> +            for iptype, rsstype in iptypes.items():
> +            #for iptype in iptypes:
>                  self.dut.send_expect("set verbose 8", "testpmd> ")
>                  self.dut.send_expect("set fwd rxonly", "testpmd> ")
>                  self.dut.send_expect(
>                      "set nbcore %d" % (queue + 1), "testpmd> ")
> 
> +                out = self.dut.send_expect(
> +                    "port config all rss %s" % rsstype, "testpmd> ")
> +                self.verify("error" not in out, "Configuration of RSS
> + hash failed: Invalid argument")
> +
>                  # configure the reta with specific mappings.
>                  if(self.nic in ["niantic", "redrockcanyou", "atwood", "boulderrapid"]):
>                      for i in range(128):
> --
> 2.5.0

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

end of thread, other threads:[~2019-01-21  6:43 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-01-15 11:09 [dts] [PATCH] tests: Add different queue number Peng Yuan
2019-01-17  7:56 ` Zhu, WenhuiX
2019-01-21  6:43 ` 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).