<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote gmail_quote_container"><div dir="ltr" class="gmail_attr">On Wed, Sep 24, 2025 at 12:47 PM Andrew Bailey <<a href="mailto:abailey@iol.unh.edu">abailey@iol.unh.edu</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Currently, the RX/TX offload test in old DTS expects the TX ports to be<br>
initially configured to use mbuf fast free. This is no longer the case<br>
and must be updated to assume mbuf fast free is not initially utilized<br>
by capable NICs. Add updated test suite to test mbuf fast free<br>
configuration.<br>
<br>
Signed-off-by: Andrew Bailey <<a href="mailto:abailey@iol.unh.edu" target="_blank">abailey@iol.unh.edu</a>><br>
---<br>
doc/api/dts/tests.TestSuite_rx_tx_offload.rst | 8 ++<br>
dts/tests/TestSuite_rx_tx_offload.py | 128 ++++++++++++++++++<br>
2 files changed, 136 insertions(+)<br>
create mode 100644 doc/api/dts/tests.TestSuite_rx_tx_offload.rst<br>
create mode 100644 dts/tests/TestSuite_rx_tx_offload.py<br>
<br>
diff --git a/doc/api/dts/tests.TestSuite_rx_tx_offload.rst b/doc/api/dts/tests.TestSuite_rx_tx_offload.rst<br>
new file mode 100644<br>
index 0000000000..225267a45f<br>
--- /dev/null<br>
+++ b/doc/api/dts/tests.TestSuite_rx_tx_offload.rst<br>
@@ -0,0 +1,8 @@<br>
+.. SPDX-License-Identifier: BSD-3-Clause<br>
+<br>
+rx_tx_offload Test Suite<br>
+=========================<br></blockquote><div><br></div><div>Did you try to build the docs after adding this file? I might be wrong but I think the syntax might be flawed. If you don't remember the docs build process is:</div><div><br></div><div>meson setup build</div><div>ninja -C build doc</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
+<br>
+.. automodule:: tests.TestSuite_rx_tx_offload<br>
+ :members:<br>
+ :show-inheritance:<br>
\ No newline at end of file<br>
diff --git a/dts/tests/TestSuite_rx_tx_offload.py b/dts/tests/TestSuite_rx_tx_offload.py<br>
new file mode 100644<br>
index 0000000000..6ae98068e4<br>
--- /dev/null<br>
+++ b/dts/tests/TestSuite_rx_tx_offload.py<br>
@@ -0,0 +1,128 @@<br>
+# SPDX-License-Identifier: BSD-3-Clause<br>
+# Copyright(c) 2025 University of New Hampshire<br>
+<br>
+"""RX TX offload test suite.<br>
+<br>
+Test the testpmd feature of configuring RX and TX offloads.<br>
+"""<br>
+<br>
+from framework.remote_session.testpmd_shell import (<br>
+ OffloadConfiguration,<br>
+ RxTxLiteralSwitch,<br>
+ TestPmdShell,<br>
+)<br></blockquote><div><br></div><div>your branch needs to be rebased off of dpdk-next-dts/for-main. We have merged Paul's DTS API patch, which means your imports are no longer valid and will need to be updated.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
+from framework.test_suite import TestSuite, func_test<br>
+from framework.testbed_model.capability import NicCapability, requires<br>
+<br>
+<br>
+class TestRxTxOffload(TestSuite):<br>
+ """RX/TX offload test suite."""<br>
+<br>
+ def _check_config(<br>
+ self,<br>
+ testpmd: TestPmdShell,<br>
+ port_offload: str | None,<br>
+ rxtx: RxTxLiteralSwitch,<br>
+ port_id: int,<br>
+ /,<br>
+ queue_offload: list[str | None] | None = None,<br>
+ verify: bool = True,<br>
+ ) -> bool:<br>
+ config: OffloadConfiguration = testpmd.get_offload_config(rxtx, port_id, verify)<br>
+ if <a href="http://config.port.name" rel="noreferrer" target="_blank">config.port.name</a> != port_offload:<br>
+ return False<br>
+<br>
+ if queue_offload:<br>
+ for i, q in enumerate(config.queues):<br>
+ if <a href="http://q.name" rel="noreferrer" target="_blank">q.name</a> != queue_offload[i]:<br>
+ return False<br>
+ return True<br>
+<br>
+ def _set_all_queues_mbuf_fast_free(<br>
+ self, testpmd: TestPmdShell, on: bool, port_id: int, num_queues: int, /, verify: bool = True<br>
+ ) -> None:<br>
+ for i in range(num_queues):<br>
+ testpmd.set_queue_mbuf_fast_free(on, port_id, i, verify)<br>
+<br>
+ @requires(NicCapability.PORT_TX_OFFLOAD_MBUF_FAST_FREE)<br>
+ @func_test<br>
+ def test_mbuf_fast_free_configuration_per_port(self) -> None:<br>
+ """Ensure mbuf_fast_free can be configured with testpmd per port.<br>
+<br>
+ Steps:<br>
+ * Start up testpmd shell.<br>
+ * Toggle mbuf_fast_free on per port.<br>
+ * Toggle mbuf_fast_free off per port.<br>
+<br>
+ Verify:<br>
+ * Mbuf_fast_free starts disabled.<br>
+ * Mbuf_fast_free can be configured on per port.<br>
+ * Mbuf_fast_free can be configured off per port.<br>
+ """<br>
+ with TestPmdShell() as testpmd:<br>
+ verify = True<br>
+ port_id = 0<br>
+ testpmd.start_all_ports()<br>
+<br>
+ # Ensure MBUF_FAST_FREE is disabled by default and verify<br>
+ self.verify(<br>
+ self._check_config(testpmd, None, "tx", port_id, verify=verify),<br>
+ "MBUF_FAST_FREE enabled on port start.",<br>
+ )<br>
+ # Enable MBUF_FAST_FREE per port and verify<br>
+ testpmd.set_port_mbuf_fast_free(True, port_id, verify)<br>
+ self.verify(<br>
+ self._check_config(<br>
+ testpmd, "TX_OFFLOAD_MBUF_FAST_FREE", "tx", port_id, verify=verify<br>
+ ),<br>
+ "Failed to enable MBUF_FAST_FREE on port.",<br>
+ )<br>
+ # Disable MBUF_FAST_FREE per port and verify<br>
+ testpmd.set_port_mbuf_fast_free(False, port_id, verify)<br>
+ self.verify(<br>
+ self._check_config(testpmd, None, "tx", port_id, verify=verify),<br>
+ "Failed to disable MBUF_FAST_FREE on port.",<br>
+ )<br>
+<br>
+ @requires(NicCapability.QUEUE_TX_OFFLOAD_MBUF_FAST_FREE)<br>
+ @func_test<br>
+ def test_mbuf_fast_free_configuration_per_queue(self) -> None:<br>
+ """Ensure mbuf_fast_free can be configured with testpmd.<br>
+<br>
+ Steps:<br>
+ * Start up testpmd shell.<br>
+ * Toggle mbuf_fast_free on per queue.<br>
+ * Toggle mbuf_fast_free off per queue.<br>
+<br>
+ Verify:<br>
+ * Mbuf_fast_free starts disabled.<br>
+ * Mbuf_fast_free can be configured on per queue.<br>
+ * Mbuf_fast_free can be configured off per queue.<br>
+ """<br>
+ with TestPmdShell() as testpmd:<br>
+ verify = True<br>
+ port_id = 0<br>
+ num_queues = 4<br>
+ queue_off: list[str | None] | None = [None] * num_queues<br>
+ queue_on: list[str | None] | None = ["TX_OFFLOAD_MBUF_FAST_FREE"] * num_queues<br>
+<br>
+ testpmd.set_ports_queues(num_queues)<br>
+ testpmd.start_all_ports()<br>
+<br>
+ # Ensure mbuf_fast_free is disabled by default on port and queues<br>
+ self.verify(<br>
+ self._check_config(testpmd, None, "tx", port_id, queue_off, verify=verify),<br>
+ "MBUF_FAST_FREE enabled on queue start.",<br>
+ )<br>
+ # Enable mbuf_fast_free per queue and verify<br>
+ self._set_all_queues_mbuf_fast_free(testpmd, True, port_id, num_queues, verify)<br>
+ self.verify(<br>
+ self._check_config(testpmd, None, "tx", port_id, queue_on, verify),<br>
+ "Failed to enable MBUF_FAST_FREE on all queues.",<br>
+ )<br>
+ # Disable mbuf_fast_free per queue and verify<br>
+ self._set_all_queues_mbuf_fast_free(testpmd, False, port_id, num_queues, verify)<br>
+ self.verify(<br>
+ self._check_config(testpmd, None, "tx", port_id, queue_off, verify),<br>
+ "Failed to disable MBUF_FAST_FREE on all queues.",<br>
+ )<br>
-- <br>
2.50.1<br>
<br></blockquote><div><br></div><div>Let's look at the testcases themselves together tomorrow or Friday. Thanks Andrew.</div><div> </div></div></div>