From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 8EAA841C8A; Mon, 13 Feb 2023 16:29:48 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 0FAF342D47; Mon, 13 Feb 2023 16:29:08 +0100 (CET) Received: from mail-wr1-f45.google.com (mail-wr1-f45.google.com [209.85.221.45]) by mails.dpdk.org (Postfix) with ESMTP id 757E342D2D for ; Mon, 13 Feb 2023 16:28:59 +0100 (CET) Received: by mail-wr1-f45.google.com with SMTP id bk16so12628697wrb.11 for ; Mon, 13 Feb 2023 07:28:59 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=pantheon-tech.20210112.gappssmtp.com; s=20210112; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=g0i4A830Q4+1QmdqGAvSBnHmdlRHXRV8YyWkgNAHKhI=; b=pcz+yqYk20jsyLCjq86wGPFymvSWvD3y/32MhfMmbFzG3/QfEghZJNSei5qoS8K3b8 QXb1VGmolxkb65IRkWHmuyV5ayn7SnRftGLrD9jLEt70gfe7VYgbdvI7ROH1HSKPNwPX ft8IJTbquEWp0Yx33o7nwWl5rYQyEkoWvGqphvnlgsYhIe+guMzBajIbMyMmwJYXP/ZY 1DvsBvVoa1LZFKtuVQWaU/clQ2Hm+91rzfRfQ4v4s458JDnfmDh2xm5x81iiA5ryJamE 8lc1773nQeCSLHXxQ87Oj7pLoRrOwTV9I0n3PZR2kcYSBBwLwYxapFOkaG/p/GvkSLVl annw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=g0i4A830Q4+1QmdqGAvSBnHmdlRHXRV8YyWkgNAHKhI=; b=gsUX/D5wAI4Gm4pTDLid/b56/08t50iFGBERZkoEiZvgW2nTnSNWDo8qE/x8JExi0m f35mQMDTopeyfOgNiJf8w40ZGnEAXJYwQzUrpi+ISQ+rl6/eJZtRSVtstxmpuzaU+QJh FyhTZcJxZjaauz30eSRusu1Ht+VppYg7wxYAgfn/DLiJP4w5VXgZBnLwtcNiPbET0Cm9 ywWXXPwbX8rTFaZEwm4rPk9vnw38m8IQI6Yp6/9PPXzb+G0wiivRTWIJbLjUzg57SHP4 XoaECKBT39Hg0BjNjw2XvD1lZZQbqq0Jq6dvURKComZLDd5bYotO7XX5nYju4Md4y/12 VNuw== X-Gm-Message-State: AO0yUKXPP6QYyoEmUJDp5K9wJ50XOdK0SVSWi8CLMSF12VI9AcYvaQxa TejNva8FSXrgy7zNTMB3BOqVuQ== X-Google-Smtp-Source: AK7set8cGhaP5TSS7Ks8okWmBQ+pP6ve3lJ+bgfkJIcb22629PF6H5VO0btnkM+QtPevufQe5ISCQA== X-Received: by 2002:a5d:4dc9:0:b0:2c5:5a6a:72af with SMTP id f9-20020a5d4dc9000000b002c55a6a72afmr2347577wru.4.1676302139233; Mon, 13 Feb 2023 07:28:59 -0800 (PST) Received: from localhost.localdomain ([84.245.121.112]) by smtp.gmail.com with ESMTPSA id d13-20020adfe88d000000b002c54f4d0f71sm5848613wrm.38.2023.02.13.07.28.58 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 13 Feb 2023 07:28:58 -0800 (PST) From: =?UTF-8?q?Juraj=20Linke=C5=A1?= To: thomas@monjalon.net, Honnappa.Nagarahalli@arm.com, ohilyard@iol.unh.edu, lijuan.tu@intel.com, bruce.richardson@intel.com, wathsala.vithanage@arm.com, probb@iol.unh.edu Cc: dev@dpdk.org, =?UTF-8?q?Juraj=20Linke=C5=A1?= Subject: [PATCH v4 07/10] dts: add hello world testsuite Date: Mon, 13 Feb 2023 16:28:43 +0100 Message-Id: <20230213152846.284191-8-juraj.linkes@pantheon.tech> X-Mailer: git-send-email 2.30.2 In-Reply-To: <20230213152846.284191-1-juraj.linkes@pantheon.tech> References: <20230117154906.860916-1-juraj.linkes@pantheon.tech> <20230213152846.284191-1-juraj.linkes@pantheon.tech> MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org The test suite implements test cases defined in the corresponding test plan. Signed-off-by: Juraj Linkeš --- dts/framework/remote_session/__init__.py | 2 +- dts/framework/remote_session/os_session.py | 16 ++++- .../remote_session/remote/__init__.py | 2 +- dts/framework/testbed_model/__init__.py | 1 + dts/framework/testbed_model/sut_node.py | 12 +++- dts/tests/TestSuite_hello_world.py | 64 +++++++++++++++++++ 6 files changed, 93 insertions(+), 4 deletions(-) create mode 100644 dts/tests/TestSuite_hello_world.py diff --git a/dts/framework/remote_session/__init__.py b/dts/framework/remote_session/__init__.py index 747316c78a..ee221503df 100644 --- a/dts/framework/remote_session/__init__.py +++ b/dts/framework/remote_session/__init__.py @@ -17,7 +17,7 @@ from .linux_session import LinuxSession from .os_session import OSSession -from .remote import RemoteSession, SSHSession +from .remote import CommandResult, RemoteSession, SSHSession def create_session( diff --git a/dts/framework/remote_session/os_session.py b/dts/framework/remote_session/os_session.py index 4753c70e91..2731a7ca0a 100644 --- a/dts/framework/remote_session/os_session.py +++ b/dts/framework/remote_session/os_session.py @@ -12,7 +12,7 @@ from framework.testbed_model import LogicalCore, MesonArgs from framework.utils import EnvVarsDict -from .remote import RemoteSession, create_remote_session +from .remote import CommandResult, RemoteSession, create_remote_session class OSSession(ABC): @@ -50,6 +50,20 @@ def is_alive(self) -> bool: """ return self.remote_session.is_alive() + def send_command( + self, + command: str, + timeout: float, + verify: bool = False, + env: EnvVarsDict | None = None, + ) -> CommandResult: + """ + An all-purpose API in case the command to be executed is already + OS-agnostic, such as when the path to the executed command has been + constructed beforehand. + """ + return self.remote_session.send_command(command, timeout, verify, env) + @abstractmethod def guess_dpdk_remote_dir(self, remote_dir) -> PurePath: """ diff --git a/dts/framework/remote_session/remote/__init__.py b/dts/framework/remote_session/remote/__init__.py index f3092f8bbe..8a1512210a 100644 --- a/dts/framework/remote_session/remote/__init__.py +++ b/dts/framework/remote_session/remote/__init__.py @@ -6,7 +6,7 @@ from framework.config import NodeConfiguration from framework.logger import DTSLOG -from .remote_session import RemoteSession +from .remote_session import CommandResult, RemoteSession from .ssh_session import SSHSession diff --git a/dts/framework/testbed_model/__init__.py b/dts/framework/testbed_model/__init__.py index 2be5169dc8..efb463f2e2 100644 --- a/dts/framework/testbed_model/__init__.py +++ b/dts/framework/testbed_model/__init__.py @@ -13,6 +13,7 @@ from .hw import ( LogicalCore, LogicalCoreAmount, + LogicalCoreAmountFilter, LogicalCoreList, LogicalCoreListFilter, VirtualDevice, diff --git a/dts/framework/testbed_model/sut_node.py b/dts/framework/testbed_model/sut_node.py index ff9d55f2d8..1bf90a448f 100644 --- a/dts/framework/testbed_model/sut_node.py +++ b/dts/framework/testbed_model/sut_node.py @@ -8,7 +8,7 @@ from pathlib import PurePath from framework.config import BuildTargetConfiguration, NodeConfiguration -from framework.remote_session import OSSession +from framework.remote_session import CommandResult, OSSession from framework.settings import SETTINGS from framework.utils import EnvVarsDict, skip_setup @@ -249,3 +249,13 @@ def create_eal_parameters( vdevs=vdevs, other_eal_param=other_eal_param, ) + + def run_dpdk_app( + self, app_path: PurePath, eal_args: EalParameters, timeout: float = 30 + ) -> CommandResult: + """ + Run DPDK application on the remote node. + """ + return self.main_session.send_command( + f"{app_path} {eal_args}", timeout, verify=True + ) diff --git a/dts/tests/TestSuite_hello_world.py b/dts/tests/TestSuite_hello_world.py new file mode 100644 index 0000000000..dcdf637a2a --- /dev/null +++ b/dts/tests/TestSuite_hello_world.py @@ -0,0 +1,64 @@ +# SPDX-License-Identifier: BSD-3-Clause +# Copyright(c) 2010-2014 Intel Corporation + +""" +Run the helloworld example app and verify it prints a message for each used core. +No other EAL parameters apart from cores are used. +""" + +from framework.test_suite import TestSuite +from framework.testbed_model import ( + LogicalCoreAmount, + LogicalCoreAmountFilter, + LogicalCoreList, +) + + +class TestHelloWorld(TestSuite): + def set_up_suite(self) -> None: + """ + Setup: + Build the app we're about to test - helloworld. + """ + self.app_helloworld_path = self.sut_node.build_dpdk_app("helloworld") + + def test_hello_world_single_core(self) -> None: + """ + Steps: + Run the helloworld app on the first usable logical core. + Verify: + The app prints a message from the used core: + "hello from core " + """ + + # get the first usable core + lcore_amount = LogicalCoreAmount(1, 1, 1) + lcores = LogicalCoreAmountFilter(self.sut_node.lcores, lcore_amount).filter() + eal_para = self.sut_node.create_eal_parameters( + lcore_filter_specifier=lcore_amount + ) + result = self.sut_node.run_dpdk_app(self.app_helloworld_path, eal_para) + self.verify( + f"hello from core {int(lcores[0])}" in result.stdout, + f"helloworld didn't start on lcore{lcores[0]}", + ) + + def test_hello_world_all_cores(self) -> None: + """ + Steps: + Run the helloworld app on all usable logical cores. + Verify: + The app prints a message from all used cores: + "hello from core " + """ + + # get the maximum logical core number + eal_para = self.sut_node.create_eal_parameters( + lcore_filter_specifier=LogicalCoreList(self.sut_node.lcores) + ) + result = self.sut_node.run_dpdk_app(self.app_helloworld_path, eal_para, 50) + for lcore in self.sut_node.lcores: + self.verify( + f"hello from core {int(lcore)}" in result.stdout, + f"helloworld didn't start on lcore{lcore}", + ) -- 2.30.2