* [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, other threads:[~2020-06-19 8:21 UTC | newest]
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
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).