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()