[lng-odp] [PATCHv2] validation: added odp_cpumask test

2015-03-16 Thread Christophe Milard
Half sunny days tests for odp cpumask: At this point, odp_cpu_count() is used to get the number of cpus a odp_cpumask_t may contain. This returns probably a too conservative (small) number Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- Second try fir this matvh, with updates

[lng-odp] [PATCH] validation: fixed memory leak in odp_cpumask.c

2015-03-20 Thread Christophe Milard
Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- test/validation/odp_cpumask.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/test/validation/odp_cpumask.c b/test/validation/odp_cpumask.c index 2285034..b7e18d6 100644 --- a/test/validation/odp_cpumask.c +++ b/test

[lng-odp] [PATCHv2] validation: fixed memory leak in odp_cpumask.c

2015-03-20 Thread Christophe Milard
CID 88273: Resource leaks (RESOURCE_LEAK) Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- now with CID in comments. test/validation/odp_cpumask.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/test/validation/odp_cpumask.c b/test/validation/odp_cpumask.c index

[lng-odp] odp_cpu_count() vs odp_cpumask size

2015-03-10 Thread Christophe Milard
Hi, I am writing validation tests for the odp_cpumask functions. The question is: How many CPUs should the cpu mask be tested for?. Of course, the answer is implementation dependent, but if we want the tests to be non platform dependent, the tests should know the max number of CPU the mask could

[lng-odp] [PATCH] validation: added odp_cpumask test

2015-03-13 Thread Christophe Milard
Half sunny days tests for odp cpumask: At this point, odp_cpu_count() is used to get the number of cpus a odp_cpumask_t may contain. This returns probably a too conservative (small) number Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- test/validation/.gitignore| 1

Re: [lng-odp] [PATCH] validation: added odp_cpumask test

2015-03-13 Thread Christophe Milard
macros. But even in that variant it's ok after removing shm. Best regards, Maxim. On 03/13/15 14:19, Christophe Milard wrote: Half sunny days tests for odp cpumask: At this point, odp_cpu_count() is used to get the number of cpus a odp_cpumask_t may contain. This returns probably a too

Re: [lng-odp] [PATCH] linux-generic: odp_linux: migrate helpers to helper dir

2015-04-14 Thread Christophe Milard
Well, helpers are quite confusing to me... For instance, I don't see why the a checksum algorythm would be in helpers while a crypto would be in ODP. both are condidate for HW acceleration. The thread part should also belong to ODP. But some part make more sense, like the definition of the

[lng-odp] [PATCH] validation: tests for module odp_ver_abt_log_dbg

2015-04-23 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 --- Applies

[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

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

2015-04-24 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] [PATCHv3 1/2] validation: rename executable to match modules

2015-04-24 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 --- same as before. just rebased. test/validation/.gitignore | 6 +++--- test

Re: [lng-odp] [PATCH] validation: thread: add odp_thread sunnyday tests

2015-04-23 Thread Christophe Milard
On 2015-04-21 17:26, Mike Holmes wrote: Add test cases for APIs in the logical ODP THREAD grouping Signed-off-by: Mike Holmes mike.hol...@linaro.org --- test/validation/.gitignore | 1 + test/validation/Makefile.am | 4 +++- test/validation/odp_thread.c | 41

Re: [lng-odp] [PATCH] validation: thread: add odp_thread sunnyday tests

2015-04-23 Thread Christophe Milard
Forget my previous comment. I was just confused. What you intend to put there is limited to the ODP API (excluding helper) right? In that case, you get my blessing... :-) On 23 April 2015 at 09:36, Christophe Milard christophe.mil...@linaro.org wrote: On 2015-04-21 17:26, Mike Holmes wrote

Re: [lng-odp] [PATCH] validation: thread: add odp_thread sunnyday tests

2015-04-23 Thread Christophe Milard
On 2015-04-21 17:26, Mike Holmes wrote: Add test cases for APIs in the logical ODP THREAD grouping Signed-off-by: Mike Holmes mike.hol...@linaro.org Reviewed-by: Christophe Milard christophe.mil...@linaro.org --- test/validation/.gitignore | 1 + test/validation/Makefile.am | 4

[lng-odp] cpumask size

2015-04-20 Thread Christophe Milard
Hi, I am not sure we came to any conclusion when it comes the getting the size of the CPU mask. There was a proposal from Petri which made sense to me... /** * @return Maximum number of CPUs a mask can hold */ int odp_cpumask_max_cpus(void) But the discussion just died off... Is that

[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] [PATCHv4 2/2] validation: tests for module odp_ver_abt_log_dbg

2015-04-27 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 --- v3/v4

[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] [PATCHv2 0/2] validation: init tests gathered and using common main

2015-04-27 Thread Christophe Milard
The first patch has already been reviewed by Mike. There was comments on the second patch, hoppefully addressed here. Christophe Milard (2): validation: init src files in single directory validation: init tests using common main test/validation/Makefile.am | 6 ++-- test

[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] [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: 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] [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: 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

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] 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] [PATCHv13 3/5] validation: move l2fwd from example to performance tests

2015-04-09 Thread Christophe Milard
Hi Maxim, I am not fully following what this patch does -I appologize for this- but I am a bit worried about the usage of scripts and other environment specific usage in the tests. I am trying to restructure the test/validation tree to enventually get one single directory per ODP API module and

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] [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

[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] [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 0/8] pktio test move to platform side

2015-06-09 Thread Christophe Milard
conversion 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

[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 v2 1/3] api: cpumask: added cpumask_setall

2015-06-09 Thread Christophe Milard
On 5 June 2015 at 12:48, Petri Savolainen petri.savolai...@nokia.com 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 --- include/odp/api/cpumask.h

[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 4/7] validation: cosmetic fixes in odp_pktio.c

2015-06-03 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] [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] [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

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] [PATCH 3/5] validation: cosmetic fixes in odp_time.c

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

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

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

[lng-odp] [PATCHv2 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] [PATCHv2 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 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 2/4] validation: renaming in odp_random.c

2015-06-23 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] [PATCH 4/4] validation: call to random_main from platform

2015-06-23 Thread Christophe Milard
random_main is now ran from the platform side. As random 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 0/4] validation: tests for random moved to plt side

2015-06-23 Thread Christophe Milard
To be applied on top of patch for system (moving time to module system called from plt side). Tests for random now go through the same modification as pktio, and gets finally called from the platform side. Christophe Milard (4): validation: own main in odp_random.c validation: renaming

[lng-odp] [PATCH 1/4] validation: own main in odp_random.c

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

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

2015-06-23 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 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

  1   2   3   4   5   6   7   8   9   10   >