Re: [dts] [PATCH V1 2/2] tests/generic_flow_api: delete test_L2_tunnel_filter

Re: [dts] [PATCH V1 2/2] tests/generic_flow_api: delete test_L2_tunnel_filter

[Peng, Yuan]

Acked-by Peng, Yuan <yuan.peng@intel.com>

-----Original Message-----
From: dts <dts-bounces@dpdk.org> On Behalf Of Yan Xia
Sent: Tuesday, October 26, 2021 10:41 PM
To: dts@dpdk.org
Cc: Xia, YanX <yanx.xia@intel.com>
Subject: [dts] [PATCH V1 2/2] tests/generic_flow_api: delete test_L2_tunnel_filter

Re: [dts] [PATCH V1 2/2] tests/generic_flow_api: delete test_L2_tunnel_filter

[Tu, Lijuan]

> -----Original Message-----
> From: dts <dts-bounces@dpdk.org> On Behalf Of Peng, Yuan
> Sent: 2021年10月26日 15:31
> To: Xia, YanX <yanx.xia@intel.com>; dts@dpdk.org
> Cc: Xia, YanX <yanx.xia@intel.com>; Peng, Yuan <yuan.peng@intel.com>
> Subject: Re: [dts] [PATCH V1 2/2] tests/generic_flow_api: delete
> test_L2_tunnel_filter
> 
> Acked-by Peng, Yuan <yuan.peng@intel.com>
> 
> -----Original Message-----
> From: dts <dts-bounces@dpdk.org> On Behalf Of Yan Xia
> Sent: Tuesday, October 26, 2021 10:41 PM
> To: dts@dpdk.org
> Cc: Xia, YanX <yanx.xia@intel.com>
> Subject: [dts] [PATCH V1 2/2] tests/generic_flow_api: delete
> test_L2_tunnel_filter

Applied

[dts] [PATCH V1 0/2] *** delete test_L2_tunnel_filter ***

[Yan Xia]

*** The case only support sagepond and sageville, but these two types NIC
have been removed, so delete the case after confirmed with Peng,Yuan.***

Yan Xia (2):
  test_plans/generic_flow_api: delete test_L2_tunnel_filter
  tests/generic_flow_api: delete test_L2_tunnel_filter

 test_plans/generic_flow_api_test_plan.rst | 61 -----------------------
 tests/TestSuite_generic_flow_api.py       | 47 -----------------
 2 files changed, 108 deletions(-)

-- 
2.33.1

[dts] [PATCH V1 1/2] test_plans/generic_flow_api: delete test_L2_tunnel_filter

[Yan Xia]

The case only support sagepond and sageville, but these two types NIC
have been removed, so delete the case after confirmed with Peng,Yuan.

Signed-off-by: Yan Xia <yanx.xia@intel.com>
---
 test_plans/generic_flow_api_test_plan.rst | 61 -----------------------
 1 file changed, 61 deletions(-)

diff --git a/test_plans/generic_flow_api_test_plan.rst b/test_plans/generic_flow_api_test_plan.rst
index 51ceb474..a07b809a 100644
--- a/test_plans/generic_flow_api_test_plan.rst
+++ b/test_plans/generic_flow_api_test_plan.rst
@@ -803,67 +803,6 @@ Test case: IXGBE ethertype
 
     testpmd> flow list 0
 
-
-Test case: IXGBE L2-tunnel(supported by x552 and x550)
-======================================================
-
-   Prerequisites:
-
-   add two vfs on dpdk pf, then bind the vfs to vfio-pci::
-
-    echo 2 >/sys/bus/pci/devices/0000:05:00.0/max_vfs
-    ./usertools/dpdk-devbind.py -b vfio-pci 05:02.0 05:02.1
-
-1. Launch the app ``testpmd`` with the following arguments::
-
-    ./x86_64-native-linuxapp-gcc/app/testpmd -c 1ffff -n 4 -a 05:00.0 --file-prefix=pf --socket-mem=1024,1024 -- -i --rxq=16 --txq=16 --disable-rss
-    testpmd> set fwd rxonly
-    testpmd> set verbose 1
-    testpmd> start
-
-    ./x86_64-native-linuxapp-gcc/app/testpmd -c 1e0000 -n 4 -a 05:02.0 --file-prefix=vf0 --socket-mem=1024,1024 -- -i --rxq=4 --txq=4 --disable-rss
-    testpmd> set fwd rxonly
-    testpmd> set verbose 1
-    testpmd> start
-
-    ./x86_64-native-linuxapp-gcc/app/testpmd -c 1e00000 -n 4 -a 05:02.1 --file-prefix=vf1 --socket-mem=1024,1024 -- -i --rxq=4 --txq=4 --disable-rss
-    testpmd> set fwd rxonly
-    testpmd> set verbose 1
-    testpmd> start
-
-   Enabling ability of parsing E-tag packet, set on pf::
-
-    testpmd> port config 0 l2-tunnel E-tag enable
-
-   Enable E-tag packet forwarding, set on pf::
-
-    testpmd> E-tag set forwarding on port 0
-
-2. create filter rules::
-
-    testpmd> flow create 0 ingress pattern e_tag grp_ecid_b is 0x1309 / end actions queue index 0 / end
-    testpmd> flow create 0 ingress pattern e_tag grp_ecid_b is 0x1308 / end actions queue index 1 / end
-    testpmd> flow create 0 ingress pattern e_tag grp_ecid_b is 0x1307 / end actions queue index 2 / end
-
-3. send packets::
-
-    pkt1 = Ether(dst="00:11:22:33:44:55")/Dot1BR(GRP=0x1, ECIDbase=0x309)/Raw('x' * 20)
-    pkt2 = Ether(dst="00:11:22:33:44:55")/Dot1BR(GRP=0x1, ECIDbase=0x308)/Raw('x' * 20)
-    pkt3 = Ether(dst="00:11:22:33:44:55")/Dot1BR(GRP=0x1, ECIDbase=0x307)/Raw('x' * 20)
-    pkt4 = Ether(dst="00:11:22:33:44:55")/Dot1BR(GRP=0x2, ECIDbase=0x309)/Raw('x' * 20)
-
-   verify pkt1 to vf0 queue0, pkt2 to vf1 queue0, pkt3 to pf queue0,
-   pkt4 can't received by pf and vfs.
-
-4. verify rules can be listed and destroyed::
-
-    testpmd> flow list 0
-    testpmd> flow destroy 0 rule 0
-    testpmd> flow list 0
-    testpmd> flow flush 0
-    testpmd> flow list 0
-
-
 Test case: IXGBE fdir for ipv4
 ==============================
 
-- 
2.33.1

[dts] [PATCH V1 2/2] tests/generic_flow_api: delete test_L2_tunnel_filter

[Yan Xia]

The case only support sagepond and sageville, but these two types NIC
have been removed, so delete the case after confirmed with Peng,Yuan.

Signed-off-by: Yan Xia <yanx.xia@intel.com>
---
 tests/TestSuite_generic_flow_api.py | 47 -----------------------------
 1 file changed, 47 deletions(-)

diff --git a/tests/TestSuite_generic_flow_api.py b/tests/TestSuite_generic_flow_api.py
index e06919c1..e6b04ab6 100644
--- a/tests/TestSuite_generic_flow_api.py
+++ b/tests/TestSuite_generic_flow_api.py
@@ -912,53 +912,6 @@ class TestGeneric_flow_api(TestCase):
             rule_num = extrapkt_rulenum['rulenum']
             self.verify_rulenum(rule_num)
 
-    def test_L2_tunnel_filter(self):
-        """
-        only supported by ixgbe
-        """
-        self.verify(self.nic in ["sagepond", "sageville", "foxville"], "%s nic not support fdir L2 tunnel filter" % self.nic)
-
-        self.setup_env()
-        # start testpmd on pf
-        self.pmdout.start_testpmd("1S/4C/1T", "--rxq=4 --txq=4", "-a %s --file-prefix=pf -m 1024" % self.pf_pci)
-        self.dut.send_expect("set fwd rxonly", "testpmd> ", 120)
-        self.dut.send_expect("set verbose 1", "testpmd> ", 120)
-        self.dut.send_expect("start", "testpmd> ", 120)
-        time.sleep(2)
-        # start testpmd on vf0
-        self.session_secondary.send_expect("%s -c 0x1e0 -n 4 -m 1024 -a %s --file-prefix=vf1 -- -i --rxq=4 --txq=4" % (self.app_path, self.sriov_vfs_port[0].pci), "testpmd>", 120)
-        self.session_secondary.send_expect("set fwd rxonly", "testpmd> ")
-        self.session_secondary.send_expect("set verbose 1", "testpmd> ")
-        self.session_secondary.send_expect("start", "testpmd> ")
-        time.sleep(2)
-        # start testpmd on vf1
-        self.session_third.send_expect("%s -c 0x1e00 -n 4 -m 1024 -a %s --file-prefix=vf2 -- -i --rxq=4 --txq=4" % (self.app_path, self.sriov_vfs_port[1].pci), "testpmd>", 120)
-        self.session_third.send_expect("set fwd rxonly", "testpmd> ")
-        self.session_third.send_expect("set verbose 1", "testpmd> ")
-        self.session_third.send_expect("start", "testpmd> ")
-        time.sleep(2)
-
-        # Enabling ability of parsing E-tag packet
-        self.dut.send_expect("port config 0 l2-tunnel E-tag enable", "testpmd> ")
-        # Enable E-tag packet forwarding, set on pf
-        self.dut.send_expect("E-tag set forwarding on port 0", "testpmd> ")
-
-        # create the flow rules
-        basic_flow_actions = [
-            {'create': 'validate', 'flows': ['etag'], 'actions': ['vf0']},
-            {'create': 'validate', 'flows': ['etag'], 'actions': ['vf1']},
-            {'create': 'validate', 'flows': ['etag'], 'actions': ['pf']},
-            {'create': 'create', 'flows': ['etag'], 'actions': ['vf0']},
-            {'create': 'create', 'flows': ['etag'], 'actions': ['vf1']},
-            {'create': 'create', 'flows': ['etag'], 'actions': ['pf']},
-        ]
-        extrapkt_rulenum = self.all_flows_process(basic_flow_actions)
-        extra_packet = extrapkt_rulenum['extrapacket']
-        self.sendpkt('Ether(dst="%s")/Dot1BR(GRP=0x2, ECIDbase=%s)/Raw("x" * 20)' % (self.pf_mac, extra_packet[0]['etag']))
-        self.verify_result("pf", expect_rxpkts="0", expect_queue="NULL", verify_mac=self.pf_mac)
-        rule_num = extrapkt_rulenum['rulenum']
-        self.verify_rulenum(rule_num)
-
     def test_fdir_for_L2_payload(self):
         """
         only supported by i40e
-- 
2.33.1