On Thu, Oct 23, 2025 at 8:32 AM Andrew Bailey <[email protected]> wrote:
> > def _set_all_queues_mbuf_fast_free( > - self, testpmd: TestPmd, port_id: int, on: bool, num_queues: int, > /, verify: bool = True > + self, > + testpmd: TestPmd, > + port_id: int, > + on: bool, > + num_queues: int, > ) -> None: > for i in range(num_queues): > testpmd.set_queue_mbuf_fast_free(on, port_id, i, verify) > 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. > @@ -61,25 +64,24 @@ def test_mbuf_fast_free_configuration_per_port(self) > -> None: > * Mbuf_fast_free can be configured on per port. > """ > with TestPmd() as testpmd: > - to_verify = True > port_id = 0 > testpmd.start_all_ports() > >
