Re: [PATCH] tests: Refactor parallel test execution

2015-03-04 Thread Sebastian Huber
On 04/03/15 16:43, Joel Sherrill wrote: Would it at least make sense to add a subdirectory with a different name in cpukit? You could add libmisc/test_support and only have to add a few lines to libmisc/Makefile.am. That would contain this stuff better. Ok, good idea. We move the stuff to this

Re: [PATCH] tests: Refactor parallel test execution

2015-03-04 Thread Joel Sherrill
Would it at least make sense to add a subdirectory with a different name in cpukit? You could add libmisc/test_support and only have to add a few lines to libmisc/Makefile.am. That would contain this stuff better. On 3/4/2015 9:41 AM, Gedare Bloom wrote: > OK. > > On Wed, Mar 4, 2015 at 9:22 AM,

Re: [PATCH] tests: Refactor parallel test execution

2015-03-04 Thread Gedare Bloom
OK. On Wed, Mar 4, 2015 at 9:22 AM, Sebastian Huber wrote: > > On 04/03/15 15:17, Gedare Bloom wrote: >> >> It seems that these changes to test.h and the new testparallel.c >> belong in the testsuites/smptests area, perhaps in a shared/ >> subdirectory. Certainly I don't see why testparallel.c is

Re: [PATCH] tests: Refactor parallel test execution

2015-03-04 Thread Sebastian Huber
On 04/03/15 15:17, Gedare Bloom wrote: It seems that these changes to test.h and the new testparallel.c belong in the testsuites/smptests area, perhaps in a shared/ subdirectory. Certainly I don't see why testparallel.c is in sapi. (I don't recall why we have testing framework in the sapi though

Re: [PATCH] tests: Refactor parallel test execution

2015-03-04 Thread Gedare Bloom
It seems that these changes to test.h and the new testparallel.c belong in the testsuites/smptests area, perhaps in a shared/ subdirectory. Certainly I don't see why testparallel.c is in sapi. (I don't recall why we have testing framework in the sapi though...) On Wed, Mar 4, 2015 at 7:33 AM, Alex

[PATCH] tests: Refactor parallel test execution

2015-03-04 Thread Alexander Krutwig
--- cpukit/sapi/Makefile.am| 1 + cpukit/sapi/include/rtems/test.h | 109 ++ cpukit/sapi/src/testparallel.c | 151 +++ testsuites/smptests/smpatomic01/init.c | 264 ++--- 4 files changed, 375 insertions(+), 150