From: "Qiu, Michael" <michael.qiu@intel.com>
To: "Liu, Yong" <yong.liu@intel.com>, "dts@dpdk.org" <dts@dpdk.org>
Subject: Re: [dts] [PATCH] TSO: Add RRC support and some bug fix
Date: Wed, 4 Nov 2015 06:24:35 +0000 [thread overview]
Message-ID: <533710CFB86FA344BFBF2D6802E6028621B658D5@SHSMSX101.ccr.corp.intel.com> (raw)
In-Reply-To: <86228AFD5BCD8E4EBFD2B90117B5E81E10F2A9FE@SHSMSX103.ccr.corp.intel.com>
Hi, Marvin
I will update dpdk_test_case_checklist.xls in one patch, so would you
please merge this first?
Thanks,
Michael
On 2015/10/14 16:30, Liu, Yong wrote:
> Hi Michael,
> For those case not supported RRC, you can add them into dpdk_test_case_checklist.xls just in conf folder.
>
>
>> -----Original Message-----
>> From: dts [mailto:dts-bounces@dpdk.org] On Behalf Of Michael Qiu
>> Sent: Wednesday, October 14, 2015 9:18 AM
>> To: dts@dpdk.org
>> Subject: [dts] [PATCH] TSO: Add RRC support and some bug fix
>>
>> Add RRC support and some bug fix.
>>
>> Signed-off-by: Mihcael Qiu <michael.qiu@intel.com>
>> ---
>> tests/TestSuite_tso.py | 37 +++++++++++++++++++++----------------
>> 1 file changed, 21 insertions(+), 16 deletions(-)
>>
>> diff --git a/tests/TestSuite_tso.py b/tests/TestSuite_tso.py
>> index 392157f..c3a13ee 100644
>> --- a/tests/TestSuite_tso.py
>> +++ b/tests/TestSuite_tso.py
>> @@ -83,7 +83,8 @@ class TestTSO(TestCase):
>> # this feature support Fortville, Niantic
>> self.verify(self.nic in ["kawela_2", "niantic", "bartonhills",
>> "82545EM",
>> "82540EM", "springfountain",
>> "fortville_eagle",
>> - "fortville_spirit",
>> "fortville_spirit_single"],
>> + "fortville_spirit",
>> "fortville_spirit_single",
>> + "redrockcanyou"],
>> "NIC Unsupported: " + str(self.nic))
>>
>> # Based on h/w type, choose how many ports to use
>> @@ -138,7 +139,7 @@ class TestTSO(TestCase):
>>
>> for iface in ifaces:
>> command = ('tcpdump -w tcpdump_{0}.pcap -i {0}
>> 2>tcpdump_{0}.out &').format(iface)
>> - self.tester.send_expect('rm -f tcpdump_{0}.pcap',
>> '#').format(iface)
>> + self.tester.send_expect(('rm -f
>> tcpdump_{0}.pcap').format(iface), '#')
>> self.tester.send_expect(command, '#')
>>
>> def tcpdump_stop_sniff(self):
>> @@ -182,7 +183,7 @@ class TestTSO(TestCase):
>> self.verify(cores is not None, "Insufficient cores for speed
>> testing")
>> self.coreMask = dts.create_mask(cores)
>>
>> - padding = self.frame_sizes[0] - self.headers_size
>> + padding = [self.frame_sizes[0] - self.headers_size,
>> self.frame_sizes[1] - self.headers_size]
>>
>> self.tester.send_expect("ethtool -K %s rx off tx off tso off gso
>> off gro off lro off" % tx_interface, "# ")
>> self.tester.send_expect("ip l set %s up" % tx_interface, "# ")
>> @@ -206,6 +207,7 @@ class TestTSO(TestCase):
>>
>> self.dut.send_expect("tso set 800 %d" % self.dut_ports[1],
>> "testpmd> ", 120)
>> self.dut.send_expect("set fwd csum", "testpmd> ", 120)
>> + self.dut.send_expect("set promisc all off", "testpmd> ", 120)
>> self.dut.send_expect("start", "testpmd> ")
>>
>> self.tester.scapy_foreground()
>> @@ -214,31 +216,34 @@ class TestTSO(TestCase):
>> # IPv4 tcp test
>>
>> self.tcpdump_start_sniffing([tx_interface, rx_interface])
>> -
>> self.tester.scapy_append('sendp([Ether(dst="%s",src="52:00:00:00:00:00")/I
>> P(src="192.168.1.1",dst="192.168.1.2")/TCP(sport=1021,dport=1021)/("X"*%s)
>> ], iface="%s")' % (mac, padding, tx_interface))
>> +
>> self.tester.scapy_append('sendp([Ether(dst="%s",src="52:00:00:00:00:00")/I
>> P(src="192.168.1.1",dst="192.168.1.2")/TCP(sport=1021,dport=1021)/("X"*%s)
>> ], iface="%s")' % (mac, padding[0], tx_interface))
>> out = self.tester.scapy_execute()
>> out = self.dut.send_expect("show port stats all", "testpmd> ",
>> 120)
>> print out
>> self.tcpdump_stop_sniff()
>> rx_stats = self.number_of_packets(rx_interface)
>> - if (rx_stats == 2):
>> - self.verify(1, "Pass")
>> + self.verify(rx_stats == 1, "FAIL")
>>
>> # IPv6 tcp test
>>
>> self.tcpdump_start_sniffing([tx_interface, rx_interface])
>> - self.tester.scapy_append('sendp([Ether(dst="%s",
>> src="52:00:00:00:00:00")/IPv6(src="FE80:0:0:0:200:1FF:FE00:200",
>> dst="3555:5555:6666:6666:7777:7777:8888:8888")/TCP(sport=1021,dport=1021)/
>> ("X"*%s)], iface="%s")' % (mac, padding, tx_interface))
>> + self.tester.scapy_append('sendp([Ether(dst="%s",
>> src="52:00:00:00:00:00")/IPv6(src="FE80:0:0:0:200:1FF:FE00:200",
>> dst="3555:5555:6666:6666:7777:7777:8888:8888")/TCP(sport=1021,dport=1021)/
>> ("X"*%s)], iface="%s")' % (mac, padding[1], tx_interface))
>> out = self.tester.scapy_execute()
>> out = self.dut.send_expect("show port stats all", "testpmd> ",
>> 120)
>> print out
>> self.tcpdump_stop_sniff()
>> rx_stats = self.number_of_packets(rx_interface)
>> - if (rx_stats == 2):
>> - self.verify(1, "Pass")
>> + self.verify(rx_stats == 2, "FAIL")
>>
>> def test_tso_tunneling(self):
>> """
>> TSO IPv4 TCP, IPv6 TCP, VXLan testing
>> """
>> + # RedRockCanyou does not support Tunneling in DPDK yet.
>> + if self.nic == "redrockcanyou":
>> + print dts.RED("fm10k not support this case\n")
>> + return
>> +
>> tx_interface =
>> self.tester.get_interface(self.tester.get_local_port(self.dut_ports[0]))
>> rx_interface =
>> self.tester.get_interface(self.tester.get_local_port(self.dut_ports[1]))
>>
>> @@ -248,7 +253,7 @@ class TestTSO(TestCase):
>> self.verify(cores is not None, "Insufficient cores for speed
>> testing")
>> self.coreMask = dts.create_mask(cores)
>>
>> - padding = self.frame_sizes[0] - self.headers_size
>> + padding = [self.frame_sizes[0] - self.headers_size,
>> self.frame_sizes[1] - self.headers_size]
>>
>> self.tester.send_expect("ethtool -K %s rx off tx off tso off gso
>> off gro off lro off" % tx_interface, "# ")
>> self.tester.send_expect("ip l set %s up" % tx_interface, "# ")
>> @@ -272,6 +277,7 @@ class TestTSO(TestCase):
>>
>> self.dut.send_expect("tso set 800 %d" % self.dut_ports[1],
>> "testpmd> ", 120)
>> self.dut.send_expect("set fwd csum", "testpmd> ", 120)
>> + self.dut.send_expect("set promisc all off", "testpmd> ", 120)
>> self.dut.send_expect("start", "testpmd> ")
>>
>> self.tester.scapy_foreground()
>> @@ -279,25 +285,23 @@ class TestTSO(TestCase):
>>
>> # Vxlan test
>> self.tcpdump_start_sniffing([tx_interface, rx_interface])
>> -
>> self.tester.scapy_append('sendp([Ether(dst="%s",src="52:00:00:00:00:00")/I
>> P(src="192.168.1.1",dst="192.168.1.2")/UDP(sport="1021",dport="4789")/VXLA
>> N()/Ether(dst=%s,src="52:00:00:00:00:00")/IP(src="192.168.1.1",dst="192.16
>> 8.1.2")/TCP(sport="1021",dport="1021")/("X"*%s)], iface="%s")' % (mac, mac,
>> padding, tx_interface))
>> +
>> self.tester.scapy_append('sendp([Ether(dst="%s",src="52:00:00:00:00:00")/I
>> P(src="192.168.1.1",dst="192.168.1.2")/UDP(sport="1021",dport="4789")/VXLA
>> N()/Ether(dst=%s,src="52:00:00:00:00:00")/IP(src="192.168.1.1",dst="192.16
>> 8.1.2")/TCP(sport="1021",dport="1021")/("X"*%s)], iface="%s")' % (mac, mac,
>> padding[0], tx_interface))
>> out = self.tester.scapy_execute()
>> out = self.dut.send_expect("show port stats all", "testpmd> ",
>> 120)
>> print out
>> self.tcpdump_stop_sniff()
>> rx_stats = self.number_of_packets(rx_interface)
>> - if (rx_stats == 2):
>> - self.verify(1, "Pass")
>> + self.verify(rx_stats == 1, "FAIL")
>>
>> # Nvgre test
>> self.tcpdump_start_sniffing([tx_interface, rx_interface])
>> -
>> self.tester.scapy_append('sendp([Ether(dst="%s",src="52:00:00:00:00:00")/I
>> P(src="192.168.1.1",dst="192.168.1.2",proto=47)/NVGRE()/Ether(dst=%s,src="
>> 52:00:00:00:00:00")/IP(src="192.168.1.1",dst="192.168.1.2")/TCP(sport="102
>> 1",dport="1021")/("X"*%s)], iface="%s")' % (mac, mac, padding,
>> tx_interface))
>> +
>> self.tester.scapy_append('sendp([Ether(dst="%s",src="52:00:00:00:00:00")/I
>> P(src="192.168.1.1",dst="192.168.1.2",proto=47)/NVGRE()/Ether(dst=%s,src="
>> 52:00:00:00:00:00")/IP(src="192.168.1.1",dst="192.168.1.2")/TCP(sport="102
>> 1",dport="1021")/("X"*%s)], iface="%s")' % (mac, mac, padding[1],
>> tx_interface))
>> out = self.tester.scapy_execute()
>> out = self.dut.send_expect("show port stats all", "testpmd> ",
>> 120)
>> print out
>> self.tcpdump_stop_sniff()
>> rx_stats = self.number_of_packets(rx_interface)
>> - if (rx_stats == 2):
>> - self.verify(1, "Pass")
>> + self.verify(rx_stats == 2, "Pass")
>>
>> def test_perf_TSO_2ports(self):
>> """
>> @@ -347,6 +351,7 @@ class TestTSO(TestCase):
>> self.dut.send_expect("csum parse_tunnel on %d" %
>> self.dut_ports[1], "testpmd> ", 120)
>> self.dut.send_expect("tso set 800 %d" % self.dut_ports[1],
>> "testpmd> ", 120)
>> self.dut.send_expect("set fwd csum", "testpmd> ", 120)
>> + self.dut.send_expect("set promisc all off", "testpmd> ", 120)
>> self.dut.send_expect("start", "testpmd> ")
>> for frame_size in self.frame_sizes:
>> wirespeed = self.wirespeed(self.nic, frame_size, 2)
>> --
>> 1.9.3
>
next prev parent reply other threads:[~2015-11-04 6:24 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-10-14 8:18 Michael Qiu
2015-10-14 8:30 ` Liu, Yong
2015-11-04 6:24 ` Qiu, Michael [this message]
2015-11-04 6:53 ` Liu, Yong
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=533710CFB86FA344BFBF2D6802E6028621B658D5@SHSMSX101.ccr.corp.intel.com \
--to=michael.qiu@intel.com \
--cc=dts@dpdk.org \
--cc=yong.liu@intel.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).