* [dts] [PATCH V2]cvl_fdir: add cvl 25g max rules support
@ 2020-06-11 4:42 Xiao Qimai
2020-06-11 4:42 ` [dts] [PATCH V1]cvl_fdir_test_plan.rst: update cvl 25g max rules and add steps in port_stop_start_reset Xiao Qimai
2020-06-11 4:50 ` [dts] [PATCH V2]cvl_fdir: add cvl 25g max rules support Xiao, QimaiX
0 siblings, 2 replies; 4+ messages in thread
From: Xiao Qimai @ 2020-06-11 4:42 UTC (permalink / raw)
To: dts; +Cc: Xiao Qimai
*. add steps for port_stop_start_reset
*. update max rules for cvl 25g
*. fix typo
Signed-off-by: Xiao Qimai <qimaix.xiao@intel.com>
---
tests/TestSuite_cvl_fdir.py | 34 +++++++++++++++++++++++-----------
1 file changed, 23 insertions(+), 11 deletions(-)
diff --git a/tests/TestSuite_cvl_fdir.py b/tests/TestSuite_cvl_fdir.py
index 70aa5e3..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")
@@ -2202,8 +2216,6 @@ class TestCVLFdir(TestCase):
self._rte_flow_validate(vectors_ipv4_udp)
def test_mac_ipv4_tcp(self):
- import pudb
- pudb.set_trace()
self._rte_flow_validate(vectors_ipv4_tcp)
def test_mac_ipv4_sctp(self):
--
1.8.3.1
^ permalink raw reply [flat|nested] 4+ messages in thread
* [dts] [PATCH V1]cvl_fdir_test_plan.rst: update cvl 25g max rules and add steps in port_stop_start_reset
2020-06-11 4:42 [dts] [PATCH V2]cvl_fdir: add cvl 25g max rules support Xiao Qimai
@ 2020-06-11 4:42 ` Xiao Qimai
2020-06-11 4:50 ` Xiao, QimaiX
2020-06-11 4:50 ` [dts] [PATCH V2]cvl_fdir: add cvl 25g max rules support Xiao, QimaiX
1 sibling, 1 reply; 4+ messages in thread
From: Xiao Qimai @ 2020-06-11 4:42 UTC (permalink / raw)
To: dts; +Cc: Xiao Qimai
Signed-off-by: Xiao Qimai <qimaix.xiao@intel.com>
---
test_plans/cvl_fdir_test_plan.rst | 13 +++++++++++++
1 file changed, 13 insertions(+)
diff --git a/test_plans/cvl_fdir_test_plan.rst b/test_plans/cvl_fdir_test_plan.rst
index 12e5041..254bb53 100644
--- a/test_plans/cvl_fdir_test_plan.rst
+++ b/test_plans/cvl_fdir_test_plan.rst
@@ -2463,6 +2463,16 @@ Subcase 1: port stop/port start/port reset
8. verify matched packet can be still redirected to queue 1 with FDIR matched ID=0x0.
+9. add a new rule::
+
+ 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
+
+10. list the rule and send matched packet::
+
+ sendp([Ether(dst="00:11:22:33:44:55")/IP(src="192.168.0.22",dst="192.168.0.23") / Raw('x' * 80)],iface="enp175s0f0")
+
+ check the packet are redirected to queue 2 with FDIR matched ID=0x1
+
Subcase 2: add/delete rules
---------------------------
@@ -2579,6 +2589,9 @@ Subcase 3: delete rules
Subcase 4: max rules
--------------------
+This case is designed based on 2*100G NIC.
+If 4*25 NIC, each PF port has 512 fdir rules guaranteed.
+So there can be created 14848 fdir rules on 1 PF port.
1. create 15360 rules on port 0::
--
1.8.3.1
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [dts] [PATCH V2]cvl_fdir: add cvl 25g max rules support
2020-06-11 4:42 [dts] [PATCH V2]cvl_fdir: add cvl 25g max rules support Xiao Qimai
2020-06-11 4:42 ` [dts] [PATCH V1]cvl_fdir_test_plan.rst: update cvl 25g max rules and add steps in port_stop_start_reset Xiao Qimai
@ 2020-06-11 4:50 ` Xiao, QimaiX
1 sibling, 0 replies; 4+ messages in thread
From: Xiao, QimaiX @ 2020-06-11 4:50 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: Thursday, June 11, 2020 12:42 PM
> To: dts@dpdk.org
> Cc: Xiao, QimaiX <qimaix.xiao@intel.com>
> Subject: [dts][PATCH V2]cvl_fdir: add cvl 25g max rules support
>
> *. add steps for port_stop_start_reset
> *. update max rules for cvl 25g
> *. fix typo
>
> Signed-off-by: Xiao Qimai <qimaix.xiao@intel.com>
> ---
> tests/TestSuite_cvl_fdir.py | 34 +++++++++++++++++++++++-----------
> 1 file changed, 23 insertions(+), 11 deletions(-)
>
> diff --git a/tests/TestSuite_cvl_fdir.py b/tests/TestSuite_cvl_fdir.py index
> 70aa5e3..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") @@ -2202,8 +2216,6
> @@ class TestCVLFdir(TestCase):
> self._rte_flow_validate(vectors_ipv4_udp)
>
> def test_mac_ipv4_tcp(self):
> - import pudb
> - pudb.set_trace()
> self._rte_flow_validate(vectors_ipv4_tcp)
>
> def test_mac_ipv4_sctp(self):
> --
> 1.8.3.1
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [dts] [PATCH V1]cvl_fdir_test_plan.rst: update cvl 25g max rules and add steps in port_stop_start_reset
2020-06-11 4:42 ` [dts] [PATCH V1]cvl_fdir_test_plan.rst: update cvl 25g max rules and add steps in port_stop_start_reset Xiao Qimai
@ 2020-06-11 4:50 ` Xiao, QimaiX
0 siblings, 0 replies; 4+ messages in thread
From: Xiao, QimaiX @ 2020-06-11 4:50 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: Thursday, June 11, 2020 12:42 PM
> To: dts@dpdk.org
> Cc: Xiao, QimaiX <qimaix.xiao@intel.com>
> Subject: [dts][PATCH V1]cvl_fdir_test_plan.rst: update cvl 25g max rules and
> add steps in port_stop_start_reset
>
> Signed-off-by: Xiao Qimai <qimaix.xiao@intel.com>
> ---
> test_plans/cvl_fdir_test_plan.rst | 13 +++++++++++++
> 1 file changed, 13 insertions(+)
>
> diff --git a/test_plans/cvl_fdir_test_plan.rst
> b/test_plans/cvl_fdir_test_plan.rst
> index 12e5041..254bb53 100644
> --- a/test_plans/cvl_fdir_test_plan.rst
> +++ b/test_plans/cvl_fdir_test_plan.rst
> @@ -2463,6 +2463,16 @@ Subcase 1: port stop/port start/port reset
>
> 8. verify matched packet can be still redirected to queue 1 with FDIR
> matched ID=0x0.
>
> +9. add a new rule::
> +
> + 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
> +
> +10. list the rule and send matched packet::
> +
> +
> sendp([Ether(dst="00:11:22:33:44:55")/IP(src="192.168.0.22",dst="192.168.0.
> 23") / Raw('x' * 80)],iface="enp175s0f0")
> +
> + check the packet are redirected to queue 2 with FDIR matched ID=0x1
> +
> Subcase 2: add/delete rules
> ---------------------------
>
> @@ -2579,6 +2589,9 @@ Subcase 3: delete rules
>
> Subcase 4: max rules
> --------------------
> +This case is designed based on 2*100G NIC.
> +If 4*25 NIC, each PF port has 512 fdir rules guaranteed.
> +So there can be created 14848 fdir rules on 1 PF port.
>
> 1. create 15360 rules on port 0::
>
> --
> 1.8.3.1
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2020-06-11 4:50 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-06-11 4:42 [dts] [PATCH V2]cvl_fdir: add cvl 25g max rules support Xiao Qimai
2020-06-11 4:42 ` [dts] [PATCH V1]cvl_fdir_test_plan.rst: update cvl 25g max rules and add steps in port_stop_start_reset Xiao Qimai
2020-06-11 4:50 ` Xiao, QimaiX
2020-06-11 4:50 ` [dts] [PATCH V2]cvl_fdir: add cvl 25g max rules support Xiao, QimaiX
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).