From: Lijuan Tu <lijuanx.a.tu@intel.com>
To: dts@dpdk.org
Cc: Lijuan Tu <lijuanx.a.tu@intel.com>
Subject: [dts] [PATCH v1] tests vf_jumboframe: revise cases to support i350
Date: Mon, 17 Apr 2017 14:06:50 +0800 [thread overview]
Message-ID: <1492409210-18669-1-git-send-email-lijuanx.a.tu@intel.com> (raw)
1. revise cases to support i350, refer to: DPDK-1117
2. dpdk vf enable crc-strip as default
3. Port should bind to default driver rather than igb_uio.
4. clear vm resouces after case or suite end.
Signed-off-by: Lijuan Tu <lijuanx.a.tu@intel.com>
---
tests/TestSuite_vf_jumboframe.py | 94 +++++++++++++++++++---------------------
1 file changed, 44 insertions(+), 50 deletions(-)
diff --git a/tests/TestSuite_vf_jumboframe.py b/tests/TestSuite_vf_jumboframe.py
index d32990a..d7241ff 100644
--- a/tests/TestSuite_vf_jumboframe.py
+++ b/tests/TestSuite_vf_jumboframe.py
@@ -60,7 +60,7 @@ class TestVfJumboFrame(TestCase):
for port in ports:
netdev = self.dut.ports_info[port]['port']
driver_now = netdev.get_nic_driver()
- if driver == "":
+ if driver == None:
driver = netdev.default_driver
if driver != driver_now:
netdev.bind_driver(driver=driver)
@@ -118,18 +118,20 @@ class TestVfJumboFrame(TestCase):
def destroy_vm_env(self):
if getattr(self, 'vm', None):
- self.vm_dut.kill_all()
+ if getattr(self, 'vm_dut', None):
+ self.vm_dut.kill_all()
self.vm_testpmd = None
self.vm_dut_ports = None
# destroy vm0
self.vm.stop()
+ self.dut.virt_exit()
+ time.sleep(3)
self.vm = None
- if getattr(self, 'used_dut_port', None):
+ if getattr(self, 'used_dut_port', None) != None:
self.dut.destroy_sriov_vfs_by_port(self.used_dut_port)
self.used_dut_port = None
-
- self.bind_nic_driver(self.dut_ports[:1], driver="igb_uio")
+ self.bind_nic_driver(self.dut_ports[:1], driver='default')
self.env_done = False
@@ -172,11 +174,16 @@ class TestVfJumboFrame(TestCase):
if received:
self.verify((rx_pkts == 1) and (tx_pkts == 1), "Packet forward assert error")
+
if self.kdriver == "ixgbe":
self.verify((rx_bytes + 4) == pktsize, "Rx packet size should be packet size - 4")
else:
self.verify(rx_bytes == pktsize, "Tx packet size should be equal to packet size")
- self.verify((tx_bytes + 4) == pktsize, "Tx packet size should be packet size - 4")
+
+ if self.kdriver == "igb":
+ self.verify(tx_bytes == pktsize, "Tx packet size should be packet size")
+ else:
+ self.verify((tx_bytes + 4) == pktsize, "Tx packet size should be packet size - 4")
else:
self.verify(rx_err == 1 or tx_pkts == 0, "Packet drop assert error")
@@ -185,15 +192,10 @@ class TestVfJumboFrame(TestCase):
This case aims to test transmitting normal size packet without jumbo enable
"""
# should enable jumbo on host
- if self.kdriver == "ixgbe":
- self.dutobj = self.dut.ports_info[self.port]['port']
- self.dutobj.enable_jumbo(framesize=ETHER_STANDARD_MTU)
- time.sleep(2)
+ self.dutobj = self.dut.ports_info[self.port]['port']
+ self.dutobj.enable_jumbo(framesize=ETHER_STANDARD_MTU)
- if self.kdriver == "i40e":
- self.vm_testpmd.start_testpmd("Default", "--max-pkt-len=%d --port-topology=loop --crc-strip --txqflags=0x0" % (ETHER_STANDARD_MTU))
- else:
- self.vm_testpmd.start_testpmd("Default", "--max-pkt-len=%d --port-topology=loop --txqflags=0x0" % (ETHER_STANDARD_MTU))
+ self.vm_testpmd.start_testpmd("Default", "--max-pkt-len=%d --port-topology=loop --txqflags=0x0" % (ETHER_STANDARD_MTU))
self.vm_testpmd.execute_cmd("set fwd mac")
self.vm_testpmd.execute_cmd("start")
@@ -210,15 +212,10 @@ class TestVfJumboFrame(TestCase):
packet forwrding should be support correct.
"""
# should enable jumbo on host
- if self.kdriver == "ixgbe":
- self.dutobj = self.dut.ports_info[self.port]['port']
- self.dutobj.enable_jumbo(framesize=ETHER_JUMBO_FRAME_MTU)
- time.sleep(2)
+ self.dutobj = self.dut.ports_info[self.port]['port']
+ self.dutobj.enable_jumbo(framesize=ETHER_JUMBO_FRAME_MTU)
- if self.kdriver == "i40e":
- self.vm_testpmd.start_testpmd("Default", "--max-pkt-len=%d --port-topology=loop --crc-strip --txqflags=0x0" % (ETHER_JUMBO_FRAME_MTU))
- else:
- self.vm_testpmd.start_testpmd("Default", "--max-pkt-len=%s --port-topology=loop --txqflags=0x0" % (ETHER_JUMBO_FRAME_MTU))
+ self.vm_testpmd.start_testpmd("Default", "--max-pkt-len=%d --port-topology=loop --txqflags=0x0" % (ETHER_JUMBO_FRAME_MTU))
self.vm_testpmd.execute_cmd("set fwd mac")
self.vm_testpmd.execute_cmd("start")
@@ -235,20 +232,24 @@ class TestVfJumboFrame(TestCase):
jumbo frame support.
"""
# should enable jumbo on host
- if self.kdriver == "ixgbe":
- self.dutobj = self.dut.ports_info[self.port]['port']
- self.dutobj.enable_jumbo(framesize=ETHER_STANDARD_MTU)
- time.sleep(2)
+ self.dutobj = self.dut.ports_info[self.port]['port']
+ self.dutobj.enable_jumbo(framesize=ETHER_STANDARD_MTU)
- if self.kdriver == "i40e":
- self.vm_testpmd.start_testpmd("Default", "--port-topology=loop --crc-strip --txqflags=0x0" )
- else:
- self.vm_testpmd.start_testpmd("Default", "--port-topology=loop --txqflags=0x0")
+ self.vm_testpmd.start_testpmd("Default", "--port-topology=loop --txqflags=0x0" )
self.vm_testpmd.execute_cmd("set fwd mac")
self.vm_testpmd.execute_cmd("start")
- self.jumboframes_send_packet(ETHER_STANDARD_MTU + 1, False)
+ # On igb, for example i350, refer to :DPDK-1117
+ # For PF, the max-pkt-len = mtu + 18 + 4(VLAN header len).
+ # For VF, the real max-pkt-len = the given max-pkt-len + 4(VLAN header len).
+ # This behavior is levelraged from kernel driver.
+ # And it means max-pkt-len is always 4 bytes longer than assumed.
+
+ if self.kdriver == "igb":
+ self.jumboframes_send_packet(ETHER_STANDARD_MTU + 1 + 4, False)
+ else:
+ self.jumboframes_send_packet(ETHER_STANDARD_MTU + 1, False)
self.vm_testpmd.execute_cmd("stop")
self.vm_testpmd.quit()
@@ -259,15 +260,10 @@ class TestVfJumboFrame(TestCase):
packet can be forwarded correct.
"""
# should enable jumbo on host
- if self.kdriver == "ixgbe":
- self.dutobj = self.dut.ports_info[self.port]['port']
- self.dutobj.enable_jumbo(framesize=ETHER_JUMBO_FRAME_MTU)
- time.sleep(2)
+ self.dutobj = self.dut.ports_info[self.port]['port']
+ self.dutobj.enable_jumbo(framesize=ETHER_JUMBO_FRAME_MTU)
- if self.kdriver == "i40e":
- self.vm_testpmd.start_testpmd("Default", "--max-pkt-len=%d --port-topology=loop --crc-strip --txqflags=0x0" % (ETHER_JUMBO_FRAME_MTU))
- else:
- self.vm_testpmd.start_testpmd("Default", "--max-pkt-len=%s --port-topology=loop --txqflags=0x0" % (ETHER_JUMBO_FRAME_MTU))
+ self.vm_testpmd.start_testpmd("Default", "--max-pkt-len=%d --port-topology=loop --txqflags=0x0" % (ETHER_JUMBO_FRAME_MTU))
self.vm_testpmd.execute_cmd("set fwd mac")
self.vm_testpmd.execute_cmd("start")
@@ -285,20 +281,19 @@ class TestVfJumboFrame(TestCase):
packet which the length bigger than MTU can not be forwarded.
"""
# should enable jumbo on host
- if self.kdriver == "ixgbe":
- self.dutobj = self.dut.ports_info[self.port]['port']
- self.dutobj.enable_jumbo(framesize=ETHER_JUMBO_FRAME_MTU)
- time.sleep(2)
+ self.dutobj = self.dut.ports_info[self.port]['port']
+ self.dutobj.enable_jumbo(framesize=ETHER_JUMBO_FRAME_MTU)
- if self.kdriver == "i40e":
- self.vm_testpmd.start_testpmd("Default", "--max-pkt-len=%d --port-topology=loop --crc-strip --txqflags=0x0" % (ETHER_JUMBO_FRAME_MTU))
- else:
- self.vm_testpmd.start_testpmd("Default", "--max-pkt-len=%s --port-topology=loop --txqflags=0x0" % (ETHER_JUMBO_FRAME_MTU))
+ self.vm_testpmd.start_testpmd("Default", "--max-pkt-len=%d --port-topology=loop --txqflags=0x0" % (ETHER_JUMBO_FRAME_MTU))
self.vm_testpmd.execute_cmd("set fwd mac")
self.vm_testpmd.execute_cmd("start")
- self.jumboframes_send_packet(ETHER_JUMBO_FRAME_MTU + 1, False)
+ # On 1G NICs, when the jubmo frame MTU set as 9000, the software adjust it to 9004.
+ if self.kdriver == "igb":
+ self.jumboframes_send_packet(ETHER_JUMBO_FRAME_MTU + 4 + 1, False)
+ else:
+ self.jumboframes_send_packet(ETHER_JUMBO_FRAME_MTU + 1, False)
self.vm_testpmd.execute_cmd("stop")
self.vm_testpmd.quit()
@@ -307,8 +302,7 @@ class TestVfJumboFrame(TestCase):
"""
Run after each test case.
"""
- self.vm_dut.kill_all()
- pass
+ self.destroy_vm_env()
def tear_down_all(self):
"""
--
1.9.3
next reply other threads:[~2017-04-17 6:05 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-04-17 6:06 Lijuan Tu [this message]
2017-04-20 8:02 ` 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=1492409210-18669-1-git-send-email-lijuanx.a.tu@intel.com \
--to=lijuanx.a.tu@intel.com \
--cc=dts@dpdk.org \
/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).