Re: [dpdk-dev] [PATCH v8] app/testpmd: support multi-process

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

On 4/12/2021 5:37 PM, Ferruh Yigit wrote:
> On 3/30/2021 2:48 AM, Min Hu (Connor) wrote:
>> From: Lijun Ou <oulijun@huawei.com>
>>
>> This patch adds multi-process support for testpmd.
>> The test cmd example as follows:
>> the primary cmd:
>> ./dpdk-testpmd -a xxx --proc-type=auto -l 0-1 -- -i \
>> --rxq=4 --txq=4 --num-procs=2 --proc-id=0
>>
>> the secondary cmd:
>> ./dpdk-testpmd -a xxx --proc-type=auto -l 2-3 -- -i \
>> --rxq=4 --txq=4 --num-procs=2 --proc-id=1
>>
> 
> Hi Connor,
> 
> Please find a few minor comments below, since they are minor issues please feel 
> free to keep the existing acks in next version.
> 

Reminder of this patch, waiting for minor updates, it would be good to have it 
before -rc1, so that the change can be tested thoroughly.

Meanwhile I have concern/question on long term testing of the feature. It is 
possible to break the multi-process support unintentionally and not recognize 
it. How can we continually verify the feature?

Are you using DTS?
Does it make sense to add a testcase into dts to test testpmd multi-process 
support so that it is verified on each release?