From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id A6EB4A0471 for ; Wed, 17 Jul 2019 08:22:46 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 9C5145B3C; Wed, 17 Jul 2019 08:22:46 +0200 (CEST) Received: from mga09.intel.com (mga09.intel.com [134.134.136.24]) by dpdk.org (Postfix) with ESMTP id 8246923D for ; Wed, 17 Jul 2019 08:22:44 +0200 (CEST) X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga004.jf.intel.com ([10.7.209.38]) by orsmga102.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 16 Jul 2019 23:22:43 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.64,273,1559545200"; d="scan'208";a="319219958" Received: from fmsmsx104.amr.corp.intel.com ([10.18.124.202]) by orsmga004.jf.intel.com with ESMTP; 16 Jul 2019 23:22:43 -0700 Received: from fmsmsx601.amr.corp.intel.com (10.18.126.81) by fmsmsx104.amr.corp.intel.com (10.18.124.202) with Microsoft SMTP Server (TLS) id 14.3.439.0; Tue, 16 Jul 2019 23:22:42 -0700 Received: from fmsmsx601.amr.corp.intel.com (10.18.126.81) by fmsmsx601.amr.corp.intel.com (10.18.126.81) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.1713.5; Tue, 16 Jul 2019 23:22:42 -0700 Received: from shsmsx106.ccr.corp.intel.com (10.239.4.159) by fmsmsx601.amr.corp.intel.com (10.18.126.81) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256) id 15.1.1713.5 via Frontend Transport; Tue, 16 Jul 2019 23:22:42 -0700 Received: from shsmsx101.ccr.corp.intel.com ([169.254.1.134]) by SHSMSX106.ccr.corp.intel.com ([169.254.10.240]) with mapi id 14.03.0439.000; Wed, 17 Jul 2019 14:22:41 +0800 From: "Ma, LihongX" To: "Wang, Yinan" , "dts@dpdk.org" Thread-Topic: [dts][PATCH V1] tests: add testsuite pvp vhost user reconnect Thread-Index: AQHVO7Ev0aVmTuS5oUKICcejTk90c6bN0V8AgACHQHA= Date: Wed, 17 Jul 2019 06:22:40 +0000 Message-ID: References: <1563239473-21171-1-git-send-email-lihongx.ma@intel.com> In-Reply-To: Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.239.127.40] Content-Type: text/plain; charset="iso-2022-jp" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Subject: Re: [dts] [PATCH V1] tests: add testsuite pvp vhost user reconnect X-BeenThere: dts@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: test suite reviews and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dts-bounces@dpdk.org Sender: "dts" Tested-by: lihong -----Original Message----- From: Wang, Yinan=20 Sent: Wednesday, July 17, 2019 2:18 PM To: Ma, LihongX ; dts@dpdk.org Subject: RE: [dts][PATCH V1] tests: add testsuite pvp vhost user reconnect Acked-by: Wang, Yinan > -----Original Message----- > From: Ma, LihongX > Sent: 2019=1B$BG/=1B(B7=1B$B7n=1B(B16=1B$BF|=1B(B 9:11 > To: dts@dpdk.org > Cc: Wang, Yinan ; Ma, LihongX=20 > > Subject: [dts][PATCH V1] tests: add testsuite pvp vhost user reconnect >=20 > Signed-off-by: lihong > --- > tests/TestSuite_pvp_vhost_user_reconnect.py | 407 > ++++++++++++++++++++++++++++ > 1 file changed, 407 insertions(+) > create mode 100644 tests/TestSuite_pvp_vhost_user_reconnect.py >=20 > diff --git a/tests/TestSuite_pvp_vhost_user_reconnect.py > b/tests/TestSuite_pvp_vhost_user_reconnect.py > new file mode 100644 > index 0000000..5568376 > --- /dev/null > +++ b/tests/TestSuite_pvp_vhost_user_reconnect.py > @@ -0,0 +1,407 @@ > +# BSD LICENSE > +# > +# Copyright(c) <2019> Intel Corporation. > +# All rights reserved. > +# > +# Redistribution and use in source and binary forms, with or without=20 > +# modification, are permitted provided that the following conditions=20 > +# 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 #=20 > +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 #=20 > +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 #=20 > +(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. > + > +Vhost reconnect two VM test suite. > +Becase this suite will use the reconnet feature, the VM will start as=20 > +server mode, so the qemu version should greater than 2.7 """ > +import re > +import utils > +import time > +from test_case import TestCase > +from settings import HEADER_SIZE > +from virt_common import VM > +from pktgen import PacketGeneratorHelper > + > + > +class TestPVPVhostUserReconnect(TestCase): > + > + def set_up_all(self): > + > + # Get and verify the ports > + self.dut_ports =3D self.dut.get_ports() > + self.verify(len(self.dut_ports) >=3D 1, "Insufficient ports for > + testing") > + > + # Get the port's socket > + self.pf =3D self.dut_ports[0] > + netdev =3D self.dut.ports_info[self.pf]['port'] > + self.socket =3D netdev.get_nic_socket() > + self.cores =3D self.dut.get_core_list("1S/2C/1T", socket=3Dself.= socket) > + self.coremask =3D utils.create_mask(self.cores) > + self.memory_channel =3D self.dut.get_memory_channels() > + self.dst_mac =3D self.dut.get_mac_address(self.dut_ports[0]) > + # set diff arg about mem_socket base on socket number > + if len(set([int(core['socket']) for core in self.dut.cores])) = =3D=3D 1: > + self.socket_mem =3D '1024' > + else: > + self.socket_mem =3D '1024,1024' > + > + self.reconnect_times =3D 5 > + self.vm_num =3D 1 > + self.frame_sizes =3D [64, 1518] > + self.virtio_ip =3D ["1.1.1.2", "1.1.1.3"] > + self.virtio_mac =3D ["52:54:00:00:00:01", > + "52:54:00:00:00:02"] > + self.src1 =3D "192.168.4.1" > + self.dst1 =3D "192.168.3.1" > + self.checked_vm =3D False > + self.header_size =3D HEADER_SIZE['eth'] + HEADER_SIZE['ip'] +=20 > + HEADER_SIZE['udp'] > + > + # create an instance to set stream field setting > + self.pktgen_helper =3D PacketGeneratorHelper() > + > + def set_up(self): > + """ > + run before each test case. > + clear the execution ENV > + """ > + self.dut.send_expect("killall -s INT testpmd", "# ") > + self.dut.send_expect("killall -s INT qemu-system-x86_64", "# ") > + self.dut.send_expect("rm -rf ./vhost-net*", "# ") > + self.vhost_user =3D self.dut.new_session(suite=3D"vhost-user") > + > + def launch_testpmd_as_vhost_user(self): > + """ > + launch the testpmd as vhost user > + """ > + vdev_info =3D "" > + for i in range(self.vm_num): > + vdev_info +=3D "--vdev > 'net_vhost%d,iface=3Dvhost-net%d,client=3D1,queues=3D1' " % (i, i) > + self.vhostapp_testcmd =3D self.dut.base_dir + \ > + "/%s/app/testpmd -c %s -n %d --socket-mem %s > --legacy-mem" + \ > + " --file-prefix=3Dvhost %s" + \ > + " -- -i --port-topology=3Dchained --nb-cores=3D1" + = \ > + " --txd=3D1024 --rxd=3D1024" > + self.vhostapp_testcmd =3D self.vhostapp_testcmd % (self.target, > + self.coremask, self.memory_channel, > + self.socket_mem, vdev_info) > + self.vhost_user.send_expect(self.vhostapp_testcmd, "testpmd> ", = 40) > + self.vhost_user.send_expect("set fwd mac", "testpmd> ", 40) > + self.vhost_user.send_expect("start", "testpmd> ", 40) > + > + def launch_testpmd_as_vhost_user_with_no_pci(self): > + """ > + launch the testpmd as vhost user > + """ > + vdev_info =3D "" > + for i in range(self.vm_num): > + vdev_info +=3D "--vdev > 'net_vhost%d,iface=3Dvhost-net%d,client=3D1,queues=3D1' " % (i, i) > + self.vhostapp_testcmd =3D self.dut.base_dir + \ > + "/%s/app/testpmd -c %s -n %d --socket-mem %s > --legacy-mem" + \ > + " --no-pci --file-prefix=3Dvhost %s" + \ > + " -- -i --nb-cores=3D1 --txd=3D1024 --rxd=3D1024" > + self.vhostapp_testcmd =3D self.vhostapp_testcmd % (self.target, > + self.coremask, self.memory_channel, > + self.socket_mem, vdev_info) > + self.vhost_user.send_expect(self.vhostapp_testcmd, "testpmd> ", = 40) > + self.vhost_user.send_expect("start", "testpmd> ", 40) > + > + def check_link_status_after_testpmd_start(self, dut_info): > + """ > + check the link status is up after testpmd start > + """ > + loop =3D 1 > + while(loop <=3D 5): > + out =3D dut_info.send_expect("show port info all",=20 > + "testpmd> ", > 120) > + port_status =3D re.findall("Link\s*status:\s*([a-z]*)", out) > + if("down" not in port_status): > + break > + time.sleep(3) > + loop =3D loop + 1 > + > + self.verify("down" not in port_status, "port can not up after > + restart") > + > + def check_qemu_version(self, vm_config): > + """ > + in this suite, the qemu version should greater 2.7 > + """ > + if self.checked_vm: > + return > + > + self.vm_qemu_version =3D vm_config.qemu_emulator > + params_number =3D len(vm_config.params) > + for i in range(params_number): > + if vm_config.params[i].keys()[0] =3D=3D 'qemu': > + self.vm_qemu_version =3D=20 > + vm_config.params[i]['qemu'][0]['path'] > + > + out =3D self.dut.send_expect("%s --version" %=20 > + self.vm_qemu_version, > "#") > + result =3D re.search("QEMU\s*emulator\s*version\s*(\d*.\d*)", ou= t) > + self.verify(result is not None, > + 'the qemu path may be not right: %s' % > self.vm_qemu_version) > + version =3D result.group(1) > + index =3D version.find('.') > + self.verify(int(version[:index]) > 2 or > + (int(version[:index]) =3D=3D 2 and=20 > + int(version[index+1:]) >=3D > 7), > + 'This qemu version should greater than 2.7 ' + \ > + 'in this suite, please config it in vhost_sample.cfg= file') > + self.checked_vm =3D True > + > + def start_vms(self): > + """ > + start two VM > + """ > + self.vm_dut =3D [] > + self.vm =3D [] > + for i in range(self.vm_num): > + vm_info =3D VM(self.dut, 'vm%d' % i, 'vhost_sample') > + vm_params =3D {} > + vm_params['driver'] =3D 'vhost-user' > + vm_params['opt_path'] =3D './vhost-net%d' % (i) > + vm_params['opt_mac'] =3D '52:54:00:00:00:0%d' % (i+1) > + vm_params['opt_server'] =3D 'server' > + vm_params['opt_settings'] =3D > 'mrg_rxbuf=3Don,rx_queue_size=3D1024,tx_queue_size=3D1024' > + vm_info.set_vm_device(**vm_params) > + self.check_qemu_version(vm_info) > + > + try: > + vm_dut =3D None > + vm_dut =3D vm_info.start() > + if vm_dut is None: > + raise Exception("Set up VM ENV failed") > + except Exception as e: > + print utils.RED("Failure for %s" % str(e)) > + self.verify(vm_dut is not None, "start vm failed") > + self.vm_dut.append(vm_dut) > + self.vm.append(vm_info) > + > + def vm_testpmd_start(self): > + """ > + start testpmd in vm > + """ > + vm_testpmd =3D self.dut.target + "/app/testpmd -c 0x3 -n 4 " + \ > + "-- -i --port-topology=3Dchained --txd=3D1024=20 > + --rxd=3D1024 > " > + for i in range(len(self.vm_dut)): > + self.vm_dut[i].send_expect(vm_testpmd, "testpmd> ", 20) > + self.vm_dut[i].send_expect("set fwd mac", "testpmd> ") > + self.vm_dut[i].send_expect("start", "testpmd> ") > + > + self.check_link_status_after_testpmd_start(self.vhost_user) > + > + def stop_all_apps(self): > + """ > + quit the testpmd in vm and stop all apps > + """ > + for i in range(len(self.vm_dut)): > + self.vm_dut[i].send_expect("stop", "testpmd> ", 20) > + self.vm_dut[i].send_expect("quit", "# ", 20) > + self.vm[i].stop() > + self.vhost_user.send_expect("quit", "# ", 20) > + > + def config_vm_intf(self): > + """ > + restore vm interfaces and config intf arp > + """ > + for i in range(len(self.vm_dut)): > + self.vm_dut[i].restore_interfaces() > + time.sleep(5) > + vm_intf =3D self.vm_dut[i].ports_info[0]['intf'] > + self.vm_dut[i].send_expect("ifconfig %s %s" % > + (vm_intf, self.virtio_ip[i]), "#", 1= 0) > + self.vm_dut[i].send_expect("ifconfig %s up" % vm_intf,=20 > + "#", > + 10) > + > + self.vm_dut[0].send_expect('arp -s %s %s' % > + (self.virtio_ip[1],=20 > + self.virtio_mac[1]), '#', > 10) > + self.vm_dut[1].send_expect('arp -s %s %s' % > + (self.virtio_ip[0],=20 > + self.virtio_mac[0]), '#', 10) > + > + def start_iperf(self): > + """ > + start iperf > + """ > + self.vm_dut[0].send_expect( > + 'iperf -s -p 12345 -i 1 > iperf_server.log &', '', 10) > + self.vm_dut[1].send_expect( > + 'iperf -c %s -p 12345 -i 1 -t 5 > iperf_client.log &' % > + self.virtio_ip[0], '', 60) > + time.sleep(20) > + > + def iperf_result_verify(self, cycle, tinfo): > + """ > + verify the Iperf test result > + """ > + # copy iperf_client file from vm1 > + self.vm_dut[1].session.copy_file_from("%s/iperf_client.log" % > + self.dut.base_dir) > + fp =3D open("./iperf_client.log") > + fmsg =3D fp.read() > + fp.close() > + iperfdata =3D re.compile('\S*\s*[M|G]bits/sec').findall(fmsg) > + self.verify(len(iperfdata) !=3D 0, "The iperf data between to vm= s is 0") > + if cycle =3D=3D 0: > + cinfo =3D "Before reconnet" > + else: > + cinfo =3D tinfo > + self.result_table_add(["vm2vm iperf", iperfdata[-1], cinfo]) > + > + def send_and_verify(self, cycle=3D0, tinfo=3D""): > + for frame_size in self.frame_sizes: > + payload =3D frame_size - self.header_size > + flow =3D > '[Ether(dst=3D"%s")/IP(src=3D"%s",dst=3D"%s")/UDP()/("X"*%d)]' % ( > + self.dst_mac, self.src1, self.dst1, payload) > + self.tester.scapy_append('wrpcap("reconnect.pcap", %s)' % fl= ow) > + self.tester.scapy_execute() > + > + tgenInput =3D [] > + port =3D self.tester.get_local_port(self.pf) > + tgenInput.append((port, port, "reconnect.pcap")) > + > + self.tester.pktgen.clear_streams() > + streams =3D > self.pktgen_helper.prepare_stream_from_tginput(tgenInput, 100, > + None, self.tester.pktgen) > + traffic_opt =3D {'delay': 30, } > + _, pps =3D > self.tester.pktgen.measure_throughput(stream_ids=3Dstreams, > options=3Dtraffic_opt) > + Mpps =3D pps / 1000000.0 > + self.verify(Mpps > 0, "can not receive packets of frame=20 > + size %d" % > (frame_size)) > + pct =3D Mpps * 100 / \ > + float(self.wirespeed(self.nic, frame_size, 1)) > + if cycle =3D=3D 0: > + data_row =3D [tinfo, frame_size, str(Mpps), str(pct), > + "Before relaunch", "1"] > + elif cycle =3D=3D 1: > + data_row =3D [tinfo, frame_size, str(Mpps), str(pct), > + "After relaunch", "1"] > + self.result_table_add(data_row) > + > + def test_perf_vhost_user_reconnet_one_vm(self): > + """ > + test reconnect stability test of one vm > + """ > + self.header_row =3D ["Mode", "FrameSize(B)", "Throughput(Mpps)", > + "LineRate(%)", "Cycle", "Queue Number"] > + self.result_table_create(self.header_row) > + vm_cycle =3D 0 > + self.vm_num =3D 1 > + self.launch_testpmd_as_vhost_user() > + self.start_vms() > + self.vm_testpmd_start() > + self.send_and_verify(vm_cycle, "reconnet one vm") > + > + vm_cycle =3D 1 > + # reconnet from vhost > + self.logger.info('now reconnect from vhost') > + for i in range(self.reconnect_times): > + self.dut.send_expect("killall -s INT testpmd", "# ") > + self.launch_testpmd_as_vhost_user() > + self.send_and_verify(vm_cycle, "reconnet from vhost") > + > + # reconnet from qemu > + self.logger.info('now reconnect from vm') > + for i in range(self.reconnect_times): > + self.dut.send_expect("killall -s INT qemu-system-x86_64", "#= ") > + self.start_vms() > + self.vm_testpmd_start() > + self.send_and_verify(vm_cycle, "reconnet from VM") > + self.result_table_print() > + self.stop_all_apps() > + > + def test_perf_vhost_user_reconnet_two_vms(self): > + """ > + test reconnect stability test of two vms > + """ > + self.header_row =3D ["Mode", "FrameSize(B)", "Throughput(Mpps)", > + "LineRate(%)", "Cycle", "Queue Number"] > + self.result_table_create(self.header_row) > + vm_cycle =3D 0 > + self.vm_num =3D 2 > + self.launch_testpmd_as_vhost_user() > + self.start_vms() > + self.vm_testpmd_start() > + self.send_and_verify(vm_cycle, "reconnet two vm") > + > + vm_cycle =3D 1 > + # reconnet from vhost > + self.logger.info('now reconnect from vhost') > + for i in range(self.reconnect_times): > + self.dut.send_expect("killall -s INT testpmd", "# ") > + self.launch_testpmd_as_vhost_user() > + self.send_and_verify(vm_cycle, "reconnet from vhost") > + > + # reconnet from qemu > + self.logger.info('now reconnect from vm') > + for i in range(self.reconnect_times): > + self.dut.send_expect("killall -s INT qemu-system-x86_64", "#= ") > + self.start_vms() > + self.vm_testpmd_start() > + self.send_and_verify(vm_cycle, "reconnet from VM") > + self.result_table_print() > + self.stop_all_apps() > + > + def test_perf_vhost_vm2vm_virtio_net_reconnet_two_vms(self): > + """ > + test the iperf traffice can resume after reconnet > + """ > + self.header_row =3D ["Mode", "[M|G]bits/sec", "Cycle"] > + self.result_table_create(self.header_row) > + self.vm_num =3D 2 > + vm_cycle =3D 0 > + self.launch_testpmd_as_vhost_user_with_no_pci() > + self.start_vms() > + self.config_vm_intf() > + self.start_iperf() > + self.iperf_result_verify(vm_cycle, 'before reconnet') > + > + vm_cycle =3D 1 > + # reconnet from vhost > + self.logger.info('now reconnect from vhost') > + for i in range(self.reconnect_times): > + self.dut.send_expect("killall -s INT testpmd", "# ") > + self.launch_testpmd_as_vhost_user_with_no_pci() > + self.start_iperf() > + self.iperf_result_verify(vm_cycle, 'reconnet from vhost') > + > + # reconnet from VM > + self.logger.info('now reconnect from vm') > + for i in range(self.reconnect_times): > + self.vm_dut[0].send_expect('rm iperf_server.log', '# ', 10) > + self.vm_dut[1].send_expect('rm iperf_client.log', '# ', 10) > + self.dut.send_expect("killall -s INT qemu-system-x86_64", "#= ") > + self.start_vms() > + self.config_vm_intf() > + self.start_iperf() > + self.iperf_result_verify(vm_cycle, 'reconnet from vm') > + self.result_table_print() > + > + def tear_down(self): > + # > + # Run after each test case. > + # > + self.dut.send_expect("killall -s INT testpmd", "# ") > + self.dut.send_expect("killall -s INT qemu-system-x86_64", "# ") > + time.sleep(2) > + > + def tear_down_all(self): > + """ > + Run after each test suite. > + """ > + pass > -- > 2.7.4