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 B75DE44161; Wed, 5 Jun 2024 19:54:53 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 9040C40DDA; Wed, 5 Jun 2024 19:54:53 +0200 (CEST) Received: from mail-oa1-f42.google.com (mail-oa1-f42.google.com [209.85.160.42]) by mails.dpdk.org (Postfix) with ESMTP id 9E338402C9 for ; Wed, 5 Jun 2024 19:54:51 +0200 (CEST) Received: by mail-oa1-f42.google.com with SMTP id 586e51a60fabf-24c9f892aeaso23334fac.2 for ; Wed, 05 Jun 2024 10:54:51 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=iol.unh.edu; s=unh-iol; t=1717610091; x=1718214891; darn=dpdk.org; h=cc:to:subject:message-id:date:from:mime-version:from:to:cc:subject :date:message-id:reply-to; bh=ycf/aVeInEf7CStXZ+dz9DAIT8ZP4qsW6QlLwPLJ8jY=; b=IoaiwGTqpiLq0M4+6JCbgUsy25NqbMbev17R9g0cpdtQon7OArvKIWB+dRpGti3hVq OnvwKj8RTSVtm3T0y9GBLcnq5E1Ll+Sd1WQsXMKQBETWs2k4aOSP1PtxvFH72BM/V6Na U4txZ146jymFDXXzyX9JSKwvpS4j//I+nqcAA= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1717610091; x=1718214891; h=cc:to:subject:message-id:date:from:mime-version:x-gm-message-state :from:to:cc:subject:date:message-id:reply-to; bh=ycf/aVeInEf7CStXZ+dz9DAIT8ZP4qsW6QlLwPLJ8jY=; b=JAj6Jmsem0S9gnkRJFTHltx8YfUOYonoISzKWN4bo7w5a3MI+GCBmBEEnkJUjKTZUN Y1raVbKMDFEXspgJHXDEQnA+Vk3NSIwC/7rhBD5QlNajgFIR/eWf9/CD0+dZrMBOIjaX 7ddv8P2AsnPnWVDFDrUVvenfOuaLCkhAyUovFGRosOwvAd+cNkwBvY2xhZCafDqkeaSl lND5SvhrZazEmRdzcjMQR9fhEp7JJfBaw/pdN50zXmxXNZftefwnD/nqy71ppeNU+YOj StS3a6JwGOrEL7xR6wOaK5ltFbULgoBel01gkmNFRRVW/lyTaBxhOepkvVgkbmEHfTe0 P+4w== X-Gm-Message-State: AOJu0YzuuH//MeqpJ1T7RQqbYfq2KevO0Lca3630WP6e2CeGEt8pzBDO eyE7BV2MI8t41DQPd7HXIH8z7W0ncNecSE/1907szi/LFsGMUXa8FdWz54wn2jZCKqZkGF3ANpi Q8ACFZO+kKeAGuei1EAH2SUQc6PP5/Rhm9udB+Mygl+wAa0XIRYP5gA== X-Google-Smtp-Source: AGHT+IGYmo1Z7eHMCRmaW+HFxXSSbGctdbSRXs/JpAMtUXTh5IUKx0Vq5c0U2HelRzpu0j/Vz6p+a775WlqX14u0alQ= X-Received: by 2002:a05:6870:8191:b0:250:7e40:4eb8 with SMTP id 586e51a60fabf-25121fed0f3mr4392033fac.37.1717610090757; Wed, 05 Jun 2024 10:54:50 -0700 (PDT) MIME-Version: 1.0 From: Patrick Robb Date: Wed, 5 Jun 2024 13:54:40 -0400 Message-ID: Subject: DTS WG Meeting Minutes - June 5, 2024 To: dev Cc: ci@dpdk.org, dts@dpdk.org Content-Type: multipart/alternative; boundary="000000000000529c68061a28435b" 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 --000000000000529c68061a28435b Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable ##################################################################### June 5, 2024 Attendees * Patrick Robb * Nicholas Pratte * Dean Marx * Luca Vizzarro * Juraj Linke=C5=A1 * Paul Szczepanek * Tomas Durovec * Jeremy Spewock ##################################################################### 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 Announcements * Testsuites which have been picked up: * Dean picked up the vlan_testsuite since the previous meeting: https://bugs.dpdk.org/show_bug.cgi?id=3D1453 * Nick has started on mac_filter =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=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 * Patches which can be merged?: * Juraj is going to try and check in with Thomas about the below patches, following this meeting * Mypy patch: https://patchwork.dpdk.org/project/dpdk/list/?series=3D320= 26 * Error and usage improvements: https://patchwork.dpdk.org/project/dpdk/list/?series=3D32038 * Hugepages patch has not been run on top of the new mypy version, so this should be checked. Will submit a subsequent version if new mypy flags anything, otherwise we will just ping Juraj indicating the patch is ready: https://patchwork.dpdk.org/project/dpdk/list/?series=3D32035 * https://patches.dpdk.org/project/dpdk/list/?series=3D31790 * https://patches.dpdk.org/project/dpdk/list/?series=3D31808 * https://patches.dpdk.org/project/dpdk/list/?series=3D31811 * Improve interactive shell output gathering: https://patchwork.dpdk.org/project/dpdk/list/?series=3D32014 * Needs a review from Juraj, but is =E2=80=9Cdone=E2=80=9D and is review= ed by Luca * Capabilities patch: * On E810 and XL710, the non-offload testcase is skipping * Will not skip on E810 if you up the MTU before checking the capability - need to also check on XL710 * When checking capabilities, one idea would be to set MTU to 9000, get scatter capability, set MTU back to 1500, so that we do not affect any other capabilities (i.e. what if another capability was disabled by a 9000 MTU) * Show rx port info 0 0 is probably still valid, but we are missing the correct port configuration which will give us the correct reading (the mtu configuration) * Pre-built DPDK feature: * Idea is to scp in a DPDK tarball with a pre-built DPDK * User should be able to specify either: * a path to a dpdk repo where DPDK has NOT been built (can be local or on SUT) * Or a path to a DPDK repo with a build dir inside * Could remove the git-ref option as it becomes redundant with this option. You can easily checkout to an older commit hash, refer to this repo when running DTS * The use case we care about for developers is making it easy for them to make local changes to their repo, and seamlessly run DTS from that. * This is even easier than the git-ref option, as they don=E2=80=99t = even need to commit anything, they can simply provide the path to their dpdk development repo * Testpmd parameters patch: * Juraj will provide a review * There is a blanket requirement for google format docstrings for every function * It would be a benefit if we had a script (should exist under dts-format-patch) which will support checking for these, and remind developers if they are submitting patches which are missing some docstrings * Pylama is being used to audit patches within dts-format-patch.sh currently, but is out of date and does not resolve this issue * Should look into using =E2=80=9CRuff=E2=80=9D for the next release * Patrick Robbmake a new ticket for this issue * API documentation building: * Expected to be merged soon * Testpmd params patch introduces a new module without an .rst, so this introduces a dependency. If both patches get merged at the same time, an additional patch for testpmd params can be included. =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=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 * No discussion today =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=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 * There are some 24.07 roadmap contains items which will not get done: * Externally compiled DPDK is starting to get worked on now, but will not be merged in this release * Dynamic Queue will probably not be merged this release, as we have shifted over to writing other testsuite which were not originally in our roadmap (like vlan) * Jeremy has indicated he can pick this up soon * More discussion about creation of a quiet wire for DTS * Will proceed with writing testsuites according to current =E2=80=9Cwir= e noise tolerant=E2=80=9D approach (send some packets with a given payload, sniff f= or a list of packets during testsuite, check for expected packets from output list) * Can provide support for quiet wire later if needed. * Next meeting Jun 19, 2024 --000000000000529c68061a28435b Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable

######################################################= ###############
June 5, 2024
Attendees
* Patrick Robb
* Nichola= s Pratte
* Dean Marx
* Luca Vizzarro
* Juraj Linke=C5=A1
* Paul= Szczepanek
* Tomas Durovec
* Jeremy Spewock

#################= ####################################################
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 Announcem= ents
* Testsuites which have been picked up:
=C2=A0 =C2=A0* Dean pick= ed up the vlan_testsuite since the previous meeting: https://bugs.dpdk.org/show_bug.cgi?id=3D= 1453
=C2=A0 =C2=A0* Nick has started on mac_filter
=C2=A0 =C2=A0<= br>=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=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 discu= ssions
* Patches which can be merged?:
=C2=A0 =C2=A0* Juraj is going = to try and check in with Thomas about the below patches, following this mee= ting
=C2=A0 =C2=A0* Mypy patch: https://patchwork.dpdk.org/project/dpdk/li= st/?series=3D32026
=C2=A0 =C2=A0* Error and usage improvements: https:= //patchwork.dpdk.org/project/dpdk/list/?series=3D32038
=C2=A0 =C2=A0= * Hugepages patch has not been run on top of the new mypy version, so this = should be checked. Will submit a subsequent version if new mypy flags anyth= ing, otherwise we will just ping Juraj indicating the patch is ready: https:/= /patchwork.dpdk.org/project/dpdk/list/?series=3D32035
=C2=A0 =C2=A0*= htt= ps://patches.dpdk.org/project/dpdk/list/?series=3D31790
=C2=A0 =C2= =A0* https://patches.dpdk.org/project/dpdk/list/?series=3D31808
=C2=A0 = =C2=A0* https://patches.dpdk.org/project/dpdk/list/?series=3D31811
* Imp= rove interactive shell output gathering: https://patchwork.dpdk.org/project/d= pdk/list/?series=3D32014
=C2=A0 =C2=A0* Needs a review from Juraj, b= ut is =E2=80=9Cdone=E2=80=9D and is reviewed by Luca
* Capabilities patc= h:
=C2=A0 =C2=A0* On E810 and XL710, the non-offload testcase is skippin= g
=C2=A0 =C2=A0 =C2=A0 * Will not skip on E810 if you up the MTU before = checking the capability - need to also check on XL710
=C2=A0 =C2=A0 =C2= =A0 * When checking capabilities, one idea would be to set MTU to 9000, get= scatter capability, set MTU back to 1500, so that we do not affect any oth= er capabilities (i.e. what if another capability was disabled by a 9000 MTU= )
=C2=A0 =C2=A0 =C2=A0 * Show rx port info 0 0 is probably still valid, = but we are missing the correct port configuration which will give us the co= rrect reading (the mtu configuration)
* Pre-built DPDK feature:
=C2= =A0 =C2=A0* Idea is to scp in a DPDK tarball with a pre-built DPDK
=C2= =A0 =C2=A0* User should be able to specify either:
=C2=A0 =C2=A0 =C2=A0= * a path to a dpdk repo where DPDK has NOT been built (can be local or on = SUT)
=C2=A0 =C2=A0 =C2=A0 * Or a path to a DPDK repo with a build dir in= side
=C2=A0 =C2=A0* Could remove the git-ref option as it becomes redund= ant with this option. You can easily checkout to an older commit hash, refe= r to this repo when running DTS
=C2=A0 =C2=A0* The use case we care abou= t for developers is making it easy for them to make local changes to their = repo, and seamlessly run DTS from that.
=C2=A0 =C2=A0 =C2=A0 * This is = even easier than the git-ref option, as they don=E2=80=99t even need to com= mit anything, they can simply provide the path to their dpdk development re= po
* Testpmd parameters patch:
=C2=A0 =C2=A0* Juraj will provide a re= view
* There is a blanket requirement for google format docstrings for e= very function
=C2=A0 =C2=A0* It would be a benefit if we had a script (s= hould exist under dts-format-patch) which will support checking for these, = and remind developers if they are submitting patches which are missing some= docstrings
=C2=A0 =C2=A0* Pylama is being used to audit patches within = dts-format-patch.sh currently, but is out of date and does not resolve this= issue
=C2=A0 =C2=A0* Should look into using =E2=80=9CRuff=E2=80=9D for = the next release
=C2=A0 =C2=A0* Patrick Robbmake a new ticket for this i= ssue
* API documentation building:
=C2=A0 =C2=A0* Expected to be mer= ged soon
=C2=A0 =C2=A0* Testpmd params patch introduces a new module wit= hout an .rst, so this introduces a dependency. If both patches get merged a= t the same time, an additional patch for testpmd params can be included.=C2=A0 =C2=A0 =C2=A0 =C2=A0
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=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
* No discussion today

=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=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 busines= s
* There are some 24.07 roadmap contains items which will not get done:=
=C2=A0 =C2=A0* =C2=A0Externally compiled DPDK is starting to get worked= on now, but will not be merged in this release
=C2=A0 =C2=A0* Dynamic Q= ueue will probably not be merged this release, as we have shifted over to w= riting other testsuite which were not originally in our roadmap (like vlan)=
=C2=A0 =C2=A0 =C2=A0 * Jeremy has indicated he can pick this up soon* More discussion about creation of a quiet wire for DTS
=C2=A0 =C2=A0*= Will proceed with writing testsuites according to current =E2=80=9Cwire no= ise tolerant=E2=80=9D approach (send some packets with a given payload, sni= ff for a list of packets during testsuite, check for expected packets from = output list)
=C2=A0 =C2=A0* Can provide support for quiet wire later if = needed.
* Next meeting Jun 19, 2024
--000000000000529c68061a28435b--