* [dts] [PATCH V4]Ethtool: update automation testing script
2016-07-27 7:12 [dts] [PATCH V4]Ethtool: update automation testing script Yufen Mo
@ 2016-07-27 7:12 ` Yufen Mo
2016-07-27 9:16 ` Liu, Yong
1 sibling, 0 replies; 3+ messages in thread
From: Yufen Mo @ 2016-07-27 7:12 UTC (permalink / raw)
To: dts; +Cc: yufengmx
From: yufengmx <yufengx.mo@intel.com>
*. close eeprom test case on FVL for FVL series firmware not supported
*. reset ring parameter after ring parameter is used
*. set testing content the same user space ethtool dumped
Signed-off-by: yufengmx <yufengx.mo@intel.com>
---
tests/TestSuite_userspace_ethtool.py | 15 +++++++++------
1 file changed, 9 insertions(+), 6 deletions(-)
diff --git a/tests/TestSuite_userspace_ethtool.py b/tests/TestSuite_userspace_ethtool.py
index c718a58..2346e3b 100644
--- a/tests/TestSuite_userspace_ethtool.py
+++ b/tests/TestSuite_userspace_ethtool.py
@@ -34,6 +34,7 @@ DPDK Test suite.
Test support of userspace ethtool feature
"""
+import os
import dts
import time
import re
@@ -128,8 +129,8 @@ class TestUserspaceEthtool(TestCase, IxiaPacketGenerator):
return 1518
def resize_linux_eeprom_file(self, dpdk_eeprom_file, linux_eeprom_file):
- basePath = self.dut.base_dir
- with open( basePath + dpdk_eeprom_file, 'rb') as fpDpdk:
+ basePath = os.sep + "root" + self.dut.base_dir[1:] + os.sep
+ with open( basePath + os.sep + dpdk_eeprom_file, 'rb') as fpDpdk:
dpdk_bytes = fpDpdk.read()
dpdk_length = len(dpdk_bytes)
@@ -220,7 +221,10 @@ class TestUserspaceEthtool(TestCase, IxiaPacketGenerator):
portsinfo = []
ori_drivers = []
-
+
+ if self.nic.startswith("fortville"):
+ return
+
for portid in range(len(self.ports)):
self.dut.send_expect("regs %d regs_%d.bin" % (portid, portid), "EthApp>")
portinfo = {'portid': portid, 'reg_file': 'regs_%d.bin' % portid}
@@ -300,8 +304,8 @@ class TestUserspaceEthtool(TestCase, IxiaPacketGenerator):
"""
Test ethtool app ring parameter getting and setting
"""
- self.dut.send_expect(self.cmd, "EthApp>", 60)
for index in range(len(self.ports)):
+ self.dut.send_expect(self.cmd, "EthApp>", 60)
port = self.ports[index]
ori_rx_pkts, ori_tx_pkts = self.strip_portstats(port)
_, rx_max, _, tx_max = self.strip_ringparam(index)
@@ -315,8 +319,7 @@ class TestUserspaceEthtool(TestCase, IxiaPacketGenerator):
pkt.send_pkt(tx_port=intf)
rx_pkts, tx_pkts = self.strip_portstats(index)
self.verify(rx_pkts == ori_rx_pkts + 1, "Failed to forward after ring parameter changed")
-
- self.dut.send_expect("quit", "# ")
+ self.dut.send_expect("quit", "# ")
def test_ethtool_vlan(self):
"""
--
1.9.3
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [dts] [PATCH V4]Ethtool: update automation testing script
2016-07-27 7:12 [dts] [PATCH V4]Ethtool: update automation testing script Yufen Mo
2016-07-27 7:12 ` Yufen Mo
@ 2016-07-27 9:16 ` Liu, Yong
1 sibling, 0 replies; 3+ messages in thread
From: Liu, Yong @ 2016-07-27 9:16 UTC (permalink / raw)
To: Mo, YufengX, dts; +Cc: Mo, YufengX
Yufen, better to add description for each reversion. It will help others understand what happened.
v2 changes:
-- what changed
v1 description:
> -----Original Message-----
> From: dts [mailto:dts-bounces@dpdk.org] On Behalf Of Yufen Mo
> Sent: Wednesday, July 27, 2016 3:12 PM
> To: dts@dpdk.org
> Cc: Mo, YufengX
> Subject: [dts] [PATCH V4]Ethtool: update automation testing script
>
> From: yufengmx <yufengx.mo@intel.com>
>
> yufengmx (1):
> Ethtool: update automation testing script
>
> tests/TestSuite_userspace_ethtool.py | 15 +++++++++------
> 1 file changed, 9 insertions(+), 6 deletions(-)
>
> --
> 1.9.3
^ permalink raw reply [flat|nested] 3+ messages in thread