[dpdk-dev] [PATCH] test/ring: code rework to reduce compilation time

2020-04-29 Thread Konstantin Ananyev
Rework test code to reduce code complexity for the compiler and bring down compilation time and memory consumption. Signed-off-by: Konstantin Ananyev --- app/test/test_ring.c | 373 +-- 1 file changed, 249 insertions(+), 124 deletions(-) diff --git a/app/

Re: [dpdk-dev] [PATCH] test/ring: code rework to reduce compilation time

2020-07-02 Thread Honnappa Nagarahalli
> > Rework test code to reduce code complexity for the compiler and bring down > compilation time and memory consumption. > > Signed-off-by: Konstantin Ananyev Thomas has a nit [1]. Otherwise, Acked-by: Honnappa Nagarahalli [1] http://mails.dpdk.org/archives/dev/2020-May/166325.html > ---

Re: [dpdk-dev] [PATCH] test/ring: code rework to reduce compilation time

2020-07-03 Thread David Marchand
On Thu, Jul 2, 2020 at 10:47 PM Honnappa Nagarahalli wrote: > > Rework test code to reduce code complexity for the compiler and bring down > > compilation time and memory consumption. > > > > Signed-off-by: Konstantin Ananyev > Acked-by: Honnappa Nagarahalli > Updated commitlog with Konstantin

Re: [dpdk-dev] [PATCH] test/ring: code rework to reduce compilation time

2020-04-29 Thread Honnappa Nagarahalli
Hi Konstantin, Thanks for the patch. I think we need to co-ordinate better on the rte_ring related work to avoid duplicating the efforts. I also have been working on a patch myself and it is undergoing internal review. I will review this patch soon. Thanks, Honnappa > -Original Mes

Re: [dpdk-dev] [PATCH] test/ring: code rework to reduce compilation time

2020-04-29 Thread Aaron Conole
Konstantin Ananyev writes: > Rework test code to reduce code complexity for the compiler > and bring down compilation time and memory consumption. > > Signed-off-by: Konstantin Ananyev > --- I'll wait for Honnappa's review, but this looks good to me. Consider that I will ACK by the end of the

Re: [dpdk-dev] [PATCH] test/ring: code rework to reduce compilation time

2020-04-29 Thread Honnappa Nagarahalli
Hi Konstantin, I like the way the tests are organized and it looks good. I am just wondering about the way it is being tested here. The intent to write the test cases the way they are currently is to mimic how the APIs would be used mostly. IMO, the APIs would be used with a constant val

Re: [dpdk-dev] [PATCH] test/ring: code rework to reduce compilation time

2020-04-30 Thread Ananyev, Konstantin
Hi Honnappa, > Hi Konstantin, > I like the way the tests are organized and it looks good. > > I am just wondering about the way it is being tested here. The intent to > write the test cases the way they are currently is to mimic how the > APIs would be used mostly. IMO, the APIs would b

Re: [dpdk-dev] [PATCH] test/ring: code rework to reduce compilation time

2020-05-01 Thread Honnappa Nagarahalli
> > > Hi Honnappa, > > > Hi Konstantin, > > I like the way the tests are organized and it looks good. > > > > I am just wondering about the way it is being tested here. The intent > > to write the test cases the way they are currently is to mimic how the > > APIs would be used mostly. IMO,

Re: [dpdk-dev] [PATCH] test/ring: code rework to reduce compilation time

2020-05-04 Thread Thomas Monjalon
30/04/2020 16:43, Ananyev, Konstantin: > > Hi Honnappa, > > > Hi Konstantin, > > I like the way the tests are organized and it looks good. > > > > I am just wondering about the way it is being tested here. The intent to > > write the test cases the way they are currently is to mimic how th

Re: [dpdk-dev] [PATCH] test/ring: code rework to reduce compilation time

2020-05-05 Thread Ananyev, Konstantin
> > > > > Hi Konstantin, > > > I like the way the tests are organized and it looks good. > > > > > > I am just wondering about the way it is being tested here. The intent > > > to write the test cases the way they are currently is to mimic how the > > > APIs would be used mostly. IMO, the APIs

Re: [dpdk-dev] [PATCH] test/ring: code rework to reduce compilation time

2020-05-05 Thread Honnappa Nagarahalli
> > > > > > > Hi Konstantin, > > > > I like the way the tests are organized and it looks good. > > > > > > > > I am just wondering about the way it is being tested here. The > > > > intent to write the test cases the way they are currently is to > > > > mimic how the APIs would be used mo

Re: [dpdk-dev] [PATCH] test/ring: code rework to reduce compilation time

2020-05-05 Thread Ananyev, Konstantin
> > > > > > > > > Hi Konstantin, > > > > > I like the way the tests are organized and it looks good. > > > > > > > > > > I am just wondering about the way it is being tested here. The > > > > > intent to write the test cases the way they are currently is to > > > > > mimic how the APIs would

Re: [dpdk-dev] [PATCH] test/ring: code rework to reduce compilation time

2020-05-05 Thread Honnappa Nagarahalli
> > > > > > > > > > > Hi Konstantin, > > > > > > I like the way the tests are organized and it looks good. > > > > > > > > > > > > I am just wondering about the way it is being tested here. The > > > > > > intent to write the test cases the way they are currently is > > > > > > to mimic how t

Re: [dpdk-dev] [PATCH] test/ring: code rework to reduce compilation time

2020-05-05 Thread Ananyev, Konstantin
> > > > > > > > > > > > > > > Hi Konstantin, > > > > > > > I like the way the tests are organized and it looks good. > > > > > > > > > > > > > > I am just wondering about the way it is being tested here. The > > > > > > > intent to write the test cases the way they are currently is > > > > > >

Re: [dpdk-dev] [PATCH] test/ring: code rework to reduce compilation time

2020-05-07 Thread Honnappa Nagarahalli
> > > > > > > > > > > > > > > > > > > Hi Konstantin, > > > > > > > > I like the way the tests are organized and it looks > > > > > > > > good. > > > > > > > > > > > > > > > > I am just wondering about the way it is being tested here. > > > > > > > > The intent to write the test cases the

Re: [dpdk-dev] [PATCH] test/ring: code rework to reduce compilation time

2020-05-08 Thread Ananyev, Konstantin
> > > > > > > > > > > > > > > > > > > > > > > Hi Konstantin, > > > > > > > > > I like the way the tests are organized and it looks > > > > > > > > > good. > > > > > > > > > > > > > > > > > > I am just wondering about the way it is being tested here. > > > > > > > > > The intent to write the

Re: [dpdk-dev] [PATCH] test/ring: code rework to reduce compilation time

2020-05-08 Thread Honnappa Nagarahalli
> > > > > > > > > > > > > > > > > > > > > > > > > > > Hi Konstantin, > > > > > > > > > > I like the way the tests are organized and it looks > good. > > > > > > > > > > > > > > > > > > > > I am just wondering about the way it is being tested here. > > > > > > > > > > The intent to write the

Re: [dpdk-dev] [PATCH] test/ring: code rework to reduce compilation time

2020-05-11 Thread Ananyev, Konstantin
> > > > > > > > > > > > > > > > > > > > > Hi Konstantin, > > > > > > > > > > > I like the way the tests are organized and it looks > > good. > > > > > > > > > > > > > > > > > > > > > > I am just wondering about the way it is being tested here. > > > > > > > > > > > The intent to write the test ca

Re: [dpdk-dev] [PATCH] test/ring: code rework to reduce compilation time

2020-05-12 Thread Honnappa Nagarahalli
> > > > > > > > > > > > > > > > > > > > > > > > Hi Konstantin, > > > > > > > > > > > > I like the way the tests are organized and it > > > > > > > > > > > > looks > > > good. > > > > > > > > > > > > > > > > > > > > > > > > I am just wondering about the way it is being tested > > > > > >

Re: [dpdk-dev] [PATCH] test/ring: code rework to reduce compilation time

2020-05-13 Thread Ananyev, Konstantin
> > > > > > > > > > > > > Hi Konstantin, > > > > > > > > > > > > > I like the way the tests are organized and it > > > > > > > > > > > > > looks > > > > good. > > > > > > > > > > > > > > > > > > > > > > > > > > I am just wondering about the way it is being tested > > > > > > > > > > > > >