test suite reviews and discussions
 help / color / Atom feed
* [dts] [PATCH V3]tests/TestSuite_cvl_fdir: add cvl 25g max rules support
@ 2020-06-12  2:38 Xiao Qimai
  2020-06-12  2:46 ` Xiao, QimaiX
  0 siblings, 1 reply; 3+ messages in thread
From: Xiao Qimai @ 2020-06-12  2:38 UTC (permalink / raw)
  To: dts; +Cc: Xiao Qimai

* 1. cvl25g only support 14848 fdir rules
* 2. fix max rules case failed caused python crash issue
* 3. add steps for case test_port_start_stop_reset according to testplan updation

Signed-off-by: Xiao Qimai <qimaix.xiao@intel.com>
---
 tests/TestSuite_cvl_fdir.py | 32 +++++++++++++++++++++++---------
 1 file changed, 23 insertions(+), 9 deletions(-)

diff --git a/tests/TestSuite_cvl_fdir.py b/tests/TestSuite_cvl_fdir.py
index 12d9ad8..f2252d0 100644
--- a/tests/TestSuite_cvl_fdir.py
+++ b/tests/TestSuite_cvl_fdir.py
@@ -1722,7 +1722,7 @@ class TestCVLFdir(TestCase):
 
     def test_void_action(self):
         rule = "flow create 0 ingress pattern eth dst is 00:11:22:33:44:55 / ipv4 src is 192.168.0.20 dst is 192.168.0.21 ttl is 2 tos is 4 / end actions end"
-        self.create_fdir_rule(rule, check_stats=False, msg="Invalid input action: Invalid argument")
+        self.create_fdir_rule(rule, check_stats=False)
         self.check_fdir_rule(stats=False)
 
     def _test_unsupported_action(self):
@@ -2068,6 +2068,11 @@ class TestCVLFdir(TestCase):
         self.check_fdir_rule(port_id=0, stats=True, rule_list=rule_li)
         out2 = self.send_pkts_getouput(pkts=pkt, port_id=0, count=1)
         rfc.check_mark(out2, pkt_num=1, check_param={"port_id": 0, "queue": 1, "mark_id": 0}, stats=True)
+        rule2 = 'flow create 0 ingress pattern eth dst is 00:11:22:33:44:55 / ipv4 src is 192.168.0.22 dst is 192.168.0.23 / end actions queue index 2 / mark id 1 / end'
+        rule_li2 = self.create_fdir_rule(rule=rule2, check_stats=True)
+        self.check_fdir_rule(rule_list=rule_li+rule_li2)
+        out3 = self.send_pkts_getouput(pkts='Ether(dst="00:11:22:33:44:55")/IP(src="192.168.0.22",dst="192.168.0.23") / Raw("x" * 80)', port_id=0, count=1)
+        rfc.check_mark(out3, pkt_num=1, check_param={"port_id": 0, "queue": 2, "mark_id": 1}, stats=True)
 
     def test_add_delete_rules(self):
         self.pmd_output.execute_cmd("stop")
@@ -2157,10 +2162,22 @@ class TestCVLFdir(TestCase):
         rules = list()
         pkt_pattern = 'Ether(dst="00:11:22:33:44:55")/IP(src="192.168.100.20",dst="192.168.%d.%d")/Raw("x" * 80)'
         pkts2 = list()
-        for i in range(60):
-            for j in range(256):
-                rules.append(rule_pattern % (i, j))
-                pkts2.append(pkt_pattern % (i, j))
+        if self.nic in ['columbiaville_100g']:
+            rule_li = list(map(str, range(15360)))
+            pkts = ['Ether(dst="00:11:22:33:44:55")/IP(src="192.168.100.20",dst="192.168.0.0")/Raw("x" * 80)',
+                    'Ether(dst="00:11:22:33:44:55")/IP(src="192.168.100.20",dst="192.168.59.255")/Raw("x" * 80)']
+            for i in range(60):
+                for j in range(256):
+                    rules.append(rule_pattern % (i, j))
+                    pkts2.append(pkt_pattern % (i, j))
+        else:
+            rule_li = list(map(str, range(14848)))
+            pkts = ['Ether(dst="00:11:22:33:44:55")/IP(src="192.168.100.20",dst="192.168.0.0")/Raw("x" * 80)',
+                    'Ether(dst="00:11:22:33:44:55")/IP(src="192.168.100.20",dst="192.168.57.255")/Raw("x" * 80)']
+            for i in range(58):
+                for j in range(256):
+                    rules.append(rule_pattern % (i, j))
+                    pkts2.append(pkt_pattern % (i, j))
         cmd_path = '/tmp/test_max_rules'
         cmd_li = map(lambda x: x + os.linesep, rules)
         with open(cmd_path, 'w') as f:
@@ -2174,15 +2191,12 @@ class TestCVLFdir(TestCase):
                                                     self.portMask, 64, 64, cmd_path),
                                                 eal_param="-w %s,flow-mark-support=1 -w %s,flow-mark-support=1" % (
                                                     self.pci0, self.pci1), socket=self.ports_socket)
-            self.verify('Failed to create file' not in out, "create some rule failed: %s" % out)
+            self.verify('Failed to create flow' not in out, "create some rule failed")
             self.config_testpmd()
             self.pmd_output.execute_cmd('start')
             rule = "flow create 0 ingress pattern eth / ipv4 src is 192.168.100.20 dst is 192.168.60.0 / end actions queue index 1 / mark / end"
             self.create_fdir_rule(rule=rule, check_stats=False, msg='Failed to create flow', validate=False)
-            rule_li = list(map(str, range(15360)))
             self.check_fdir_rule(port_id=0, stats=True, rule_list=rule_li)
-            pkts = ['Ether(dst="00:11:22:33:44:55")/IP(src="192.168.100.20",dst="192.168.0.0")/Raw("x" * 80)',
-                    'Ether(dst="00:11:22:33:44:55")/IP(src="192.168.100.20",dst="192.168.59.255")/Raw("x" * 80)']
             out1 = self.send_pkts_getouput(pkts=pkts, port_id=0, count=1)
             rfc.check_mark(out1, pkt_num=2, check_param={"port_id": 0, "queue": 1, "mark_id": 0}, stats=True)
             self.pmd_output.execute_cmd("flow flush 0")
-- 
1.8.3.1


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

* Re: [dts] [PATCH V3]tests/TestSuite_cvl_fdir: add cvl 25g max rules support
  2020-06-12  2:38 [dts] [PATCH V3]tests/TestSuite_cvl_fdir: add cvl 25g max rules support Xiao Qimai
@ 2020-06-12  2:46 ` Xiao, QimaiX
  2020-06-19  8:21   ` Peng, Yuan
  0 siblings, 1 reply; 3+ messages in thread
From: Xiao, QimaiX @ 2020-06-12  2:46 UTC (permalink / raw)
  To: dts

Tested-by: Xiao, QimaiX <qimaix.xiao@intel.com>

Regards,
Xiao Qimai

> -----Original Message-----
> From: Xiao, QimaiX <qimaix.xiao@intel.com>
> Sent: Friday, June 12, 2020 10:38 AM
> To: dts@dpdk.org
> Cc: Xiao, QimaiX <qimaix.xiao@intel.com>
> Subject: [dts][PATCH V3]tests/TestSuite_cvl_fdir: add cvl 25g max rules
> support
> 
> * 1. cvl25g only support 14848 fdir rules
> * 2. fix max rules case failed caused python crash issue
> * 3. add steps for case test_port_start_stop_reset according to testplan
> updation
> 
> Signed-off-by: Xiao Qimai <qimaix.xiao@intel.com>
> ---
>  tests/TestSuite_cvl_fdir.py | 32 +++++++++++++++++++++++---------
>  1 file changed, 23 insertions(+), 9 deletions(-)
> 
> diff --git a/tests/TestSuite_cvl_fdir.py b/tests/TestSuite_cvl_fdir.py index
> 12d9ad8..f2252d0 100644
> --- a/tests/TestSuite_cvl_fdir.py
> +++ b/tests/TestSuite_cvl_fdir.py
> @@ -1722,7 +1722,7 @@ class TestCVLFdir(TestCase):
> 
>      def test_void_action(self):
>          rule = "flow create 0 ingress pattern eth dst is 00:11:22:33:44:55 / ipv4 src
> is 192.168.0.20 dst is 192.168.0.21 ttl is 2 tos is 4 / end actions end"
> -        self.create_fdir_rule(rule, check_stats=False, msg="Invalid input action:
> Invalid argument")
> +        self.create_fdir_rule(rule, check_stats=False)
>          self.check_fdir_rule(stats=False)
> 
>      def _test_unsupported_action(self):
> @@ -2068,6 +2068,11 @@ class TestCVLFdir(TestCase):
>          self.check_fdir_rule(port_id=0, stats=True, rule_list=rule_li)
>          out2 = self.send_pkts_getouput(pkts=pkt, port_id=0, count=1)
>          rfc.check_mark(out2, pkt_num=1, check_param={"port_id": 0, "queue":
> 1, "mark_id": 0}, stats=True)
> +        rule2 = 'flow create 0 ingress pattern eth dst is 00:11:22:33:44:55 / ipv4
> src is 192.168.0.22 dst is 192.168.0.23 / end actions queue index 2 / mark id 1 /
> end'
> +        rule_li2 = self.create_fdir_rule(rule=rule2, check_stats=True)
> +        self.check_fdir_rule(rule_list=rule_li+rule_li2)
> +        out3 =
> self.send_pkts_getouput(pkts='Ether(dst="00:11:22:33:44:55")/IP(src="192.1
> 68.0.22",dst="192.168.0.23") / Raw("x" * 80)', port_id=0, count=1)
> +        rfc.check_mark(out3, pkt_num=1, check_param={"port_id": 0,
> + "queue": 2, "mark_id": 1}, stats=True)
> 
>      def test_add_delete_rules(self):
>          self.pmd_output.execute_cmd("stop")
> @@ -2157,10 +2162,22 @@ class TestCVLFdir(TestCase):
>          rules = list()
>          pkt_pattern =
> 'Ether(dst="00:11:22:33:44:55")/IP(src="192.168.100.20",dst="192.168.%d.%d
> ")/Raw("x" * 80)'
>          pkts2 = list()
> -        for i in range(60):
> -            for j in range(256):
> -                rules.append(rule_pattern % (i, j))
> -                pkts2.append(pkt_pattern % (i, j))
> +        if self.nic in ['columbiaville_100g']:
> +            rule_li = list(map(str, range(15360)))
> +            pkts =
> ['Ether(dst="00:11:22:33:44:55")/IP(src="192.168.100.20",dst="192.168.0.0")/
> Raw("x" * 80)',
> +
> 'Ether(dst="00:11:22:33:44:55")/IP(src="192.168.100.20",dst="192.168.59.255"
> )/Raw("x" * 80)']
> +            for i in range(60):
> +                for j in range(256):
> +                    rules.append(rule_pattern % (i, j))
> +                    pkts2.append(pkt_pattern % (i, j))
> +        else:
> +            rule_li = list(map(str, range(14848)))
> +            pkts =
> ['Ether(dst="00:11:22:33:44:55")/IP(src="192.168.100.20",dst="192.168.0.0")/
> Raw("x" * 80)',
> +
> 'Ether(dst="00:11:22:33:44:55")/IP(src="192.168.100.20",dst="192.168.57.255"
> )/Raw("x" * 80)']
> +            for i in range(58):
> +                for j in range(256):
> +                    rules.append(rule_pattern % (i, j))
> +                    pkts2.append(pkt_pattern % (i, j))
>          cmd_path = '/tmp/test_max_rules'
>          cmd_li = map(lambda x: x + os.linesep, rules)
>          with open(cmd_path, 'w') as f:
> @@ -2174,15 +2191,12 @@ class TestCVLFdir(TestCase):
>                                                      self.portMask, 64, 64, cmd_path),
>                                                  eal_param="-w %s,flow-mark-support=1 -
> w %s,flow-mark-support=1" % (
>                                                      self.pci0, self.pci1), socket=self.ports_socket)
> -            self.verify('Failed to create file' not in out, "create some rule
> failed: %s" % out)
> +            self.verify('Failed to create flow' not in out, "create
> + some rule failed")
>              self.config_testpmd()
>              self.pmd_output.execute_cmd('start')
>              rule = "flow create 0 ingress pattern eth / ipv4 src is 192.168.100.20 dst
> is 192.168.60.0 / end actions queue index 1 / mark / end"
>              self.create_fdir_rule(rule=rule, check_stats=False, msg='Failed to
> create flow', validate=False)
> -            rule_li = list(map(str, range(15360)))
>              self.check_fdir_rule(port_id=0, stats=True, rule_list=rule_li)
> -            pkts =
> ['Ether(dst="00:11:22:33:44:55")/IP(src="192.168.100.20",dst="192.168.0.0")/
> Raw("x" * 80)',
> -
> 'Ether(dst="00:11:22:33:44:55")/IP(src="192.168.100.20",dst="192.168.59.255"
> )/Raw("x" * 80)']
>              out1 = self.send_pkts_getouput(pkts=pkts, port_id=0, count=1)
>              rfc.check_mark(out1, pkt_num=2, check_param={"port_id": 0,
> "queue": 1, "mark_id": 0}, stats=True)
>              self.pmd_output.execute_cmd("flow flush 0")
> --
> 1.8.3.1


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

* Re: [dts] [PATCH V3]tests/TestSuite_cvl_fdir: add cvl 25g max rules support
  2020-06-12  2:46 ` Xiao, QimaiX
@ 2020-06-19  8:21   ` Peng, Yuan
  0 siblings, 0 replies; 3+ messages in thread
From: Peng, Yuan @ 2020-06-19  8:21 UTC (permalink / raw)
  To: Xiao, QimaiX, dts

Comment inline

-----Original Message-----
From: dts <dts-bounces@dpdk.org> On Behalf Of Xiao, QimaiX
Sent: Friday, June 12, 2020 10:47 AM
To: dts@dpdk.org
Subject: Re: [dts] [PATCH V3]tests/TestSuite_cvl_fdir: add cvl 25g max rules support

Tested-by: Xiao, QimaiX <qimaix.xiao@intel.com>

Regards,
Xiao Qimai

> -----Original Message-----
> From: Xiao, QimaiX <qimaix.xiao@intel.com>
> Sent: Friday, June 12, 2020 10:38 AM
> To: dts@dpdk.org
> Cc: Xiao, QimaiX <qimaix.xiao@intel.com>
> Subject: [dts][PATCH V3]tests/TestSuite_cvl_fdir: add cvl 25g max 
> rules support
> 
> * 1. cvl25g only support 14848 fdir rules
> * 2. fix max rules case failed caused python crash issue
> * 3. add steps for case test_port_start_stop_reset according to 
> testplan updation
> 
> Signed-off-by: Xiao Qimai <qimaix.xiao@intel.com>
> ---
>  tests/TestSuite_cvl_fdir.py | 32 +++++++++++++++++++++++---------
>  1 file changed, 23 insertions(+), 9 deletions(-)
> 
> diff --git a/tests/TestSuite_cvl_fdir.py b/tests/TestSuite_cvl_fdir.py 
> index
> 12d9ad8..f2252d0 100644
> --- a/tests/TestSuite_cvl_fdir.py
> +++ b/tests/TestSuite_cvl_fdir.py
> @@ -1722,7 +1722,7 @@ class TestCVLFdir(TestCase):
> 
>      def test_void_action(self):
>          rule = "flow create 0 ingress pattern eth dst is 
> 00:11:22:33:44:55 / ipv4 src is 192.168.0.20 dst is 192.168.0.21 ttl is 2 tos is 4 / end actions end"
> -        self.create_fdir_rule(rule, check_stats=False, msg="Invalid input action:
> Invalid argument")
> +        self.create_fdir_rule(rule, check_stats=False)
>          self.check_fdir_rule(stats=False)
> 
>      def _test_unsupported_action(self):
> @@ -2068,6 +2068,11 @@ class TestCVLFdir(TestCase):
>          self.check_fdir_rule(port_id=0, stats=True, rule_list=rule_li)
>          out2 = self.send_pkts_getouput(pkts=pkt, port_id=0, count=1)
>          rfc.check_mark(out2, pkt_num=1, check_param={"port_id": 0, "queue":
> 1, "mark_id": 0}, stats=True)
> +        rule2 = 'flow create 0 ingress pattern eth dst is 
> + 00:11:22:33:44:55 / ipv4
> src is 192.168.0.22 dst is 192.168.0.23 / end actions queue index 2 / 
> mark id 1 / end'
> +        rule_li2 = self.create_fdir_rule(rule=rule2, check_stats=True)
> +        self.check_fdir_rule(rule_list=rule_li+rule_li2)
> +        out3 =
> self.send_pkts_getouput(pkts='Ether(dst="00:11:22:33:44:55")/IP(src="1
> 92.1
> 68.0.22",dst="192.168.0.23") / Raw("x" * 80)', port_id=0, count=1)
> +        rfc.check_mark(out3, pkt_num=1, check_param={"port_id": 0,
> + "queue": 2, "mark_id": 1}, stats=True)
> 
>      def test_add_delete_rules(self):
>          self.pmd_output.execute_cmd("stop")
> @@ -2157,10 +2162,22 @@ class TestCVLFdir(TestCase):
>          rules = list()
>          pkt_pattern =
> 'Ether(dst="00:11:22:33:44:55")/IP(src="192.168.100.20",dst="192.168.%
> d.%d
> ")/Raw("x" * 80)'
>          pkts2 = list()
> -        for i in range(60):
> -            for j in range(256):
> -                rules.append(rule_pattern % (i, j))
> -                pkts2.append(pkt_pattern % (i, j))
> +        if self.nic in ['columbiaville_100g']:
> +            rule_li = list(map(str, range(15360)))
> +            pkts =
> ['Ether(dst="00:11:22:33:44:55")/IP(src="192.168.100.20",dst="192.168.
> 0.0")/
> Raw("x" * 80)',
> +
> 'Ether(dst="00:11:22:33:44:55")/IP(src="192.168.100.20",dst="192.168.59.255"
> )/Raw("x" * 80)']
> +            for i in range(60):
> +                for j in range(256):
> +                    rules.append(rule_pattern % (i, j))
> +                    pkts2.append(pkt_pattern % (i, j))
> +        else:
[Yuan]: I think it should identify the NIC type.
> +            rule_li = list(map(str, range(14848)))
> +            pkts =
> ['Ether(dst="00:11:22:33:44:55")/IP(src="192.168.100.20",dst="192.168.
> 0.0")/
> Raw("x" * 80)',
> +
> 'Ether(dst="00:11:22:33:44:55")/IP(src="192.168.100.20",dst="192.168.57.255"
> )/Raw("x" * 80)']
> +            for i in range(58):
> +                for j in range(256):
> +                    rules.append(rule_pattern % (i, j))
> +                    pkts2.append(pkt_pattern % (i, j))
>          cmd_path = '/tmp/test_max_rules'
>          cmd_li = map(lambda x: x + os.linesep, rules)
>          with open(cmd_path, 'w') as f:
> @@ -2174,15 +2191,12 @@ class TestCVLFdir(TestCase):
>                                                      self.portMask, 64, 64, cmd_path),
>                                                  eal_param="-w 
> %s,flow-mark-support=1 - w %s,flow-mark-support=1" % (
>                                                      self.pci0, self.pci1), socket=self.ports_socket)
> -            self.verify('Failed to create file' not in out, "create some rule
> failed: %s" % out)
> +            self.verify('Failed to create flow' not in out, "create 
> + some rule failed")
>              self.config_testpmd()
>              self.pmd_output.execute_cmd('start')
>              rule = "flow create 0 ingress pattern eth / ipv4 src is 
> 192.168.100.20 dst is 192.168.60.0 / end actions queue index 1 / mark / end"
>              self.create_fdir_rule(rule=rule, check_stats=False, 
> msg='Failed to create flow', validate=False)
> -            rule_li = list(map(str, range(15360)))
>              self.check_fdir_rule(port_id=0, stats=True, rule_list=rule_li)
> -            pkts =
> ['Ether(dst="00:11:22:33:44:55")/IP(src="192.168.100.20",dst="192.168.
> 0.0")/
> Raw("x" * 80)',
> -
> 'Ether(dst="00:11:22:33:44:55")/IP(src="192.168.100.20",dst="192.168.59.255"
> )/Raw("x" * 80)']
>              out1 = self.send_pkts_getouput(pkts=pkts, port_id=0, count=1)
>              rfc.check_mark(out1, pkt_num=2, check_param={"port_id": 
> 0,
> "queue": 1, "mark_id": 0}, stats=True)
>              self.pmd_output.execute_cmd("flow flush 0")
> --
> 1.8.3.1


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

end of thread, back to index

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-06-12  2:38 [dts] [PATCH V3]tests/TestSuite_cvl_fdir: add cvl 25g max rules support Xiao Qimai
2020-06-12  2:46 ` Xiao, QimaiX
2020-06-19  8:21   ` Peng, Yuan

test suite reviews and discussions

Archives are clonable:
	git clone --mirror http://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/ http://inbox.dpdk.org/dts \
		dts@dpdk.org
	public-inbox-index dts


Newsgroup available over NNTP:
	nntp://inbox.dpdk.org/inbox.dpdk.dts


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