04/11/2020 09:04, Wisam Monther:
> Hi,
>
> You can add my ack:
>
> Acked-by: Wisam Jaddo mailto:wis...@nvidia.com>>
>
> Thomas,
> Do you have comments left here?
No comment, it looks very good.
Applied, thanks
I think Thomas can add it if he want to merge it in this version.
Moreover I think you need to keep the ack between versions and the person who
acked before should comment if newer version not ok with him, otherwise he is
ok with it. “Using git commit —amend”.
Thomas, correct me if I’m wrong, A
Hi,
Is what you are asking possible with a simple git commit --amend?
Thanks,
Georgios
On Wed, Nov 4, 2020 at 10:04 AM Wisam Monther wrote:
> Hi,
>
>
>
> You can add my ack:
>
> Acked-by: Wisam Jaddo
>
>
>
> Thomas,
>
> Do you have comments left here?
>
>
>
> BRs,
>
> Wisam Jaddo
>
>
>
> *Fro
Hi,
You can add my ack:
Acked-by: Wisam Jaddo mailto:wis...@nvidia.com>>
Thomas,
Do you have comments left here?
BRs,
Wisam Jaddo
From: Georgios Katsikas
Sent: Tuesday, November 3, 2020 1:26 PM
To: wis...@mellanox.com
Cc: dev@dpdk.org
Subject: Re: [PATCH] app/flow-perf: configurable rule batc
Hi,
Any news on this patch?
Is there anything else I could do?
Thanks,
Georgios
On Sun, Oct 11, 2020 at 1:03 PM Georgios Katsikas
wrote:
> Currently, flow-perf measures the performance of
> rule installation/deletion operations by breaking
> down the entire number of operations into windows
>
Hi,
Thanks for the feedback Thomas.
Please see my updated patch, I hope it better explains the scope of this
work.
Best regards,
Georgios
On Tue, Oct 6, 2020 at 1:25 AM Thomas Monjalon wrote:
> 05/10/2020 19:16, Georgios Katsikas:
> > Hi,
> >
> > What is the conclusion?
> > I haven't seen this
Currently, flow-perf measures the performance of
rule installation/deletion operations by breaking
down the entire number of operations into windows
of fixed size (i.e., 10 operations per window).
Then, flow-perf measures the total time per window
and computes an average time across all windows
05/10/2020 19:16, Georgios Katsikas:
> Hi,
>
> What is the conclusion?
> I haven't seen this patch being committed yet.
You missed updating the file doc/guides/tools/flow-perf.rst
> On Thu, Sep 24, 2020 at 3:01 PM Wisam Monther wrote:
> >From: Georgios Katsikas
> > >
> > >* One can now configu
Hi,
What is the conclusion?
I haven't seen this patch being committed yet.
Thanks,
Georigos
On Thu, Sep 24, 2020 at 3:01 PM Wisam Monther wrote:
> Hi,
>
> >-Original Message-
> >From: george@gmail.com On Behalf Of Georgios
> >Katsikas
> >Sent: Thursday, September 24, 2020 12:11 PM
Hi,
>-Original Message-
>From: george@gmail.com On Behalf Of Georgios
>Katsikas
>Sent: Thursday, September 24, 2020 12:11 PM
>To: wis...@mellanox.com
>Cc: dev@dpdk.org; Georgios Katsikas
>Subject: [PATCH] app/flow-perf: configurable rule batches
>
>* One can now configure the number
* One can now configure the number of rules per batch
* Refactored flow_count variable to rules_count as it is
related to the newly added rules_batch variable
* Added default values to usage function
Signed-off-by: Georgios Katsikas
---
app/test-flow-perf/main.c | 87 +++-
11 matches
Mail list logo
