[PATCH] app/testpmd: revert primary process polling all queues fix

Ali Alnubani alialnu at nvidia.com
Thu Jul 6 12:01:10 CEST 2023


> -----Original Message-----
> From: Ferruh Yigit <ferruh.yigit at amd.com>
> Sent: Wednesday, July 5, 2023 5:32 PM
> To: Aman Singh <aman.deep.singh at intel.com>; Yuying Zhang
> <yuying.zhang at intel.com>; Anatoly Burakov <anatoly.burakov at intel.com>;
> Jie Hai <haijie1 at huawei.com>
> Cc: dev at dpdk.org; NBU-Contact-Thomas Monjalon (EXTERNAL)
> <thomas at monjalon.net>; David Marchand <david.marchand at redhat.com>;
> stable at dpdk.org; songx.jiale at intel.com; qiming.yang at intel.com
> Subject: [PATCH] app/testpmd: revert primary process polling all queues fix
> 
> For some drivers [1], testpmd forwarding is broken with commit [2].
> 
> This is because with [2] testpmd gets queue state from ethdev and
> forwarding is done only on queues in started state, but some drivers
> don't update queue status properly, and this breaks forwarding for those
> drivers.
> 
> Drivers should be fixed but more time is required to verify drivers
> again, instead reverting [2] for now to not break drivers.
> Target is to merge [2] back at the beginning of next release cycle and
> fix drivers accordingly.
> 
> [1]
> Bugzilla ID: 1259
> 
> [2]
> Fixes: 141a520b35f7 ("app/testpmd: fix primary process not polling all
> queues")
> Cc: stable at dpdk.org
> 
> Signed-off-by: Ferruh Yigit <ferruh.yigit at amd.com>
> ---

Thanks, Ferruh. Tested on top of latest main (df60837ccd65).

Tested-by: Ali Alnubani <alialnu at nvidia.com>


More information about the dev mailing list