Skip to contentSkip to conversation

app/testpmd: support multi-process

PATCH Singh, Aman Deep 3 participants 4 messages Apr 28, 2021 – Apr 29, 2021

lore
Share & tools
lore

Singh, Aman Deep

author Acked-by Xiaoyun Li Acked-by Ajit Khaparde Reviewed-by Ferruh Yigit Acked-by Aman Deep Singh Tested-by Aman Deep Singh
#0 lore raw

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

<snip>

#1 lore raw

Thanks Aman.

Hi, Thomas, Ferruh, any other comments about this patch?

Quoted 32 lines · 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) <[email protected] <mailto:[email protected]>>
Signed-off-by Lijun Ou <[email protected] <mailto:[email protected]>>
Acked-by Xiaoyun Li <[email protected] <mailto:[email protected]>>
Acked-by Ajit Khaparde <[email protected] <mailto:[email protected]>>
Reviewed-by Ferruh Yigit <[email protected] <mailto:[email protected]>>

<snip>

Acked-by Aman Deep Singh <[email protected] <mailto:[email protected]>>
Tested-by Aman Deep Singh <[email protected] <mailto:[email protected]>>
#2 lore raw
Quoted 4 lines · Min Hu (Connor)

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

Quoted 37 lines · Aman Deep
Quoted 36 lines

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) <[email protected] <mailto:[email protected]>>
Signed-off-by Lijun Ou <[email protected] <mailto:[email protected]>>
Acked-by Xiaoyun Li <[email protected] <mailto:[email protected]>>

<mailto:[email protected]>>

<mailto:[email protected]>>

<snip>

<mailto:[email protected]>>

<mailto:[email protected]>>

#3 lore raw

Got it, thanks Ferruh.

Quoted 59 lines · Ferruh Yigit
Quoted 4 lines · Min Hu (Connor)

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

Quoted 29 lines · Aman Deep
Quoted 28 lines

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) <[email protected] <mailto:[email protected]>>
Signed-off-by Lijun Ou <[email protected] <mailto:[email protected]>>
Acked-by Xiaoyun Li <[email protected] <mailto:[email protected]>>

<mailto:[email protected]>>

<mailto:[email protected]>>

Quoted 8 lines
Quoted 8 lines

<snip>

<mailto:[email protected]>>

<mailto:[email protected]>>

.