> -----Original Message----- > From: Ferruh Yigit <[email protected]> > Sent: Wednesday, July 5, 2023 5:32 PM > To: Aman Singh <[email protected]>; Yuying Zhang > <[email protected]>; Anatoly Burakov <[email protected]>; > Jie Hai <[email protected]> > Cc: [email protected]; NBU-Contact-Thomas Monjalon (EXTERNAL) > <[email protected]>; David Marchand <[email protected]>; > [email protected]; [email protected]; [email protected] > 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: [email protected] > > Signed-off-by: Ferruh Yigit <[email protected]> > ---
Thanks, Ferruh. Tested on top of latest main (df60837ccd65). Tested-by: Ali Alnubani <[email protected]>
