Re: [dts] [PATCH V6] tests/virtio_smoke:add virtio smoke test suite sync with test plan

["Wang, Yinan" <yinan.wang@intel.com>]

Acked-by: Wang, Yinan <yinan.wang@intel.com>

> -----Original Message-----
> From: dts <dts-bounces@dpdk.org> On Behalf Of Ling, WeiX
> Sent: 2021?2?2? 15:52
> To: Ling, WeiX <weix.ling@intel.com>; dts@dpdk.org
> Subject: Re: [dts] [PATCH V6] tests/virtio_smoke:add virtio smoke test suite
> sync with test plan
> 
> Tested-by: Wei Ling <weix.ling@intel.com>
> 
> Regards,
> Ling Wei
> 
> > -----Original Message-----
> > From: Ling Wei <weix.ling@intel.com>
> > Sent: Tuesday, February 2, 2021 11:23 PM
> > To: dts@dpdk.org
> > Cc: Ling, WeiX <weix.ling@intel.com>
> > Subject: [dts][PATCH V6] tests/virtio_smoke:add virtio smoke test suite sync
> > with test plan
> >
> > v1:
> > Add virtio smoke test suite sync with test plan.
> >
> > v2:
> > 1.Modiry Copyrint(c)<2019> to Copyrint(c)<2021>.
> > 2.Fix CentOS check hugepagesize return ' 2048' dts issue.
> > 3.Add test case run exception handing 'kill dpdk-testpmd'
> >   in set_up and tear_down.
> >
> > v3:
> > Modify patch subject and comments description error.
> >
> > v4:
> > Modify set_up method,kill testpmd then remove vhost-net*.
> >
> > v5:
> > Fix patch spelling error.
> >
> > v6:
> > Fix patch spelling error.
> >
> > Signed-off-by: Ling Wei <weix.ling@intel.com>
> > ---
> >  tests/TestSuite_virtio_smoke.py | 202
> > ++++++++++++++++++++++++++++++++
> >  1 file changed, 202 insertions(+)
> >  create mode 100644 tests/TestSuite_virtio_smoke.py
> >
> > diff --git a/tests/TestSuite_virtio_smoke.py
> > b/tests/TestSuite_virtio_smoke.py new file mode 100644 index
> > 00000000..3c9e96fc
> > --- /dev/null
> > +++ b/tests/TestSuite_virtio_smoke.py
> > @@ -0,0 +1,202 @@
> > +# BSD LICENSE+#+# Copyright(c) <2021> 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.+"""++import re+from test_case import TestCase+from pmd_output
> > import PmdOutput+DEFAULT_MTU = 1500+TSO_MTU = 9000++class
> > TestVirtioSmoke(TestCase):+++    def set_up_all(self):+        """+        Run at
> > the start of each test suite.+        """+        self.dst_mac =
> "00:01:02:03:04:05"+
> > self.dut_ports = self.dut.get_ports()+        self.txItf =
> > self.tester.get_interface(self.tester.get_local_port(self.dut_ports[0]))+
> > self.socket = self.dut.get_numa_id(self.dut_ports[0])+        self.cores =
> > self.dut.get_core_list("all", socket=self.socket)+        self.vhost_cores =
> > self.cores[0:3]+        self.virtio1_cores = self.cores[3:6]+        self.base_dir =
> > self.dut.base_dir.replace('~', '/root')+        self.path =
> > self.dut.apps_name['test-pmd']+        self.testpmd_name =
> > self.path.split("/")[-1]+        self.vhost_user =
> > self.dut.new_session(suite="vhost-user")+        self.virtio_user1 =
> > self.dut.new_session(suite="virtio-user1")+        self.pmdout_vhost_user =
> > PmdOutput(self.dut, self.vhost_user)+        self.pmdout_virtio_user1 =
> > PmdOutput(self.dut, self.virtio_user1)++    def set_up(self):+        """+
> > Run before each test case.+        """+        self.dut.send_expect("killall -s
> > INT %s" % self.testpmd_name, "#")+        self.dut.send_expect("rm -
> > rf %s/vhost-net*" % self.base_dir, "#")++    @property+    def
> > check_2M_env(self):+        out = self.dut.send_expect("cat /proc/meminfo
> > |grep Hugepagesize|awk '{print($2)}'", "# ")+        # On CentOS, sometimes
> > return ' 2048'+        resp = out.replace(' ', '')+        return True if resp ==
> '2048'
> > else False++    def launch_testpmd_as_vhost_user(self, param,
> > cores="Default", udev="", ports=[], no_pci=True):+
> > self.pmdout_vhost_user.start_testpmd(cores=cores, param=param,
> > vdevs=[udev], ports=ports, prefix="vhost",+
> > fixed_prefix=True, no_pci=no_pci)+
> > self.pmdout_vhost_user.execute_cmd('set fwd mac')++    def
> > launch_testpmd_as_virtio_user1(self, param, cores="Default", udev="",
> > no_pci=True):+        eal_param = ""+        if self.check_2M_env:+
> > eal_param += " --single-file-segments"+        if 'vectorized' in
> > self.running_case:+            eal_param += " --force-max-simd-bitwidth=512"+
> > self.pmdout_virtio_user1.start_testpmd(cores=cores, param=param,
> > vdevs=[udev], ports=[], prefix="virtio1",+
> > fixed_prefix=True, eal_param=eal_param, no_pci=no_pci)++    def
> > verify_vhost_queue_rx_tx_pkts(self, queue_list):+        out =
> > self.pmdout_vhost_user.execute_cmd('stop')+        for queue_index in
> > queue_list:+            queue = "Queue= %d" % queue_index+            index =
> > out.find(queue)+            rx = re.search("RX-packets:\s*(\d*)", out[index:])+
> > tx = re.search("TX-packets:\s*(\d*)", out[index:])+            rx_packets =
> > int(rx.group(1))+            tx_packets = int(tx.group(1))+
> > self.verify(rx_packets > 0 and tx_packets > 0,+                        "The queue %d
> rx-
> > packets or tx-packets is 0 about " % queue_index ++                        "rx-
> > packets:%d, tx-packets:%d" % (rx_packets, tx_packets))++    def
> > test_virtio_loopback(self):+        param = " --nb-cores={} --rxq={} --txq={}"+
> > other_param = " --tx-offloads=0x0 --enable-hw-vlan-strip --rss-ip"+
> > vhost_dev = f"'eth_vhost0,iface=vhost-net,client=1,queues=8'"+
> > virtio_dev = f"'net_virtio_user0,mac={self.dst_mac},path=./vhost-
> > net,server=1,queues=8,mrg_rxbuf=1,in_order=1'"++
> > self.logger.info("Launch vhost as client mode with 2 queues")+        nb_core
> =
> > 2+        vhost_rxq_txq = 2+        virtio_rxq_txq = 8+        vhost_param =
> > param.format(nb_core, vhost_rxq_txq, vhost_rxq_txq)+        virtio_param =
> > (other_param + param).format(nb_core, virtio_rxq_txq, virtio_rxq_txq)+
> > self.launch_testpmd_as_vhost_user(param=vhost_param,
> > cores=self.vhost_cores, udev=vhost_dev, no_pci=True)+
> > self.pmdout_vhost_user.execute_cmd('start')+        self.logger.info("Launch
> > virtio-user as server mode with 8 queues")+
> > self.launch_testpmd_as_virtio_user1(param=virtio_param,
> > cores=self.virtio1_cores, udev=virtio_dev, no_pci=True)+
> > self.pmdout_virtio_user1.execute_cmd('set fwd mac')+
> > self.pmdout_virtio_user1.execute_cmd('start tx_first 32')+
> > self.verify_vhost_queue_rx_tx_pkts(queue_list=range(vhost_rxq_txq))+
> > self.pmdout_vhost_user.execute_cmd('quit', '#')++
> > self.logger.info("Relaunch vhost with 8 queues and send packets")+
> > vhost_rxq_txq = 8+        vhost_param = param.format(nb_core,
> > vhost_rxq_txq, vhost_rxq_txq)+
> > self.launch_testpmd_as_vhost_user(param=vhost_param,
> > cores=self.cores[0:3], udev=vhost_dev, no_pci=True)+
> > self.pmdout_vhost_user.execute_cmd('start tx_first 32')+
> > self.pmdout_vhost_user.execute_cmd('stop')+
> > self.pmdout_vhost_user.execute_cmd('set burst 1')+
> > self.pmdout_vhost_user.execute_cmd('start tx_first 1')+
> > self.verify_vhost_queue_rx_tx_pkts(queue_list=range(vhost_rxq_txq))++
> > self.pmdout_virtio_user1.execute_cmd('quit', '#')+
> > self.pmdout_vhost_user.execute_cmd('quit', '#')++    def send_packets(self,
> > frame_size, pkt_count):+        pkt =
> > "Ether(dst='%s')/IP()/('x'*%d)" %(self.dst_mac, frame_size)+
> > self.tester.scapy_append('sendp([%s], iface="%s", count=%s)' % (pkt,
> > self.txItf, pkt_count))+        self.tester.scapy_execute()++    def
> > verify_virtio_user_receive_packets(self, pkt_count):+        out =
> > self.pmdout_virtio_user1.execute_cmd('show port stats all')+        rx =
> > re.search("RX-packets:\s*(\d*)", out)+        tx = re.search("TX-
> > packets:\s*(\d*)", out)+        rx_packets = int(rx.group(1))+        tx_packets =
> > int(tx.group(1))+        self.verify(rx_packets >= pkt_count and tx_packets >=
> > pkt_count,+                    "Virtio-user receive no enough packets, RX-packets:
> > {},RX-packets: {}".format(rx_packets, tx_packets))++    def
> > test_virtio_pvp(self):+        param = " --nb-cores={} --txd={} --rxd={}"+
> > vhost_dev = f"'net_vhost0,iface=vhost-net,queues=1'"+        virtio_dev =
> > f"'net_virtio_user0,mac={self.dst_mac},path=./vhost-
> > net,packed_vq=1,mrg_rxbuf=0,in_order=1," \+
> > f"vectorized=1,queue_size=1024'"+        self.logger.info("Launch vhost")+
> > nb_core = 2+        vhost_rxd_txd = 1024+        vhost_param =
> > param.format(nb_core, vhost_rxd_txd, vhost_rxd_txd)+        port =
> > self.dut.ports_info[self.dut_ports[0]]['pci']+
> > self.launch_testpmd_as_vhost_user(param=vhost_param,
> > cores=self.vhost_cores, udev=vhost_dev,+
> ports=[port],
> > no_pci=False)+        self.pmdout_vhost_user.execute_cmd('start')++
> > self.logger.info("Launch virtio")+        nb_core = 1+        virtio_param =
> > param.format(nb_core, vhost_rxd_txd, vhost_rxd_txd)+
> > self.launch_testpmd_as_virtio_user1(param=virtio_param,
> > cores=self.virtio1_cores, udev=virtio_dev, no_pci=True)+
> > self.pmdout_virtio_user1.execute_cmd('set fwd mac')+
> > self.pmdout_virtio_user1.execute_cmd('start')++        self.logger.info("Start
> > send packets and verify")+        # set tester port MTU=9000 when need to
> > send big packets.+        self.tester.send_expect("ifconfig %s mtu %s" %
> > (self.txItf, TSO_MTU), "# ")+        # set vhost testpmd port MTU=9000+
> > self.pmdout_vhost_user.execute_cmd('stop')+
> > self.pmdout_vhost_user.execute_cmd('port stop 0')+
> > self.pmdout_vhost_user.execute_cmd('port config mtu 0 %s' % TSO_MTU)+
> > self.pmdout_vhost_user.execute_cmd('port start 0')+
> > self.pmdout_vhost_user.execute_cmd('start')+
> > self.send_packets(frame_size=64, pkt_count=10)+
> > self.send_packets(frame_size=1518, pkt_count=10)+
> > self.verify_virtio_user_receive_packets(pkt_count=20)++
> > self.pmdout_virtio_user1.execute_cmd('quit', '#')+
> > self.pmdout_vhost_user.execute_cmd('quit', '#')++    def tear_down(self):+
> > """+        Run after each test case.+        """+        self.dut.send_expect("killall -
> s
> > INT %s" % self.testpmd_name, "#")++    def tear_down_all(self):+        """+
> > Run after each test suite.+        """+
> > self.dut.close_session(self.vhost_user)+
> > self.dut.close_session(self.virtio_user1)+
> > self.tester.send_expect("ifconfig %s mtu %s" % (self.txItf, DEFAULT_MTU),
> > "# ")--
> > 2.25.1