[dts] [PATCH V1] Use tcpdump_sniff_packets instead of sniff method

[dts] [PATCH V1] Use tcpdump_sniff_packets instead of sniff method

[lihong]

About the test case test_vxlan_ipv6_checksum_offload,the sniff
method may not work well when add filter arguments, so use
tcpdump_sniff_packets instead of it.

Signed-off-by: lihong <lihongx.ma@intel.com>
---
 tests/TestSuite_vxlan.py | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

diff --git a/tests/TestSuite_vxlan.py b/tests/TestSuite_vxlan.py
index d5a9d77..8de8384 100644
--- a/tests/TestSuite_vxlan.py
+++ b/tests/TestSuite_vxlan.py
@@ -447,16 +447,14 @@ class TestVxlan(TestCase, IxiaPacketGenerator):
         config.create_pcap()
 
         # remove tempory files
+        config.capture_file = "/tmp/sniff_%s.pcap" % self.recv_iface
         self.tester.send_expect("rm -rf %s" % config.capture_file, "# ")
         # save the capture packet into pcap format
         self.tester.scapy_background()
-        self.tester.scapy_append(
-            'p=sniff(iface="%s",filter="ether[12:2]!=0x88cc",count=1,timeout=5)' % self.recv_iface)
-        self.tester.scapy_append(
-            'wrpcap(\"%s\", p)' % config.capture_file)
-        self.tester.scapy_foreground()
 
+        inst = self.tester.tcpdump_sniff_packets(self.recv_iface, timeout=5)
         config.send_pcap(self.tester_iface)
+        self.tester.load_tcpdump_sniff_packets(inst)
         time.sleep(5)
 
         # extract the checksum offload from saved pcap file
-- 
2.7.4

Re: [dts] [PATCH V1] Use tcpdump_sniff_packets instead of sniff method

[Zhu, WenhuiX]

-----Original Message-----
From: dts [mailto:dts-bounces@dpdk.org] On Behalf Of lihong
Sent: Wednesday, December 5, 2018 9:37 AM
To: dts@dpdk.org
Cc: Ma, LihongX <lihongx.ma@intel.com>
Subject: [dts] [PATCH V1] Use tcpdump_sniff_packets instead of sniff method

About the test case test_vxlan_ipv6_checksum_offload,the sniff method may not work well when add filter arguments, so use tcpdump_sniff_packets instead of it.

Signed-off-by: lihong <lihongx.ma@intel.com>
Tested-by: wenhui<wenhuix.zhu@intel.com>

Re: [dts] [PATCH V1] Use tcpdump_sniff_packets instead of sniff method

[Tu, Lijuan]

Sorry, applied V2 instead of V1

> -----Original Message-----
> From: dts [mailto:dts-bounces@dpdk.org] On Behalf Of Lijuan Tu
> Sent: Wednesday, December 12, 2018 3:00 AM
> To: Ma, LihongX <lihongx.ma@intel.com>; dts@dpdk.org
> Subject: Re: [dts] [PATCH V1] Use tcpdump_sniff_packets instead of sniff
> method
> 
> Applied, thanks
> 
> 
> On 2018年12月05日 09:36, lihong wrote:
> > About the test case test_vxlan_ipv6_checksum_offload,the sniff method
> > may not work well when add filter arguments, so use
> > tcpdump_sniff_packets instead of it.
> >
> > Signed-off-by: lihong <lihongx.ma@intel.com>
> > ---
> >   tests/TestSuite_vxlan.py | 8 +++-----
> >   1 file changed, 3 insertions(+), 5 deletions(-)
> >
> > diff --git a/tests/TestSuite_vxlan.py b/tests/TestSuite_vxlan.py index
> > d5a9d77..8de8384 100644
> > --- a/tests/TestSuite_vxlan.py
> > +++ b/tests/TestSuite_vxlan.py
> > @@ -447,16 +447,14 @@ class TestVxlan(TestCase, IxiaPacketGenerator):
> >           config.create_pcap()
> >
> >           # remove tempory files
> > +        config.capture_file = "/tmp/sniff_%s.pcap" % self.recv_iface
> >           self.tester.send_expect("rm -rf %s" % config.capture_file, "#
> ")
> >           # save the capture packet into pcap format
> >           self.tester.scapy_background()
> > -        self.tester.scapy_append(
> > -
> 'p=sniff(iface="%s",filter="ether[12:2]!=0x88cc",count=1,timeout=5)' %
> self.recv_iface)
> > -        self.tester.scapy_append(
> > -            'wrpcap(\"%s\", p)' % config.capture_file)
> > -        self.tester.scapy_foreground()
> >
> > +        inst = self.tester.tcpdump_sniff_packets(self.recv_iface,
> > + timeout=5)
> >           config.send_pcap(self.tester_iface)
> > +        self.tester.load_tcpdump_sniff_packets(inst)
> >           time.sleep(5)
> >
> >           # extract the checksum offload from saved pcap file

Re: [dts] [PATCH V1] Use tcpdump_sniff_packets instead of sniff method

[Lijuan Tu]

Applied, thanks


On 2018年12月05日 09:36, lihong wrote:
> About the test case test_vxlan_ipv6_checksum_offload,the sniff
> method may not work well when add filter arguments, so use
> tcpdump_sniff_packets instead of it.
>
> Signed-off-by: lihong <lihongx.ma@intel.com>
> ---
>   tests/TestSuite_vxlan.py | 8 +++-----
>   1 file changed, 3 insertions(+), 5 deletions(-)
>
> diff --git a/tests/TestSuite_vxlan.py b/tests/TestSuite_vxlan.py
> index d5a9d77..8de8384 100644
> --- a/tests/TestSuite_vxlan.py
> +++ b/tests/TestSuite_vxlan.py
> @@ -447,16 +447,14 @@ class TestVxlan(TestCase, IxiaPacketGenerator):
>           config.create_pcap()
>   
>           # remove tempory files
> +        config.capture_file = "/tmp/sniff_%s.pcap" % self.recv_iface
>           self.tester.send_expect("rm -rf %s" % config.capture_file, "# ")
>           # save the capture packet into pcap format
>           self.tester.scapy_background()
> -        self.tester.scapy_append(
> -            'p=sniff(iface="%s",filter="ether[12:2]!=0x88cc",count=1,timeout=5)' % self.recv_iface)
> -        self.tester.scapy_append(
> -            'wrpcap(\"%s\", p)' % config.capture_file)
> -        self.tester.scapy_foreground()
>   
> +        inst = self.tester.tcpdump_sniff_packets(self.recv_iface, timeout=5)
>           config.send_pcap(self.tester_iface)
> +        self.tester.load_tcpdump_sniff_packets(inst)
>           time.sleep(5)
>   
>           # extract the checksum offload from saved pcap file