[dts][PATCH V2 1/2] tests/userspace_ethtool:remove duplicate vlan test case.

[dts][PATCH V2 1/2] tests/userspace_ethtool:remove duplicate vlan test case.

[Weiyuan Li]

This vlan test case is repeated.
Remove the test case.

Signed-off-by: Weiyuan Li <weiyuanx.li@intel.com>
---

v2:
-Remove added sleep time

 tests/TestSuite_userspace_ethtool.py | 57 ----------------------------
 1 file changed, 57 deletions(-)

diff --git a/tests/TestSuite_userspace_ethtool.py b/tests/TestSuite_userspace_ethtool.py
index 4f104dca..2b84b0a7 100755
--- a/tests/TestSuite_userspace_ethtool.py
+++ b/tests/TestSuite_userspace_ethtool.py
@@ -443,63 +443,6 @@ class TestUserspaceEthtool(TestCase):
             self.verify(rx_pkts == ori_rx_pkts + 4, "Failed to forward after ring parameter changed")
             self.dut.send_expect("quit", "# ")
 
-    def test_ethtool_vlan(self):
-        """
-        Test ethtool app vlan add and delete
-        """
-        main_file = "examples/ethtool/ethtool-app/main.c"
-        # enable vlan filter
-        self.dut.send_expect("sed -i -e '/cfg_port.txmode.mq_mode = ETH_MQ_TX_NONE;$/a\\cfg_port.rxmode.offloads|=DEV_RX_OFFLOAD_VLAN_FILTER;' %s" % main_file, "# ")
-
-        # build sample app
-        self.build_ethtool()
-
-        self.dut.send_expect(self.cmd, "EthApp>", 60)
-        for index in range(len(self.ports)):
-            port = self.ports[index]
-            dst_mac =  self.dut.get_mac_address(port)
-            # generate random vlan
-            vlan = random.randrange(0, 4095)
-            # add vlan on port, record original statistic
-            self.dut.send_expect("vlan %d add %d" % (index, vlan), "EthApp>")
-            ori_rx_pkts, ori_tx_pkts = self.strip_portstats(port)
-
-            # send correct vlan packet to port
-            pkt = Packet(pkt_type='VLAN_UDP')
-            pkt.config_layer('ether', {'dst': dst_mac})
-            pkt.config_layer('vlan', {'vlan': vlan})
-            tester_port = self.tester.get_local_port(port)
-            intf = self.tester.get_interface(tester_port)
-            self.verify(self.ethapp_check_link_status(index, 'Up') == True,
-                    'Fail to Open port{}'.format(index))
-
-            pkt.send_pkt(self.tester, tx_port=intf, count=4)
-            rx_pkts, tx_pkts = self.strip_portstats(port)
-            self.verify(rx_pkts == ori_rx_pkts + 4, "Failed to Rx vlan packet")
-            self.verify(tx_pkts == ori_tx_pkts + 4, "Failed to Tx vlan packet")
-
-            # send incorrect vlan packet to port
-            wrong_vlan = (vlan + 1) % 4096
-            pkt.config_layer('vlan', {'vlan': wrong_vlan})
-            pkt.send_pkt(self.tester, tx_port=intf, count=4)
-            time.sleep(2)
-            rx_pkts_wrong, tx_pkts_wrong = self.strip_portstats(port)
-            self.verify(tx_pkts_wrong == rx_pkts, "Failed to filter Rx vlan packet")
-
-            # remove vlan
-            self.dut.send_expect("vlan %d del %d" % (index, vlan), "EthApp>")
-            # send same packet and make sure not received
-            pkt.config_layer('vlan', {'vlan': vlan})
-            pkt.send_pkt(self.tester, tx_port=intf, count=4)
-            time.sleep(2)
-            rx_pkts_del, tx_pkts_del = self.strip_portstats(port)
-            self.verify(tx_pkts_del == rx_pkts, "Failed to remove Rx vlan filter")
-
-        self.dut.send_expect("quit", "# ")
-        self.dut.send_expect("sed -i -e '/cfg_port.rxmode.offloads|=DEV_RX_OFFLOAD_VLAN_FILTER;$/d' %s" % main_file, "# ")
-        # build sample app
-        self.build_ethtool()
-
     def test_mac_address(self):
         """
         Test ethtool app mac function
-- 
2.25.1

[dts][PATCH V2 2/2] test_plans/userspace_ethtool:remove duplicate vlan test case.

[Weiyuan Li]

Sync remove duplicate test plans.

Signed-off-by: Weiyuan Li <weiyuanx.li@intel.com>
---

v2:
-Remove added sleep time

 test_plans/userspace_ethtool_test_plan.rst | 35 ----------------------
 1 file changed, 35 deletions(-)

diff --git a/test_plans/userspace_ethtool_test_plan.rst b/test_plans/userspace_ethtool_test_plan.rst
index 790e25e8..ab3fbc8f 100644
--- a/test_plans/userspace_ethtool_test_plan.rst
+++ b/test_plans/userspace_ethtool_test_plan.rst
@@ -159,41 +159,6 @@ send packet by scapy on Tester::
    check tx/rx packets
    EthApp>  portstats 0
 
-Test Case: Vlan test
-====================
-
-enable vlan filter flag in main.c of dpdk's ethtool::
-
-    sed -i -e '/cfg_port.txmode.mq_mode = ETH_MQ_TX_NONE;$/a\\cfg_port.rxmode.offloads|=DEV_RX_OFFLOAD_VLAN_FILTER;' examples/ethtool/ethtool-app/main.c
-
-re-compile examples/ethtool::
-
-    meson configure -Dexamples=ethtool x86_64-native-linuxapp-gcc
-    ninja -C x86_64-native-linuxapp-gcc
-
-Add vlan 0 to port 0 and vlan 1 to port1, send packet without vlan to port0,1
-Verify port0 and port1 received vlan packets::
-
-    EthApp> vlan 0 add 0
-    VLAN vid 0 added
-
-    EthApp> vlan 1 add 1
-    VLAN vid 1 added
-
-Send packet with vlan0,1 to port0&1. Verify port0 and port1 received vlan
-packets
-
-Send packet with vlan1,0 to port0&1. Verify port0 and port1 can not receive
-vlan packets
-
-Remove vlan 0,1 from port0&1, send packet with vlan0,1 to port0,1. Verify
-port0 and port1 can not receive vlan packet::
-
-    EthApp> vlan 0 del 0
-    VLAN vid 0 removed
-    EthApp> vlan 1 del 1
-    VLAN vid 1 removed
-
 Test Case: Mac address test
 ===========================
 Use "macaddr" command to dump port mac address and then check that dumped
-- 
2.25.1

RE: [dts][PATCH V2 2/2] test_plans/userspace_ethtool:remove duplicate vlan test case.

[Tu, Lijuan]

> -----Original Message-----
> From: Li, WeiyuanX <weiyuanx.li@intel.com>
> Sent: 2022年2月11日 14:26
> To: dts@dpdk.org; Tu, Lijuan <lijuan.tu@intel.com>
> Cc: Li, WeiyuanX <weiyuanx.li@intel.com>
> Subject: [dts][PATCH V2 2/2] test_plans/userspace_ethtool:remove duplicate
> vlan test case.
> 
> Sync remove duplicate test plans.
> 
> Signed-off-by: Weiyuan Li <weiyuanx.li@intel.com>

Applied