> -----Original Message----- > From: Jie Hai <[email protected]> > Sent: Tuesday, July 4, 2023 11:45 AM > To: Aman Singh <[email protected]>; Yuying Zhang > <[email protected]>; Ferruh Yigit <[email protected]>; Shiyang He > <[email protected]> > Cc: [email protected]; [email protected]; Ali Alnubani > <[email protected]> > Subject: [PATCH v2] app/testpmd: fix invalid queue ID when start port > > Function update_queue_state updates queue state of all queues > of all ports, using the queue num nb_rxq|nb_txq stored locally > by testpmd. An error on the invalid queue ID occurs if we run > testpmd with two ports and detach-attach one of them and start > the other port first. This is because the attached port has not > been configured and has no queues, which differs from nb_rxq|nb_txq. > The similar error happens in multi-process senoris if secondary > process attaches a port and starts it. > > This patch updates queue state of the specified port, which has > been configured by primary process. As the secondary process > cannot configure the ports, make sure that the secondary process > starts the port only after the primary process has done so. > > Fixes: 141a520b35f7 ("app/testpmd: fix primary process not polling all > queues") > Fixes: 5028f207a4fa ("app/testpmd: fix secondary process packet > forwarding") > Cc: [email protected] > > Signed-off-by: Jie Hai <[email protected]> > --- > app/test-pmd/testpmd.c | 9 ++++++--- > 1 file changed, 6 insertions(+), 3 deletions(-) > > --
Thanks Jie. Tested-by: Ali Alnubani <[email protected]>
