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 0D10C45F24; Mon, 23 Dec 2024 19:52:57 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id A00BC4026C; Mon, 23 Dec 2024 19:52:56 +0100 (CET) Received: from mail-pj1-f47.google.com (mail-pj1-f47.google.com [209.85.216.47]) by mails.dpdk.org (Postfix) with ESMTP id DE6A140263 for ; Mon, 23 Dec 2024 19:52:54 +0100 (CET) Received: by mail-pj1-f47.google.com with SMTP id 98e67ed59e1d1-2ef8c012913so3179466a91.3 for ; Mon, 23 Dec 2024 10:52:54 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=iol.unh.edu; s=unh-iol; t=1734979974; x=1735584774; darn=dpdk.org; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:from:to:cc:subject:date:message-id:reply-to; bh=Ecf11WFF7kUSpwDmQ+J3UO+KlrZAX+JyhIWD8DNkXrY=; b=gVkbZau2IBHfzWccIy5vGoIeXADxXOPB8ngDEEpaD4b8PbSZoQaYOMATlmH1NdtAbf wtapRyNUTzlgFmlTiPkFzimtW93kTfrgMBv6gBXb51fA6tRo+K+wO+9OvkfBx6cXuJTu hv5bp/NaFq/KLJ7dhUuU8fGFzOM38CRhH13U0= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1734979974; x=1735584774; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=Ecf11WFF7kUSpwDmQ+J3UO+KlrZAX+JyhIWD8DNkXrY=; b=LXW0TgOC/wwMnm1ZFZyd8PiHnzVjvQffXr/7h0lX+BBHHEzRatkfn2YzSSWIHjFNIj y50HkgpiqAfph9EX9KBZSpggcnQqL/7QbU88DhVmt3G2pDEqf3kGGaeo9PhfBahssYgm qQ7e7xlHx15Z3TAvq3ChWbjjZlEWYEA7mu+GtFZmUzNlRhXTrtlGG0LO5zjssp/CUpk/ mQNQLDuD5m8HcKoqKpb9umaAMcPT93f1lzlKGY5agyyc9P9oMCWffSckplCXDgu6Seqh 7pEy7qN0cT+QTAK19dNP9QrjsO96N1mzIzX2mU1QE8q9OHkJ11P63rnXe7TWXeZ7bfmb 3tWw== X-Forwarded-Encrypted: i=1; AJvYcCWMkeSMrCxQWCbhq3Nv07PEYKSCyWDL8i7jbNlrX0/BV/HYhY73OV7xllNYCeigTOfigho=@dpdk.org X-Gm-Message-State: AOJu0YyOZ2f5bnkUl9x+2EsDJAGUW6Pb9ouMTkoqb32KHKQaytPNCEFg 68HBgTuaaMhsqgSUSsMvLPEZhvPutIHbEw767UGPu70dCkiyEK3+l7kh4/OLfvSMP830Uk6QReM vOzbzMq9uCwY8OBF+7otZFCQglFr94pi36ZvgCg== X-Gm-Gg: ASbGncvmcDu2ifGRRiRQjFTpZ+NOX53fslswOldHp+oSuC5snCllW2a+Y5Cj4RxHlic mkbyGw+3811jlTv5Zn+qBKj6JR3B1Mx3XAUJH4FefYgHi267tFn6QOMjpQht0MbdgXLZmj6c= X-Google-Smtp-Source: AGHT+IGIEhjUZVvOeSCgpWB7dJER4py16RC7EkDZcGHZthAZ4QhzTfAfG7Pzd0XXC74nogythUJiGUcdpynQ3nlAoTw= X-Received: by 2002:a17:90b:51c1:b0:2ee:8031:cdbc with SMTP id 98e67ed59e1d1-2f452ee634bmr18693732a91.23.1734979973814; Mon, 23 Dec 2024 10:52:53 -0800 (PST) MIME-Version: 1.0 References: <20241209215406.6982-2-dmarx@iol.unh.edu> <20241223185739.29621-1-dmarx@iol.unh.edu> In-Reply-To: <20241223185739.29621-1-dmarx@iol.unh.edu> From: Patrick Robb Date: Mon, 23 Dec 2024 13:50:23 -0500 Message-ID: Subject: Re: [PATCH v2 1/1] dts: add EAL confidence check suite To: Dean Marx Cc: npratte@iol.unh.edu, luca.vizzarro@arm.com, yoan.picchi@foss.arm.com, Honnappa.Nagarahalli@arm.com, paul.szczepanek@arm.com, dev@dpdk.org Content-Type: multipart/alternative; boundary="00000000000008388b0629f481bd" 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 --00000000000008388b0629f481bd Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Hey Dean, I would retain the hello world name if I were you, in place of "eal" as you use here. I don't want to give off the impression that we are testing EAL in some intentional way, as opposed to providing a reference testsuite. On Mon, Dec 23, 2024 at 12:56=E2=80=AFPM Dean Marx wrot= e: > Add a test suite to replace hello_world which simply > starts and stops a testpmd session. The user can use > this as a confidence check to verify their configuration. > > Signed-off-by: Dean Marx > Reviewed-by: Paul Szczepanek > --- > dts/tests/TestSuite_eal.py | 27 +++++++++++++++++++++++++++ > 1 file changed, 27 insertions(+) > create mode 100644 dts/tests/TestSuite_eal.py > > diff --git a/dts/tests/TestSuite_eal.py b/dts/tests/TestSuite_eal.py > new file mode 100644 > index 0000000000..7cf96e61cd > --- /dev/null > +++ b/dts/tests/TestSuite_eal.py > @@ -0,0 +1,27 @@ > +# SPDX-License-Identifier: BSD-3-Clause > +# Copyright(c) 2024 University of New Hampshire > + > +"""DPDK EAL confidence check suite. > + > +Starts and stops a testpmd session to verify EAL parameters > +are properly configured. > +""" > + > +from framework.remote_session.testpmd_shell import TestPmdShell > +from framework.test_suite import TestSuite, func_test > + > + > +class TestEal(TestSuite): > + """EAL test suite. One test case, which starts and stops a testpmd > session.""" > + > + @func_test > + def test_verify_eal(self) -> None: > + """EAL confidence test. > + > + Steps: > + Start testpmd session and check status. > + Verify: > + The testpmd session is alive after starting. > + """ > + with TestPmdShell(node=3Dself.sut_node) as testpmd: > + testpmd.start() > -- > 2.44.0 > > --00000000000008388b0629f481bd Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable
Hey Dean,

I would retain the hello worl= d name if I were you,=C2=A0in place of "eal" as you use here. I d= on't want to give off the impression that we are testing EAL in some in= tentional way,=C2=A0as opposed to providing a reference testsuite.

On Mon, Dec 23, 2024 at 12:56=E2=80=AFPM Dean Marx <<= a href=3D"mailto:dmarx@iol.unh.edu">dmarx@iol.unh.edu> wrote:
Add a test suite to re= place hello_world which simply
starts and stops a testpmd session. The user can use
this as a confidence check to verify their configuration.

Signed-off-by: Dean Marx <dmarx@iol.unh.edu>
Reviewed-by: Paul Szczepanek <paul.szczepanek@arm.com>
---
=C2=A0dts/tests/TestSuite_eal.py | 27 +++++++++++++++++++++++++++
=C2=A01 file changed, 27 insertions(+)
=C2=A0create mode 100644 dts/tests/TestSuite_eal.py

diff --git a/dts/tests/TestSuite_eal.py b/dts/tests/TestSuite_eal.py
new file mode 100644
index 0000000000..7cf96e61cd
--- /dev/null
+++ b/dts/tests/TestSuite_eal.py
@@ -0,0 +1,27 @@
+# SPDX-License-Identifier: BSD-3-Clause
+# Copyright(c) 2024 University of New Hampshire
+
+"""DPDK EAL confidence check suite.
+
+Starts and stops a testpmd session to verify EAL parameters
+are properly configured.
+"""
+
+from framework.remote_session.testpmd_shell import TestPmdShell
+from framework.test_suite import TestSuite, func_test
+
+
+class TestEal(TestSuite):
+=C2=A0 =C2=A0 """EAL test suite. One test case, which start= s and stops a testpmd session."""
+
+=C2=A0 =C2=A0 @func_test
+=C2=A0 =C2=A0 def test_verify_eal(self) -> None:
+=C2=A0 =C2=A0 =C2=A0 =C2=A0 """EAL confidence test.
+
+=C2=A0 =C2=A0 =C2=A0 =C2=A0 Steps:
+=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 Start testpmd session and check = status.
+=C2=A0 =C2=A0 =C2=A0 =C2=A0 Verify:
+=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 The testpmd session is alive aft= er starting.
+=C2=A0 =C2=A0 =C2=A0 =C2=A0 """
+=C2=A0 =C2=A0 =C2=A0 =C2=A0 with TestPmdShell(node=3Dself.sut_node) as tes= tpmd:
+=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 testpmd.start()
--
2.44.0

--00000000000008388b0629f481bd--