Re: [PATCH 2/2] app/testpmd: add testpmd based sleeping

2023-03-21 Thread Ferruh Yigit
On 3/17/2023 11:09 AM, Anthony Harivel wrote: > Ferruh Yigit, Mar 16, 2023 at 18:05: >> Hi Anthony, >> >> What is the motivation here? > > Hi Ferruh, > > AFAIK testpmd is the reference tool used for CI and tests whether it is > for functional or performance tests and I think it would be in > ever

Re: [PATCH 2/2] app/testpmd: add testpmd based sleeping

2023-03-17 Thread Robin Jarry
Anthony Harivel, Mar 16, 2023 at 16:14: > Sleep for an incremental amount of time if the fwd engine has processed > less than at least half a burst of packets (i.e 16pkts with default > setting) on a polling iteration of testpmd. > > Upon detecting the threshold of >= 16 pkts on an Rxq, reset the s

Re: [PATCH 2/2] app/testpmd: add testpmd based sleeping

2023-03-17 Thread Stephen Hemminger
On Fri, 17 Mar 2023 12:09:04 +0100 "Anthony Harivel" wrote: > Ferruh Yigit, Mar 16, 2023 at 18:05: > > Hi Anthony, > > > > What is the motivation here? > > Hi Ferruh, > > AFAIK testpmd is the reference tool used for CI and tests whether it is > for functional or performance tests and I think

Re: [PATCH 2/2] app/testpmd: add testpmd based sleeping

2023-03-17 Thread Anthony Harivel
Ferruh Yigit, Mar 16, 2023 at 18:05: > Hi Anthony, > > What is the motivation here? Hi Ferruh, AFAIK testpmd is the reference tool used for CI and tests whether it is for functional or performance tests and I think it would be in everyone's interest to consume less CPU during them. Moreover, all

Re: [PATCH 2/2] app/testpmd: add testpmd based sleeping

2023-03-16 Thread Ferruh Yigit
On 3/16/2023 3:14 PM, Anthony Harivel wrote: > Sleep for an incremental amount of time if the fwd engine has processed > less than at least half a burst of packets (i.e 16pkts with default > setting) on a polling iteration of testpmd. > > Upon detecting the threshold of >= 16 pkts on an Rxq, reset

Re: [PATCH 2/2] app/testpmd: add testpmd based sleeping

2023-03-16 Thread Stephen Hemminger
On Thu, 16 Mar 2023 16:14:38 +0100 Anthony Harivel wrote: > Sleep for an incremental amount of time if the fwd engine has processed > less than at least half a burst of packets (i.e 16pkts with default > setting) on a polling iteration of testpmd. > > Upon detecting the threshold of >= 16 pkts o

[PATCH 2/2] app/testpmd: add testpmd based sleeping

2023-03-16 Thread Anthony Harivel
Sleep for an incremental amount of time if the fwd engine has processed less than at least half a burst of packets (i.e 16pkts with default setting) on a polling iteration of testpmd. Upon detecting the threshold of >= 16 pkts on an Rxq, reset the sleep time to zero (i.e. no sleep). Sleep time wi