Re: [lng-odp] [PATCH] test:linux-generic: run odp_scheduling in process mode

2016-08-10 Thread Maxim Uvarov
Merged, Maxim. On 08/04/16 16:18, Mike Holmes wrote: Set up the environment to allow calling the performance tests in process mode as part of make check when enabled. To run the tests use --enable-test-perf-proc Initial patch using odp_scheduling as a proof Signed-off-by: Mike Holmes

Re: [lng-odp] [PATCH] test:linux-generic: run odp_scheduling in process mode

2016-08-10 Thread Mike Holmes
scho...@linaro.org> > > Cc: Savolainen, Petri (Nokia - FI/Espoo) <petri.savolainen@nokia-bell- > > labs.com>; Mike Holmes <mike.hol...@linaro.org>; LNG ODP Mailman List > > <lng-odp@lists.linaro.org> > > Subject: Re: [lng-odp] [PATCH] test:linux-generi

Re: [lng-odp] [PATCH] test:linux-generic: run odp_scheduling in process mode

2016-08-10 Thread Savolainen, Petri (Nokia - FI/Espoo)
y, August 09, 2016 8:33 PM > To: Bill Fischofer <bill.fischo...@linaro.org> > Cc: Savolainen, Petri (Nokia - FI/Espoo) <petri.savolainen@nokia-bell- > labs.com>; Mike Holmes <mike.hol...@linaro.org>; LNG ODP Mailman List > <lng-odp@lists.linaro.org> > Subjec

Re: [lng-odp] [PATCH] test:linux-generic: run odp_scheduling in process mode

2016-08-09 Thread Mike Holmes
Thanks Brian We have https://bugs.linaro.org/show_bug.cgi?id=2457 for this issue to continue the discussion. This patch is reviewed and should go in so that CI can help find the next thing we missed! Mike On 9 August 2016 at 13:33, Brian Brooks wrote: > On 08/04

Re: [lng-odp] [PATCH] test:linux-generic: run odp_scheduling in process mode

2016-08-05 Thread Maxim Uvarov
On 5 August 2016 at 19:41, Bill Fischofer wrote: > > > On Fri, Aug 5, 2016 at 11:31 AM, Maxim Uvarov > wrote: > >> On 08/05/16 19:29, Mike Holmes wrote: >> >>> >>> >>> On 5 August 2016 at 12:27, Maxim Uvarov >>

Re: [lng-odp] [PATCH] test:linux-generic: run odp_scheduling in process mode

2016-08-05 Thread Maxim Uvarov
clear_sched_queues(); is missing just before destroying queues + fix form Bill with account all term errors. Maxim. On 08/05/16 19:01, Bill Fischofer wrote: On Fri, Aug 5, 2016 at 10:59 AM, Mike Holmes wrote: On 4 August 2016 at 11:26, Brian Brooks

Re: [lng-odp] [PATCH] test:linux-generic: run odp_scheduling in process mode

2016-08-05 Thread Maxim Uvarov
On 08/05/16 19:29, Mike Holmes wrote: On 5 August 2016 at 12:27, Maxim Uvarov > wrote: clear_sched_queues(); is missing just before destroying queues + fix form Bill with account all term errors. Should this go on the

Re: [lng-odp] [PATCH] test:linux-generic: run odp_scheduling in process mode

2016-08-05 Thread Mike Holmes
On 5 August 2016 at 12:27, Maxim Uvarov wrote: > clear_sched_queues(); > > is missing just before destroying queues + fix form Bill with account all > term errors. Should this go on the bug ? This this thread should be about this patch to enable process mode or it

Re: [lng-odp] [PATCH] test:linux-generic: run odp_scheduling in process mode

2016-08-05 Thread Bill Fischofer
On Fri, Aug 5, 2016 at 10:59 AM, Mike Holmes wrote: > On 4 August 2016 at 11:26, Brian Brooks wrote: > > > Reviewed-by: Brian Brooks > > > > Just wanted to follow up on this, this patch has also highlighted a bug in >

Re: [lng-odp] [PATCH] test:linux-generic: run odp_scheduling in process mode

2016-08-05 Thread Mike Holmes
On 4 August 2016 at 11:26, Brian Brooks wrote: > Reviewed-by: Brian Brooks > Just wanted to follow up on this, this patch has also highlighted a bug in the original code and the thread below documents it, also a bug for it has been created

Re: [lng-odp] [PATCH] test:linux-generic: run odp_scheduling in process mode

2016-08-04 Thread Bill Fischofer
On Thu, Aug 4, 2016 at 11:17 AM, Brian Brooks wrote: > On 08/04 11:01:09, Bill Fischofer wrote: > > On Thu, Aug 4, 2016 at 10:59 AM, Mike Holmes > wrote: > > > > > > > > > > > On 4 August 2016 at 11:47, Bill Fischofer

Re: [lng-odp] [PATCH] test:linux-generic: run odp_scheduling in process mode

2016-08-04 Thread Brian Brooks
On 08/04 11:01:09, Bill Fischofer wrote: > On Thu, Aug 4, 2016 at 10:59 AM, Mike Holmes wrote: > > > > > > > On 4 August 2016 at 11:47, Bill Fischofer > > wrote: > > > >> > >> On Thu, Aug 4, 2016 at 10:36 AM, Mike Holmes

Re: [lng-odp] [PATCH] test:linux-generic: run odp_scheduling in process mode

2016-08-04 Thread Maxim Uvarov
On 08/04/16 18:26, Brian Brooks wrote: Reviewed-by: Brian Brooks On 08/04 09:18:14, Mike Holmes wrote: +ret=0 + +run() +{ + echo odp_scheduling_run_proc starts with $1 worker threads + echo = + +

Re: [lng-odp] [PATCH] test:linux-generic: run odp_scheduling in process mode

2016-08-04 Thread Bill Fischofer
On Thu, Aug 4, 2016 at 10:59 AM, Mike Holmes wrote: > > > On 4 August 2016 at 11:47, Bill Fischofer > wrote: > >> >> On Thu, Aug 4, 2016 at 10:36 AM, Mike Holmes >> wrote: >> >>> On my vanilla x86 I don't get any

Re: [lng-odp] [PATCH] test:linux-generic: run odp_scheduling in process mode

2016-08-04 Thread Bill Fischofer
On Thu, Aug 4, 2016 at 11:03 AM, Maxim Uvarov wrote: > On 08/04/16 18:26, Brian Brooks wrote: > >> Reviewed-by: Brian Brooks >> >> On 08/04 09:18:14, Mike Holmes wrote: >> >>> +ret=0 >>> + >>> +run() >>> +{ >>> + echo

Re: [lng-odp] [PATCH] test:linux-generic: run odp_scheduling in process mode

2016-08-04 Thread Mike Holmes
On 4 August 2016 at 11:47, Bill Fischofer wrote: > > On Thu, Aug 4, 2016 at 10:36 AM, Mike Holmes > wrote: > >> On my vanilla x86 I don't get any issues, keen to get this in and have >> CI run it on lots of HW to see what happens, many of the

Re: [lng-odp] [PATCH] test:linux-generic: run odp_scheduling in process mode

2016-08-04 Thread Bill Fischofer
Quick update. I can repro this in v1.10.0.1 as well, however v1.10.0.0 seems good. On Thu, Aug 4, 2016 at 10:47 AM, Bill Fischofer wrote: > > On Thu, Aug 4, 2016 at 10:36 AM, Mike Holmes > wrote: > >> On my vanilla x86 I don't get any issues,

Re: [lng-odp] [PATCH] test:linux-generic: run odp_scheduling in process mode

2016-08-04 Thread Bill Fischofer
On Thu, Aug 4, 2016 at 10:36 AM, Mike Holmes wrote: > On my vanilla x86 I don't get any issues, keen to get this in and have CI > run it on lots of HW to see what happens, many of the other tests > completely fail in process mode so we will expose a lot as we add them I

Re: [lng-odp] [PATCH] test:linux-generic: run odp_scheduling in process mode

2016-08-04 Thread Mike Holmes
On my vanilla x86 I don't get any issues, keen to get this in and have CI run it on lots of HW to see what happens, many of the other tests completely fail in process mode so we will expose a lot as we add them I think. On 4 August 2016 at 11:33, Bill Fischofer wrote:

Re: [lng-odp] [PATCH] test:linux-generic: run odp_scheduling in process mode

2016-08-04 Thread Bill Fischofer
On Thu, Aug 4, 2016 at 10:26 AM, Brian Brooks wrote: > Reviewed-by: Brian Brooks > > On 08/04 09:18:14, Mike Holmes wrote: > > +ret=0 > > + > > +run() > > +{ > > + echo odp_scheduling_run_proc starts with $1 worker threads > > + echo

Re: [lng-odp] [PATCH] test:linux-generic: run odp_scheduling in process mode

2016-08-04 Thread Brian Brooks
Reviewed-by: Brian Brooks On 08/04 09:18:14, Mike Holmes wrote: > +ret=0 > + > +run() > +{ > + echo odp_scheduling_run_proc starts with $1 worker threads > + echo = > + > +

[lng-odp] [PATCH] test:linux-generic: run odp_scheduling in process mode

2016-08-04 Thread Mike Holmes
Set up the environment to allow calling the performance tests in process mode as part of make check when enabled. To run the tests use --enable-test-perf-proc Initial patch using odp_scheduling as a proof Signed-off-by: Mike Holmes --- configure.ac