test suite reviews and discussions
 help / color / mirror / Atom feed
From: <thaq@marvell.com>
To: <dts@dpdk.org>
Cc: <fmasood@marvell.com>, <avijay@marvell.com>,
	Thanseerulhaq <thaq@marvell.com>
Subject: [dts] [PATCH] tests/TestSuite_ipgre.py: Modified ipgre-GRE_packet_chksum_offload testcase to Uncheck sctp chksum if nic-card is cavium_a063. Setting send packet count as 4 to support Vector mode.
Date: Tue, 18 Jun 2019 20:06:33 +0530	[thread overview]
Message-ID: <1560868593-853-1-git-send-email-thaq@marvell.com> (raw)

From: Thanseerulhaq <thaq@marvell.com>

Signed-off-by: Thanseerulhaq <thaq@marvell.com>
---
 tests/TestSuite_ipgre.py | 51 ++++++++++++++++++++++++------------------------
 1 file changed, 26 insertions(+), 25 deletions(-)

diff --git a/tests/TestSuite_ipgre.py b/tests/TestSuite_ipgre.py
index f188df0..de366b0 100644
--- a/tests/TestSuite_ipgre.py
+++ b/tests/TestSuite_ipgre.py
@@ -99,7 +99,7 @@ class TestIpgre(TestCase):
                 for layer in layer_configs.keys():
                     pkt.config_layer(layer, layer_configs[layer])
             inst = self.tester.tcpdump_sniff_packets(self.tester_iface, count=1, timeout=8)
-            pkt.send_pkt(tx_port=self.tester_iface)
+            pkt.send_pkt(tx_port=self.tester_iface, count = 4)
             out = self.dut.get_session_output(timeout=2)
             time.sleep(1)
             self.tester.load_tcpdump_sniff_packets(inst)
@@ -315,7 +315,8 @@ class TestIpgre(TestCase):
         self.dut.send_expect("port stop all", "testpmd>")
         self.dut.send_expect("csum set ip hw 0", "testpmd>")
         self.dut.send_expect("csum set udp hw 0", "testpmd>")
-        self.dut.send_expect("csum set sctp hw 0", "testpmd>")
+        if self.nic != "cavium_a063":
+            self.dut.send_expect("csum set sctp hw 0", "testpmd>")
         self.dut.send_expect("csum set outer-ip hw 0", "testpmd>")
         self.dut.send_expect("csum set tcp hw 0", "testpmd>")
         self.dut.send_expect("csum parse-tunnel on 0", "testpmd>")
@@ -407,29 +408,29 @@ class TestIpgre(TestCase):
                          'udp': {'chksum': 0xffff}}
         self.check_packet_transmission(pkt_types, config_layers)
         self.compare_checksum()
-    
-        # Send packet with wrong inner SCTP checksum and check forwarded packet SCTP checksum is correct
-        pkt_types = { "MAC_IP_GRE_IPv4-TUNNEL_SCTP_PKT": ["PKT_TX_SCTP_CKSUM"]}
-        config_layers = {'ether': {'src': self.outer_mac_src, 
-                                   'dst': self.outer_mac_dst},
-                         'ipv4': {'proto': 'gre',
-                                  'src': self.outer_ip_src,
-                                  'dst': self.outer_ip_dst},
-                         'inner_ipv4':{'src':self.inner_ip_src,
-                                       'dst':self.inner_ip_dst},
-                         'sctp': {'src': 53,
-                                  'dst': 53}}
-        self.save_ref_packet(pkt_types, config_layers)
-        config_layers = {'ether': {'src': self.outer_mac_src, 
-                                   'dst': self.outer_mac_dst},
-                         'ipv4': {'proto': 'gre',
-                                  'src': self.outer_ip_src,
-                                  'dst': self.outer_ip_dst},
-                         'inner_ipv4':{'src':self.inner_ip_src,
-                                       'dst':self.inner_ip_dst},
-                         'sctp': {'chksum': 0x0}}
-        self.check_packet_transmission(pkt_types, config_layers)
-        self.compare_checksum()
+        if self.nic != "cavium_a063":
+            # Send packet with wrong inner SCTP checksum and check forwarded packet SCTP checksum is correct
+            pkt_types = { "MAC_IP_GRE_IPv4-TUNNEL_SCTP_PKT": ["PKT_TX_SCTP_CKSUM"]}
+            config_layers = {'ether': {'src': self.outer_mac_src, 
+                                       'dst': self.outer_mac_dst},
+                             'ipv4': {'proto': 'gre',
+                                      'src': self.outer_ip_src,
+                                      'dst': self.outer_ip_dst},
+                             'inner_ipv4':{'src':self.inner_ip_src,
+                                           'dst':self.inner_ip_dst},
+                             'sctp': {'src': 53,
+                                      'dst': 53}}
+            self.save_ref_packet(pkt_types, config_layers)
+            config_layers = {'ether': {'src': self.outer_mac_src, 
+                                       'dst': self.outer_mac_dst},
+                             'ipv4': {'proto': 'gre',
+                                      'src': self.outer_ip_src,
+                                      'dst': self.outer_ip_dst},
+                             'inner_ipv4':{'src':self.inner_ip_src,
+                                           'dst':self.inner_ip_dst},
+                             'sctp': {'chksum': 0x0}}
+            self.check_packet_transmission(pkt_types, config_layers)
+            self.compare_checksum()
 
         self.dut.send_expect("quit", "#")
 
-- 
1.8.3.1


             reply	other threads:[~2019-06-18 14:37 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-06-18 14:36 thaq [this message]
2019-06-26  8:41 ` 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=1560868593-853-1-git-send-email-thaq@marvell.com \
    --to=thaq@marvell.com \
    --cc=avijay@marvell.com \
    --cc=dts@dpdk.org \
    --cc=fmasood@marvell.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).