test suite reviews and discussions
 help / color / mirror / Atom feed
From: "Tu, Lijuan" <lijuan.tu@intel.com>
To: "thaq@marvell.com" <thaq@marvell.com>, "dts@dpdk.org" <dts@dpdk.org>
Cc: "fmasood@marvell.com" <fmasood@marvell.com>,
	"avijay@marvell.com" <avijay@marvell.com>
Subject: Re: [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: Wed, 26 Jun 2019 08:41:22 +0000	[thread overview]
Message-ID: <8CE3E05A3F976642AAB0F4675D0AD20E0BAC533C@SHSMSX101.ccr.corp.intel.com> (raw)
In-Reply-To: <1560868593-853-1-git-send-email-thaq@marvell.com>

Applied, thanks

> -----Original Message-----
> From: dts [mailto:dts-bounces@dpdk.org] On Behalf Of thaq@marvell.com
> Sent: Tuesday, June 18, 2019 10:37 PM
> 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.
> 
> 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-26  8:41 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-06-18 14:36 thaq
2019-06-26  8:41 ` Tu, Lijuan [this message]

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=8CE3E05A3F976642AAB0F4675D0AD20E0BAC533C@SHSMSX101.ccr.corp.intel.com \
    --to=lijuan.tu@intel.com \
    --cc=avijay@marvell.com \
    --cc=dts@dpdk.org \
    --cc=fmasood@marvell.com \
    --cc=thaq@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).