On Thu, Oct 23, 2025 at 8:32 AM Andrew Bailey <abailey@iol.unh.edu> 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()
>
>