From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id 89258A0350; Mon, 22 Jun 2020 05:06:54 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 765E91C11D; Mon, 22 Jun 2020 05:06:54 +0200 (CEST) Received: from mga02.intel.com (mga02.intel.com [134.134.136.20]) by dpdk.org (Postfix) with ESMTP id 5DC451C113 for ; Mon, 22 Jun 2020 05:06:52 +0200 (CEST) IronPort-SDR: iDwrk27roSLtzja1UCtQ80lE0+LyHtqZ5n1ktAEBhZf3lGkdV6droez5a01jACAPnF1mlGIQj/ icxabR5iSq6Q== X-IronPort-AV: E=McAfee;i="6000,8403,9659"; a="132078066" X-IronPort-AV: E=Sophos;i="5.75,265,1589266800"; d="scan'208";a="132078066" X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga005.jf.intel.com ([10.7.209.41]) by orsmga101.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 21 Jun 2020 20:06:51 -0700 IronPort-SDR: 5oNc11jZVwCGsN5Efw76WSNQP4PvtNVbTy0xIsO9+CUbH9uB9AufcJQ7zgjf6K6q+3MhQoIQqf tMHf5xqrJnmw== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.75,265,1589266800"; d="scan'208";a="451683139" Received: from npg_dpdk_nic_yuanpeng_tester62.sh.intel.com ([10.67.119.61]) by orsmga005.jf.intel.com with ESMTP; 21 Jun 2020 20:06:50 -0700 From: Peng Yuan To: dts@dpdk.org Cc: pengyuan Date: Mon, 22 Jun 2020 10:14:04 +0000 Message-Id: <1592820844-18157-1-git-send-email-yuan.peng@intel.com> X-Mailer: git-send-email 2.7.4 Subject: [dts] [PATCH v2] add a max number case to iavf_fdir X-BeenThere: dts@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: test suite reviews and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dts-bounces@dpdk.org Sender: "dts" From: pengyuan Add a max number case to iavf_fdir_test_plan.rst Add a max number case to TestSuite_iavf_fdir.py Signed-off-by: pengyuan diff --git a/test_plans/iavf_fdir_test_plan.rst b/test_plans/iavf_fdir_test_plan.rst index a201b9d..9cf5efe 100644 --- a/test_plans/iavf_fdir_test_plan.rst +++ b/test_plans/iavf_fdir_test_plan.rst @@ -2689,7 +2689,7 @@ Test case: Max number ===================== All the max number cases are designed based on 2*100G NIC. If the hardware is 4*25G NIC, the guaranteed rule number of PF is 512. -So in subcase 3, there can be created at most 14848 rules on 1pf and 2vfs. +So in subcase 3 and subcase 4, there can be created at most 14848 rules on 1pf and 2vfs. Subcase 1: 14336 rules on 1 vf ------------------------------ @@ -2764,11 +2764,13 @@ Subcase 2: 14336 rules on 2 vfs of 2pfs 5. verify matched packet received without FDIR matched ID. -Subcase 3: 15360 rules on 1pf and 2vfs --------------------------------------- +Subcase 3: 1025 rules on 1pf and 14335 rules on 2vfs +---------------------------------------------------- each pf can create 1024 rules at least in 2 ports card. +each pf can create 512 rules at least in 4 ports card. there are 14k rules shared by pfs and vfs. -so 1 pf and 2 vfs can create 15360 rules at most. +so 1 pf and 2 vfs can create 15360 rules at most on 2 ports card. +1 pf and 2 vfs can create 14848 rules at most on 4 ports card. 1. create 1025 rules on pf0:: @@ -2830,7 +2832,79 @@ so 1 pf and 2 vfs can create 15360 rules at most. 8. verify matched packet received without FDIR matched ID. -Subcase 4: 128 profiles +Subcase 4: 15360 rules on 1pf and 0 rules on 2vfs +------------------------------------------------- +each pf can create 1024 rules at least in 2 ports card. +each pf can create 512 rules at least in 4 ports card. +there are 14k rules shared by pfs and vfs. +so 1 pf and 2 vfs can create 15360 rules at most on 2 ports card. +1 pf and 2 vfs can create 14848 rules at most on 4 ports card. +so if create 15360/14848 rules on 1 pf, there can't create rule on vf successfully. + +1. create 15360 rules on pf0:: + + ethtool -N enp134s0f0 flow-type tcp4 src-ip 192.168.0.0 dst-ip 192.168.100.2 src-port 32 dst-port 33 action 8 + ethtool -N enp134s0f0 flow-type tcp4 src-ip 192.168.0.1 dst-ip 192.168.100.2 src-port 32 dst-port 33 action 8 + ...... + ethtool -N enp134s0f0 flow-type tcp4 src-ip 192.168.57.255 dst-ip 192.168.100.2 src-port 32 dst-port 33 action 8 + + all the rules can be created successfully:: + + Added rule with ID + +2. failed to create one more rule on pf0:: + + ethtool -N enp134s0f0 flow-type tcp4 src-ip 192.168.58.0 dst-ip 192.168.100.2 src-port 32 dst-port 33 action 8 + +3. start testpmd on vf00 and vf10:: + + ./testpmd -c 0xf -n 6 -w 86:01.0 -w 86:11.0 --file-prefix=vf00 -- -i --rxq=4 --txq=4 + + create 1 rule on vf00:: + + flow create 0 ingress pattern eth / ipv4 src is 192.168.0.20 dst is 192.168.0.0 / end actions queue index 1 / mark / end + + failed to create the rule, check there is no rule listed. + + create 1 rule on vf10:: + + flow create 1 ingress pattern eth / ipv4 src is 192.168.0.20 dst is 192.168.0.0 / end actions queue index 1 / mark / end + + failed to create the rule, check there is no rule listed. + +4. delete 1 rule on pf0:: + + ethtool -N enp134s0f0 delete + +5. create 1 rule on vf00:: + + flow create 0 ingress pattern eth / ipv4 src is 192.168.0.20 dst is 192.168.55.254 / end actions queue index 1 / mark / end + + the rule can be created successfully. + + create 1 rule on vf10:: + + flow create 1 ingress pattern eth / ipv4 src is 192.168.0.20 dst is 192.168.0.0 / end actions queue index 1 / mark / end + + failed to create the rule, check there is no rule listed. + +6. send matched packet to vf00, it can be redirected to queue 1 with FDIR matched ID=0x0. + send matched packet to vf10, it is received without FDIR matched ID. + +7. delete 1 more rule on pf0:: + + ethtool -N enp134s0f0 delete + +8. create 1 rule on vf10:: + + flow create 1 ingress pattern eth / ipv4 src is 192.168.0.20 dst is 192.168.0.0 / end actions queue index 1 / mark / end + + the rule can be created successfully. + +9. send matched packet to vf00, it can be redirected to queue 1 with FDIR matched ID=0x0. + send matched packet to vf10, it can be redirected to queue 1 with FDIR matched ID=0x0. + +Subcase 5: 128 profiles ----------------------- 1. create 16 vfs on pf0:: diff --git a/tests/TestSuite_iavf_fdir.py b/tests/TestSuite_iavf_fdir.py index 2ea411c..f1adc0f 100644 --- a/tests/TestSuite_iavf_fdir.py +++ b/tests/TestSuite_iavf_fdir.py @@ -1381,7 +1381,7 @@ class TestIAVFFdir(TestCase): """ remove and reload the ice driver """ - self.dut.send_expect("rmmod ice", "# ", 20) + self.dut.send_expect("rmmod ice", "# ", 40) ice_driver_file_location = self.suite_config["ice_driver_file_location"] self.dut.send_expect("insmod %s" % ice_driver_file_location, "# ") self.dut.send_expect("ifconfig %s up" % self.tester_iface0, "# ", 15) @@ -2320,6 +2320,13 @@ class TestIAVFFdir(TestCase): out_14336 = self.send_pkts_getouput(pkts=pkt_14336, pf_id=0) rfc.check_iavf_fdir_mark(out_14336, pkt_num=1, check_param={"port_id": 0, "queue": 5}, stats=False) + # delete one rule of vf0 + self.dut.send_expect("flow destroy 0 rule 0", "testpmd> ", timeout=200) + self.create_fdir_rule(rule_0_vf1, check_stats=True) + pkt_0_vf1 = 'Ether(dst="00:11:22:33:44:66")/IP(src="192.168.0.20",dst="192.168.56.0")/Raw("x" * 80)' + out_0_vf1 = self.send_pkts_getouput(pkts=pkt_0_vf1, pf_id=0) + rfc.check_iavf_fdir_mark(out_0_vf1, pkt_num=1, check_param={"port_id": 1, "mark_id": 0, "queue": 5}, stats=True) + # flush all the rules self.dut.send_expect("flow flush 0", "testpmd> ", timeout=200) self.check_fdir_rule(port_id=0, stats=False) @@ -2329,11 +2336,9 @@ class TestIAVFFdir(TestCase): rfc.check_iavf_fdir_mark(out_14335, pkt_num=1, check_param={"port_id": 0, "mark_id": 0, "queue": 5}, stats=False) self.create_fdir_rule(rule_14336_vf0, check_stats=True) - self.create_fdir_rule(rule_0_vf1, check_stats=True) out_14336 = self.send_pkts_getouput(pkts=pkt_14336, pf_id=0) rfc.check_iavf_fdir_mark(out_14336, pkt_num=1, check_param={"port_id": 0, "mark_id": 0, "queue": 5}, stats=True) - def test_maxnum_14336rules_2vf(self): """ vfs share 14336 rules table @@ -2424,40 +2429,38 @@ class TestIAVFFdir(TestCase): self.dut.kill_all() self.session_secondary = self.dut.new_session() - #create 1025 rules on pf0 - src_file = 'iavf_fdir_15360_kernel_rules' + #create 1025 rules on pf1 + src_file = 'create_1025_kernel_rules' flows=open(self.src_file_dir + src_file,mode='w') count=0 if self.nic in ["columbiaville_100g"]: for i in range(4): for j in range(256): - flows.write('ethtool -N enp134s0f1 flow-type tcp4 src-ip 192.168.%d.%d dst-ip 192.168.100.2 src-port 32 dst-port 33 action 8 \n'%(i,j)) + flows.write('ethtool -N %s flow-type tcp4 src-ip 192.168.%d.%d dst-ip 192.168.100.2 src-port 32 dst-port 33 action 8 \n' % (self.pf1_intf,i,j)) count=count+1 - flows.write('ethtool -N enp134s0f1 flow-type tcp4 src-ip 192.168.100.0 dst-ip 192.168.100.2 src-port 32 dst-port 33 action 8 \n') + flows.write('ethtool -N %s flow-type tcp4 src-ip 192.168.100.0 dst-ip 192.168.100.2 src-port 32 dst-port 33 action 8 \n' % self.pf1_intf) count=count+1 flows.close() self.verify(count == 1025, "failed to create 1025 fdir rules on pf.") elif self.nic in ["columbiaville_25g"]: for i in range(2): for j in range(256): - flows.write('ethtool -N enp134s0f1 flow-type tcp4 src-ip 192.168.%d.%d dst-ip 192.168.100.2 src-port 32 dst-port 33 action 8 \n'%(i,j)) + flows.write('ethtool -N %s flow-type tcp4 src-ip 192.168.%d.%d dst-ip 192.168.100.2 src-port 32 dst-port 33 action 8 \n' % (self.pf1_intf,i,j)) count=count+1 - flows.write('ethtool -N enp134s0f1 flow-type tcp4 src-ip 192.168.100.0 dst-ip 192.168.100.2 src-port 32 dst-port 33 action 8 \n') + flows.write('ethtool -N %s flow-type tcp4 src-ip 192.168.100.0 dst-ip 192.168.100.2 src-port 32 dst-port 33 action 8 \n' % self.pf1_intf) count=count+1 flows.close() self.verify(count == 513, "failed to create 1025 fdir rules on pf.") self.dut.session.copy_file_to(self.src_file_dir + src_file, self.dut_file_dir) - # create 1025 rules on pf0 - fkr = open(self.dut_file_dir + "iavf_fdir_15360_kernel_rules", "r+") + # create 1025 rules on pf1 + fkr = open(self.dut_file_dir + src_file, "r+") kernel_rules = fkr.read() fkr.close() - self.dut.send_expect(kernel_rules, "# ", 300) - # write the kernel rules result to file - fkw = open("1025_kernel_rules_result.txt", "w") - fkw.write(self.dut.send_expect("ethtool -n %s" % self.pf1_intf, "# ", 300)) + self.dut.send_expect(kernel_rules, "# ") + time.sleep(15) #create 1 rule on vf00, and 14334 rules on vf01 - src_file_vf = 'iavf_fdir_15360_vf_rules' + src_file_vf = 'create_14335_rules_on_2vfs' flows = open(self.src_file_dir + src_file_vf, mode='w') flows.write('flow create 0 ingress pattern eth / ipv4 src is 192.168.0.20 dst is 192.168.0.0 / end actions queue index 5 / mark / end \n') count=1 @@ -2494,7 +2497,6 @@ class TestIAVFFdir(TestCase): # delete a rule on pf0 self.session_secondary.send_expect("ethtool -N %s delete 14847" % self.pf1_intf, "# ") - time.sleep(3) # then can create one more rule on vf01 self.create_fdir_rule(rule_14334_vf1, check_stats=True) @@ -2511,6 +2513,88 @@ class TestIAVFFdir(TestCase): self.dut.send_expect("quit", "# ") self.dut.close_session(self.session_secondary) self.re_load_ice_driver() + self.setup_2pf_4vf_env() + + def test_maxnum_15360rules_1pf_0_rules_vf(self): + """ + 2*100G NIC, each pf can create 1024 rules at least, vfs share 14336 rules table + 4*25G NIC, each pf can create 512 rules at least, vfs share 14336 rules table + so if 2*25G NIC, max number is 14848 on 1pf and vfs. + create 15360/14848 rules on pf1, check failed to create rule on vf00 and vf10 + """ + self.dut.kill_all() + self.session_secondary = self.dut.new_session() + + #create 15360 rules on pf1 + src_file = 'create_15360_kernel_rules' + flows=open(self.src_file_dir + src_file,mode='w') + count=0 + if self.nic in ["columbiaville_100g"]: + for i in range(60): + for j in range(256): + flows.write('ethtool -N %s flow-type tcp4 src-ip 192.168.%d.%d dst-ip 192.168.100.2 src-port 32 dst-port 33 action 8 \n' % (self.pf1_intf,i,j)) + count=count+1 + flows.close() + self.verify(count == 15360, "failed to create 15360 fdir rules on pf.") + elif self.nic in ["columbiaville_25g"]: + for i in range(58): + for j in range(256): + flows.write('ethtool -N %s flow-type tcp4 src-ip 192.168.%d.%d dst-ip 192.168.100.2 src-port 32 dst-port 33 action 8 \n' % (self.pf1_intf,i,j)) + count=count+1 + flows.close() + self.verify(count == 14848, "failed to create 14848 fdir rules on pf.") + self.dut.session.copy_file_to(self.src_file_dir + src_file, self.dut_file_dir) + # create 15360 rules on pf1 + fkr = open(self.dut_file_dir + src_file, "r+") + kernel_rules = fkr.read() + fkr.close() + self.dut.send_expect(kernel_rules, "# ") + time.sleep(200) + # failed to create 1 more rule on pf1 + self.dut.send_expect("ethtool -N %s flow-type tcp4 src-ip 192.168.100.0 dst-ip 192.168.100.2 src-port 32 dst-port 33 action 8" % self.pf1_intf, "Cannot insert RX class rule: No space left on device") + # start testpmd with creating rules in commandline + eal_param = "-c f -n 6 -w %s -w %s" % (self.sriov_vfs_pf0[0].pci,self.sriov_vfs_pf1[0].pci) + command = "./%s/app/testpmd %s -- -i %s" % (self.dut.target, eal_param, "--rxq=16 --txq=16") + self.dut.send_expect(command, "testpmd> ", 20) + + self.config_testpmd() + rule_0_vf00 = "flow create 0 ingress pattern eth / ipv4 src is 192.168.0.20 dst is 192.168.56.0 / end actions queue index 5 / mark / end" + pkt_0_vf00 = 'Ether(dst="00:11:22:33:44:55")/IP(src="192.168.0.20",dst="192.168.56.0")/Raw("x" * 80)' + rule_0_vf10 = "flow create 1 ingress pattern eth / ipv4 src is 192.168.0.20 dst is 192.168.56.0 / end actions queue index 5 / mark / end" + pkt_0_vf10 = 'Ether(dst="00:11:22:33:44:77")/IP(src="192.168.0.20",dst="192.168.56.0")/Raw("x" * 80)' + + #failed to create rule on vf00 and vf10 + self.create_fdir_rule(rule_0_vf00, check_stats=False) + self.create_fdir_rule(rule_0_vf10, check_stats=False) + self.check_rule_number(port_id=0, num=0) + self.check_rule_number(port_id=1, num=0) + + # delete a rule on pf1 + self.session_secondary.send_expect("ethtool -N %s delete 14847" % self.pf1_intf, "# ") + + # then can create one rule on vf00 + self.create_fdir_rule(rule_0_vf00, check_stats=True) + self.create_fdir_rule(rule_0_vf10, check_stats=False) + self.check_rule_number(port_id=0, num=1) + self.check_rule_number(port_id=1, num=0) + + # delete a rule on pf1 + self.session_secondary.send_expect("ethtool -N %s delete 14846" % self.pf1_intf, "# ") + + # then can create one rule on vf10 + self.create_fdir_rule(rule_0_vf10, check_stats=True) + self.check_rule_number(port_id=1, num=1) + + out_0_vf00 = self.send_pkts_getouput(pkts=pkt_0_vf00, pf_id=0) + rfc.check_iavf_fdir_mark(out_0_vf00, pkt_num=1, check_param={"port_id": 0, "mark_id": 0, "queue": 5}, stats=True) + + out_0_vf10 = self.send_pkts_getouput(pkts=pkt_0_vf10, pf_id=1) + rfc.check_iavf_fdir_mark(out_0_vf10, pkt_num=1, check_param={"port_id": 1, "mark_id": 0, "queue": 5}, stats=True) + + self.dut.send_expect("quit", "# ") + self.dut.close_session(self.session_secondary) + self.re_load_ice_driver() + self.setup_2pf_4vf_env() def test_maxnum_128_profiles(self): """ @@ -3270,7 +3354,11 @@ class TestIAVFFdir(TestCase): def tear_down(self): # destroy all flow rule on port 0 self.dut.kill_all() + if getattr(self, 'session_secondary', None): + self.dut.close_session(self.session_secondary) + if getattr(self, 'session_third', None): + self.dut.close_session(self.session_third) def tear_down_all(self): - self.destroy_env() self.dut.kill_all() + self.destroy_env() -- 2.14.3