* [dts] [PATCH V2] Use tcpdump_sniff_packets instead of sniff method
@ 2018-12-10 18:24 lihong
2018-12-11 5:29 ` Zhu, WenhuiX
2018-12-11 10:52 ` Tu, Lijuan
0 siblings, 2 replies; 3+ messages in thread
From: lihong @ 2018-12-10 18:24 UTC (permalink / raw)
To: dts; +Cc: lihong
The sniff method may not work well when add filter ether arguments,
so use tcpdump_sniff_packets instead of it.
Signed-off-by: lihong <lihongx.ma@intel.com>
---
tests/TestSuite_short_live.py | 9 ++++-----
tests/TestSuite_vxlan.py | 8 +++-----
2 files changed, 7 insertions(+), 10 deletions(-)
diff --git a/tests/TestSuite_short_live.py b/tests/TestSuite_short_live.py
index ca3643f..b3ec131 100644
--- a/tests/TestSuite_short_live.py
+++ b/tests/TestSuite_short_live.py
@@ -102,10 +102,7 @@ class TestShortLiveApp(TestCase):
if (txPort == rxPort):
count = 2
- self.tester.scapy_append('p=sniff(iface="%s", filter="ether[12:2]!=0x88cc", count=%d, timeout=5)' % (rxitf, count))
- self.tester.scapy_append('RESULT=str(p[%d].show)' % (count-1))
-
- self.tester.scapy_foreground()
+ inst = self.tester.tcpdump_sniff_packets(rxitf, count=count, timeout=5)
pktlen = pktSize - 14
padding = pktlen - 20
@@ -114,7 +111,9 @@ class TestShortLiveApp(TestCase):
self.tester.scapy_execute()
time.sleep(3)
- out = self.tester.scapy_get_result()
+ pkts = self.tester.load_tcpdump_sniff_packets(inst)
+ out = str(pkts[0].pktgen.pkt.show)
+ self.logger.info('SCAPY Result:\n' + out + '\n\n\n')
if received:
self.verify(('PPP' in out) and 'src=%s'% Dut_tx_mac in out, "Receive test failed")
else:
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
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [dts] [PATCH V2] Use tcpdump_sniff_packets instead of sniff method
2018-12-10 18:24 [dts] [PATCH V2] Use tcpdump_sniff_packets instead of sniff method lihong
@ 2018-12-11 5:29 ` Zhu, WenhuiX
2018-12-11 10:52 ` Tu, Lijuan
1 sibling, 0 replies; 3+ messages in thread
From: Zhu, WenhuiX @ 2018-12-11 5:29 UTC (permalink / raw)
To: Ma, LihongX, dts; +Cc: Ma, LihongX, Wang, FengqinX
-----Original Message-----
From: dts [mailto:dts-bounces@dpdk.org] On Behalf Of lihong
Sent: Tuesday, December 11, 2018 2:24 AM
To: dts@dpdk.org
Cc: Ma, LihongX <lihongx.ma@intel.com>
Subject: [dts] [PATCH V2] Use tcpdump_sniff_packets instead of sniff method
The sniff method may not work well when add filter ether arguments, so use tcpdump_sniff_packets instead of it.
Signed-off-by: lihong <lihongx.ma@intel.com>
Test-by: Zhu, WenhuiX <wenhuix.zhu@intel.com>
---
tests/TestSuite_short_live.py | 9 ++++-----
tests/TestSuite_vxlan.py | 8 +++-----
2 files changed, 7 insertions(+), 10 deletions(-)
diff --git a/tests/TestSuite_short_live.py b/tests/TestSuite_short_live.py index ca3643f..b3ec131 100644
--- a/tests/TestSuite_short_live.py
+++ b/tests/TestSuite_short_live.py
@@ -102,10 +102,7 @@ class TestShortLiveApp(TestCase):
if (txPort == rxPort):
count = 2
- self.tester.scapy_append('p=sniff(iface="%s", filter="ether[12:2]!=0x88cc", count=%d, timeout=5)' % (rxitf, count))
- self.tester.scapy_append('RESULT=str(p[%d].show)' % (count-1))
-
- self.tester.scapy_foreground()
+ inst = self.tester.tcpdump_sniff_packets(rxitf, count=count,
+ timeout=5)
pktlen = pktSize - 14
padding = pktlen - 20
@@ -114,7 +111,9 @@ class TestShortLiveApp(TestCase):
self.tester.scapy_execute()
time.sleep(3)
- out = self.tester.scapy_get_result()
+ pkts = self.tester.load_tcpdump_sniff_packets(inst)
+ out = str(pkts[0].pktgen.pkt.show)
+ self.logger.info('SCAPY Result:\n' + out + '\n\n\n')
if received:
self.verify(('PPP' in out) and 'src=%s'% Dut_tx_mac in out, "Receive test failed")
else:
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
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [dts] [PATCH V2] Use tcpdump_sniff_packets instead of sniff method
2018-12-10 18:24 [dts] [PATCH V2] Use tcpdump_sniff_packets instead of sniff method lihong
2018-12-11 5:29 ` Zhu, WenhuiX
@ 2018-12-11 10:52 ` Tu, Lijuan
1 sibling, 0 replies; 3+ messages in thread
From: Tu, Lijuan @ 2018-12-11 10:52 UTC (permalink / raw)
To: Ma, LihongX, dts; +Cc: Ma, LihongX
Applied V2, and supersede V1, thanks
> -----Original Message-----
> From: dts [mailto:dts-bounces@dpdk.org] On Behalf Of lihong
> Sent: Tuesday, December 11, 2018 2:24 AM
> To: dts@dpdk.org
> Cc: Ma, LihongX <lihongx.ma@intel.com>
> Subject: [dts] [PATCH V2] Use tcpdump_sniff_packets instead of sniff
> method
>
> The sniff method may not work well when add filter ether arguments, so use
> tcpdump_sniff_packets instead of it.
>
> Signed-off-by: lihong <lihongx.ma@intel.com>
> ---
> tests/TestSuite_short_live.py | 9 ++++-----
> tests/TestSuite_vxlan.py | 8 +++-----
> 2 files changed, 7 insertions(+), 10 deletions(-)
>
> diff --git a/tests/TestSuite_short_live.py b/tests/TestSuite_short_live.py
> index ca3643f..b3ec131 100644
> --- a/tests/TestSuite_short_live.py
> +++ b/tests/TestSuite_short_live.py
> @@ -102,10 +102,7 @@ class TestShortLiveApp(TestCase):
> if (txPort == rxPort):
> count = 2
>
> - self.tester.scapy_append('p=sniff(iface="%s",
> filter="ether[12:2]!=0x88cc", count=%d, timeout=5)' % (rxitf, count))
> - self.tester.scapy_append('RESULT=str(p[%d].show)' % (count-1))
> -
> - self.tester.scapy_foreground()
> + inst = self.tester.tcpdump_sniff_packets(rxitf, count=count,
> + timeout=5)
>
> pktlen = pktSize - 14
> padding = pktlen - 20
> @@ -114,7 +111,9 @@ class TestShortLiveApp(TestCase):
> self.tester.scapy_execute()
> time.sleep(3)
>
> - out = self.tester.scapy_get_result()
> + pkts = self.tester.load_tcpdump_sniff_packets(inst)
> + out = str(pkts[0].pktgen.pkt.show)
> + self.logger.info('SCAPY Result:\n' + out + '\n\n\n')
> if received:
> self.verify(('PPP' in out) and 'src=%s'% Dut_tx_mac in out,
> "Receive test failed")
> else:
> 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
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2018-12-11 10:52 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-12-10 18:24 [dts] [PATCH V2] Use tcpdump_sniff_packets instead of sniff method lihong
2018-12-11 5:29 ` Zhu, WenhuiX
2018-12-11 10:52 ` Tu, Lijuan
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).