> -----Original Message----- > From: Ferruh Yigit <ferruh.yi...@amd.com> > Sent: Wednesday, July 5, 2023 5:32 PM > To: Aman Singh <aman.deep.si...@intel.com>; Yuying Zhang > <yuying.zh...@intel.com>; Anatoly Burakov <anatoly.bura...@intel.com>; > Jie Hai <haij...@huawei.com> > Cc: dev@dpdk.org; NBU-Contact-Thomas Monjalon (EXTERNAL) > <tho...@monjalon.net>; David Marchand <david.march...@redhat.com>; > sta...@dpdk.org; songx.ji...@intel.com; qiming.y...@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: sta...@dpdk.org > > Signed-off-by: Ferruh Yigit <ferruh.yi...@amd.com> > ---
Thanks, Ferruh. Tested on top of latest main (df60837ccd65). Tested-by: Ali Alnubani <alia...@nvidia.com>