On 4/29/2021 11:57 AM, Min Hu (Connor) wrote:
> Thanks Aman.
> 
> Hi, Thomas, Ferruh, any other comments about this patch?
> 

Hi Connor,

There is no outstanding comments, also the patch has review/test tags in place.

But the multi-process support may affect many operations, and assuming it is not
critical for this release, to prevent any unexpected side affects I am for
getting this patch early next release instead of this release.

This gives more time in next release to address any unexpected issue.

Thanks,
ferruh

> 在 2021/4/28 20:10, Singh, Aman Deep 写道:
>> 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
>>
>> Signed-off-by: Min Hu (Connor) <humi...@huawei.com 
>> <mailto:humi...@huawei.com>>
>>
>> Signed-off-by: Lijun Ou <ouli...@huawei.com <mailto:ouli...@huawei.com>>
>>
>> Acked-by: Xiaoyun Li <xiaoyun...@intel.com <mailto:xiaoyun...@intel.com>>
>>
>> Acked-by: Ajit Khaparde <ajit.khapa...@broadcom.com
>> <mailto:ajit.khapa...@broadcom.com>>
>>
>> Reviewed-by: Ferruh Yigit <ferruh.yi...@intel.com
>> <mailto:ferruh.yi...@intel.com>>
>>
>> <snip>
>>
>> Acked-by: Aman Deep Singh <aman.deep.si...@intel.com
>> <mailto:aman.deep.si...@intel.com>>
>>
>> Tested-by: Aman Deep Singh <aman.deep.si...@intel.com
>> <mailto:aman.deep.si...@intel.com>>
>>

Reply via email to