Re: [dpdk-dev] [PATCH v16] app/testpmd: support multi-process
From: Ferruh Yigit <hidden>
Date: 2021-08-24 12:18:45
On 8/16/2021 7:12 PM, Singh, Aman Deep wrote:
On 8/2/2021 1:33 PM, Thomas Monjalon wrote:quoted
02/08/2021 03:51, Min Hu (Connor):quoted
在 2021/7/26 15:28, Min Hu (Connor) 写道:quoted
在 2021/7/26 14:30, Thomas Monjalon 写道:quoted
26/07/2021 02:26, Min Hu (Connor):quoted
在 2021/7/24 19:45, Thomas Monjalon 写道:quoted
10/07/2021 05:50, Min Hu (Connor):quoted
This patch adds multi-process support for testpmd. For example the following commands run two testpmd processes: * the primary process: ./dpdk-testpmd --proc-type=auto -l 0-1 -- -i \ --rxq=4 --txq=4 --num-procs=2 --proc-id=0 * the secondary process: ./dpdk-testpmd --proc-type=auto -l 2-3 -- -i \ --rxq=4 --txq=4 --num-procs=2 --proc-id=1 Signed-off-by: Min Hu (Connor) <redacted> Signed-off-by: Lijun Ou <redacted> Signed-off-by: Andrew Rybchenko <redacted> Acked-by: Xiaoyun Li <redacted> Acked-by: Ajit Khaparde <ajit.khaparde@broadcom.com> Reviewed-by: Ferruh Yigit <redacted> --- V16: * revert unrelated changes. * add some restrictions in doc.I didn't see clear agreement to integrate this feature in DPDK 21.08. BTW, the testpmd maintainer was not Cc'ed.Hi, Thomas, This patch has two "acked" and one "reviewed-by". And Xiaoyun Li is the maintainer, I has already Cc'ed to him/her, please check it out, thanks.Andrew's comment from v15: " IMHO, as I state above, current state is insufficient to consider is a start point to be applied. "Hi, Andrew, Thomos, What is the least thing should be done for this patch to meet the state sufficient to be applied?Hi, all, How about this patch, could it be applied?I was not confident enough to apply it in 21.08. When others will be back, we'll discuss it again for 21.11.For 21.11 release. Acked-by: Aman Deep Singh <redacted>
Hi Connor, Can you please rebase it to latest dpdk (v21.11-rc0) so we can proceed with it? I am for getting the patch as early as possible in the release cycle, specially before it is getting busy for -rc1. Thanks, ferruh