test suite reviews and discussions
 help / color / mirror / Atom feed
From: "Tu, Lijuan" <lijuan.tu@intel.com>
To: "Zeng, XiaoxiaoX" <xiaoxiaox.zeng@intel.com>,
	"dts@dpdk.org" <dts@dpdk.org>
Cc: "Zeng, XiaoxiaoX" <xiaoxiaox.zeng@intel.com>
Subject: Re: [dts] [PATCH V1] tests/port_control: automation port_contol
Date: Thu, 2 Jan 2020 05:16:26 +0000	[thread overview]
Message-ID: <8CE3E05A3F976642AAB0F4675D0AD20E0BBA104B@SHSMSX101.ccr.corp.intel.com> (raw)
In-Reply-To: <1577782742-84298-1-git-send-email-xiaoxiaox.zeng@intel.com>

applied

> -----Original Message-----
> From: dts [mailto:dts-bounces@dpdk.org] On Behalf Of Zeng Xiaoxiao
> Sent: Tuesday, December 31, 2019 4:59 PM
> To: dts@dpdk.org
> Cc: Zeng, XiaoxiaoX <xiaoxiaox.zeng@intel.com>
> Subject: [dts] [PATCH V1] tests/port_control: automation port_contol
> 
> *.according to port_control_test_plan.rst,automation suite port_control
> 
> Signed-off-by: Zeng Xiaoxiao <xiaoxiaox.zeng@intel.com>
> ---
>  tests/TestSuite_port_control.py | 260
> ++++++++++++++++++++++++++++++++++++++++
>  1 file changed, 260 insertions(+)
>  create mode 100644 tests/TestSuite_port_control.py
> 
> diff --git a/tests/TestSuite_port_control.py b/tests/TestSuite_port_control.py
> new file mode 100644 index 0000000..a458add
> --- /dev/null
> +++ b/tests/TestSuite_port_control.py
> @@ -0,0 +1,260 @@
> +# BSD LICENSE
> +#
> +# Copyright(c) <2019> 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.
> +
> +import os
> +import utils
> +import time
> +import re
> +import packet
> +from test_case import TestCase
> +from pmd_output import PmdOutput
> +from virt_common import VM
> +
> +
> +class TestPortControl(TestCase):
> +
> +    def set_up_all(self):
> +        """
> +        Run before each test suite
> +        """
> +        # initialize ports topology
> +        self.vm0 = None
> +        self.env_done = False
> +        self.port_id_0 = 0
> +        self.pkt_count = 1000
> +        self.dut_ports = self.dut.get_ports(self.nic)
> +        # Verify that enough ports are available
> +        self.verify(len(self.dut_ports) >= 1, "Insufficient ports for testing")
> +        self.pf_mac = self.dut.get_mac_address(self.dut_ports[0])
> +        self.vf_mac = "00:01:23:45:67:89"
> +        self.txitf =
> self.tester.get_interface(self.tester.get_local_port(self.dut_ports[0]))
> +        self.host_testpmd = PmdOutput(self.dut)
> +        self.vf_assign_method = 'vfio-pci'
> +        self.dut.send_expect('modprobe vfio-pci', '#')
> +        self.socket = self.dut.get_numa_id(self.dut_ports[0])
> +        port = self.dut.ports_info[0]['port']
> +        self.pf_default_driver = port.get_nic_driver()
> +
> +    def set_up(self):
> +        """
> +        Run before each test case.
> +        """
> +        pass
> +
> +    def setup_vm_env(self, driver='default'):
> +        """
> +        Create testing environment with 1VF generated from 1PF
> +        """
> +        if self.env_done:
> +            return
> +
> +        # bind to default driver
> +        self.bind_nic_driver(self.dut_ports[:1], driver="")
> +        self.used_dut_port = self.dut_ports[0]
> +        self.host_intf = self.dut.ports_info[self.used_dut_port]['intf']
> +        self.dut.generate_sriov_vfs_by_port(self.used_dut_port, 1,
> driver=driver)
> +        self.sriov_vfs_port = self.dut.ports_info[self.used_dut_port]['vfs_port']
> +        self.dut.send_expect("ip link set %s vf 0 mac %s" % (self.host_intf,
> self.vf_mac), "# ")
> +        try:
> +            for port in self.sriov_vfs_port:
> +                port.bind_driver(self.vf_assign_method)
> +            time.sleep(1)
> +            vf_popt = {'opt_host': self.sriov_vfs_port[0].pci}
> +
> +            # set up VM ENV
> +            self.vm = VM(self.dut, 'vm0', 'port_control')
> +            self.vm.set_vm_device(driver=self.vf_assign_method, **vf_popt)
> +            self.vm_dut = self.vm.start()
> +            if self.vm_dut is None:
> +                raise Exception("Set up VM ENV failed!")
> +
> +            self.vm_testpmd = PmdOutput(self.vm_dut)
> +
> +        except Exception as e:
> +            self.destroy_vm_env()
> +            raise Exception(e)
> +
> +        self.env_done = True
> +
> +    def destroy_vm_env(self):
> +        if getattr(self, 'vm', None):
> +            if getattr(self, 'vm_dut', None):
> +                self.vm_dut.kill_all()
> +            self.vm_testpmd = None
> +            self.vm_dut_ports = None
> +            # destroy vm0
> +            self.vm.stop()
> +            self.dut.virt_exit()
> +            time.sleep(3)
> +            self.vm = None
> +
> +        if getattr(self, 'used_dut_port', None) != None:
> +            self.dut.destroy_sriov_vfs_by_port(self.used_dut_port)
> +            self.used_dut_port = None
> +        self.bind_nic_driver(self.dut_ports[:1],
> + driver=self.pf_default_driver)
> +
> +        self.env_done = False
> +
> +    def bind_nic_driver(self, ports, driver=""):
> +        # modprobe vfio driver
> +        if driver == "vfio-pci":
> +            for port in ports:
> +                netdev = self.dut.ports_info[port]['port']
> +                driver = netdev.get_nic_driver()
> +                if driver != 'vfio-pci':
> +                    netdev.bind_driver(driver=self.d)
> +
> +        elif driver == "igb_uio":
> +            # igb_uio should insmod as default, no need to check
> +            for port in ports:
> +                netdev = self.dut.ports_info[port]['port']
> +                driver = netdev.get_nic_driver()
> +                if driver != 'igb_uio':
> +                    netdev.bind_driver(driver='igb_uio')
> +        else:
> +            for port in ports:
> +                netdev = self.dut.ports_info[port]['port']
> +                driver_now = netdev.get_nic_driver()
> +                if driver is None:
> +                    driver = netdev.default_driver
> +                if driver != driver_now:
> +                    netdev.bind_driver(driver=driver)
> +
> +    def start_testpmd(self, terminal):
> +        terminal.start_testpmd(ports=[0], socket=self.socket)
> +        res = terminal.wait_link_status_up('all', timeout=5)
> +        self.verify(res is True, 'there have port link is down')
> +        terminal.execute_cmd('set fwd mac')
> +        terminal.execute_cmd('set promisc all off')
> +
> +    def start_pmd_port(self, terminal):
> +        terminal.execute_cmd("port start all")
> +        terminal.execute_cmd("start")
> +        terminal.wait_link_status_up('all', timeout=5)
> +        ret = terminal.get_port_link_status(self.port_id_0)
> +        self.verify(ret == "up", "port not up!")
> +
> +    def stop_pmd_port(self, terminal):
> +        terminal.execute_cmd("stop")
> +        terminal.execute_cmd("port stop all")
> +        ret = terminal.get_port_link_status(self.port_id_0)
> +        self.verify(ret == "down", "port not down!")
> +
> +    def reset_pmd_port(self, terminal):
> +        terminal.execute_cmd("port reset all")
> +        ret = terminal.get_port_link_status(self.port_id_0)
> +        self.verify(ret == "down", "port reset fail!")
> +
> +    def close_pmd_port(self, terminal):
> +        terminal.execute_cmd("port close all")
> +        ret = terminal.execute_cmd("show port info all")
> +        ret = ret.split('\r')
> +        self.verify(ret[1] == '', "close all port fail!")
> +
> +    def calculate_stats(self, start_stats, end_stats):
> +        ret_stats = {}
> +        ret_stats['RX-packets'] = int(end_stats['RX-packets']) -
> int(start_stats['RX-packets'])
> +        ret_stats['TX-packets'] = int(end_stats['TX-packets']) -
> int(start_stats['TX-packets'])
> +        return ret_stats
> +
> +    def send_and_verify_packets(self, terminal):
> +        """
> +        Send packets according to parameters.
> +        """
> +        if terminal is self.host_testpmd:
> +            self.dts_mac = self.pf_mac
> +        else:
> +            self.dts_mac = self.vf_mac
> +
> +        self.pkt = packet.Packet('Ether(dst="%s")/IP()/Raw("x"*40)' %
> + self.dts_mac)
> +
> +        pf_start_stats = terminal.get_pmd_stats(self.port_id_0)
> +        self.pkt.send_pkt(crb=self.tester, tx_port=self.txitf,
> count=self.pkt_count, timeout=30)
> +        pf_end_stats = terminal.get_pmd_stats(self.port_id_0)
> +        pf_ret_stats = self.calculate_stats(pf_start_stats,
> + pf_end_stats)
> +
> +        self.verify(pf_ret_stats['RX-packets'] == self.pkt_count and
> pf_ret_stats['TX-packets'] == self.pkt_count,
> +                    "Packets receive and forward fail!")
> +
> +    def test_pf_start_stop_reset_close(self):
> +        self.start_testpmd(self.host_testpmd)
> +        # start port
> +        self.start_pmd_port(self.host_testpmd)
> +        self.send_and_verify_packets(self.host_testpmd)
> +        # stop port and start port
> +        self.stop_pmd_port(self.host_testpmd)
> +        self.start_pmd_port(self.host_testpmd)
> +        self.send_and_verify_packets(self.host_testpmd)
> +        # reset port
> +        self.stop_pmd_port(self.host_testpmd)
> +        self.reset_pmd_port(self.host_testpmd)
> +        self.start_pmd_port(self.host_testpmd)
> +        self.send_and_verify_packets(self.host_testpmd)
> +        # close all port
> +        self.stop_pmd_port(self.host_testpmd)
> +        self.close_pmd_port(self.host_testpmd)
> +
> +    def test_e1000_start_stop_reset_close(self):
> +        self.setup_vm_env()
> +        self.start_testpmd(self.vm_testpmd)
> +        # start port
> +        self.start_pmd_port(self.vm_testpmd)
> +        self.send_and_verify_packets(self.vm_testpmd)
> +        # stop port and start port
> +        self.stop_pmd_port(self.vm_testpmd)
> +        self.start_pmd_port(self.vm_testpmd)
> +        self.send_and_verify_packets(self.vm_testpmd)
> +        # reset port
> +        self.stop_pmd_port(self.vm_testpmd)
> +        self.reset_pmd_port(self.vm_testpmd)
> +        self.start_pmd_port(self.vm_testpmd)
> +        self.send_and_verify_packets(self.vm_testpmd)
> +        # close all port
> +        self.stop_pmd_port(self.vm_testpmd)
> +        self.close_pmd_port(self.vm_testpmd)
> +
> +    def tear_down(self):
> +        """
> +        Run after each test case.
> +        """
> +        if self.env_done:
> +            self.vm_testpmd.quit()
> +            self.destroy_vm_env()
> +        else:
> +            self.host_testpmd.quit()
> +
> +    def tear_down_all(self):
> +        """
> +        Run after each test suite.
> +        """
> +        if self.env_done:
> +            self.destroy_vm_env()
> +        self.dut.kill_all()
> --
> 1.8.3.1


      parent reply	other threads:[~2020-01-02  5:16 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-12-31  8:59 Zeng Xiaoxiao
2019-12-31  9:05 ` Zeng, XiaoxiaoX
2019-12-31  9:31 ` Lu, Nannan
2020-01-02  5:16 ` Tu, Lijuan [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=8CE3E05A3F976642AAB0F4675D0AD20E0BBA104B@SHSMSX101.ccr.corp.intel.com \
    --to=lijuan.tu@intel.com \
    --cc=dts@dpdk.org \
    --cc=xiaoxiaox.zeng@intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).