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 9FEB746D06; Tue, 12 Aug 2025 18:06:22 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 7E91F40270; Tue, 12 Aug 2025 18:06:22 +0200 (CEST) Received: from mail-pf1-f173.google.com (mail-pf1-f173.google.com [209.85.210.173]) by mails.dpdk.org (Postfix) with ESMTP id 99FA0400EF for ; Tue, 12 Aug 2025 18:06:21 +0200 (CEST) Received: by mail-pf1-f173.google.com with SMTP id d2e1a72fcca58-76bd050184bso7296395b3a.2 for ; Tue, 12 Aug 2025 09:06:21 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=iol.unh.edu; s=unh-iol; t=1755014780; x=1755619580; darn=dpdk.org; h=cc:to:subject:message-id:date:from:mime-version:from:to:cc:subject :date:message-id:reply-to; bh=9kawyV+b4uZ0D+wWO10Wgqa4uqzieLD1md67Ix1ACig=; b=ROokCsZjH66f6HoID6KyYo1ezaSEUZOsUatUQEERu7cri8b1mBA8kiDr38Nbht5YUq K8p+HPDa8c/oyMaJVkX3dnAMwJooeMDd3PUG7PCJwtHt+RyEXsDmKQmXSFT3CKtpeC7u io+Hqwe+T5PpQ3+eAl9a7EdU6sShnT3u0zfnM= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1755014780; x=1755619580; h=cc:to:subject:message-id:date:from:mime-version:x-gm-message-state :from:to:cc:subject:date:message-id:reply-to; bh=9kawyV+b4uZ0D+wWO10Wgqa4uqzieLD1md67Ix1ACig=; b=ayY7i2RnZsyA64GbRYUvWMSifiqM7XSZXjk2SGikcxXD+1rxZpi16J9c8KrP4rXCUE A40C2cRNHe+UIlzTc0J34GaJd7Kq/9RVwp6C/u8+HQ1D7pBegXjdcfm13+irnLwGUb7G QxWvOlJTVzwX7ST73e2OTXDVK8FYP3gK/16Ub6S9vsUF2MqQ5Zai1PP5bLoUM8O/9siU MypS58kIKZlkGOM+8zRDHSojOhKlxX9LWWH0ru+/EVgCAGcRVQ++/aV1VHvZTXJM4ZDK vL5JfTUh6s4x7nCalC2b5gkfJtYdO1Szt29XRudnMtMWFyne8vAK9CI1FZqzGGGNfcDB p3xg== X-Gm-Message-State: AOJu0YzWy/BuEIGu3ZTFPWWCTQTPnd32ZADkagHw6biPTxm/2SE88KOZ EpHcFjrYTzqPj13axYmiJMbIK3K7a5Ou1lba6cXfvDDxlm/EZ3SFgc+8QkwjqVndjZ6nMUmEtA+ HxNqA8z+QQbU8xn78mnj9SdcORkW7Bf7k3YeTmg54gg== X-Gm-Gg: ASbGncuXYbOAm82eT+x9PBIl3mPhyCj9XHYuFCLvMwUAFhvbtJhwclfnadfzaG3NElu TYu0YITZgfCJv+sitJD8DtKLHEHjgC4Lvde9XxhRAV77Wloo7c6T7x/dyUYwAwiT521MGIpGFPo gO1Ogp+QGUZbr43O0O+E/hfQiHA7AYUXMcLpKDS9dGPxICoLJdKbEvUBkXI4mjaOUGg3P3DJmzb u1a1XyRIwL40DsfbN9XHQ== X-Google-Smtp-Source: AGHT+IGCckUNuS1LQw/tZyB3VEpdWaP03RBVLObA9W1ZVWMqpK8/oDaBkhF56tKCxk3BQQFOIYyKqu7xk0VxNv5c2B8= X-Received: by 2002:a17:903:2b03:b0:240:a889:554d with SMTP id d9443c01a7336-2430c102eefmr3193665ad.45.1755014780325; Tue, 12 Aug 2025 09:06:20 -0700 (PDT) MIME-Version: 1.0 From: Patrick Robb Date: Tue, 12 Aug 2025 11:59:59 -0400 X-Gm-Features: Ac12FXwz0NmTM6W8wEecT_a1EaTS_5YYAzVtTzt2RbkmC7LpdgBYA7TrJtxOcng Message-ID: Subject: DTS WG Meeting Minutes - July 31, 2025 To: dev Cc: ci@dpdk.org, Dean Marx , Paul Szczepanek , Luca Vizzarro , Andrew Bailey , Thomas Wilks Content-Type: multipart/alternative; boundary="0000000000008eacf8063c2d385b" X-BeenThere: ci@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK CI discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: ci-bounces@dpdk.org --0000000000008eacf8063c2d385b Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable #################################################################### July 31, 2025 Attendees * Patrick Robb * Luca Vizzarro * Paul Szczepanek * Andrew Bailey #################################################################### Minutes =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D General Discussion * RFC deadline for 25.11 development is August 31. =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D Patch discussions * RSS: * Updated and rebased. Ivan=E2=80=99s comments have been addressed. * Patrick Robband Andrew Baileyto review this and test it on lab devices (must include mellanox) * dts: add file management * Patrick Robbto review * Provides abstraction which defines a file on TG, SUT, or DTS engine system * There are some stylistic differences between how testsuite docstrings are written * Currently, the only =E2=80=9Crule=E2=80=9D is that testsuite developer= s write =E2=80=9Csteps=E2=80=9D and =E2=80=9Cverify=E2=80=9D section * Steps and Verify should also have an empty line included between them * Otherwise, there are various discrepancies (example, ordered vs unordered list for providing =E2=80=9Csteps=E2=80=9D) and we need to provid= e more comprehensive rules to bring the testsuite docstrings in alignment * Also some missing =E2=80=9CAttributes=E2=80=9D or =E2=80=9CReturns=E2= =80=9D sections for various class and method docstrings * There is major duplication between module, class, and method docstrings in each testsuite. * Module docstring should be a few sentences briefly identifying the testsuite and the DPDK functionality validated in the testsuite * Testsuite class can just be a stock phrase to satisfy the linter, like =E2=80=9Ctestsuite class=E2=80=9D * Testcase docstrings contain the actual details * Thomas Wilks is looking through testsuites and looking for problems with code style. We should provide a code style guide going forward. * Example: Some functions which return None have no typehinted return, when technically they should return None like =E2=80=9C -> None =E2=80=9C * Testcase naming convention * The old requirement of prefixing testcases with =E2=80=9Ctest_=E2=80= =9D is still present in some testcases. Although it=E2=80=99s not a =E2=80=9Cproblem=E2= =80=9D for testcases to be named in this way, we should stop doing this going forward and do a 1 time re-naming of all the existing testcases that are named in this way. * Docs builds: * We have some private methods which are not actually made private (they are not prefixed with an underscore), so these methods are showing up in the testsuite docs unnecessarily. * Patrick Robb and luca.vizzarro@arm.comreviewed change requested patches * Confirmed all of those patches are tracked * It may be better to not use the change requested status at all. Just provide comments on series, and let the submitter change the series to =E2=80=9Csuperseded=E2=80=9D when they send a new version. =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D Bugzilla discussions * Reviewed bugzilla tickets: * https://bugs.dpdk.org/buglist.cgi?quicksearch=3Dcomponent%3Adts&list_id=3D9= 444 =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D Any other business * Andrew at UNH is looking at TG driver binding, which was previously removed from DTS. * Intel CI Docs build had a failure for some DTS docs on a patch applied to next-dts * Need to remove the autodoc pydantic dependency from the standard docs build, which Luca is going to do * Next meeting is Aug 14, 2025 --0000000000008eacf8063c2d385b Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable
##########################################################= ##########
July 31, 2025
Attendees
* Patrick Robb
* Luca Vizzar= ro
* Paul Szczepanek
* Andrew Bailey

#########################= ###########################################
Minutes

=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D
General Discussion
* RF= C deadline for 25.11 development is August 31.

=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D
Patch discussions
* RSS:
=C2= =A0 =C2=A0* Updated and rebased. Ivan=E2=80=99s comments have been addresse= d.
=C2=A0 =C2=A0* Patrick Robband Andrew Baileyto review this and test i= t on lab devices (must include mellanox)
* dts: add file management
= =C2=A0 =C2=A0* Patrick Robbto review
=C2=A0 =C2=A0* Provides abstraction= which defines a file on TG, SUT, or DTS engine system
* There are some = stylistic differences between how testsuite docstrings are written
=C2= =A0 =C2=A0* Currently, the only =E2=80=9Crule=E2=80=9D is that testsuite de= velopers write =E2=80=9Csteps=E2=80=9D and =E2=80=9Cverify=E2=80=9D section=
=C2=A0 =C2=A0 =C2=A0 * Steps and Verify should also have an empty line = included between them
=C2=A0 =C2=A0* Otherwise, there are various discre= pancies (example, ordered vs unordered list for providing =E2=80=9Csteps=E2= =80=9D) and we need to provide more comprehensive rules to bring the testsu= ite docstrings in alignment
=C2=A0 =C2=A0* Also some missing =E2=80=9CAt= tributes=E2=80=9D or =E2=80=9CReturns=E2=80=9D sections for various class a= nd method docstrings
=C2=A0 =C2=A0* There is major duplication between m= odule, class, and method docstrings in each testsuite.
=C2=A0 =C2=A0 =C2= =A0 * Module docstring should be a few sentences briefly identifying the te= stsuite and the DPDK functionality validated in the testsuite
=C2=A0 =C2= =A0 =C2=A0 * Testsuite class can just be a stock phrase to satisfy the lint= er, like =E2=80=9Ctestsuite class=E2=80=9D
=C2=A0 =C2=A0 =C2=A0 * Testca= se docstrings contain the actual details
* Thomas Wilks is looking throu= gh testsuites and looking for problems with code style. We should provide a= code style guide going forward.
=C2=A0 =C2=A0* Example: Some functions = which return None have no typehinted return, when technically they should r= eturn None like =E2=80=9C -> None =E2=80=9C
* Testcase naming convent= ion
=C2=A0 =C2=A0* The old requirement of prefixing testcases with =E2= =80=9Ctest_=E2=80=9D is still present in some testcases. Although it=E2=80= =99s not a =E2=80=9Cproblem=E2=80=9D for testcases to be named in this way,= we should stop doing this going forward and do a 1 time re-naming of all t= he existing testcases that are named in this way.
* Docs builds:
=C2= =A0 =C2=A0* We have some private methods which are not actually made privat= e (they are not prefixed with an underscore), so these methods are showing = up in the testsuite docs unnecessarily.
* Patrick Robb and luca.vizzarro= @arm.comreviewed change requested patches
=C2=A0 =C2=A0* Confirmed all o= f those patches are tracked
=C2=A0 =C2=A0* It may be better to not use t= he change requested status at all. Just provide comments on series, and let= the submitter change the series to =E2=80=9Csuperseded=E2=80=9D when they = send a new version.

=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D
Bugzilla discussions
* Reviewed bugzilla tickets:
=C2=A0= =C2=A0* https://bugs.dpdk.org/buglist.cgi?quicksearch= =3Dcomponent%3Adts&list_id=3D9444

=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D
Any other business
* Andrew at UNH = is looking at TG driver binding, which was previously removed from DTS. * Intel CI Docs build had a failure for some DTS docs on a patch applied t= o next-dts
=C2=A0 =C2=A0* Need to remove the autodoc pydantic dependency= from the standard docs build, which Luca is going to do
* Next meeting = is Aug 14, 2025
--0000000000008eacf8063c2d385b--