Re: [dpdk-dev] [PATCH] app/flow-perf: configurable rule batches

[Wisam Monther <wisamm@nvidia.com>]

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, At least this how I understood the procedure here.

Moreover I noticed that you sending the versions as new patch all the time, but for history and comments tracking you should send with versions. V2-v3 ...etc with reply to older version of the patch.

BRs,
Wisam Jaddo

Get Outlook for iOS<https://aka.ms/o0ukef>
________________________________
From: Georgios Katsikas <katsikas.gp@gmail.com>
Sent: Wednesday, November 4, 2020 1:11:09 PM
To: Wisam Monther <wisamm@nvidia.com>
Cc: wisamm@mellanox.com <wisamm@mellanox.com>; NBU-Contact-Thomas Monjalon <thomas@monjalon.net>; dev@dpdk.org <dev@dpdk.org>
Subject: Re: [PATCH] app/flow-perf: configurable rule batches

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 <wisamm@nvidia.com<mailto:wisamm@nvidia.com>> wrote:

Hi,



You can add my ack:

Acked-by: Wisam Jaddo <wisamm@nvidia.com<mailto:wisamm@nvidia.com>>



Thomas,

Do you have comments left here?



BRs,

Wisam Jaddo



From: Georgios Katsikas <katsikas.gp@gmail.com<mailto:katsikas.gp@gmail.com>>
Sent: Tuesday, November 3, 2020 1:26 PM
To: wisamm@mellanox.com<mailto:wisamm@mellanox.com>
Cc: dev@dpdk.org<mailto:dev@dpdk.org>
Subject: Re: [PATCH] app/flow-perf: configurable rule batches



Hi,



Any news on this patch?

Is there anything else I could do?



Thanks,

Georgios