Re: [PATCH v2] app/testpmd: fix invalid queue ID when start port

[Ferruh Yigit <ferruh.yigit@amd.com>]

On 7/5/2023 4:16 AM, lihuisong (C) wrote:
> 
> 在 2023/7/4 18:59, Ferruh Yigit 写道:
>> On 7/4/2023 9:45 AM, Jie Hai wrote:
>>> 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.
> Now look good to me.
> Acked-by: Huisong Li <lihuisong@huawei.com>
>>>
>>> Fixes: 141a520b35f7 ("app/testpmd: fix primary process not polling
>>> all queues")
>>> Fixes: 5028f207a4fa ("app/testpmd: fix secondary process packet
>>> forwarding")
>>> Cc: stable@dpdk.org
>>>
>>> Signed-off-by: Jie Hai <haijie1@huawei.com>
>>>
>> The problem description and solution looks reasonable to me, but Intel
>> testing still reporting the issue.
>>
>> There is a chance that the issue Intel side observing is different,
>> waiting for more information from Intel test team.
>>
>> .

Hi Song,

As far as I understand this patch works with an update from ixgbevf
driver, can you please confirm?
And can we have the ixgbevf fix soon, to not block the -rc3?

Thanks,
ferruh