test suite reviews and discussions
 help / color / mirror / Atom feed
* [dts] [PATCH v1] tests vf_jumboframe: revise cases to support i350
@ 2017-04-17  6:06 Lijuan Tu
  2017-04-20  8:02 ` Liu, Yong
  0 siblings, 1 reply; 2+ messages in thread
From: Lijuan Tu @ 2017-04-17  6:06 UTC (permalink / raw)
  To: dts; +Cc: Lijuan Tu

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

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [dts] [PATCH v1] tests vf_jumboframe: revise cases to support i350
  2017-04-17  6:06 [dts] [PATCH v1] tests vf_jumboframe: revise cases to support i350 Lijuan Tu
@ 2017-04-20  8:02 ` Liu, Yong
  0 siblings, 0 replies; 2+ messages in thread
From: Liu, Yong @ 2017-04-20  8:02 UTC (permalink / raw)
  To: Lijuan Tu, dts

Thanks, applied.

On 04/17/2017 02:06 PM, Lijuan Tu wrote:
> 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>
> ---

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2017-04-20  8:13 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-04-17  6:06 [dts] [PATCH v1] tests vf_jumboframe: revise cases to support i350 Lijuan Tu
2017-04-20  8:02 ` Liu, Yong

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).