Hi Wisam,

-----Original Message-----
From: Wisam Monther <redacted>
Sent: Tuesday, October 12, 2021 3:53 PM
To: Sean Zhang (Networking SW) <redacted>; NBU-Contact-
Thomas Monjalon [off-list ref]
Cc: dev@dpdk.org; Raslan Darawsheh <redacted>
Subject: RE: [PATCH] app/flow-perf: add destination ports parameter

Hi,

quoted

-----Original Message-----
From: Sean Zhang (Networking SW) <redacted>
Sent: Tuesday, October 12, 2021 6:43 AM
To: NBU-Contact-Thomas Monjalon <redacted>; Wisam

Monther

quoted

[off-list ref]
Cc: dev@dpdk.org; Raslan Darawsheh <redacted>
Subject: [PATCH] app/flow-perf: add destination ports parameter

Add destination ports(dst-ports) parameter for port-id action, the
parameter is only valid for port-id action. The parameter is not Must,
and the value is 1 by default as before if not provided.

For example:

$ dpdk-test-flow-perf -w 08:00.0,representor=[0,1] -- --transfer \

quoted

--ingress --transfer --ether --portmask=0x2 --vxlan-encap \
--port-id
--dst-ports=0

This command means the rule created on representor 0 with port 0 as
destination, since the portmask is 0x2 and dst-ports is 0:

$ dpdk-test-flow-perf -w 08:00.0,representor=[0,1] \

quoted

-w 08:00.1,representor=[0,1]-- --transfer --ingress --transfer \
--ether --portmask=0x12 --vxlan-encap --port-id --dst-ports=0,3

This command means the rules created on both representor 0 of PF 0 and
PF 1, the destination port for the first represontor is PF 0, and the
destination port for the other one it PF 1.

Signed-off-by: Sean Zhang <redacted>

Do you think it's applicable to use the same option?
I mean, usage:
--port-id: means default.
--port-id=N,M: use the parsing ports?

Yes, since the argument can be optional, your suggestion is applicable. So do you suggest to change to --port-id=N,M?

BRs,
Wisam Jaddo