<div dir="ltr"><div><br></div><div><br></div><div><br></div><div class="gmail_quote gmail_quote_container"><div dir="ltr" class="gmail_attr">On Thu, Oct 23, 2025 at 8:32 AM 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"><br>
     def _set_all_queues_mbuf_fast_free(<br>
-        self, testpmd: TestPmd, port_id: int, on: bool, num_queues: int, /, verify: bool = True<br>
+        self,<br>
+        testpmd: TestPmd,<br>
+        port_id: int,<br>
+        on: bool,<br>
+        num_queues: int,<br>
     ) -> None:<br>
         for i in range(num_queues):<br>
             testpmd.set_queue_mbuf_fast_free(on, port_id, i, verify)<br></blockquote><div><br></div><div>on and port_id are swapped from where they should be in the function call above. I swapped these and reran the test and it's now working on Intel E810. Normally I would just fixup and merge but since you have to submit a new version anyways because of the issue noted in the 1/3 patch you can send the new version with this correction.</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">
@@ -61,25 +64,24 @@ def test_mbuf_fast_free_configuration_per_port(self) -> None:<br>
             * Mbuf_fast_free can be configured on per port.<br>
         """<br>
         with TestPmd() as testpmd:<br>
-            to_verify = True<br>
             port_id = 0<br>
             testpmd.start_all_ports()<br>
<br></blockquote><div><br></div></div></div>