On Tue, 20 Jan 2026 at 10:13, David Marchand <[email protected]> wrote: > > Hello dts guys, > > On Fri, 7 Nov 2025 at 22:50, Dean Marx <[email protected]> wrote: > > > > Add start tx_first method to testpmd shell, which sends > > a specified number of burst packets prior to starting > > packet forwarding. > > > > Signed-off-by: Dean Marx <[email protected]> > > Reviewed-by: Luca Vizzarro <[email protected]> > > --- > > dts/api/testpmd/__init__.py | 21 +++++++++++++++++++++ > > 1 file changed, 21 insertions(+) > > > > diff --git a/dts/api/testpmd/__init__.py b/dts/api/testpmd/__init__.py > > index aadb7f4e70..6d70927660 100644 > > --- a/dts/api/testpmd/__init__.py > > +++ b/dts/api/testpmd/__init__.py > > @@ -204,6 +204,27 @@ def start(self, verify: bool = True) -> None: > > self._logger.debug(f"Failed to start packet forwarding: > > \n{start_cmd_output}") > > raise InteractiveCommandExecutionError("Testpmd failed to > > start packet forwarding.") > > > > + @_requires_started_ports > > + def start_tx_first(self, burst_num: int, verify: bool = True) -> None: > > + """Start packet forwarding after sending specified number of > > bursts of packets. > > + > > + Args: > > + burst_num: Number of packets to send before stopping > > transmission. > > + verify: If :data:`True` , a second start command will be sent > > in an attempt to verify > > + packet forwarding started as expected. > > + > > + Raises: > > + InteractiveCommandExecutionError: If `verify` is :data:`True` > > and forwarding fails to > > + start or ports fail to come up. > > + """ > > + self.send_command(f"start tx_first {burst_num if burst_num is not > > None else ""}") > > + if verify: > > + # If forwarding was already started, sending "start" again > > should tell us > > + start_cmd_output = self.send_command("start") > > + if "Packet forwarding already started" not in start_cmd_output: > > + self._logger.debug(f"Failed to start packet forwarding: > > \n{start_cmd_output}") > > + raise InteractiveCommandExecutionError("Testpmd failed to > > start packet forwarding.") > > + > > def stop(self, verify: bool = True) -> str: > > """Stop packet forwarding. > > > > I can see (uncaught) warnings in GHA when generating the documentation. > I don't know if this is something new, or if it was like this from the start. > > Please have a look. > > 2026-01-20T08:16:58.5130393Z [250/252] Generating dts_api_html with a > custom command > 2026-01-20T08:16:58.5131324Z WARNING: autodoc: failed to import module > 'testpmd' from module 'api'; the following exception was raised: > 2026-01-20T08:16:58.5132132Z Traceback (most recent call last): > 2026-01-20T08:16:58.5132676Z File > "/usr/lib/python3/dist-packages/sphinx/ext/autodoc/importer.py", line > 70, in import_module > 2026-01-20T08:16:58.5133175Z return importlib.import_module(modname) > 2026-01-20T08:16:58.5133807Z File > "/usr/lib/python3.10/importlib/__init__.py", line 126, in > import_module > 2026-01-20T08:16:58.5134351Z return > _bootstrap._gcd_import(name[level:], package, level) > 2026-01-20T08:16:58.5134771Z File "<frozen importlib._bootstrap>", > line 1050, in _gcd_import > 2026-01-20T08:16:58.5135202Z File "<frozen importlib._bootstrap>", > line 1027, in _find_and_load > 2026-01-20T08:16:58.5135997Z File "<frozen importlib._bootstrap>", > line 1006, in _find_and_load_unlocked > 2026-01-20T08:16:58.5136465Z File "<frozen importlib._bootstrap>", > line 688, in _load_unlocked > 2026-01-20T08:16:58.5136900Z File "<frozen > importlib._bootstrap_external>", line 879, in exec_module > 2026-01-20T08:16:58.5137348Z File "<frozen > importlib._bootstrap_external>", line 1017, in get_code > 2026-01-20T08:16:58.5137795Z File "<frozen > importlib._bootstrap_external>", line 947, in source_to_code > 2026-01-20T08:16:58.5138259Z File "<frozen importlib._bootstrap>", > line 241, in _call_with_frames_removed > 2026-01-20T08:16:58.5138733Z File > "/home/runner/work/dpdk/dpdk/dts/api/testpmd/__init__.py", line 230 > 2026-01-20T08:16:58.5139388Z self.send_command(f"start tx_first > {burst_num if burst_num is not None else ""}") > 2026-01-20T08:16:58.5139819Z > ^ > 2026-01-20T08:16:58.5140133Z SyntaxError: f-string: expecting '}' > 2026-01-20T08:16:58.5140317Z > > ... > > https://github.com/DPDK/dpdk/actions/runs/21163780224/job/60863594703#step:19:9587
Ah, it seems Lukas has a fix. https://patchwork.dpdk.org/project/dpdk/patch/366f384e90cce0867652231494d8c873066b5fe6.1768518003.git.sis...@dyna-nic.com/ Can this series be reviewed? Thanks. -- David Marchand
