Re: [PATCH v4 3/3] dts: add virtio forwarding test suite

[Dean Marx <dmarx@iol.unh.edu>]

On Fri, Nov 7, 2025 at 3:49 PM Patrick Robb <probb@iol.unh.edu> wrote:
>
>
>
> Is it the case that if I already have tap0 created on my SUT machine, this testsuite will create a tap1 and then the testsuite will fail to run properly on the line above (because tap1 won't be brought up)? If there is no way of getting around this limitation, we should add a new capability for "tap interface names available" or something and skip if tap0 is found in "ip a" or a similar solution.

Yes, I've added a workaround which sends a delete tap0 command prior
to the testpmd session starting. If the tap0 interface doesn't exist
it fails silently and continues with the rest of the test case.

>>
>
>
> Otherwise, change the pvp requirement to ONE_LINK, set portlist to 0, 2, 1 (2 is the vdev) when 2 link topology, and 0, 1 when 1 link, and set forwarding to chained mode so that the packet forwarding works as expected regardless of whether we have 1 or 2 physical interfaces the testpmd instance is using.

Done

>
> Thanks.
>
> Reviewed-by: Patrick Robb <probb@iol.unh.edu>

Thanks for the review!