> -----Original Message----- > From: Jie Hai <haij...@huawei.com> > Sent: Tuesday, July 4, 2023 11:45 AM > To: Aman Singh <aman.deep.si...@intel.com>; Yuying Zhang > <yuying.zh...@intel.com>; Ferruh Yigit <ferruh.yi...@amd.com>; Shiyang He > <shiyangx...@intel.com> > Cc: dev@dpdk.org; liudongdo...@huawei.com; Ali Alnubani > <alia...@nvidia.com> > 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: sta...@dpdk.org > > Signed-off-by: Jie Hai <haij...@huawei.com> > --- > app/test-pmd/testpmd.c | 9 ++++++--- > 1 file changed, 6 insertions(+), 3 deletions(-) > > --
Thanks Jie. Tested-by: Ali Alnubani <alia...@nvidia.com>