* [dts] [PATCH V1] tests/kni: remove case ethtool from kni
@ 2019-06-28 8:51 changqingxwu
2019-06-28 9:01 ` Zhu, ShuaiX
2019-07-01 1:15 ` Ma, LihongX
0 siblings, 2 replies; 3+ messages in thread
From: changqingxwu @ 2019-06-28 8:51 UTC (permalink / raw)
To: dts; +Cc: changqingxwu
Signed-off-by: changqingxwu <changqingx.wu@intel.com>
---
tests/TestSuite_kni.py | 87 ------------------------------------------
1 file changed, 87 deletions(-)
diff --git a/tests/TestSuite_kni.py b/tests/TestSuite_kni.py
index 6bf388c..7969464 100644
--- a/tests/TestSuite_kni.py
+++ b/tests/TestSuite_kni.py
@@ -659,93 +659,6 @@ class TestKni(TestCase):
self.verify("%d packets captured" % len(packets_to_send) in out,
"Wrong number of packets captured")
- def test_ethtool(self):
- """
- Ethtool support KNI.
- """
-
- # Ports and cores configuration set in set_up_all function
- # For each virtual interface perform the following operations
- for port in self.config['ports']:
-
- virtual_interface = self.virtual_interface_name(port)
-
- # Request settings
- out = self.dut.send_expect("ethtool %s" % virtual_interface,
- "# ")
- self.verify("Settings for %s" % virtual_interface in out,
- "ethtool not supported")
- self.verify("Operation not supported" not in out,
- "ethtool not supported")
-
- # Request driver information
- out = self.dut.send_expect("ethtool -i %s" % virtual_interface,
- "# ")
- self.verify("driver: ixgbe" or "driver: igb" in out,
- "'ethtool -i' not supported")
- self.verify("Operation not supported" not in out,
- "'ethtool -i' not supported")
-
- # Request pause parameters
- with open("/usr/include/linux/ethtool.h", "r") as ethtool_h:
- ethtool_contents = ethtool_h.read()
- GSET = "ETHTOOL_GLINKSETTINGS"
- SSET = "ETHTOOL_SLINKSETTINGS"
- if (GSET in ethtool_contents) and (SSET in ethtool_contents):
- out = self.dut.send_expect("ethtool -a %s" % virtual_interface,
- "# ")
- self.verify("Pause parameters for %s" % virtual_interface in out,
- "'ethtool -a' not supported")
- self.verify("Operation not supported" not in out,
- "ethtool '-a' not supported")
-
- # Request statistics
- out = self.dut.send_expect("ethtool -S %s" % virtual_interface,
- "# ")
- self.verify("NIC statistics" in out,
- "'ethtool -S' not supported")
- self.verify("Operation not supported" not in out,
- "ethtool '-S' not supported")
-
- # Request features status
- out = self.dut.send_expect("ethtool -k %s" % virtual_interface, "# ")
- self.verify(("Features for %s" % virtual_interface in out) or
- ("Offload parameters for %s" %
- virtual_interface in out),
- "'ethtool -k' not supported")
- self.verify("Operation not supported" not in out,
- "'ethtool -k' not supported")
-
- # Request ring parameters
- out = self.dut.send_expect("ethtool -g %s" % virtual_interface,
- "# ")
- self.verify("Ring parameters for %s" % virtual_interface in out,
- "'ethtool -g' not supported")
- self.verify("Operation not supported" not in out,
- "'ethtool -g' not supported")
-
- # Request coalesce parameters. NOT SUPPORTED
- out = self.dut.send_expect("ethtool -c %s" % virtual_interface,
- "# ")
- self.verify("Operation not supported" in out,
- "'ethtool -c' seems to be supported. Check it.")
-
- # Request register dump
- out = self.dut.send_expect("ethtool -d %s" % virtual_interface,
- "# ")
- expectstring = "0x00000: CTRL.*0x00008: STATUS"
- self.verify(len(re.findall(expectstring, out, re.DOTALL)) > 0, "'ethtool -d' not supported")
- self.verify("Operation not supported" not in out,
- "'ethtool -d' not supported")
-
- # Request eeprom dump
- out = self.dut.send_expect("ethtool -e %s" % virtual_interface,
- "# ")
- self.verify("Offset" and "Values" in out,
- "'ethtool -e' not support")
- self.verify("Operation not supported" not in out,
- "'ethtool -e' not support")
-
def test_statistics(self):
"""
KNI Statistics test.
--
2.17.2
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [dts] [PATCH V1] tests/kni: remove case ethtool from kni
2019-06-28 8:51 [dts] [PATCH V1] tests/kni: remove case ethtool from kni changqingxwu
@ 2019-06-28 9:01 ` Zhu, ShuaiX
2019-07-01 1:15 ` Ma, LihongX
1 sibling, 0 replies; 3+ messages in thread
From: Zhu, ShuaiX @ 2019-06-28 9:01 UTC (permalink / raw)
To: Wu, ChangqingX, dts; +Cc: Wu, ChangqingX, Zhu, ShuaiX
Tested-by: Zhu, ShuaiX <shuaix.zhu@intel.com>
> -----Original Message-----
> From: dts [mailto:dts-bounces@dpdk.org] On Behalf Of changqingxwu
> Sent: Friday, June 28, 2019 4:51 PM
> To: dts@dpdk.org
> Cc: Wu, ChangqingX <changqingx.wu@intel.com>
> Subject: [dts] [PATCH V1] tests/kni: remove case ethtool from kni
>
> Signed-off-by: changqingxwu <changqingx.wu@intel.com>
> ---
> tests/TestSuite_kni.py | 87 ------------------------------------------
> 1 file changed, 87 deletions(-)
>
> diff --git a/tests/TestSuite_kni.py b/tests/TestSuite_kni.py index
> 6bf388c..7969464 100644
> --- a/tests/TestSuite_kni.py
> +++ b/tests/TestSuite_kni.py
> @@ -659,93 +659,6 @@ class TestKni(TestCase):
> self.verify("%d packets captured" % len(packets_to_send) in
> out,
> "Wrong number of packets captured")
>
> - def test_ethtool(self):
> - """
> - Ethtool support KNI.
> - """
> -
> - # Ports and cores configuration set in set_up_all function
> - # For each virtual interface perform the following operations
> - for port in self.config['ports']:
> -
> - virtual_interface = self.virtual_interface_name(port)
> -
> - # Request settings
> - out = self.dut.send_expect("ethtool %s" % virtual_interface,
> - "# ")
> - self.verify("Settings for %s" % virtual_interface in out,
> - "ethtool not supported")
> - self.verify("Operation not supported" not in out,
> - "ethtool not supported")
> -
> - # Request driver information
> - out = self.dut.send_expect("ethtool -i %s" % virtual_interface,
> - "# ")
> - self.verify("driver: ixgbe" or "driver: igb" in out,
> - "'ethtool -i' not supported")
> - self.verify("Operation not supported" not in out,
> - "'ethtool -i' not supported")
> -
> - # Request pause parameters
> - with open("/usr/include/linux/ethtool.h", "r") as ethtool_h:
> - ethtool_contents = ethtool_h.read()
> - GSET = "ETHTOOL_GLINKSETTINGS"
> - SSET = "ETHTOOL_SLINKSETTINGS"
> - if (GSET in ethtool_contents) and (SSET in
> ethtool_contents):
> - out = self.dut.send_expect("ethtool -a %s" %
> virtual_interface,
> - "# ")
> - self.verify("Pause parameters for %s" %
> virtual_interface in out,
> - "'ethtool -a' not supported")
> - self.verify("Operation not supported" not in out,
> - "ethtool '-a' not supported")
> -
> - # Request statistics
> - out = self.dut.send_expect("ethtool -S %s" % virtual_interface,
> - "# ")
> - self.verify("NIC statistics" in out,
> - "'ethtool -S' not supported")
> - self.verify("Operation not supported" not in out,
> - "ethtool '-S' not supported")
> -
> - # Request features status
> - out = self.dut.send_expect("ethtool -k %s" % virtual_interface,
> "# ")
> - self.verify(("Features for %s" % virtual_interface in out) or
> - ("Offload parameters for %s" %
> - virtual_interface in out),
> - "'ethtool -k' not supported")
> - self.verify("Operation not supported" not in out,
> - "'ethtool -k' not supported")
> -
> - # Request ring parameters
> - out = self.dut.send_expect("ethtool -g %s" % virtual_interface,
> - "# ")
> - self.verify("Ring parameters for %s" % virtual_interface in out,
> - "'ethtool -g' not supported")
> - self.verify("Operation not supported" not in out,
> - "'ethtool -g' not supported")
> -
> - # Request coalesce parameters. NOT SUPPORTED
> - out = self.dut.send_expect("ethtool -c %s" % virtual_interface,
> - "# ")
> - self.verify("Operation not supported" in out,
> - "'ethtool -c' seems to be supported. Check it.")
> -
> - # Request register dump
> - out = self.dut.send_expect("ethtool -d %s" % virtual_interface,
> - "# ")
> - expectstring = "0x00000: CTRL.*0x00008: STATUS"
> - self.verify(len(re.findall(expectstring, out, re.DOTALL)) > 0,
> "'ethtool -d' not supported")
> - self.verify("Operation not supported" not in out,
> - "'ethtool -d' not supported")
> -
> - # Request eeprom dump
> - out = self.dut.send_expect("ethtool -e %s" % virtual_interface,
> - "# ")
> - self.verify("Offset" and "Values" in out,
> - "'ethtool -e' not support")
> - self.verify("Operation not supported" not in out,
> - "'ethtool -e' not support")
> -
> def test_statistics(self):
> """
> KNI Statistics test.
> --
> 2.17.2
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [dts] [PATCH V1] tests/kni: remove case ethtool from kni
2019-06-28 8:51 [dts] [PATCH V1] tests/kni: remove case ethtool from kni changqingxwu
2019-06-28 9:01 ` Zhu, ShuaiX
@ 2019-07-01 1:15 ` Ma, LihongX
1 sibling, 0 replies; 3+ messages in thread
From: Ma, LihongX @ 2019-07-01 1:15 UTC (permalink / raw)
To: Wu, ChangqingX, dts; +Cc: Wu, ChangqingX
Hi, changing
I think you should explanation why remove this case.
-----Original Message-----
From: dts [mailto:dts-bounces@dpdk.org] On Behalf Of changqingxwu
Sent: Friday, June 28, 2019 4:51 PM
To: dts@dpdk.org
Cc: Wu, ChangqingX <changqingx.wu@intel.com>
Subject: [dts] [PATCH V1] tests/kni: remove case ethtool from kni
Signed-off-by: changqingxwu <changqingx.wu@intel.com>
---
tests/TestSuite_kni.py | 87 ------------------------------------------
1 file changed, 87 deletions(-)
diff --git a/tests/TestSuite_kni.py b/tests/TestSuite_kni.py index 6bf388c..7969464 100644
--- a/tests/TestSuite_kni.py
+++ b/tests/TestSuite_kni.py
@@ -659,93 +659,6 @@ class TestKni(TestCase):
self.verify("%d packets captured" % len(packets_to_send) in out,
"Wrong number of packets captured")
- def test_ethtool(self):
- """
- Ethtool support KNI.
- """
-
- # Ports and cores configuration set in set_up_all function
- # For each virtual interface perform the following operations
- for port in self.config['ports']:
-
- virtual_interface = self.virtual_interface_name(port)
-
- # Request settings
- out = self.dut.send_expect("ethtool %s" % virtual_interface,
- "# ")
- self.verify("Settings for %s" % virtual_interface in out,
- "ethtool not supported")
- self.verify("Operation not supported" not in out,
- "ethtool not supported")
-
- # Request driver information
- out = self.dut.send_expect("ethtool -i %s" % virtual_interface,
- "# ")
- self.verify("driver: ixgbe" or "driver: igb" in out,
- "'ethtool -i' not supported")
- self.verify("Operation not supported" not in out,
- "'ethtool -i' not supported")
-
- # Request pause parameters
- with open("/usr/include/linux/ethtool.h", "r") as ethtool_h:
- ethtool_contents = ethtool_h.read()
- GSET = "ETHTOOL_GLINKSETTINGS"
- SSET = "ETHTOOL_SLINKSETTINGS"
- if (GSET in ethtool_contents) and (SSET in ethtool_contents):
- out = self.dut.send_expect("ethtool -a %s" % virtual_interface,
- "# ")
- self.verify("Pause parameters for %s" % virtual_interface in out,
- "'ethtool -a' not supported")
- self.verify("Operation not supported" not in out,
- "ethtool '-a' not supported")
-
- # Request statistics
- out = self.dut.send_expect("ethtool -S %s" % virtual_interface,
- "# ")
- self.verify("NIC statistics" in out,
- "'ethtool -S' not supported")
- self.verify("Operation not supported" not in out,
- "ethtool '-S' not supported")
-
- # Request features status
- out = self.dut.send_expect("ethtool -k %s" % virtual_interface, "# ")
- self.verify(("Features for %s" % virtual_interface in out) or
- ("Offload parameters for %s" %
- virtual_interface in out),
- "'ethtool -k' not supported")
- self.verify("Operation not supported" not in out,
- "'ethtool -k' not supported")
-
- # Request ring parameters
- out = self.dut.send_expect("ethtool -g %s" % virtual_interface,
- "# ")
- self.verify("Ring parameters for %s" % virtual_interface in out,
- "'ethtool -g' not supported")
- self.verify("Operation not supported" not in out,
- "'ethtool -g' not supported")
-
- # Request coalesce parameters. NOT SUPPORTED
- out = self.dut.send_expect("ethtool -c %s" % virtual_interface,
- "# ")
- self.verify("Operation not supported" in out,
- "'ethtool -c' seems to be supported. Check it.")
-
- # Request register dump
- out = self.dut.send_expect("ethtool -d %s" % virtual_interface,
- "# ")
- expectstring = "0x00000: CTRL.*0x00008: STATUS"
- self.verify(len(re.findall(expectstring, out, re.DOTALL)) > 0, "'ethtool -d' not supported")
- self.verify("Operation not supported" not in out,
- "'ethtool -d' not supported")
-
- # Request eeprom dump
- out = self.dut.send_expect("ethtool -e %s" % virtual_interface,
- "# ")
- self.verify("Offset" and "Values" in out,
- "'ethtool -e' not support")
- self.verify("Operation not supported" not in out,
- "'ethtool -e' not support")
-
def test_statistics(self):
"""
KNI Statistics test.
--
2.17.2
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2019-07-01 1:15 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-06-28 8:51 [dts] [PATCH V1] tests/kni: remove case ethtool from kni changqingxwu
2019-06-28 9:01 ` Zhu, ShuaiX
2019-07-01 1:15 ` Ma, LihongX
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).