* [dts] [PATCH V1] tests: add test for vhost/virtio loopback performance
@ 2016-12-22 3:29 lei,yao
2016-12-28 5:28 ` Liu, Yong
0 siblings, 1 reply; 4+ messages in thread
From: lei,yao @ 2016-12-22 3:29 UTC (permalink / raw)
To: dts; +Cc: lei yao
From: lei yao <lei.a.yao@intel.com>
This test won't use qemu, but will use two testpmd thread to launch vhost-user and
virtio-user to simulate the qemu user case.
Signed-off-by: lei yao <lei.a.yao@intel.com>
---
...Suite_vhost_loopback_performance_virtio_user.py | 256 +++++++++++++++++++++
1 file changed, 256 insertions(+)
create mode 100644 tests/TestSuite_vhost_loopback_performance_virtio_user.py
diff --git a/tests/TestSuite_vhost_loopback_performance_virtio_user.py b/tests/TestSuite_vhost_loopback_performance_virtio_user.py
new file mode 100644
index 0000000..46d2110
--- /dev/null
+++ b/tests/TestSuite_vhost_loopback_performance_virtio_user.py
@@ -0,0 +1,256 @@
+#
+# BSD LICENSE
+#
+# Copyright(c) 2010-2016 Intel Corporation. All rights reserved.
+# All rights reserved.
+#
+# Redistribution and use in source and binary forms, with or without
+# modification, are permitted provided that the following conditions
+# are met:
+#
+# * Redistributions of source code must retain the above copyright
+# notice, this list of conditions and the following disclaimer.
+# * Redistributions in binary form must reproduce the above copyright
+# notice, this list of conditions and the following disclaimer in
+# the documentation and/or other materials provided with the
+# distribution.
+# * Neither the name of Intel Corporation nor the names of its
+# contributors may be used to endorse or promote products derived
+# from this software without specific prior written permission.
+#
+# THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
+# "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
+# LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
+# A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
+# OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
+# SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
+# LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
+# DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
+# THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
+# (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
+# OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+
+
+"""
+DPDK Test suite.
+Test Vhost loopback performance for Mergeable, normal , vector Path
+"""
+import string
+import utils
+import time
+import re
+from test_case import TestCase
+
+
+class TestVhostLoopback(TestCase):
+
+ def set_up_all(self):
+ """
+ Run at the start of each test suite.
+ """
+ # Clean the execution environment
+ self.dut.send_expect("rm -rf ./vhost.out", "#")
+ self.dut.send_expect("rm -rf ./vhost-net*", "#")
+ self.dut.send_expect("killall -s INT vhost-switch", "#")
+ self.dut.send_expect("killall -s INT testpmd", "#")
+ self.header_row = ["FrameSize(B)", "Mode", "Throughput(Mpps)", "Virtio Version"]
+ self.frame_sizes = [64, 128, 260, 520, 1024, 1500]
+ self.test_cycles = {'Mpps': {}, 'pct': {}}
+
+ def set_up(self):
+ """
+ Run before each test case.
+ """
+ self.dut.restore_interfaces()
+ pass
+
+ def test_perf_vhost_mergeable_loopback(self):
+ """
+ Benchmark performance for frame_sizes.
+ """
+
+ # Modify the driver file to disbale the Mergeable, then re-compile the DPDK and back up the original driver file
+ for frame_size in self.frame_sizes:
+ # Back up the original driver file
+ self.dut.send_expect("cp ./drivers/net/virtio/virtio_ethdev.h ./", "#", 30)
+ # Change the packet size sent by the testpmd
+ self.change_testpmd_size = "sed -i -e 's/#define TXONLY_DEF_PACKET_LEN.*$/#define TXONLY_DEF_PACKET_LEN %d/' ./app/test-pmd/testpmd.h" % frame_size
+ self.dut.send_expect(self.change_testpmd_size, "#", 30)
+ self.dut.build_install_dpdk(self.dut.target)
+
+ # Start the vhost user side
+ cmd = self.target + "/app/testpmd -n 4 -c 0x03 " + \
+ "-m 2048 --file-prefix=vhost --vdev 'net_vhost0,iface=vhost-net,queues=1,client=0' -- -i"
+ self.dut.send_expect(cmd, "testpmd>", 120)
+ # Start the virtio_user side
+ vhost_user = self.dut.new_session()
+ command_line_user = self.target + "/app/testpmd -n 4 -c 0x0c " + \
+ " -m 2048 --no-pci --file-prefix=virtio " + \
+ " --vdev=net_virtio_user0,mac=00:01:02:03:04:05,path=./vhost-net" + \
+ " -- -i --txqflags=0xf01 --disable-hw-vlan-filter"
+
+ vhost_user.send_expect(command_line_user, "testpmd>", 120)
+
+ self.dut.send_expect("set fwd mac retry", "testpmd>", 60)
+ self.dut.send_expect("start tx_first 32", "testpmd>", 60)
+ vhost_user.send_expect("start tx_first 32", "testpmd> ", 120)
+ results = 0.0
+ out = self.dut.send_expect("show port stats all", "testpmd>", 60)
+ time.sleep(5)
+ # Get throughput 10 times and calculate the average throughput
+ for i in range(10):
+ out = self.dut.send_expect("show port stats all", "testpmd>", 60)
+ time.sleep(5)
+ lines = re.search("Rx-pps:\s*(\d*)", out)
+ result = lines.group(1)
+ results += float(result)
+ Mpps = results / (1000000 * 10)
+ self.result_table_create(self.header_row)
+
+ self.dut.send_expect("quit", "#", 60)
+ vhost_user.send_expect("quit", "#", 60)
+ # Restore the driver file
+ self.dut.send_expect("rm -rf ./drivers/net/virtio/virtio_ethdev.h", "#", 30)
+ self.dut.send_expect("mv ./virtio_ethdev.h ./drivers/net/virtio/", "#", 30)
+ self.test_cycles['Mpps'][frame_size] = Mpps
+ self.test_cycles['pct'][frame_size] = "Virtio 0.95"
+
+ for frame_size in self.frame_sizes:
+ results_row = [frame_size]
+ results_row.append("Mergeable on")
+ results_row.append(self.test_cycles['Mpps'][frame_size])
+ results_row.append(self.test_cycles['pct'][frame_size])
+ self.result_table_add(results_row)
+
+ self.result_table_print()
+
+ def test_perf_vhost_vector_loopback(self):
+ """
+ Benchmark performance for frame_sizes.
+ """
+ for frame_size in self.frame_sizes:
+ # Modify the driver file to disbale the Mergeable, then re-compile the DPDK and back up the original driver file
+ self.dut.send_expect("cp ./drivers/net/virtio/virtio_ethdev.h ./", "#", 30)
+ self.dut.send_expect("sed -i '/VIRTIO_NET_F_MRG_RXBUF/d' ./drivers/net/virtio/virtio_ethdev.h", "#", 30)
+ self.change_testpmd_size = "sed -i -e 's/#define TXONLY_DEF_PACKET_LEN .*$/#define TXONLY_DEF_PACKET_LEN %d/' ./app/test-pmd/testpmd.h" % frame_size
+ self.dut.send_expect(self.change_testpmd_size, "#", 30)
+ self.dut.build_install_dpdk(self.dut.target)
+
+ # Start the vhost user side
+ cmd = self.target + "/app/testpmd -n 4 -c 0x03 " + \
+ "-m 2048 --file-prefix=vhost --vdev 'net_vhost0,iface=vhost-net,queues=1,client=0' -- -i"
+ self.dut.send_expect(cmd, "testpmd>", 120)
+ # Start the virtio_user side
+ vhost_user = self.dut.new_session()
+ command_line_user = self.target + "/app/testpmd -n 4 -c 0x0c " + \
+ " -m 2048 --no-pci --file-prefix=virtio " + \
+ " --vdev=net_virtio_user0,mac=00:01:02:03:04:05,path=./vhost-net " + \
+ " -- -i --txqflags=0xf01 --disable-hw-vlan-filter"
+
+ vhost_user.send_expect(command_line_user, "testpmd>", 120)
+
+ self.dut.send_expect("set fwd mac retry", "testpmd>", 60)
+ self.dut.send_expect("start tx_first 32", "testpmd>", 60)
+ vhost_user.send_expect("start tx_first 32", "testpmd> ", 120)
+ results = 0.0
+ out = self.dut.send_expect("show port stats all", "testpmd>", 60)
+ time.sleep(5)
+ # Get throughput 10 times and calculate the average throughput
+ for i in range(10):
+ out = self.dut.send_expect("show port stats all", "testpmd>", 60)
+ time.sleep(5)
+ lines = re.search("Rx-pps:\s*(\d*)", out)
+ result = lines.group(1)
+ results += float(result)
+ Mpps = results / (1000000 * 10)
+ self.result_table_create(self.header_row)
+
+ self.dut.send_expect("quit", "#", 60)
+ vhost_user.send_expect("quit", "#", 60)
+ # Restore the driver file
+ self.dut.send_expect("rm -rf ./drivers/net/virtio/virtio_ethdev.h", "#", 30)
+ self.dut.send_expect("mv ./virtio_ethdev.h ./drivers/net/virtio/", "#", 30)
+ self.test_cycles['Mpps'][frame_size] = Mpps
+ self.test_cycles['pct'][frame_size] = "Virtio 0.95"
+
+ for frame_size in self.frame_sizes:
+ results_row = [frame_size]
+ results_row.append("Vector on")
+ results_row.append(self.test_cycles['Mpps'][frame_size])
+ results_row.append(self.test_cycles['pct'][frame_size])
+ self.result_table_add(results_row)
+
+ self.result_table_print()
+
+ def test_perf_vhost_normal_loopback(self):
+ """
+ Benchmark performance for frame_sizes.
+ """
+
+ self.result_table_create(self.header_row)
+ for frame_size in self.frame_sizes:
+ # Modify the driver file to disbale the Mergeable, then re-compile the DPDK and back up the original driver file
+ self.dut.send_expect("cp ./drivers/net/virtio/virtio_ethdev.h ./", "#", 30)
+ self.dut.send_expect("sed -i '/VIRTIO_NET_F_MRG_RXBUF/d' ./drivers/net/virtio/virtio_ethdev.h", "#", 30)
+ self.change_testpmd_size = "sed -i -e 's/#define TXONLY_DEF_PACKET_LEN .*$/#define TXONLY_DEF_PACKET_LEN %d/' ./app/test-pmd/testpmd.h" % frame_size
+ self.dut.send_expect(self.change_testpmd_size, "#", 30)
+ self.dut.build_install_dpdk(self.dut.target)
+
+ # Start the vhost user side
+ cmd = self.target + "/app/testpmd -n 4 -c 0x03 " + \
+ "-m 2048 --file-prefix=vhost --vdev 'net_vhost0,iface=vhost-net,queues=1,client=0' -- -i"
+ self.dut.send_expect(cmd, "testpmd>", 120)
+ # Start the virtio_user side
+ vhost_user = self.dut.new_session()
+ command_line_user = self.target + "/app/testpmd -n 4 -c 0x0c " + \
+ " -m 2048 --no-pci --file-prefix=virtio " + \
+ " --vdev=net_virtio_user0,mac=00:01:02:03:04:05,path=./vhost-net" + \
+ " -- -i --txqflags=0xf00 --disable-hw-vlan-filter"
+
+ vhost_user.send_expect(command_line_user, "testpmd>", 120)
+
+ self.dut.send_expect("set fwd mac retry", "testpmd>", 60)
+ self.dut.send_expect("start tx_first 32", "testpmd>", 60)
+ vhost_user.send_expect("start tx_first 32", "testpmd> ", 120)
+ results = 0.0
+ out = self.dut.send_expect("show port stats all", "testpmd>", 60)
+ time.sleep(5)
+ # Get throughput 10 times and calculate the average throughput
+ for i in range(10):
+ out = self.dut.send_expect("show port stats all", "testpmd>", 60)
+ time.sleep(5)
+ lines = re.search("Rx-pps:\s*(\d*)", out)
+ result = lines.group(1)
+ results += float(result)
+ Mpps = results / (1000000 * 10)
+ self.result_table_create(self.header_row)
+
+ self.dut.send_expect("quit", "#", 60)
+ vhost_user.send_expect("quit", "#", 60)
+ # Restore the driver file
+ self.dut.send_expect("rm -rf ./drivers/net/virtio/virtio_ethdev.h", "#", 30)
+ self.dut.send_expect("mv ./virtio_ethdev.h ./drivers/net/virtio/", "#", 30)
+ self.test_cycles['Mpps'][frame_size] = Mpps
+ self.test_cycles['pct'][frame_size] = "Virtio 0.95"
+
+ for frame_size in self.frame_sizes:
+ results_row = [frame_size]
+ results_row.append("Normal")
+ results_row.append(self.test_cycles['Mpps'][frame_size])
+ results_row.append(self.test_cycles['pct'][frame_size])
+ self.result_table_add(results_row)
+
+ self.result_table_print()
+
+ def tear_down(self):
+ """
+ Run after each test case.
+ """
+ time.sleep(2)
+
+ def tear_down_all(self):
+ """
+ Run after each test suite.
+ """
+ # Recompile the dpdk because we change the source code during the test
+ self.dut.build_install_dpdk(self.dut.target)
--
2.7.4
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [dts] [PATCH V1] tests: add test for vhost/virtio loopback performance
2016-12-22 3:29 [dts] [PATCH V1] tests: add test for vhost/virtio loopback performance lei,yao
@ 2016-12-28 5:28 ` Liu, Yong
2016-12-28 5:56 ` Yao, Lei A
0 siblings, 1 reply; 4+ messages in thread
From: Liu, Yong @ 2016-12-28 5:28 UTC (permalink / raw)
To: Yao, Lei A, dts; +Cc: Yao, Lei A
Lei, some comments below.
> -----Original Message-----
> From: dts [mailto:dts-bounces@dpdk.org] On Behalf Of lei,yao
> Sent: Thursday, December 22, 2016 11:29 AM
> To: dts@dpdk.org
> Cc: Yao, Lei A
> Subject: [dts] [PATCH V1] tests: add test for vhost/virtio loopback
> performance
>
> From: lei yao <lei.a.yao@intel.com>
>
> This test won't use qemu, but will use two testpmd thread to launch vhost-
> user and
> virtio-user to simulate the qemu user case.
>
> Signed-off-by: lei yao <lei.a.yao@intel.com>
> ---
> ...Suite_vhost_loopback_performance_virtio_user.py | 256
> +++++++++++++++++++++
> 1 file changed, 256 insertions(+)
> create mode 100644
> tests/TestSuite_vhost_loopback_performance_virtio_user.py
>
> diff --git a/tests/TestSuite_vhost_loopback_performance_virtio_user.py
> b/tests/TestSuite_vhost_loopback_performance_virtio_user.py
> new file mode 100644
> index 0000000..46d2110
> --- /dev/null
> +++ b/tests/TestSuite_vhost_loopback_performance_virtio_user.py
> @@ -0,0 +1,256 @@
> +#
> +# BSD LICENSE
> +#
> +# Copyright(c) 2010-2016 Intel Corporation. All rights reserved.
> +# All rights reserved.
> +#
> +# Redistribution and use in source and binary forms, with or without
> +# modification, are permitted provided that the following conditions
> +# are met:
> +#
> +# * Redistributions of source code must retain the above copyright
> +# notice, this list of conditions and the following disclaimer.
> +# * Redistributions in binary form must reproduce the above copyright
> +# notice, this list of conditions and the following disclaimer in
> +# the documentation and/or other materials provided with the
> +# distribution.
> +# * Neither the name of Intel Corporation nor the names of its
> +# contributors may be used to endorse or promote products derived
> +# from this software without specific prior written permission.
> +#
> +# THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
> +# "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
> +# LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
> +# A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
> +# OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
> +# SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
> +# LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
> +# DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
> +# THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
> +# (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
> +# OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
> +
> +
> +"""
> +DPDK Test suite.
> +Test Vhost loopback performance for Mergeable, normal , vector Path
> +"""
> +import string
> +import utils
> +import time
> +import re
> +from test_case import TestCase
> +
> +
> +class TestVhostLoopback(TestCase):
> +
> + def set_up_all(self):
> + """
> + Run at the start of each test suite.
> + """
> + # Clean the execution environment
> + self.dut.send_expect("rm -rf ./vhost.out", "#")
> + self.dut.send_expect("rm -rf ./vhost-net*", "#")
> + self.dut.send_expect("killall -s INT vhost-switch", "#")
> + self.dut.send_expect("killall -s INT testpmd", "#")
Dpdk application will be automated killed by framework. There's no need to kill dpdk application in set_up_all function.
> + self.header_row = ["FrameSize(B)", "Mode", "Throughput(Mpps)",
> "Virtio Version"]
> + self.frame_sizes = [64, 128, 260, 520, 1024, 1500]
> + self.test_cycles = {'Mpps': {}, 'pct': {}}
> +
> + def set_up(self):
> + """
> + Run before each test case.
> + """
> + self.dut.restore_interfaces()
> + pass
This is critical function which will bind all ports to default driver.
You can use simple code like "port.bind_driver()" to bind port to default driver and "port.bind_driver(driver=self.drivername)" to igb_uio driver.
Just in this suite, I can't see any requirement to do this.
> +
> + def test_perf_vhost_mergeable_loopback(self):
> + """
> + Benchmark performance for frame_sizes.
> + """
> +
> + # Modify the driver file to disbale the Mergeable, then re-
> compile the DPDK and back up the original driver file
> + for frame_size in self.frame_sizes:
> + # Back up the original driver file
> +
> self.dut.send_expect("cp ./drivers/net/virtio/virtio_ethdev.h ./", "#", 30)
> + # Change the packet size sent by the testpmd
> + self.change_testpmd_size = "sed -i -e 's/#define
> TXONLY_DEF_PACKET_LEN.*$/#define TXONLY_DEF_PACKET_LEN %d/' ./app/test-
> pmd/testpmd.h" % frame_size
If changed any code in testpmd, please restore it back for other suites may depend on that.
> + self.dut.send_expect(self.change_testpmd_size, "#", 30)
> + self.dut.build_install_dpdk(self.dut.target)
> +
> + # Start the vhost user side
> + cmd = self.target + "/app/testpmd -n 4 -c 0x03 " + \
> + "-m 2048 --file-prefix=vhost --vdev
> 'net_vhost0,iface=vhost-net,queues=1,client=0' -- -i"
> + self.dut.send_expect(cmd, "testpmd>", 120)
> + # Start the virtio_user side
> + vhost_user = self.dut.new_session()
> + command_line_user = self.target + "/app/testpmd -n 4 -c 0x0c
> " + \
> + " -m 2048 --no-pci --file-prefix=virtio "
> + \
> + " --
> vdev=net_virtio_user0,mac=00:01:02:03:04:05,path=./vhost-net" + \
> + " -- -i --txqflags=0xf01 --disable-hw-
> vlan-filter"
> +
> + vhost_user.send_expect(command_line_user, "testpmd>", 120)
> +
> + self.dut.send_expect("set fwd mac retry", "testpmd>", 60)
> + self.dut.send_expect("start tx_first 32", "testpmd>", 60)
> + vhost_user.send_expect("start tx_first 32", "testpmd> ", 120)
> + results = 0.0
> + out = self.dut.send_expect("show port stats all", "testpmd>",
> 60)
> + time.sleep(5)
> + # Get throughput 10 times and calculate the average
> throughput
> + for i in range(10):
> + out = self.dut.send_expect("show port stats all",
> "testpmd>", 60)
> + time.sleep(5)
> + lines = re.search("Rx-pps:\s*(\d*)", out)
> + result = lines.group(1)
> + results += float(result)
> + Mpps = results / (1000000 * 10)
> + self.result_table_create(self.header_row)
> +
> + self.dut.send_expect("quit", "#", 60)
> + vhost_user.send_expect("quit", "#", 60)
> + # Restore the driver file
> + self.dut.send_expect("rm -
> rf ./drivers/net/virtio/virtio_ethdev.h", "#", 30)
> +
> self.dut.send_expect("mv ./virtio_ethdev.h ./drivers/net/virtio/", "#", 30)
> + self.test_cycles['Mpps'][frame_size] = Mpps
> + self.test_cycles['pct'][frame_size] = "Virtio 0.95"
> +
> + for frame_size in self.frame_sizes:
> + results_row = [frame_size]
> + results_row.append("Mergeable on")
> + results_row.append(self.test_cycles['Mpps'][frame_size])
> + results_row.append(self.test_cycles['pct'][frame_size])
> + self.result_table_add(results_row)
> +
> + self.result_table_print()
> +
> + def test_perf_vhost_vector_loopback(self):
> + """
> + Benchmark performance for frame_sizes.
> + """
> + for frame_size in self.frame_sizes:
> + # Modify the driver file to disbale the Mergeable, then re-
> compile the DPDK and back up the original driver file
> +
> self.dut.send_expect("cp ./drivers/net/virtio/virtio_ethdev.h ./", "#", 30)
> + self.dut.send_expect("sed -i
> '/VIRTIO_NET_F_MRG_RXBUF/d' ./drivers/net/virtio/virtio_ethdev.h", "#", 30)
> + self.change_testpmd_size = "sed -i -e 's/#define
> TXONLY_DEF_PACKET_LEN .*$/#define TXONLY_DEF_PACKET_LEN %d/' ./app/test-
> pmd/testpmd.h" % frame_size
Same comments as previous.
> + self.dut.send_expect(self.change_testpmd_size, "#", 30)
> + self.dut.build_install_dpdk(self.dut.target)
> +
> + # Start the vhost user side
> + cmd = self.target + "/app/testpmd -n 4 -c 0x03 " + \
> + "-m 2048 --file-prefix=vhost --vdev
> 'net_vhost0,iface=vhost-net,queues=1,client=0' -- -i"
> + self.dut.send_expect(cmd, "testpmd>", 120)
> + # Start the virtio_user side
> + vhost_user = self.dut.new_session()
> + command_line_user = self.target + "/app/testpmd -n 4 -c 0x0c
> " + \
> + " -m 2048 --no-pci --file-prefix=virtio "
> + \
> + " --
> vdev=net_virtio_user0,mac=00:01:02:03:04:05,path=./vhost-net " + \
> + " -- -i --txqflags=0xf01 --disable-hw-
> vlan-filter"
> +
> + vhost_user.send_expect(command_line_user, "testpmd>", 120)
> +
> + self.dut.send_expect("set fwd mac retry", "testpmd>", 60)
> + self.dut.send_expect("start tx_first 32", "testpmd>", 60)
> + vhost_user.send_expect("start tx_first 32", "testpmd> ", 120)
> + results = 0.0
> + out = self.dut.send_expect("show port stats all", "testpmd>",
> 60)
> + time.sleep(5)
> + # Get throughput 10 times and calculate the average
> throughput
> + for i in range(10):
> + out = self.dut.send_expect("show port stats all",
> "testpmd>", 60)
> + time.sleep(5)
> + lines = re.search("Rx-pps:\s*(\d*)", out)
> + result = lines.group(1)
> + results += float(result)
> + Mpps = results / (1000000 * 10)
> + self.result_table_create(self.header_row)
Code look like not aligned here, please check again.
> +
> + self.dut.send_expect("quit", "#", 60)
> + vhost_user.send_expect("quit", "#", 60)
> + # Restore the driver file
> + self.dut.send_expect("rm -
> rf ./drivers/net/virtio/virtio_ethdev.h", "#", 30)
> +
> self.dut.send_expect("mv ./virtio_ethdev.h ./drivers/net/virtio/", "#", 30)
> + self.test_cycles['Mpps'][frame_size] = Mpps
> + self.test_cycles['pct'][frame_size] = "Virtio 0.95"
> +
> + for frame_size in self.frame_sizes:
> + results_row = [frame_size]
> + results_row.append("Vector on")
> + results_row.append(self.test_cycles['Mpps'][frame_size])
> + results_row.append(self.test_cycles['pct'][frame_size])
> + self.result_table_add(results_row)
> +
> + self.result_table_print()
> +
> + def test_perf_vhost_normal_loopback(self):
> + """
> + Benchmark performance for frame_sizes.
> + """
> +
> + self.result_table_create(self.header_row)
> + for frame_size in self.frame_sizes:
> + # Modify the driver file to disbale the Mergeable, then re-
> compile the DPDK and back up the original driver file
> +
> self.dut.send_expect("cp ./drivers/net/virtio/virtio_ethdev.h ./", "#", 30)
> + self.dut.send_expect("sed -i
> '/VIRTIO_NET_F_MRG_RXBUF/d' ./drivers/net/virtio/virtio_ethdev.h", "#", 30)
> + self.change_testpmd_size = "sed -i -e 's/#define
> TXONLY_DEF_PACKET_LEN .*$/#define TXONLY_DEF_PACKET_LEN %d/' ./app/test-
> pmd/testpmd.h" % frame_size
> + self.dut.send_expect(self.change_testpmd_size, "#", 30)
> + self.dut.build_install_dpdk(self.dut.target)
> +
> + # Start the vhost user side
> + cmd = self.target + "/app/testpmd -n 4 -c 0x03 " + \
> + "-m 2048 --file-prefix=vhost --vdev
> 'net_vhost0,iface=vhost-net,queues=1,client=0' -- -i"
> + self.dut.send_expect(cmd, "testpmd>", 120)
> + # Start the virtio_user side
> + vhost_user = self.dut.new_session()
> + command_line_user = self.target + "/app/testpmd -n 4 -c 0x0c
> " + \
> + " -m 2048 --no-pci --file-prefix=virtio "
> + \
> + " --
> vdev=net_virtio_user0,mac=00:01:02:03:04:05,path=./vhost-net" + \
> + " -- -i --txqflags=0xf00 --disable-hw-
> vlan-filter"
> +
> + vhost_user.send_expect(command_line_user, "testpmd>", 120)
> +
> + self.dut.send_expect("set fwd mac retry", "testpmd>", 60)
> + self.dut.send_expect("start tx_first 32", "testpmd>", 60)
> + vhost_user.send_expect("start tx_first 32", "testpmd> ", 120)
> + results = 0.0
> + out = self.dut.send_expect("show port stats all", "testpmd>",
> 60)
> + time.sleep(5)
> + # Get throughput 10 times and calculate the average
> throughput
> + for i in range(10):
> + out = self.dut.send_expect("show port stats all",
> "testpmd>", 60)
> + time.sleep(5)
> + lines = re.search("Rx-pps:\s*(\d*)", out)
> + result = lines.group(1)
> + results += float(result)
> + Mpps = results / (1000000 * 10)
> + self.result_table_create(self.header_row)
> +
> + self.dut.send_expect("quit", "#", 60)
> + vhost_user.send_expect("quit", "#", 60)
> + # Restore the driver file
> + self.dut.send_expect("rm -
> rf ./drivers/net/virtio/virtio_ethdev.h", "#", 30)
> +
> self.dut.send_expect("mv ./virtio_ethdev.h ./drivers/net/virtio/", "#", 30)
> + self.test_cycles['Mpps'][frame_size] = Mpps
> + self.test_cycles['pct'][frame_size] = "Virtio 0.95"
> +
> + for frame_size in self.frame_sizes:
> + results_row = [frame_size]
> + results_row.append("Normal")
> + results_row.append(self.test_cycles['Mpps'][frame_size])
> + results_row.append(self.test_cycles['pct'][frame_size])
> + self.result_table_add(results_row)
> +
> + self.result_table_print()
> +
> + def tear_down(self):
> + """
> + Run after each test case.
> + """
> + time.sleep(2)
> +
> + def tear_down_all(self):
> + """
> + Run after each test suite.
> + """
> + # Recompile the dpdk because we change the source code during the
> test
Here need to restore default Tx packet size.
> + self.dut.build_install_dpdk(self.dut.target)
> --
> 2.7.4
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [dts] [PATCH V1] tests: add test for vhost/virtio loopback performance
2016-12-28 5:28 ` Liu, Yong
@ 2016-12-28 5:56 ` Yao, Lei A
2016-12-28 7:21 ` Liu, Yong
0 siblings, 1 reply; 4+ messages in thread
From: Yao, Lei A @ 2016-12-28 5:56 UTC (permalink / raw)
To: Liu, Yong, dts
Some update. Thanks.
> -----Original Message-----
> From: Liu, Yong
> Sent: Wednesday, December 28, 2016 1:28 PM
> To: Yao, Lei A <lei.a.yao@intel.com>; dts@dpdk.org
> Cc: Yao, Lei A <lei.a.yao@intel.com>
> Subject: RE: [dts] [PATCH V1] tests: add test for vhost/virtio loopback
> performance
>
> Lei, some comments below.
>
> > -----Original Message-----
> > From: dts [mailto:dts-bounces@dpdk.org] On Behalf Of lei,yao
> > Sent: Thursday, December 22, 2016 11:29 AM
> > To: dts@dpdk.org
> > Cc: Yao, Lei A
> > Subject: [dts] [PATCH V1] tests: add test for vhost/virtio loopback
> > performance
> >
> > From: lei yao <lei.a.yao@intel.com>
> >
> > This test won't use qemu, but will use two testpmd thread to launch vhost-
> > user and
> > virtio-user to simulate the qemu user case.
> >
> > Signed-off-by: lei yao <lei.a.yao@intel.com>
> > ---
> > ...Suite_vhost_loopback_performance_virtio_user.py | 256
> > +++++++++++++++++++++
> > 1 file changed, 256 insertions(+)
> > create mode 100644
> > tests/TestSuite_vhost_loopback_performance_virtio_user.py
> >
> > diff --git a/tests/TestSuite_vhost_loopback_performance_virtio_user.py
> > b/tests/TestSuite_vhost_loopback_performance_virtio_user.py
> > new file mode 100644
> > index 0000000..46d2110
> > --- /dev/null
> > +++ b/tests/TestSuite_vhost_loopback_performance_virtio_user.py
> > @@ -0,0 +1,256 @@
> > +#
> > +# BSD LICENSE
> > +#
> > +# Copyright(c) 2010-2016 Intel Corporation. All rights reserved.
> > +# All rights reserved.
> > +#
> > +# Redistribution and use in source and binary forms, with or without
> > +# modification, are permitted provided that the following conditions
> > +# are met:
> > +#
> > +# * Redistributions of source code must retain the above copyright
> > +# notice, this list of conditions and the following disclaimer.
> > +# * Redistributions in binary form must reproduce the above copyright
> > +# notice, this list of conditions and the following disclaimer in
> > +# the documentation and/or other materials provided with the
> > +# distribution.
> > +# * Neither the name of Intel Corporation nor the names of its
> > +# contributors may be used to endorse or promote products derived
> > +# from this software without specific prior written permission.
> > +#
> > +# THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND
> CONTRIBUTORS
> > +# "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT
> NOT
> > +# LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND
> FITNESS FOR
> > +# A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE
> COPYRIGHT
> > +# OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT,
> INCIDENTAL,
> > +# SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT
> NOT
> > +# LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS
> OF USE,
> > +# DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED
> AND ON ANY
> > +# THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR
> TORT
> > +# (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT
> OF THE USE
> > +# OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH
> DAMAGE.
> > +
> > +
> > +"""
> > +DPDK Test suite.
> > +Test Vhost loopback performance for Mergeable, normal , vector Path
> > +"""
> > +import string
> > +import utils
> > +import time
> > +import re
> > +from test_case import TestCase
> > +
> > +
> > +class TestVhostLoopback(TestCase):
> > +
> > + def set_up_all(self):
> > + """
> > + Run at the start of each test suite.
> > + """
> > + # Clean the execution environment
> > + self.dut.send_expect("rm -rf ./vhost.out", "#")
> > + self.dut.send_expect("rm -rf ./vhost-net*", "#")
> > + self.dut.send_expect("killall -s INT vhost-switch", "#")
> > + self.dut.send_expect("killall -s INT testpmd", "#")
>
> Dpdk application will be automated killed by framework. There's no need to
> kill dpdk application in set_up_all function.
>
So both vhost-switch and testpmd will be cleaned by framework, right?
> > + self.header_row = ["FrameSize(B)", "Mode", "Throughput(Mpps)",
> > "Virtio Version"]
> > + self.frame_sizes = [64, 128, 260, 520, 1024, 1500]
> > + self.test_cycles = {'Mpps': {}, 'pct': {}}
> > +
> > + def set_up(self):
> > + """
> > + Run before each test case.
> > + """
> > + self.dut.restore_interfaces()
> > + pass
>
> This is critical function which will bind all ports to default driver.
> You can use simple code like "port.bind_driver()" to bind port to default
> driver and "port.bind_driver(driver=self.drivername)" to igb_uio driver.
> Just in this suite, I can't see any requirement to do this.
>
Because this test cases don't need any NIC binned to pmd driver. So I use this function
to clean it.
> > +
> > + def test_perf_vhost_mergeable_loopback(self):
> > + """
> > + Benchmark performance for frame_sizes.
> > + """
> > +
> > + # Modify the driver file to disbale the Mergeable, then re-
> > compile the DPDK and back up the original driver file
> > + for frame_size in self.frame_sizes:
> > + # Back up the original driver file
> > +
> > self.dut.send_expect("cp ./drivers/net/virtio/virtio_ethdev.h ./", "#", 30)
> > + # Change the packet size sent by the testpmd
> > + self.change_testpmd_size = "sed -i -e 's/#define
> > TXONLY_DEF_PACKET_LEN.*$/#define
> TXONLY_DEF_PACKET_LEN %d/' ./app/test-
> > pmd/testpmd.h" % frame_size
>
> If changed any code in testpmd, please restore it back for other suites may
> depend on that.
Sure. I will restore the testpmd for every test in V2.
> > + self.dut.send_expect(self.change_testpmd_size, "#", 30)
> > + self.dut.build_install_dpdk(self.dut.target)
> > +
> > + # Start the vhost user side
> > + cmd = self.target + "/app/testpmd -n 4 -c 0x03 " + \
> > + "-m 2048 --file-prefix=vhost --vdev
> > 'net_vhost0,iface=vhost-net,queues=1,client=0' -- -i"
> > + self.dut.send_expect(cmd, "testpmd>", 120)
> > + # Start the virtio_user side
> > + vhost_user = self.dut.new_session()
> > + command_line_user = self.target + "/app/testpmd -n 4 -c 0x0c
> > " + \
> > + " -m 2048 --no-pci --file-prefix=virtio "
> > + \
> > + " --
> > vdev=net_virtio_user0,mac=00:01:02:03:04:05,path=./vhost-net" + \
> > + " -- -i --txqflags=0xf01 --disable-hw-
> > vlan-filter"
> > +
> > + vhost_user.send_expect(command_line_user, "testpmd>", 120)
> > +
> > + self.dut.send_expect("set fwd mac retry", "testpmd>", 60)
> > + self.dut.send_expect("start tx_first 32", "testpmd>", 60)
> > + vhost_user.send_expect("start tx_first 32", "testpmd> ", 120)
> > + results = 0.0
> > + out = self.dut.send_expect("show port stats all", "testpmd>",
> > 60)
> > + time.sleep(5)
> > + # Get throughput 10 times and calculate the average
> > throughput
> > + for i in range(10):
> > + out = self.dut.send_expect("show port stats all",
> > "testpmd>", 60)
> > + time.sleep(5)
> > + lines = re.search("Rx-pps:\s*(\d*)", out)
> > + result = lines.group(1)
> > + results += float(result)
> > + Mpps = results / (1000000 * 10)
> > + self.result_table_create(self.header_row)
> > +
> > + self.dut.send_expect("quit", "#", 60)
> > + vhost_user.send_expect("quit", "#", 60)
> > + # Restore the driver file
> > + self.dut.send_expect("rm -
> > rf ./drivers/net/virtio/virtio_ethdev.h", "#", 30)
> > +
> > self.dut.send_expect("mv ./virtio_ethdev.h ./drivers/net/virtio/", "#", 30)
> > + self.test_cycles['Mpps'][frame_size] = Mpps
> > + self.test_cycles['pct'][frame_size] = "Virtio 0.95"
> > +
> > + for frame_size in self.frame_sizes:
> > + results_row = [frame_size]
> > + results_row.append("Mergeable on")
> > + results_row.append(self.test_cycles['Mpps'][frame_size])
> > + results_row.append(self.test_cycles['pct'][frame_size])
> > + self.result_table_add(results_row)
> > +
> > + self.result_table_print()
> > +
> > + def test_perf_vhost_vector_loopback(self):
> > + """
> > + Benchmark performance for frame_sizes.
> > + """
> > + for frame_size in self.frame_sizes:
> > + # Modify the driver file to disbale the Mergeable, then re-
> > compile the DPDK and back up the original driver file
> > +
> > self.dut.send_expect("cp ./drivers/net/virtio/virtio_ethdev.h ./", "#", 30)
> > + self.dut.send_expect("sed -i
> > '/VIRTIO_NET_F_MRG_RXBUF/d' ./drivers/net/virtio/virtio_ethdev.h", "#",
> 30)
> > + self.change_testpmd_size = "sed -i -e 's/#define
> > TXONLY_DEF_PACKET_LEN .*$/#define
> TXONLY_DEF_PACKET_LEN %d/' ./app/test-
> > pmd/testpmd.h" % frame_size
>
> Same comments as previous.
>
> > + self.dut.send_expect(self.change_testpmd_size, "#", 30)
> > + self.dut.build_install_dpdk(self.dut.target)
> > +
> > + # Start the vhost user side
> > + cmd = self.target + "/app/testpmd -n 4 -c 0x03 " + \
> > + "-m 2048 --file-prefix=vhost --vdev
> > 'net_vhost0,iface=vhost-net,queues=1,client=0' -- -i"
> > + self.dut.send_expect(cmd, "testpmd>", 120)
> > + # Start the virtio_user side
> > + vhost_user = self.dut.new_session()
> > + command_line_user = self.target + "/app/testpmd -n 4 -c 0x0c
> > " + \
> > + " -m 2048 --no-pci --file-prefix=virtio "
> > + \
> > + " --
> > vdev=net_virtio_user0,mac=00:01:02:03:04:05,path=./vhost-net " + \
> > + " -- -i --txqflags=0xf01 --disable-hw-
> > vlan-filter"
> > +
> > + vhost_user.send_expect(command_line_user, "testpmd>", 120)
> > +
> > + self.dut.send_expect("set fwd mac retry", "testpmd>", 60)
> > + self.dut.send_expect("start tx_first 32", "testpmd>", 60)
> > + vhost_user.send_expect("start tx_first 32", "testpmd> ", 120)
> > + results = 0.0
> > + out = self.dut.send_expect("show port stats all", "testpmd>",
> > 60)
> > + time.sleep(5)
> > + # Get throughput 10 times and calculate the average
> > throughput
> > + for i in range(10):
> > + out = self.dut.send_expect("show port stats all",
> > "testpmd>", 60)
> > + time.sleep(5)
> > + lines = re.search("Rx-pps:\s*(\d*)", out)
> > + result = lines.group(1)
> > + results += float(result)
> > + Mpps = results / (1000000 * 10)
> > + self.result_table_create(self.header_row)
>
> Code look like not aligned here, please check again.
>
> > +
> > + self.dut.send_expect("quit", "#", 60)
> > + vhost_user.send_expect("quit", "#", 60)
> > + # Restore the driver file
> > + self.dut.send_expect("rm -
> > rf ./drivers/net/virtio/virtio_ethdev.h", "#", 30)
> > +
> > self.dut.send_expect("mv ./virtio_ethdev.h ./drivers/net/virtio/", "#", 30)
> > + self.test_cycles['Mpps'][frame_size] = Mpps
> > + self.test_cycles['pct'][frame_size] = "Virtio 0.95"
> > +
> > + for frame_size in self.frame_sizes:
> > + results_row = [frame_size]
> > + results_row.append("Vector on")
> > + results_row.append(self.test_cycles['Mpps'][frame_size])
> > + results_row.append(self.test_cycles['pct'][frame_size])
> > + self.result_table_add(results_row)
> > +
> > + self.result_table_print()
> > +
> > + def test_perf_vhost_normal_loopback(self):
> > + """
> > + Benchmark performance for frame_sizes.
> > + """
> > +
> > + self.result_table_create(self.header_row)
> > + for frame_size in self.frame_sizes:
> > + # Modify the driver file to disbale the Mergeable, then re-
> > compile the DPDK and back up the original driver file
> > +
> > self.dut.send_expect("cp ./drivers/net/virtio/virtio_ethdev.h ./", "#", 30)
> > + self.dut.send_expect("sed -i
> > '/VIRTIO_NET_F_MRG_RXBUF/d' ./drivers/net/virtio/virtio_ethdev.h", "#",
> 30)
> > + self.change_testpmd_size = "sed -i -e 's/#define
> > TXONLY_DEF_PACKET_LEN .*$/#define
> TXONLY_DEF_PACKET_LEN %d/' ./app/test-
> > pmd/testpmd.h" % frame_size
> > + self.dut.send_expect(self.change_testpmd_size, "#", 30)
> > + self.dut.build_install_dpdk(self.dut.target)
> > +
> > + # Start the vhost user side
> > + cmd = self.target + "/app/testpmd -n 4 -c 0x03 " + \
> > + "-m 2048 --file-prefix=vhost --vdev
> > 'net_vhost0,iface=vhost-net,queues=1,client=0' -- -i"
> > + self.dut.send_expect(cmd, "testpmd>", 120)
> > + # Start the virtio_user side
> > + vhost_user = self.dut.new_session()
> > + command_line_user = self.target + "/app/testpmd -n 4 -c 0x0c
> > " + \
> > + " -m 2048 --no-pci --file-prefix=virtio "
> > + \
> > + " --
> > vdev=net_virtio_user0,mac=00:01:02:03:04:05,path=./vhost-net" + \
> > + " -- -i --txqflags=0xf00 --disable-hw-
> > vlan-filter"
> > +
> > + vhost_user.send_expect(command_line_user, "testpmd>", 120)
> > +
> > + self.dut.send_expect("set fwd mac retry", "testpmd>", 60)
> > + self.dut.send_expect("start tx_first 32", "testpmd>", 60)
> > + vhost_user.send_expect("start tx_first 32", "testpmd> ", 120)
> > + results = 0.0
> > + out = self.dut.send_expect("show port stats all", "testpmd>",
> > 60)
> > + time.sleep(5)
> > + # Get throughput 10 times and calculate the average
> > throughput
> > + for i in range(10):
> > + out = self.dut.send_expect("show port stats all",
> > "testpmd>", 60)
> > + time.sleep(5)
> > + lines = re.search("Rx-pps:\s*(\d*)", out)
> > + result = lines.group(1)
> > + results += float(result)
> > + Mpps = results / (1000000 * 10)
> > + self.result_table_create(self.header_row)
> > +
> > + self.dut.send_expect("quit", "#", 60)
> > + vhost_user.send_expect("quit", "#", 60)
> > + # Restore the driver file
> > + self.dut.send_expect("rm -
> > rf ./drivers/net/virtio/virtio_ethdev.h", "#", 30)
> > +
> > self.dut.send_expect("mv ./virtio_ethdev.h ./drivers/net/virtio/", "#", 30)
> > + self.test_cycles['Mpps'][frame_size] = Mpps
> > + self.test_cycles['pct'][frame_size] = "Virtio 0.95"
> > +
> > + for frame_size in self.frame_sizes:
> > + results_row = [frame_size]
> > + results_row.append("Normal")
> > + results_row.append(self.test_cycles['Mpps'][frame_size])
> > + results_row.append(self.test_cycles['pct'][frame_size])
> > + self.result_table_add(results_row)
> > +
> > + self.result_table_print()
> > +
> > + def tear_down(self):
> > + """
> > + Run after each test case.
> > + """
> > + time.sleep(2)
> > +
> > + def tear_down_all(self):
> > + """
> > + Run after each test suite.
> > + """
> > + # Recompile the dpdk because we change the source code during the
> > test
>
> Here need to restore default Tx packet size.
Sure. I will add it.
>
> > + self.dut.build_install_dpdk(self.dut.target)
> > --
> > 2.7.4
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [dts] [PATCH V1] tests: add test for vhost/virtio loopback performance
2016-12-28 5:56 ` Yao, Lei A
@ 2016-12-28 7:21 ` Liu, Yong
0 siblings, 0 replies; 4+ messages in thread
From: Liu, Yong @ 2016-12-28 7:21 UTC (permalink / raw)
To: Yao, Lei A, dts
Thanks, Lei.
> -----Original Message-----
> From: Yao, Lei A
> Sent: Wednesday, December 28, 2016 1:56 PM
> To: Liu, Yong; dts@dpdk.org
> Subject: RE: [dts] [PATCH V1] tests: add test for vhost/virtio loopback
> performance
>
> Some update. Thanks.
>
> > -----Original Message-----
> > From: Liu, Yong
> > Sent: Wednesday, December 28, 2016 1:28 PM
> > To: Yao, Lei A <lei.a.yao@intel.com>; dts@dpdk.org
> > Cc: Yao, Lei A <lei.a.yao@intel.com>
> > Subject: RE: [dts] [PATCH V1] tests: add test for vhost/virtio loopback
> > performance
> >
> > Lei, some comments below.
> >
> > > -----Original Message-----
> > > From: dts [mailto:dts-bounces@dpdk.org] On Behalf Of lei,yao
> > > Sent: Thursday, December 22, 2016 11:29 AM
> > > To: dts@dpdk.org
> > > Cc: Yao, Lei A
> > > Subject: [dts] [PATCH V1] tests: add test for vhost/virtio loopback
> > > performance
> > >
> > > From: lei yao <lei.a.yao@intel.com>
> > >
> > > + self.dut.send_expect("killall -s INT vhost-switch", "#")
> > > + self.dut.send_expect("killall -s INT testpmd", "#")
> >
> > Dpdk application will be automated killed by framework. There's no need
> to
> > kill dpdk application in set_up_all function.
> >
> So both vhost-switch and testpmd will be cleaned by framework, right?
>
Yes.
> > > + self.header_row = ["FrameSize(B)", "Mode", "Throughput(Mpps)",
> > > "Virtio Version"]
> > > + self.frame_sizes = [64, 128, 260, 520, 1024, 1500]
> > > + self.test_cycles = {'Mpps': {}, 'pct': {}}
> > > +
> > > + def set_up(self):
> > > + """
> > > + Run before each test case.
> > > + """
> > > + self.dut.restore_interfaces()
> > > + pass
> >
> > This is critical function which will bind all ports to default driver.
> > You can use simple code like "port.bind_driver()" to bind port to
> default
> > driver and "port.bind_driver(driver=self.drivername)" to igb_uio driver.
> > Just in this suite, I can't see any requirement to do this.
> >
> Because this test cases don't need any NIC binned to pmd driver. So I use
> this function
> to clean it.
By now if you do that, all later suites will be failed. I remember that Lijuan is developing one patch for restore all ports before running suite.
Based on that patch, you can do this function in set_up_all function.
But now I recommend that change drivername to blank and then run this suite.
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2016-12-28 7:21 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-12-22 3:29 [dts] [PATCH V1] tests: add test for vhost/virtio loopback performance lei,yao
2016-12-28 5:28 ` Liu, Yong
2016-12-28 5:56 ` Yao, Lei A
2016-12-28 7:21 ` 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).