test suite reviews and discussions
 help / color / mirror / Atom feed
From: qifu <qi.fu@intel.com>
To: dts@dpdk.org
Cc: qifu <qi.fu@intel.com>
Subject: [dts] [PATCH V2]test_plans/cvl_advanced_iavf_rss_test_plan:delete steps after destroying rss rules
Date: Thu, 28 Jan 2021 10:56:40 -0500	[thread overview]
Message-ID: <20210128155640.27524-1-qi.fu@intel.com> (raw)

synced with developer, the behavior after destroy rss rules are not unified.
so delete the steps after destroying rss rules. 

Signed-off-by: qifu <qi.fu@intel.com>
---
 .../cvl_advanced_iavf_rss_test_plan.rst       | 78 +------------------
 1 file changed, 3 insertions(+), 75 deletions(-)

diff --git a/test_plans/cvl_advanced_iavf_rss_test_plan.rst b/test_plans/cvl_advanced_iavf_rss_test_plan.rst
index c4661550..6376a140 100644
--- a/test_plans/cvl_advanced_iavf_rss_test_plan.rst
+++ b/test_plans/cvl_advanced_iavf_rss_test_plan.rst
@@ -334,9 +334,8 @@ all the test cases run the same test steps as below:
    check the received packet have same hash value with the basic packet.
    check the packet is distributed to queues by rss.
    note: if there is not this type packet in the case, omit this step.
-6. destroy the rule and list rule.
-7. send same packet with step 3.
-   check the received packet have different hash value with basic packet.
+6. destroy the rule and list rule. check the flow list has no rule.
+
 
 Test case: MAC_IPV4
 ===================
@@ -1503,7 +1502,7 @@ all the test cases run the same test steps as below:
    note: if there is not this type packet in the case, omit this step.
 6. destroy the rule and list rule.
 7. send same packets with step 2.
-   check the received packets have different hash value, or have no hash value.
+   check the received packets have different hash value.
 
 Test case: symmetric MAC_IPV4
 =============================
@@ -1760,40 +1759,6 @@ Subcase 1: two rules with same pattern but different hash input set, not hit def
 
    check the hash value is different from the first packet.
 
-5. destroy the rule 1::
-
-     testpmd> flow destroy 0 rule 1
-     testpmd> flow list 0
-
-   check the rule 1 not exists in the list.
-   send a MAC_IPV4_UDP packet, you can find it hit default ipv4 profile::
-
-     sendp([Ether(dst="00:11:22:33:44:55")/IP(src="192.168.0.3",dst="192.168.0.5")/UDP(dport=45)/Raw("x"*480)], iface="enp134s0f0")
-
-   change the fields [Source IP] or [Dest IP], send a packet::
-
-     sendp([Ether(dst="00:11:22:33:44:55")/IP(src="192.168.0.7",dst="192.168.0.5")/UDP(dport=45)/Raw("x"*480)], iface="enp134s0f0")
-     sendp([Ether(dst="00:11:22:33:44:55")/IP(src="192.168.0.3",dst="192.168.0.9")/UDP(dport=45)/Raw("x"*480)], iface="enp134s0f0")
-     sendp([Ether(dst="00:11:22:33:44:55")/IP(src="192.168.0.7",dst="192.168.0.9")/UDP(dport=45)/Raw("x"*480)], iface="enp134s0f0")
-
-   check the hash values of the last two packets are different from the first packet.
-   destroy the rule 0::
-
-     testpmd> flow destroy 0 rule 0
-     testpmd> flow list 0
-
-   check the rule 0 not exists in the list.
-   send a MAC_IPV4_UDP packet, you can find it hit default ipv4 profile::
-
-     sendp([Ether(dst="00:11:22:33:44:55")/IP(src="192.168.0.3",dst="192.168.0.5")/UDP(dport=45)/Raw("x"*480)], iface="enp134s0f0")
-
-   change the fields [Source IP] or [Dest IP], send a packet::
-
-     sendp([Ether(dst="00:11:22:33:44:55")/IP(src="192.168.0.7",dst="192.168.0.5")/UDP(dport=45)/Raw("x"*480)], iface="enp134s0f0")
-     sendp([Ether(dst="00:11:22:33:44:55")/IP(src="192.168.0.3",dst="192.168.0.9")/UDP(dport=45)/Raw("x"*480)], iface="enp134s0f0")
-     sendp([Ether(dst="00:11:22:33:44:55")/IP(src="192.168.0.7",dst="192.168.0.9")/UDP(dport=45)/Raw("x"*480)], iface="enp134s0f0")
-
-   check the hash values of the last two packets are different from the first packet.
 
 Subcase 2: two rules with same pattern but different hash input set, hit default profile
 ----------------------------------------------------------------------------------------
@@ -1842,17 +1807,6 @@ Subcase 2: two rules with same pattern but different hash input set, hit default
 
    check the hash value is different from the first packet.
 
-5. destroy the rule 1::
-
-     testpmd> flow destroy 0 rule 1
-     testpmd> flow list 0
-
-   check the rule 1 not exists in the list.
-   send the MAC_IPV4_PAY packet::
-
-     sendp([Ether(dst="00:11:22:33:44:55")/IP(src="192.168.0.3",dst="192.168.0.5")/Raw("x"*480)], iface="enp134s0f0")
-
-   check the hash value is not exist.
 
 Subcase 3: two rules, scope smaller created first, and the larger one created later
 -----------------------------------------------------------------------------------
@@ -1901,19 +1855,6 @@ Subcase 3: two rules, scope smaller created first, and the larger one created la
 
    check the hash value is the same as the first packet.
 
-5. destroy the rule 1::
-
-     testpmd> flow destroy 0 rule 1
-     testpmd> flow list 0
-
-   check the rule 1 not exists in the list.
-   repeat step 2, get the same result.
-   destroy the rule 0::
-
-     testpmd> flow destroy 0 rule 0
-     testpmd> flow list 0
-
-   send a MAC_IPV4_UDP_PAY packet, check the hash values not exists.
 
 Subcase 4: two rules, scope larger created first, and the smaller one created later
 -----------------------------------------------------------------------------------
@@ -1962,16 +1903,3 @@ Subcase 4: two rules, scope larger created first, and the smaller one created la
 
    check the hash value is the same as the first packet.
 
-5. destroy the rule 1::
-
-     testpmd> flow destroy 0 rule 1
-     testpmd> flow list 0
-
-   check the rule 1 not exists in the list.
-   repeat step 2, hit ipv4 profile, get the same result.
-   destroy the rule 0::
-
-     testpmd> flow destroy 0 rule 0
-     testpmd> flow list 0
-
-   send a MAC_IPV4_UDP_PAY packet, check the hash values not exists.
-- 
2.25.1


             reply	other threads:[~2021-01-28  7:11 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-01-28 15:56 qifu [this message]
2021-02-02  5:48 ` 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=20210128155640.27524-1-qi.fu@intel.com \
    --to=qi.fu@intel.com \
    --cc=dts@dpdk.org \
    /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).