test suite reviews and discussions
 help / color / mirror / Atom feed
From: sunqin <qinx.sun@intel.com>
To: dts@dpdk.org
Cc: qi.fu@intel.com, sunqin <qinx.sun@intel.com>
Subject: [dts] [PATCH V1] tests/flexible_rxd: remove one unsupported case
Date: Fri, 12 Mar 2021 13:30:04 +0000	[thread overview]
Message-ID: <20210312133004.1339-1-qinx.sun@intel.com> (raw)

Confirmed with developer,only type=0x8100 needs to be supported in single vlan mode,

other type values do not need to be verified, so remove the type=0x88A8 case from

test suite and plan.

Signed-off-by: sunqin <qinx.sun@intel.com>
---
 test_plans/flexible_rxd_test_plan.rst | 38 +++++++--------------------
 tests/TestSuite_flexible_rxd.py       |  6 -----
 2 files changed, 10 insertions(+), 34 deletions(-)

diff --git a/test_plans/flexible_rxd_test_plan.rst b/test_plans/flexible_rxd_test_plan.rst
index c3a0e1c6..b2ca34a4 100644
--- a/test_plans/flexible_rxd_test_plan.rst
+++ b/test_plans/flexible_rxd_test_plan.rst
@@ -123,25 +123,7 @@ Check the output in testpmd, **ctag=1:0:23** is expected, which is consistent wi
   ol_flags: PKT_RX_RSS_HASH PKT_RX_L4_CKSUM_GOOD PKT_RX_IP_CKSUM_GOOD PKT_RX_OUTER_L4_CKSUM_UNKNOWN
 
 
-Test Case 02: Check single VLAN fields in RXD (802.1ad)
-=======================================================
-
-Test steps are same to ``Test Case 01``, just change the launch command of testpmd, test packet and expected output
-
-Launch testpmd command::
-
-  ./x86_64-native-linux-gcc/app/testpmd -l 6-9 -n 4 -w 18:00.0,proto_xtr=vlan -- -i --rxq=32 --txq=32 --portmask=0x1 --nb-cores=2
-
-Test packet::
-
-  p = Ether(src='3c:fd:fe:bc:f6:78', dst='68:05:ca:a3:13:4c', type=0x88A8)/Dot1Q(prio=1,vlan=23)/IP()/UDP()/DNS()
-
-Expected output in testpmd::
-
-  stag=1:0:23
-
-
-Test Case 03: Check double VLAN fields in RXD (802.1Q) only 1 VLAN tag
+Test Case 02: Check double VLAN fields in RXD (802.1Q) only 1 VLAN tag
 ======================================================================
 
 Test steps are same to ``Test Case 01``, just change the launch command of testpmd, test packet and expected output
@@ -159,7 +141,7 @@ Expected output in testpmd::
   stag=1:0:23
 
 
-Test Case 04: Check double VLAN fields in RXD (802.1Q) 2 VLAN tags
+Test Case 03: Check double VLAN fields in RXD (802.1Q) 2 VLAN tags
 ==================================================================
 
 Test steps are same to ``Test Case 01``, just change the launch command of testpmd, test packet and expected output
@@ -178,7 +160,7 @@ Expected output in testpmd::
   ctag=4:0:56
 
 
-Test Case 05: Check double VLAN fields in RXD (802.1ad)
+Test Case 04: Check double VLAN fields in RXD (802.1ad)
 =======================================================
 
 Test steps are same to ``Test Case 01``, just change the launch command of testpmd, test packet and expected output
@@ -197,7 +179,7 @@ Expected output in testpmd::
   ctag=4:0:56
 
 
-Test Case 06: Check IPv4 fields in RXD
+Test Case 05: Check IPv4 fields in RXD
 ======================================
 
 Test steps are same to ``Test Case 01``, just change the launch command of testpmd, test packet and expected output
@@ -219,7 +201,7 @@ Expected output in testpmd::
   proto=17
 
 
-Test Case 07: Check IPv6 fields in RXD 
+Test Case 06: Check IPv6 fields in RXD
 =======================================================
 
 Test steps are same to ``Test Case 01``, just change the launch command of testpmd, test packet and expected output
@@ -241,7 +223,7 @@ Expected output in testpmd::
   hoplimit=34
 
 
-Test Case 08: Check IPv6 flow field in RXD
+Test Case 07: Check IPv6 flow field in RXD
 =======================================================
 
 Test steps are same to ``Test Case 01``, just change the launch command of testpmd, test packet and expected output
@@ -261,7 +243,7 @@ Expected output in testpmd::
   flow=0x98765
 
 
-Test Case 09: Check TCP fields in IPv4 in RXD
+Test Case 08: Check TCP fields in IPv4 in RXD
 =======================================================
 
 Test steps are same to ``Test Case 01``, just change the launch command of testpmd, test packet and expected output
@@ -280,7 +262,7 @@ Expected output in testpmd::
   flags=AS
 
 
-Test Case 10: Check TCP fields in IPv6 in RXD
+Test Case 09: Check TCP fields in IPv6 in RXD
 =======================================================
 
 Test steps are same to ``Test Case 01``, just change the launch command of testpmd, test packet and expected output
@@ -299,7 +281,7 @@ Expected output in testpmd::
   flags=S
 
 
-Test Case 11: Check IPv4, IPv6, TCP fields in RXD on specific queues
+Test Case 10: Check IPv4, IPv6, TCP fields in RXD on specific queues
 ====================================================================
 
 Test steps are same to ``Test Case 01``, just change the launch command of testpmd, test packet and expected output
@@ -341,7 +323,7 @@ Expected output in testpmd::
   flags=AS
 
 
-Test Case 12: Check effect of replacing pkg from RXID #22 to RXID #16
+Test Case 11: Check effect of replacing pkg from RXID #22 to RXID #16
 =====================================================================
 
 Put the ice.pkg with RXID #16(ice-1.3.7.0.pkg and more) to /lib/firmware/updates/intel/ice/ddp/ice.pkg, then reload ice driver::
diff --git a/tests/TestSuite_flexible_rxd.py b/tests/TestSuite_flexible_rxd.py
index 6ce0a091..648173e7 100644
--- a/tests/TestSuite_flexible_rxd.py
+++ b/tests/TestSuite_flexible_rxd.py
@@ -117,12 +117,6 @@ class TestFlexibleRxd(TestCase, FlexibleRxdBase):
         """
         self.check_single_VLAN_fields_in_RXD_8021Q()
 
-    def test_check_single_VLAN_fields_in_RXD_8021ad(self):
-        """
-        Check single VLAN fields in RXD (802.1ad)
-        """
-        self.check_single_VLAN_fields_in_RXD_8021ad()
-
     def test_check_double_VLAN_fields_in_RXD_8021Q_1_VLAN_tag(self):
         """
         Check double VLAN fields in RXD (802.1Q) only 1 VLAN tag
-- 
2.17.1


             reply	other threads:[~2021-03-12  5:04 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-03-12 13:30 sunqin [this message]
2021-03-12  6:08 ` Sun, QinX
2021-03-15  7:25 ` Tu, Lijuan

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20210312133004.1339-1-qinx.sun@intel.com \
    --to=qinx.sun@intel.com \
    --cc=dts@dpdk.org \
    --cc=qi.fu@intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).