[lng-odp] [PATCH 0/5] moving time to module system called from plt side

2015-06-22 Thread Christophe Milard
has moved to the system directory (file 'system.c' as 'system' only has time functionality at this time). Christophe Milard (5): validation: own main in odp_time.c validation: renaming in odp_time.c validation: cosmetic fixes in odp_time.c validation: creating own dir and lib for system

[lng-odp] [PATCHv2 3/5] validation: renaming in odp_classification.c

2015-06-22 Thread Christophe Milard
: Module_suites[_*] where the possible suffix identifies the executable using it, if many. -Main function(s), are called: Module_main[_*] where the possible suffix identifies the executable using it Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- .../classification

[lng-odp] [PATCHv2 0/5] validation: classification tests moved to platform side

2015-06-22 Thread Christophe Milard
since v1: rebased. Classification tests now go through the same modification as pktio, and gets finally called from the platform side. Christophe Milard (5): validation: own main in odp_classification.c validation: cosmetic change for check-patch validation: renaming

[lng-odp] [PATCHv2 1/5] validation: own main in odp_classification.c

2015-06-22 Thread Christophe Milard
Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- test/validation/Makefile.am | 3 +++ test/validation/odp_classification.c | 15 ++- 2 files changed, 17 insertions(+), 1 deletion(-) diff --git a/test/validation/Makefile.am b/test/validation/Makefile.am

Re: [lng-odp] [PATCH 0/5] validation: classification tests moved to platform side

2015-06-22 Thread Christophe Milard
... that is, if you don't mind... :-) On 22 June 2015 at 16:57, Stuart Haslam stuart.has...@linaro.org wrote: On Mon, Jun 22, 2015 at 01:41:40PM +0200, Christophe Milard wrote: Classification tests now go through the same modification as pktio, and gets finally called from the platform side. Christophe

[lng-odp] [PATCH 5/5] validation: call to system_main from platform

2015-06-22 Thread Christophe Milard
system_main is now ran from the platform side. As system tests do not have any platform dependency, the call to the platform agnostic tests is performed directly in the Makefele.am (TESTS) (At present, the system module just contains time tests) Signed-off-by: Christophe Milard christophe.mil

[lng-odp] [PATCH 4/5] validation: creating own dir and lib for system

2015-06-22 Thread Christophe Milard
Module system now gets its own directory and create its own lib (currentely only containing its executable) Note that module system, at this time, only contains time functionality. Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- configure.ac

[lng-odp] [PATCH 2/5] validation: renaming in odp_time.c

2015-06-22 Thread Christophe Milard
it Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- test/validation/odp_time.c | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/test/validation/odp_time.c b/test/validation/odp_time.c index 38d940e..3005a26 100644 --- a/test/validation/odp_time.c

[lng-odp] [PATCHv2 2/5] validation: cosmetic change for check-patch

2015-06-22 Thread Christophe Milard
Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- test/validation/classification/odp_classification_tests.c | 1 + test/validation/classification/odp_classification_testsuites.h | 4 ++-- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/test/validation

[lng-odp] [PATCH 4/5] validation: creating own dir and lib for pktio

2015-06-22 Thread Christophe Milard
Module classification now gets its own directory and create its own lib (currentely only containing its executable) Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- configure.ac| 1 + test/validation/.gitignore

[lng-odp] [PATCH 5/5] validation: call to classification_main from platform

2015-06-22 Thread Christophe Milard
classification_main is now ran from the platform side. As classification 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.mil...@linaro.org --- platform/linux-generic

[lng-odp] [PATCH 2/5] validation: cosmetic change for check-patch

2015-06-22 Thread Christophe Milard
Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- test/validation/classification/odp_classification_tests.c | 1 + test/validation/classification/odp_classification_testsuites.h | 4 ++-- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/test/validation

[lng-odp] [PATCH 1/5] validation: own main in odp_classification.c

2015-06-22 Thread Christophe Milard
Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- test/validation/Makefile.am | 3 +++ test/validation/odp_classification.c | 15 ++- 2 files changed, 17 insertions(+), 1 deletion(-) diff --git a/test/validation/Makefile.am b/test/validation/Makefile.am

[lng-odp] [PATCH 3/5] validation: renaming in odp_classification.c

2015-06-22 Thread Christophe Milard
: Module_suites[_*] where the possible suffix identifies the executable using it, if many. -Main function(s), are called: Module_main[_*] where the possible suffix identifies the executable using it Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- .../classification

[lng-odp] [PATCH 5/5] validation: call to crypto_main from platform

2015-06-17 Thread Christophe Milard
crypto_main is now ran from the platform side. As crypto 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.mil...@linaro.org --- platform/linux-generic/test/Makefile.am

[lng-odp] [PATCH 2/5] validation: renaming in odp_crypto.c

2015-06-17 Thread Christophe Milard
: Module_suites[_*] where the possible suffix identifies the executable using it, if many. -Main function(s), are called: Module_main[_*] where the possible suffix identifies the executable using it Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- test/validation/crypto

[lng-odp] [PATCH 1/5] validation: own main in odp_crypto.c

2015-06-17 Thread Christophe Milard
Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- test/validation/Makefile.am | 2 ++ test/validation/odp_crypto.c | 13 - 2 files changed, 14 insertions(+), 1 deletion(-) diff --git a/test/validation/Makefile.am b/test/validation/Makefile.am index 6968301..e40bc7d

[lng-odp] [PATCH 3/5] validation: cosmetic fixes in odp_crypto.c

2015-06-17 Thread Christophe Milard
Preparing for the next patch where this file is moved and check-odp would yell if these things were still there. Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- test/validation/odp_crypto.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/test

[lng-odp] [PATCH 0/5] moving crypto tests call to platform side

2015-06-17 Thread Christophe Milard
crypto is now going throw the same modifications as pktio, but its main is directely called from the makefile.am as this module has nothing platform specific. Christophe Milard (5): validation: own main in odp_crypto.c validation: renaming in odp_crypto.c validation: cosmetic fixes

[lng-odp] [PATCH 4/5] validation: creating own dir and lib for crypto

2015-06-17 Thread Christophe Milard
Module crypto now gets its own directory and create its own lib (currentely only containing its executable) Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- configure.ac | 1 + test/validation/.gitignore| 1 - test

Re: [lng-odp] [PATCH] linux-generic: crypto: eliminate buffer type hack for completions

2015-06-10 Thread Christophe Milard
On 10 June 2015 at 04:50, Bill Fischofer bill.fischo...@linaro.org wrote: odp_crypto_operation() should not change underlying buffer types for completions as these are purely internal and not needed, and doing so is confusing and error-prome. Yes, Bill, I would prefer your approach: my patch

Re: [lng-odp] [PATCHv4 7/8] validation: moving pktio plt specific to platform

2015-06-10 Thread Christophe Milard
+++ b/platform/linux-generic/test/Makefile.am @@ -2,7 +2,9 @@ TESTS_ENVIRONMENT = TEST_DIR=${top_builddir}/test/validation if test_vald TESTS = pktio/pktio_run -endif ODP_MODULES = pktio SUBDIRS = $(ODP_MODULES) +endif Maxim. On 06/09/15 14:44, Christophe Milard wrote

Re: [lng-odp] [PATCHv4 7/8] validation: moving pktio plt specific to platform

2015-06-10 Thread Christophe Milard
On 10 June 2015 at 11:17, Maxim Uvarov maxim.uva...@linaro.org wrote: On 06/10/15 11:37, Christophe Milard wrote: Hi, I would agree with you, but saddly pktio_env (which is defined under platform/linux-generic/test/pktio) is also used in the performance tests. pktio_env was before packed

[lng-odp] [PATCHv5 0/8] pktio test move to platform side

2015-06-10 Thread Christophe Milard
are a subpart of ODP, which is not true. When this patch is accepted, other modeles conversion will follows. Christophe Milard (8): validation: preparing for main in tests validation: own main in odp_pktio.c validation: renaming in odp_pktio.c validation: cosmetic fixes in odp_pktio.c

[lng-odp] [PATCHv5 1/8] validation: preparing for main in tests

2015-06-10 Thread Christophe Milard
with MODULE_HAS_OWN_MAIN defined (to be used in the future, by tests which define their own main) and one with MODULE_HAS_OWN_MAIN undefined, used by all tests at this stage. Signed-off-by: Christophe Milard christophe.mil...@linaro.org Reviewed-by: Mike Holmes mike.hol...@linaro.org Reviewed-by: Stuart

[lng-odp] [PATCHv5 2/8] validation: own main in odp_pktio.c

2015-06-10 Thread Christophe Milard
Signed-off-by: Christophe Milard christophe.mil...@linaro.org Reviewed-by: Mike Holmes mike.hol...@linaro.org Reviewed-by: Stuart Haslam stuart.has...@linaro.org --- test/validation/Makefile.am | 2 ++ test/validation/odp_pktio.c | 15 +-- 2 files changed, 15 insertions(+), 2

[lng-odp] [PATCHv5 3/8] validation: renaming in odp_pktio.c

2015-06-10 Thread Christophe Milard
: Module_suites[_*] where the possible suffix identifies the executable using it, if many. -Main function(s), are called: Module_main[_*] where the possible suffix identifies the executable using it Signed-off-by: Christophe Milard christophe.mil...@linaro.org Reviewed-by: Mike Holmes

[lng-odp] [PATCHv5 7/8] validation: moving pktio plt specific to platform

2015-06-10 Thread Christophe Milard
The platform specific scripts for pktio are now moved from the validation to the platform side. Pktio tests are now initiated from the platform side. Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- configure.ac | 1 + platform/linux

[lng-odp] [PATCHv5 8/8] validation: removing obsolete environment variable

2015-06-10 Thread Christophe Milard
These are no longer referenced and should not be used as they introduce platform dependency which should no longer occur in this platform agnostic part. Signed-off-by: Christophe Milard christophe.mil...@linaro.org Reviewed-by: Mike Holmes mike.hol...@linaro.org Reviewed-by: Stuart Haslam

[lng-odp] [PATCHv5 6/8] validation: changing build order

2015-06-10 Thread Christophe Milard
-by: Christophe Milard christophe.mil...@linaro.org Reviewed-by: Mike Holmes mike.hol...@linaro.org Reviewed-by: Stuart Haslam stuart.has...@linaro.org --- Makefile.am| 9 - configure.ac | 3 ++- platform/Makefile.am | 1 - platform

[lng-odp] [PATCHv5 5/8] validation: creating own dir and lib for pktio

2015-06-10 Thread Christophe Milard
Module pktio now gets its own directory and create its own lib (currentely only containing its executable) Startup scripting stuff is just moved to the pktio directory but remains untouched at this stage (test is still ran from validation side) Signed-off-by: Christophe Milard christophe.mil

[lng-odp] [PATCHv5 4/8] validation: cosmetic fixes in odp_pktio.c

2015-06-10 Thread Christophe Milard
Preparing for the next patch where this file is moved and check-odp would yell if these things were still there. Signed-off-by: Christophe Milard christophe.mil...@linaro.org Reviewed-by: Mike Holmes mike.hol...@linaro.org Reviewed-by: Stuart Haslam stuart.has...@linaro.org --- test/validation

[lng-odp] [PATCHv4 3/8] validation: renaming in odp_pktio.c

2015-06-09 Thread Christophe Milard
: Module_suites[_*] where the possible suffix identifies the executable using it, if many. -Main function(s), are called: Module_main[_*] where the possible suffix identifies the executable using it Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- test/validation

[lng-odp] [PATCHv4 1/8] validation: preparing for main in tests

2015-06-09 Thread Christophe Milard
with MODULE_HAS_OWN_MAIN defined (to be used in the future, by tests which define their own main) and one with MODULE_HAS_OWN_MAIN undefined, used by all tests at this stage. Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- configure.ac | 1 + test/Makefile.inc

[lng-odp] [PATCHv4 2/8] validation: own main in odp_pktio.c

2015-06-09 Thread Christophe Milard
Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- test/validation/Makefile.am | 2 ++ test/validation/odp_pktio.c | 15 +-- 2 files changed, 15 insertions(+), 2 deletions(-) diff --git a/test/validation/Makefile.am b/test/validation/Makefile.am index e1474aa..d4bbdb5

[lng-odp] [PATCHv4 4/8] validation: cosmetic fixes in odp_pktio.c

2015-06-09 Thread Christophe Milard
Preparing for the next patch where this file is moved and check-odp would yell if these things were still there. Signed-off-by: Christophe Milard christophe.mil...@linaro.org Reviewed-by: Stuart Haslam stuart.has...@linaro.org --- test/validation/odp_pktio.c | 13 - 1 file changed, 4

[lng-odp] [PATCHv4 8/8] validation: removing obsolete environment variable

2015-06-09 Thread Christophe Milard
These are no longer referenced and should not be used as they introduce platform dependency which should no longer occur in this platform agnostic part. Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- test/validation/Makefile.am | 2 -- 1 file changed, 2 deletions(-) diff

[lng-odp] [PATCHv4 6/8] validation: changing build order

2015-06-09 Thread Christophe Milard
-by: Christophe Milard christophe.mil...@linaro.org Reviewed-by: Stuart Haslam stuart.has...@linaro.org --- Makefile.am| 9 - configure.ac | 3 ++- platform/Makefile.am | 1 - platform/linux-generic/Makefile.am | 2 -- 4 files changed

[lng-odp] [PATCHv4 5/8] validation: creating own dir and lib for pktio

2015-06-09 Thread Christophe Milard
Module pktio now gets its own directory and create its own lib (currentely only containing its executable) Startup scripting stuff is just moved to the pktio directory but remains untouched at this stage (test is still ran from validation side) Signed-off-by: Christophe Milard christophe.mil

[lng-odp] [PATCHv4 7/8] validation: moving pktio plt specific to platform

2015-06-09 Thread Christophe Milard
The platform specific scripts for pktio are now moved from the validation to the platform side. Pktio tests are now initiated from the platform side. Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- configure.ac | 1 + platform/linux

Re: [lng-odp] [API-NEXT PATCH v3 3/3] api: helper: deleted odph_linux_cpumask_default

2015-06-09 Thread Christophe Milard
On 2015-06-09 11:19, Petri Savolainen wrote: The linux specific helper was replaced by default cpumask calls (odp_cpumask_def_worker()) in the cpumask API. Signed-off-by: Petri Savolainen petri.savolai...@nokia.com Reviewed-by: Christophe Milard christophe.mil...@linaro.org --- example

[lng-odp] [PATCH-RFC] linux-generic: pool: Fix for buffer type casting

2015-06-09 Thread Christophe Milard
-by: Christophe Milard christophe.mil...@linaro.org --- platform/linux-generic/odp_pool.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/platform/linux-generic/odp_pool.c b/platform/linux-generic/odp_pool.c index 35e79a0..6e65926 100644 --- a/platform/linux-generic/odp_pool.c +++ b/platform/linux

Re: [lng-odp] [API-NEXT PATCH v3 1/3] api: cpumask: added cpumask_setall

2015-06-09 Thread Christophe Milard
On 2015-06-09 11:19, Petri Savolainen wrote: The call sets all possible CPUs in the mask. It's system specific which CPUs are actually available to the application. Signed-off-by: Petri Savolainen petri.savolai...@nokia.com Reviewed-by: Christophe Milard christophe.mil...@linaro.org

Re: [lng-odp] [API-NEXT PATCH v3 2/3] api: cpumask: added default masks

2015-06-09 Thread Christophe Milard
-by: Christophe Milard christophe.mil...@linaro.org --- include/odp/api/cpumask.h| 24 +++ platform/linux-generic/odp_cpumask.c | 38 2 files changed, 62 insertions(+) diff --git a/include/odp/api/cpumask.h b/include/odp/api

[lng-odp] [PATCHv3 0/8] pktio test move to platform side

2015-06-04 Thread Christophe Milard
will follows. thanks for your time. Christophe Milard (8): validation: preparing for main in tests validation: own main in odp_pktio.c validation: renaming in odp_pktio.c validation: cosmetic fixes in odp_pktio.c validation: creating own dir and lib for pktio validation: changing build

[lng-odp] [PATCHv3 1/8] validation: preparing for main in tests

2015-06-04 Thread Christophe Milard
with MODULE_HAS_OWN_MAIN defined (to be used in the future, by tests which define their own main) and one with MODULE_HAS_OWN_MAIN undefined, used by all tests at this stage. Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- configure.ac | 1 + test/Makefile.inc

[lng-odp] [PATCHv3 3/8] validation: renaming in odp_pktio.c

2015-06-04 Thread Christophe Milard
: Module_suites[_*] where the possible suffix identifies the executable using it, if many. -Main function(s), are called: Module_main[_*] where the possible suffix identifies the executable using it Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- test/validation

[lng-odp] [PATCHv3 2/8] validation: own main in odp_pktio.c

2015-06-04 Thread Christophe Milard
Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- test/validation/Makefile.am | 2 ++ test/validation/odp_pktio.c | 15 +-- 2 files changed, 15 insertions(+), 2 deletions(-) diff --git a/test/validation/Makefile.am b/test/validation/Makefile.am index e1474aa..d4bbdb5

[lng-odp] [PATCHv3 5/8] validation: creating own dir and lib for pktio

2015-06-04 Thread Christophe Milard
Module pktio now gets its own directory and create its own lib (currentely only containing its executable) Startup scripting stuff is just moved to the pktio directory but remains untouched at this stage (test is still ran from validation side) Signed-off-by: Christophe Milard christophe.mil

[lng-odp] [PATCHv3 4/8] validation: cosmetic fixes in odp_pktio.c

2015-06-04 Thread Christophe Milard
Preparing for the next patch where this file is moved and check-odp would yell if these things were still there. Signed-off-by: Christophe Milard christophe.mil...@linaro.org Reviewed-by: Stuart Haslam stuart.has...@linaro.org --- test/validation/odp_pktio.c | 13 - 1 file changed, 4

[lng-odp] [PATCHv3 8/8] validation: removing obsolete environment variable

2015-06-04 Thread Christophe Milard
These are no longer referenced and should not be used as they introduce platform dependency which should no longer occur in this platform agnostic part. Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- test/validation/Makefile.am | 2 -- 1 file changed, 2 deletions(-) diff

[lng-odp] [PATCHv3 6/8] validation: changing build order

2015-06-04 Thread Christophe Milard
-by: Christophe Milard christophe.mil...@linaro.org Reviewed-by: Stuart Haslam stuart.has...@linaro.org --- Makefile.am| 9 - configure.ac | 3 ++- platform/Makefile.am | 1 - platform/linux-generic/Makefile.am | 2 -- 4 files changed

[lng-odp] [PATCHv3 7/8] validation: moving pktio plt specific to platform

2015-06-04 Thread Christophe Milard
The platform specific scripts for pktio are now moved from the validation to the platform side. Pktio tests are now initiated from the platform side. Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- configure.ac | 1 + platform/linux

[lng-odp] [PATCHv2 2/7] validation: own main in odp_pktio.c

2015-06-03 Thread Christophe Milard
Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- test/Makefile.inc | 7 ++- test/validation/Makefile.am | 8 +++- test/validation/odp_pktio.c | 15 +-- 3 files changed, 22 insertions(+), 8 deletions(-) diff --git a/test/Makefile.inc b/test

[lng-odp] [PATCHv2 1/7] validation: preparing for main in tests

2015-06-03 Thread Christophe Milard
with MODULE_HAS_OWN_MAIN defined (to be used in the future, by tests which define their own main) and one with MODULE_HAS_OWN_MAIN undefined, used by all tests at this stage. Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- configure.ac | 1 + test/validation

[lng-odp] [PATCHv2 0/7] validation: pktio test move to platform side

2015-06-03 Thread Christophe Milard
that this directory structure will change: Having the ODP code in platform/platform/* and the tests in platform/platform/tests seems to imply that tests are a subpart of ODP, which is not true. When this patch is accepted, other modeles conversion will follows. thanks for your time. Christophe

[lng-odp] [PATCHv2 6/7] validation: changing build order

2015-06-03 Thread Christophe Milard
-by: Christophe Milard christophe.mil...@linaro.org --- Makefile.am| 8 +++- configure.ac | 3 ++- platform/Makefile.am | 1 - platform/linux-generic/Makefile.am | 2 -- 4 files changed, 9 insertions(+), 5 deletions(-) delete mode

[lng-odp] [PATCHv2 7/7] validation: moving pktio plt specific to platform

2015-06-03 Thread Christophe Milard
The platform specific scripts for pktio are now moved from the validation to the platform side. Pktio tests are now initiated from the platform side. Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- configure.ac | 1 + platform/linux

[lng-odp] [PATCHv2 3/7] validation: renaming in odp_pktio.c

2015-06-03 Thread Christophe Milard
: Module_suites[_*] where the possible suffix identifies the executable using it, if many. -Main function(s), are called: Module_main[_*] where the possible suffix identifies the executable using it Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- test/validation

[lng-odp] [PATCHv2 5/7] validation: creating own dir and lib for pktio

2015-06-03 Thread Christophe Milard
Module pktio now gets its own directory and create its own lib (currentely only containing its executable) Startup scripting stuff is just moved to the pktio directory but remains untouched at this stage (test is still ran from validation side) Signed-off-by: Christophe Milard christophe.mil

Re: [lng-odp] [PATCH 2/6] validation: own main and renaming in odp_pktio.c

2015-06-02 Thread Christophe Milard
On 2 June 2015 at 11:22, Stuart Haslam stuart.has...@linaro.org wrote: On Thu, May 28, 2015 at 12:58:08PM +0200, Christophe Milard wrote: Renaming of things which may be, one day, exported in a lib. This renaming is important, as it creates consistency between test symbols, which is needed

Re: [lng-odp] [PATCH 4/6] validation: creating own dir and lib for pktio

2015-06-02 Thread Christophe Milard
On 2 June 2015 at 11:31, Stuart Haslam stuart.has...@linaro.org wrote: On Thu, May 28, 2015 at 12:58:10PM +0200, Christophe Milard wrote: Module pktio now gets its own directory and create its own lib (currentely only containing its executable) Startup scripting stuff is just moved

Re: [lng-odp] [PATCH 0/6] validation: pktio test move to platform side

2015-06-02 Thread Christophe Milard
, but thanks for testing. I DO appreciate that. Christophe. Best regards, Maxim. On 05/28/15 13:58, Christophe Milard wrote: This series of patches comes following the request from Stuart to see the effect of the new test structure on module pktio rather than on a simpler module, such as 'random

[lng-odp] [PATCHv2] apply-and-build.sh: cosmetic fix

2015-05-29 Thread Christophe Milard
function apply_patch() now returns a status rather than directely breaking the loop of its caller. (for loop in main, continue statment in apply_patch()) Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- apply-and-build.sh | 7 ++- 1 file changed, 6 insertions(+), 1 deletion

[lng-odp] [PATCHv2] apply-and-build.sh: fixing cover-letter issue

2015-05-29 Thread Christophe Milard
When an empty patch (such as a cover letter) was given to apply-and-build, the latter would fail applying the patch and all following patches (saying OK when it could be wrong) Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- apply-and-build.sh | 10 -- 1 file changed, 4

Re: [lng-odp] [PATCH 1/2] test: helper: add ring test

2015-05-28 Thread Christophe Milard
Hi Mike, Got: PASS: odp_thread PASS: odp_process ../../../test-driver: line 107: 7705 Segmentation fault (core dumped) $@ $log_file 21 FAIL: odp_ring when running PATCH_DIR=/home/erachmi/incoming ./apply-and-build.sh (check-odp) On 27 May 2015 at 19:39, Mike Holmes mike.hol...@linaro.org

[lng-odp] [PATCH 6/6] validation: moving pktio plt specific to platform

2015-05-28 Thread Christophe Milard
The platform specific scripts for pktio are now moved from the validation to the platform side. Pktio tests are now initiated from the platform side. Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- configure.ac | 1 + platform/linux

[lng-odp] [PATCH 3/6] validation: cosmetic fixes in odp_pktio.c

2015-05-28 Thread Christophe Milard
Preparing for the next patch where this file is moved and check-odp would yell if these things were still there. Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- test/validation/odp_pktio.c | 13 - 1 file changed, 4 insertions(+), 9 deletions(-) diff --git a/test

[lng-odp] [PATCH 1/6] validation: preparing for main in tests

2015-05-28 Thread Christophe Milard
with MODULE_HAS_OWN_MAIN defined (to be used in the future, by tests which define their own main) and one with MODULE_HAS_OWN_MAIN undefined, used by all tests at this stage. Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- configure.ac | 1 + test/validation

[lng-odp] [PATCH] apply-and-build.sh: fixing cover-letter issue

2015-05-28 Thread Christophe Milard
When an empty patch (such as a cover letter) was given to apply-and-build, the latter would fail applying the patch and all following patches (saying OK when it could be wrong) Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- apply-and-build.sh | 36

[lng-odp] [PATCH] apply-and-build.sh: cosmetic fix

2015-05-28 Thread Christophe Milard
function apply_patch() now returns a status rather than directely breaking the loop of its caller. (for loop in main, continue statment in apply_patch()) Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- apply-and-build.sh | 9 +++-- 1 file changed, 7 insertions(+), 2

Re: [lng-odp] [PATCHv3 2/2] validation: init tests using common main

2015-05-25 Thread Christophe Milard
Hi, Any reviewer for this? Christophe. On 20 May 2015 at 18:11, Christophe Milard christophe.mil...@linaro.org wrote: The 3 init tests (init, abort,log) now links with common/odp_cunit_common, as other tests. In main, ODP init is now performed via weak functions which

[lng-odp] [PATCHv3 0/2] validation: init tests gathered and using common main

2015-05-20 Thread Christophe Milard
checkpatch fixes since v2. Christophe Milard (2): validation: init src files in single directory validation: init tests using common main test/validation/Makefile.am | 6 ++-- test/validation/common/odp_cunit_common.c | 45 ++--- test/validation

[lng-odp] [PATCHv3 1/2] validation: init src files in single directory

2015-05-20 Thread Christophe Milard
The 3 tests for initialisation still generate 3 executables, but their sources are simply gathered in a single directory called as the module itself, init. And other cosmetic fixes to please check-odp. Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- test/validation/Makefile.am

[lng-odp] [PATCHv3 2/2] validation: init tests using common main

2015-05-20 Thread Christophe Milard
The 3 init tests (init, abort,log) now links with common/odp_cunit_common, as other tests. In main, ODP init is now performed via weak functions which are overloaded (to do nothing) by the 3 init tests. And other cosmetic fixes to please check-odp. Signed-off-by: Christophe

Re: [lng-odp] [PATCHv2 2/2] validation: init tests using common main

2015-05-19 Thread Christophe Milard
On 2015-05-07 18:09, Mike Holmes wrote: On 5 May 2015 at 12:31, Christophe Milard christophe.mil...@linaro.org wrote: This is the one I'd like to be reviewed... (v2) sorry for the confusion. On 27 April 2015 at 17:54, Christophe Milard christophe.mil...@linaro.org wrote: The 3

[lng-odp] [PATCH] test: helper: add process and thread tests

2015-05-13 Thread Christophe Milard
On 2015-05-12 17:32, Mike Holmes wrote: The helpers need to be tested independently from the ODP API, create a folder to contain helper tests, adding tests for process and thread creation. Signed-off-by: Mike Holmes mike.hol...@linaro.org Reviewed-by: Christophe Milard christophe.mil

Re: [lng-odp] [PATCH 0/3] validation: changing module random

2015-05-13 Thread Christophe Milard
On 12 May 2015 at 19:28, Stuart Haslam stuart.has...@linaro.org wrote: On Tue, May 12, 2015 at 09:02:18AM +0200, Christophe Milard wrote: On 11 May 2015 at 20:23, Stuart Haslam stuart.has...@linaro.org wrote: On Mon, May 11, 2015 at 12:48:24PM +0200, Christophe Milard wrote: Hopefully

Re: [lng-odp] [PATCH 0/3] validation: changing module random

2015-05-12 Thread Christophe Milard
On 11 May 2015 at 20:23, Stuart Haslam stuart.has...@linaro.org wrote: On Mon, May 11, 2015 at 12:48:24PM +0200, Christophe Milard wrote: Hopefully according to the agreement of last week. The changes are applyed to module random, one of the simplest. Please review carefully as all modules

[lng-odp] [PATCH 0/3] validation: changing module random

2015-05-11 Thread Christophe Milard
as to keep the make check as usual. The plan is to call all the executable from the platform side in one go in a final patch, when all the modules are ready. Christophe Milard (3): validation: preparing for main in tests validation: own main and renaming in odp_random.c validation: creating

[lng-odp] [PATCH 3/3] validation: creating own dir and lib for random

2015-05-11 Thread Christophe Milard
Module random now gets its own directory and create its own lib (currentely only containing its executable) Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- configure.ac | 1 + test/validation/.gitignore| 1 - test

[lng-odp] [PATCH 2/3] validation: own main and renaming in odp_random.c

2015-05-11 Thread Christophe Milard
-by: Christophe Milard christophe.mil...@linaro.org --- test/validation/Makefile.am | 1 + test/validation/odp_random.c | 21 - 2 files changed, 17 insertions(+), 5 deletions(-) diff --git a/test/validation/Makefile.am b/test/validation/Makefile.am index 7470d9d..6ddda79

[lng-odp] [PATCH 1/3] validation: preparing for main in tests

2015-05-11 Thread Christophe Milard
to be removed. Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- test/validation/common/odp_cunit_common.c | 20 +++- test/validation/common/odp_cunit_common.h | 3 +++ 2 files changed, 14 insertions(+), 9 deletions(-) diff --git a/test/validation/common

Re: [lng-odp] validation: running the tests from the platform side (again :-) )

2015-05-07 Thread Christophe Milard
On 6 May 2015 at 22:48, Mike Holmes mike.hol...@linaro.org wrote: On 6 May 2015 at 11:26, Christophe Milard christophe.mil...@linaro.org wrote: Hi, Yet another try to reach a acceptable solution regarding the test environment, following tuesday meeting. Here comes the n+2th proposal

[lng-odp] validation: running the tests from the platform side (again :-) )

2015-05-06 Thread Christophe Milard
Hi, Yet another try to reach a acceptable solution regarding the test environment, following tuesday meeting. Here comes the n+2th proposal...:-) The goal is now to improve this proposal with your comments, and then finally pick up the best alternative between the proposal from last week (running

[lng-odp] [PATCHv5 1/2] validation: rename executable to match modules

2015-05-05 Thread Christophe Milard
Just the executables whose contents already matched a module have been renamed. Module names are taken from doxygens defgroup, with the exepection of pktio, as of Maxims requirement. Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- test/validation/.gitignore

Re: [lng-odp] validation: init tests using common main

2015-05-05 Thread Christophe Milard
OK sorry. The link point to version1. On 5 May 2015 at 18:06, Christophe Milard christophe.mil...@linaro.org wrote: This patch https://lists.linaro.org/pipermail/lng-odp/2015-April/011266.html has been on the list since the 27/4, following a few comments from Mike...Anyone willing to review

[lng-odp] [PATCHv5 0/2] validation: renaming and regrouping tests

2015-05-05 Thread Christophe Milard
Same as version 4 but excluding the renaming of the pktio module, which remains, for the time being, inconsistent with the doxygen defgroup (as required by Maxim). Christophe Milard (2): validation: rename executable to match modules validation: tests for module odp_ver_abt_log_dbg test

[lng-odp] [PATCHv5 2/2] validation: tests for module odp_ver_abt_log_dbg

2015-05-05 Thread Christophe Milard
Created directory for module odp_ver_abt_log_dbg, populated with file odp_errno.c and odp_system.c which, despite its confusing name is not testing the system module but the sys* function of the odp_ver_abt_log_dbg module. Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- test

[lng-odp] validation: init tests using common main

2015-05-05 Thread Christophe Milard
This patch https://lists.linaro.org/pipermail/lng-odp/2015-April/011266.html has been on the list since the 27/4, following a few comments from Mike...Anyone willing to review it (v2)? Thanks, Christophe. ___ lng-odp mailing list lng-odp@lists.linaro.org

[lng-odp] Some reading before the validation meeting at 14 (utc+1)

2015-05-05 Thread Christophe Milard
Those who feel they need some background for the discussion can keep reading. This is more like a status mail. Mike and Stuart can stop reading here :-) The meeting has two main goals: -decide what to do with the test environment. -decide where to place the helper tests (test/helpers vs

Re: [lng-odp] [PATCHv4 1/2] validation: rename executable to match modules

2015-05-05 Thread Christophe Milard
On 5 May 2015 at 11:15, Maxim Uvarov maxim.uva...@linaro.org wrote: 2 comments bellow On 04/27/2015 11:53, Christophe Milard wrote: Just the executables whose contents already matched a module have been renamed. Module names are taken from doxygens defgroup Signed-off-by: Christophe

[lng-odp] proposal for test enviroment

2015-04-29 Thread Christophe Milard
It looks that Stuart and I now agree on what we hope would be an improvment of the test environment. I guess you (Maxim, Mike, Anders) have been following the conversation, but here comes some summary. Does this looks reasonable for you as well? Christophe. 1) validation would contain 1

[lng-odp] validation: one directory per module?

2015-04-28 Thread Christophe Milard
Hi, As we go towards doing a single lib with all tests (according to Mikes latest proposal), there will be a need for having at least 2 C files for each module: One for linking to the common C lib and another containing the main for that module To this, one can add at least one executable per

[lng-odp] [PATCHv4 1/2] validation: rename executable to match modules

2015-04-27 Thread Christophe Milard
Just the executables whose contents already matched a module have been renamed. Module names are taken from doxygens defgroup Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- v3/v4 delta: fixed odp_scheduler.c warnings (had too old version of check-odp... thanks Mike) test

Re: [lng-odp] [PATCH 2/2] validation: init tests using common main

2015-04-27 Thread Christophe Milard
On 24 April 2015 at 20:39, Mike Holmes mike.hol...@linaro.org wrote: On 24 April 2015 at 08:54, Christophe Milard christophe.mil...@linaro.org wrote: The 3 init tests (init, abort,log) now links with common/odp_cunit_common, as other tests. In main, ODP init is now performed via weak

[lng-odp] [PATCHv2 1/2] validation: init src files in single directory

2015-04-27 Thread Christophe Milard
The 3 tests for initialisation still generate 3 executables, but their sources are simply gathered in a single directory called as the module itself, init. Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- test/validation/Makefile.am | 6 +++--- test/validation

[lng-odp] Adding test attributes

2015-04-27 Thread Christophe Milard
/unset starting at a certain suite in hierarchy and downwards. Christophe Milard. ___ lng-odp mailing list lng-odp@lists.linaro.org https://lists.linaro.org/mailman/listinfo/lng-odp

[lng-odp] [PATCHv3 2/2] validation: tests for module odp_ver_abt_log_dbg

2015-04-24 Thread Christophe Milard
Created directory for module odp_ver_abt_log_dbg, populated with file odp_errno.c and odp_system.c which, despite its confusing name is not testing the system module but the sys* function of the odp_ver_abt_log_dbg module. Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- same

[lng-odp] [PATCH 2/2] validation: init tests using common main

2015-04-24 Thread Christophe Milard
The 3 init tests (init, abort,log) now links with common/odp_cunit_common, as other tests. In main, ODP init is now performed via weak functions which are overloaded (to do nothing) by the 3 init tests. Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- test/validation/Makefile.am

<    10   11   12   13   14   15   16   >