diff mbox

[PATCHv2,5/5] validation: call to queue_main from platform

Message ID 1435592374-3517-6-git-send-email-christophe.milard@linaro.org
State New
Headers show

Commit Message

Christophe Milard June 29, 2015, 3:39 p.m. UTC
queue_main is now ran from the platform side.
As queue tests do not have
any platform dependency, the call to the platform agnostic tests is
performed directly in the Makefele.am (TESTS).

Signed-off-by: Christophe Milard <christophe.milard@linaro.org>
Reviewed-by: Stuart Haslam <stuart.haslam@linaro.org>
---
 platform/linux-generic/test/Makefile.am | 1 +
 test/validation/Makefile.am             | 3 +--
 2 files changed, 2 insertions(+), 2 deletions(-)

Comments

Christophe Milard June 30, 2015, 7:33 a.m. UTC | #1
I am rebasing right now. I am in the process to move all the tests modules
to the platform side (i.e.  be called from the platfoem side, as you have
noticed for pktio and crypto.) I have quite a few modules ready here, but
they have not been all reviewed/merged yet. I need to rebase to get that in.


On 30 June 2015 at 09:26, Nicolas Morey-Chaisemartin <nmorey@kalray.eu>
wrote:

>
>
> On 06/29/2015 05:39 PM, Christophe Milard wrote:
> > queue_main is now ran from the platform side.
> > As queue tests do not have
> > any platform dependency, the call to the platform agnostic tests is
> > performed directly in the Makefele.am (TESTS).
> >
> > Signed-off-by: Christophe Milard <christophe.milard@linaro.org>
> > Reviewed-by: Stuart Haslam <stuart.haslam@linaro.org>
> > ---
> >  platform/linux-generic/test/Makefile.am | 1 +
> >  test/validation/Makefile.am             | 3 +--
> >  2 files changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/platform/linux-generic/test/Makefile.am
> b/platform/linux-generic/test/Makefile.am
> > index d129e43..583a449 100644
> > --- a/platform/linux-generic/test/Makefile.am
> > +++ b/platform/linux-generic/test/Makefile.am
> > @@ -9,6 +9,7 @@ TESTS = pktio/pktio_run \
> >       ${top_builddir}/test/validation/init/init_main_ok \
> >       ${top_builddir}/test/validation/init/init_main_abort \
> >       ${top_builddir}/test/validation/init/init_main_log \
> > +     ${top_builddir}/test/validation/queue/queue_main \
> >       ${top_builddir}/test/validation/random/random_main \
> >       ${top_builddir}/test/validation/system/system_main
> >
> >
> All these would need $(EXEEXT) extensions to match with the other binaries.
> I guess you don't necessary have to rebase all your patch series and we
> could just add another patch on top of them to fix the extensions.
>
diff mbox

Patch

diff --git a/platform/linux-generic/test/Makefile.am b/platform/linux-generic/test/Makefile.am
index d129e43..583a449 100644
--- a/platform/linux-generic/test/Makefile.am
+++ b/platform/linux-generic/test/Makefile.am
@@ -9,6 +9,7 @@  TESTS = pktio/pktio_run \
 	${top_builddir}/test/validation/init/init_main_ok \
 	${top_builddir}/test/validation/init/init_main_abort \
 	${top_builddir}/test/validation/init/init_main_log \
+	${top_builddir}/test/validation/queue/queue_main \
 	${top_builddir}/test/validation/random/random_main \
 	${top_builddir}/test/validation/system/system_main
 
diff --git a/test/validation/Makefile.am b/test/validation/Makefile.am
index 8a24449..b07f5bb 100644
--- a/test/validation/Makefile.am
+++ b/test/validation/Makefile.am
@@ -16,8 +16,7 @@  EXECUTABLES = odp_buffer \
 	      odp_ver_abt_log_dbg
 
 if test_vald
-TESTS = $(EXECUTABLES) \
-	queue/queue_main
+TESTS = $(EXECUTABLES)
 endif
 
 bin_PROGRAMS = $(EXECUTABLES) $(COMPILE_ONLY)