On 8/11/2022 7:57 AM, Guo, Junfeng wrote:


-----Original Message-----
From: Singh, Aman Deep <[email protected]>
Sent: Friday, June 17, 2022 00:40
To: Guo, Junfeng <[email protected]>; Zhang, Qi Z
<[email protected]>; Wu, Jingjing <[email protected]>; Xing,
Beilei <[email protected]>
Cc: [email protected]; Wang, Xiao W <[email protected]>
Subject: Re: [PATCH v2] app/testpmd: add throughput stats for
forward streams

Hi Junfeng,

Thanks for the patch.


On 6/6/2022 3:09 PM, Junfeng Guo wrote:
1. add throughput statistics (in pps) for forward streams.
2. display the forward statistics for every forward stream.

v2:
add parameter descriptions and fix commit title.

Signed-off-by: Xiao Wang <[email protected]>
Signed-off-by: Junfeng Guo <[email protected]>
---

We do have per port Rx/Tx pps in display stats.
For per forward stream we can enable "--record-burst-stats".
With it we can get per fwd stream Rx/Tx pps.

Please check if this patch is adding any additional functionality.

Sorry for the late reply.
Seems that "--record-burst-stats" could only show the stats of Rx/Tx bursts.


Hi Junfeng,

What is the impact of this change to the performance?

And what do you think enabling it with an command,
like existing "set record-*" ones?

Actually this patch can enable the throughput stats, which has not been 
supported in current testpmd.
So we hope this functionality could be added.
Thanks!


<snip>


Reply via email to