Re: [PATCH] dts: add l2fwd test suite

2024-09-09 Thread Nicholas Pratte
Just a few clarifying questions below. Otherwise, (Test cases was run on Connect_x5 and Broadcom P225p devices). Reviewed-by: Nicholas Pratte Tested-by: Nicholas Pratte > + > +with TestPmdShell( > +self.sut_node, > +lcore_filter_specifier=LogicalCoreCount(cores

Re: [PATCH] dts: add l2fwd test suite

2024-09-09 Thread Patrick Robb
Reviewed-by: Patrick Robb Tested-by: Patrick Robb Ran on a Broadcom 57414 2x25G NIC on an Intel x86 server. I'll follow up with a review on the pktgen and testpmd changes series as obviously we want to move that to next-dts. Thanks! On Tue, Aug 6, 2024 at 8:53 AM Luca Vizzarro wrote: > +d

Re: [PATCH] dts: add l2fwd test suite

2024-09-09 Thread Luca Vizzarro
On 09/08/2024 16:27, Jeremy Spewock wrote: Signed-off-by: Luca Vizzarro Reviewed-by: Paul Szczepanek --- Depends-on: series-32714 ("dts: add pktgen and testpmd changes") Out of my own curiosity, are depends on supposed to be outside of the commit body? I don't think it really matters for auto

Re: [PATCH] dts: add l2fwd test suite

2024-09-04 Thread Dean Marx
On Tue, Aug 6, 2024 at 8:53 AM Luca Vizzarro wrote: > Add a basic L2 forwarding test suite which tests the correct > functionality of the forwarding facility built-in in the DPDK. > > The tests are performed with different queues numbers per port. > > Bugzilla ID: 1481 > > Signed-off-by: Luca Viz

Re: [PATCH] dts: add l2fwd test suite

2024-08-09 Thread Jeremy Spewock
Hey Luca, Thanks for the patch! Just a few pretty minor comments below. On Tue, Aug 6, 2024 at 8:53 AM Luca Vizzarro wrote: > > Add a basic L2 forwarding test suite which tests the correct > functionality of the forwarding facility built-in in the DPDK. > > The tests are performed with different

[PATCH] dts: add l2fwd test suite

2024-08-06 Thread Luca Vizzarro
Add a basic L2 forwarding test suite which tests the correct functionality of the forwarding facility built-in in the DPDK. The tests are performed with different queues numbers per port. Bugzilla ID: 1481 Signed-off-by: Luca Vizzarro Reviewed-by: Paul Szczepanek --- Depends-on: series-32714 (