[PATCH v3 2/4] dts: remove unnecessary testpmd verification

Dean Marx dmarx at iol.unh.edu
Wed Jul 2 18:23:29 CEST 2025


Remove unnecessary verification step in the start method
of TestPmdShell which checks the link status.

Signed-off-by: Dean Marx <dmarx at iol.unh.edu>
---
 dts/framework/remote_session/testpmd_shell.py | 7 -------
 1 file changed, 7 deletions(-)

diff --git a/dts/framework/remote_session/testpmd_shell.py b/dts/framework/remote_session/testpmd_shell.py
index 299887dd80..6d75f89969 100644
--- a/dts/framework/remote_session/testpmd_shell.py
+++ b/dts/framework/remote_session/testpmd_shell.py
@@ -1578,13 +1578,6 @@ 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.")
 
-            number_of_ports = len(self._app_params.allowed_ports or [])
-            for port_id in range(number_of_ports):
-                if not self.wait_link_status_up(port_id):
-                    raise InteractiveCommandExecutionError(
-                        "Not all ports came up after starting packet forwarding in testpmd."
-                    )
-
     def stop(self, verify: bool = True) -> str:
         """Stop packet forwarding.
 
-- 
2.49.0



More information about the dev mailing list