From patchwork Mon Jun 29 20:09:10 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Daniel Kirichok X-Patchwork-Id: 72435 X-Patchwork-Delegate: ci@dpdk.org Return-Path: X-Original-To: patchwork@inbox.dpdk.org Delivered-To: patchwork@inbox.dpdk.org Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id 847F2A0350; Mon, 29 Jun 2020 22:09:31 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id BD4281BEC3; Mon, 29 Jun 2020 22:09:30 +0200 (CEST) Received: from mail-qt1-f194.google.com (mail-qt1-f194.google.com [209.85.160.194]) by dpdk.org (Postfix) with ESMTP id 269F91BEC0 for ; Mon, 29 Jun 2020 22:09:29 +0200 (CEST) Received: by mail-qt1-f194.google.com with SMTP id z2so13888170qts.5 for ; Mon, 29 Jun 2020 13:09:29 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=iol.unh.edu; s=unh-iol; h=from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=LXAEmbKWVbSCKQiVFzXXqlTOYGmTrNqZdgnZ+Kd1fbY=; b=DECadsUAy4508IBrfQtGZYWu897vEX0g2svDlLivXeJKWTK94iAZ9PI2NMtlDingBe GgJz7fig+znwcptEwNQZL2EHqR+GE6XXpgUCTnEpPiocIlzWIzb5j+1NCeOxNWWiEZfl Y8mbaSL+2seTt1Urxydtys8HvcF0AkEkbB2WI= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=LXAEmbKWVbSCKQiVFzXXqlTOYGmTrNqZdgnZ+Kd1fbY=; b=cWhbkZCkdA5NdsOyCSk8zeok+b0osxcAHELhRk1sGGxuhnq1NaeHc5bgy43BsS5GCY v1YxaKi9NXHne8PGtWZIIHHlexhun5ficbv1ak3luaqakuwRWRgSkdpmcg1vBynoXlDf gUX9nyqhOWSIUl52pDwnD6riQdtjZlVpOc32O3RB9VrX3Iz+0Mw8Y/Pg0ZATEiw+fmfY 4LcqQr4h9SuNV78fpmzN39lIfgAssnkU+Ou9F3yakxfoT7SXCkiSD4JUq3N5wU6w8DCG fMbB3A3F2tw9FG9+7W4OsgK2+A+eiCNq2zoN63eYnliEH+6ts8aDxR8KSZoFqzuoYqao lHpg== X-Gm-Message-State: AOAM533h/Ac9bvwkS/Akj4OjfhTVFXvR8UVCk6qXEaWaz1ch/RqkVNpy eFH0tHnRJr7PuNr1MnqoXQ9/ X-Google-Smtp-Source: ABdhPJz8AdFExpZXG451KHhG+y3zHTBiulgXVeC5khWFNB4CaNxOH4XzxUIS3eTsToe8c1IgWZmSMA== X-Received: by 2002:ac8:1c36:: with SMTP id a51mr17625141qtk.138.1593461368225; Mon, 29 Jun 2020 13:09:28 -0700 (PDT) Received: from localhost.localdomain ([72.71.241.216]) by smtp.gmail.com with ESMTPSA id q18sm861660qkq.20.2020.06.29.13.09.26 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 29 Jun 2020 13:09:27 -0700 (PDT) From: dkirichok@iol.unh.edu To: dts@dpdk.org Cc: mb@smartsharesystems.com, lylavoie@iol.unh.edu, thomas@monjalon.net, dev@dpdk.org, david.marchand@redhat.com, ferruh.yigit@intel.com, arybchenko@solarflare.com, i.dyukov@samsung.com, rasland@mellanox.com, j.hendergart@f5.com, ohilyard@iol.unh.edu, Daniel Kirichok Date: Mon, 29 Jun 2020 16:09:10 -0400 Message-Id: <20200629200910.4993-1-dkirichok@iol.unh.edu> X-Mailer: git-send-email 2.17.1 MIME-Version: 1.0 Subject: [dpdk-dev] [PATCH] speed-capabilities: Added Speed Capabilities test case X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" From: Daniel Kirichok Added Speed Capabilities test which compares detected interface speeds with what is expected in the config file. Signed-off-by: Daniel Kirichok --- conf/speed_capabilities.cfg | 4 + test_plans/speed_capabilities_test_plan.rst | 84 ++++++++++++++ tests/TestSuite_speed_capabilities.py | 120 ++++++++++++++++++++ 3 files changed, 208 insertions(+) create mode 100644 conf/speed_capabilities.cfg create mode 100644 test_plans/speed_capabilities_test_plan.rst create mode 100644 tests/TestSuite_speed_capabilities.py diff --git a/conf/speed_capabilities.cfg b/conf/speed_capabilities.cfg new file mode 100644 index 0000000..c22f531 --- /dev/null +++ b/conf/speed_capabilities.cfg @@ -0,0 +1,4 @@ +[suite] +# The format for entering expected speeds is {'':'',...} +# Unit for speed must be in G or M +expected_speeds={'interface1': '10G', 'interface2': '100M'} diff --git a/test_plans/speed_capabilities_test_plan.rst b/test_plans/speed_capabilities_test_plan.rst new file mode 100644 index 0000000..2821612 --- /dev/null +++ b/test_plans/speed_capabilities_test_plan.rst @@ -0,0 +1,84 @@ +.. # BSD LICENSE + # + # Copyright(c) 2010-2014 Intel Corporation. All rights reserved. + # Copyright © 2018[, 2019] The University of New Hampshire. 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. + +======================= +Speed Capabilities Test +======================= + +The Speed Capabilities Test checks the speeds that the device lists and compares +it to a user-defined set of expected speeds set in the ``speed_capabilities.cfg``. + +Prerequisites +============= + +If using vfio the kernel must be >= 3.6+ and VT-d must be enabled in bios.When +using vfio, use the following commands to load the vfio driver and bind it +to the device under test:: + + modprobe vfio + modprobe vfio-pci + usertools/dpdk-devbind.py --bind=vfio-pci device_bus_id + +Assuming that ports ``0`` and ``1`` of the test target are directly connected +to the traffic generator, launch the ``testpmd`` application with the following +arguments:: + + ./build/app/testpmd -- -i --portmask=0x3 + +Start packet forwarding in the ``testpmd`` application with the ``start`` +command. Then, for each port on the target make the Traffic Generator +transmit a packet to the port of arbitrary size less than the MTU of +the target port, checking that the same amount of frames and bytes +are received back by the Traffic Generator from the port. + +Ensure that the ``speed_capabilities.cfg`` file correctly reflects the speeds +the device is expected to run at. + + +Test Case : Speed Capabilities Test +=================================== + +1. Use testpmd to retrieve the speed information that the interface is linked at:: + + testpmd> show port info + +2. Compares the speed value against the rate in the speed_capabilities.cfg file. + The supported options for expected speeds are 1G, 10G, 25G, 40G, and 100G. Each interface must have an expected speed associated with it. + Example file below:: + + [suite] + expected_speeds={'interface1': '10G', 'interface2': '100M'} + + +3. Verifies that the speeds matches accordingly. + +4. Repeats the verification for each NIC and interface found on the system. diff --git a/tests/TestSuite_speed_capabilities.py b/tests/TestSuite_speed_capabilities.py new file mode 100644 index 0000000..25c45e4 --- /dev/null +++ b/tests/TestSuite_speed_capabilities.py @@ -0,0 +1,120 @@ +# BSD LICENSE +# +# Copyright(c) 2010-2014 Intel Corporation. All rights reserved. +# Copyright © 2018[, 2019] The University of New Hampshire. 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 utils +from pmd_output import PmdOutput +from test_case import TestCase + + +class TestSpeedCapabilities(TestCase): + def set_up_all(self): + """ + Run at the start of each test suite. + """ + self.ports = self.dut.get_ports() + self.ports_socket = self.dut.get_numa_id(self.ports[0]) + + for port in self.ports: + self.tester.send_expect(f"ifconfig {self.tester.get_interface(self.tester.get_local_port(port))} mtu 5000" + , "# ") + + self.pmdout = PmdOutput(self.dut) + self.vm_env_done = False + + def test_speed_capabilities(self): + self.pmdout.start_testpmd("Default") + + cfg_content = self.get_suite_cfg() + expected_speeds = cfg_content.get('expected_speeds') + + detected_interfaces = [] + + for port in self.ports: + interface_name = self.tester.get_interface(self.tester.get_local_port(port)) + + # Gives the speed in Mb/s + interface_speed = self.pmdout.get_port_link_speed(port) + + self.verify(interface_name in expected_speeds, f"The interface {interface_name} does not have an expected " + f"speed associated with it.") + + detected_interfaces.append(interface_name) + + expected_speed = expected_speeds[interface_name] + + # Isolates the unit (Either M or G) + expected_speed_unit = ''.join(i for i in expected_speed if not i.isdigit()) + + # Removes the unit from the speed + expected_speed = ''.join(i for i in expected_speed if i.isdigit()) + + self.verify(len(interface_speed) > 0, + f"A valid speed could not be read for the interface {interface_name}.") + + # Converts Gb/s to Mb/s for consistent comparison + if expected_speed_unit == "G": + expected_speed += "000" + + self.verify(interface_speed == expected_speed, + f"Detected speed: {interface_speed} Mb/s for the interface {interface_name}, " + f"but expected speed: {expected_speed} Mb/s") + + for key, value in expected_speeds.items(): + self.verify(key in detected_interfaces, f"The interface {key} expected the speed {value} in " + "speed_capabilities.cfg file, but it did not detect that interface.") + + def tear_down(self): + """ + Run after each test case. + """ + self.dut.kill_all() + self.pmdout.start_testpmd("Default", "--portmask=%s --port-topology=loop" % utils.create_mask(self.ports), + socket=self.ports_socket) + ports_num = len(self.ports) + # link up test, to avoid failing further tests if link was down + for i in range(ports_num): + # sometimes output text messing up testpmd prompt so trimmed prompt + self.dut.send_expect("set link-up port %d" % i, ">") + # start ports, to avoid failing further tests if ports are stopped + self.dut.send_expect("port start all", "testpmd> ", 100) + self.dut.send_expect("quit", "# ") + + def tear_down_all(self): + """ + Run after each test suite. + """ + if self.vm_env_done: + self.destroy_vm_env() + self.dut.kill_all()