[lng-odp] [PATCHv2 06/20] validation: shmem: lib populated

2015-07-24 Thread Christophe Milard
test symbols are now exported in the module lib. Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- test/validation/shmem/shmem.c | 6 +++--- test/validation/shmem/shmem.h | 12 2 files changed, 15 insertions(+), 3 deletions(-) diff --git a/test/validation/shmem

[lng-odp] [PATCHv2 00/20] validation: populating libraries

2015-07-24 Thread Christophe Milard
Since v1: added the 3 first patches to work around the problem of using filename errno.h locally and populated errno lib. Also rebased. This series of patch populate the module test libraries with the related test symbols. Christophe Milard (20): validation: errno: adding missing include

[lng-odp] [PATCHv2 01/20] validation: errno: adding missing include

2015-07-24 Thread Christophe Milard
Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- test/validation/errno/errno.c | 1 + 1 file changed, 1 insertion(+) diff --git a/test/validation/errno/errno.c b/test/validation/errno/errno.c index ac1d93c..65da40f 100644 --- a/test/validation/errno/errno.c +++ b/test/validation

[lng-odp] [PATCHv2 02/20] test: removing current dir from -I

2015-07-24 Thread Christophe Milard
-by: Christophe Milard christophe.mil...@linaro.org --- test/Makefile.inc| 5 ++--- test/validation/Makefile.inc | 5 + 2 files changed, 7 insertions(+), 3 deletions(-) diff --git a/test/Makefile.inc b/test/Makefile.inc index d5b4770..2700b18 100644 --- a/test/Makefile.inc +++ b/test

[lng-odp] [PATCHv2 07/20] validation: init: lib populated

2015-07-24 Thread Christophe Milard
test symbols are now exported in the module lib. Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- test/validation/init/init.c | 18 +- test/validation/init/init.h | 18 ++ 2 files changed, 27 insertions(+), 9 deletions(-) diff --git a/test

[lng-odp] [PATCHv2 04/20] validation: synchronizers: lib populated

2015-07-24 Thread Christophe Milard
test symbols are now exported in the module lib. Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- test/validation/synchronizers/synchronizers.c | 44 +-- test/validation/synchronizers/synchronizers.h | 35 + 2 files changed, 57

[lng-odp] [PATCHv2 05/20] validation: crypto: lib populated

2015-07-24 Thread Christophe Milard
test symbols are now exported in the module lib. Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- test/validation/crypto/crypto.c | 6 +++--- test/validation/crypto/crypto.h | 24 test/validation/crypto/odp_crypto_test_inp.c

[lng-odp] [PATCHv2 13/20] validation: pool: lib populated

2015-07-24 Thread Christophe Milard
test symbols are now exported in the module lib. Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- test/validation/pool/pool.c | 12 ++-- test/validation/pool/pool.h | 16 2 files changed, 22 insertions(+), 6 deletions(-) diff --git a/test/validation

Re: [lng-odp] [API-NEXT PATCHv3 5/9] validation: split out cpumask tests into separate file

2015-07-22 Thread Christophe Milard
On 2015-07-21 14:32, Stuart Haslam wrote: Split the majority of the test cases out into a separate file in preparation for them being reused to perform thrmask tests. Signed-off-by: Stuart Haslam stuart.has...@linaro.org --- test/validation/Makefile.inc | 7 +-

Re: [lng-odp] [API-NEXT PATCHv3 6/9] validation: refactor cpumask tests

2015-07-22 Thread Christophe Milard
On 2015-07-21 14:32, Stuart Haslam wrote: Refactor the cpumask tests a bit so that they may be reused to perform the thrmask testing. Signed-off-by: Stuart Haslam stuart.has...@linaro.org Reviewed-by: Christophe Milard christophe.mil...@linaro.org --- test/validation/common/mask_common.c

Re: [lng-odp] [API-NEXT PATCHv3 4/9] validation: cpumask: def_worker and def_control fixes

2015-07-22 Thread Christophe Milard
- don't return a value greater than the requested number of CPUs Signed-off-by: Stuart Haslam stuart.has...@linaro.org Reviewed-by: Christophe Milard christophe.mil...@linaro.org --- test/validation/cpumask/cpumask.c | 44 +-- 1 file changed, 33 insertions

Re: [lng-odp] [API-NEXT PATCHv3 7/9] validation: thrmask: introduce thrmask tests

2015-07-22 Thread Christophe Milard
On 2015-07-21 14:32, Stuart Haslam wrote: Signed-off-by: Stuart Haslam stuart.has...@linaro.org --- test/validation/Makefile.inc | 1 + test/validation/common/.gitignore| 1 + test/validation/common/Makefile.am | 5 - test/validation/common/mask_common.h | 12

Re: [lng-odp] [API-NEXT PATCHv3 1/9] validation: add missing include to odp_cunit_common.h

2015-07-22 Thread Christophe Milard
On 2015-07-21 14:32, Stuart Haslam wrote: stdint.h is required for uint32_t Signed-off-by: Stuart Haslam stuart.has...@linaro.org Reviewed-by: Christophe Milard christophe.mil...@linaro.org --- test/validation/common/odp_cunit_common.h | 1 + 1 file changed, 1 insertion(+) diff --git

Re: [lng-odp] [API-NEXT PATCHv3 9/9] validation: thrmask: test worker and control APIs

2015-07-22 Thread Christophe Milard
On 2015-07-21 14:32, Stuart Haslam wrote: Add tests for odp_thrmask_worker() and odp_thrmask_control() Signed-off-by: Stuart Haslam stuart.has...@linaro.org Reviewed-by: Christophe Milard christophe.mil...@linaro.org --- test/validation/thread/thread.c | 70

Re: [lng-odp] [API-NEXT PATCHv3 2/9] validation: cpumask: use odp_cpumask_setall() to determine mask capacity

2015-07-22 Thread Christophe Milard
it. Signed-off-by: Stuart Haslam stuart.has...@linaro.org Reviewed-by: Christophe Milard christophe.mil...@linaro.org --- test/validation/cpumask/cpumask.c | 52 --- 1 file changed, 27 insertions(+), 25 deletions(-) diff --git a/test/validation/cpumask

Re: [lng-odp] [API-NEXT PATCHv3 8/9] validation: common: return number of threads created

2015-07-22 Thread Christophe Milard
On 2015-07-21 14:32, Stuart Haslam wrote: odph_linux_pthread_create() now returns the number of threads created, so modify odp_cunit_thread_create() to do the same. Signed-off-by: Stuart Haslam stuart.has...@linaro.org Reviewed-by: Christophe Milard christophe.mil...@linaro.org --- test

Re: [lng-odp] [API-NEXT PATCHv3 3/9] linux-generic: cpumask: zero mask before setting

2015-07-22 Thread Christophe Milard
On 2015-07-21 14:32, Stuart Haslam wrote: odp_cpumask_def_control() was neglecting to zero out the provided mask. Signed-off-by: Stuart Haslam stuart.has...@linaro.org Reviewed-by: Christophe Milard christophe.mil...@linaro.org --- platform/linux-generic/odp_cpumask.c | 1 + 1 file

[lng-odp] [RFC-PATCH 2/3] test: removing current dir from -I

2015-07-23 Thread Christophe Milard
-by: Christophe Milard christophe.mil...@linaro.org --- test/Makefile.inc| 5 ++--- test/validation/Makefile.inc | 2 ++ 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/test/Makefile.inc b/test/Makefile.inc index d5b4770..2700b18 100644 --- a/test/Makefile.inc +++ b/test/Makefile.inc

[lng-odp] [RFC-PATCH 3/3] validation: errno: populate lib

2015-07-23 Thread Christophe Milard
Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- test/validation/errno/errno.c | 2 +- test/validation/errno/errno.h | 12 2 files changed, 13 insertions(+), 1 deletion(-) diff --git a/test/validation/errno/errno.c b/test/validation/errno/errno.c index 65da40f

[lng-odp] [RFC-PATCH 0/3] validation: errno: populating lib

2015-07-23 Thread Christophe Milard
the convention by inserting some prefix to the names... Nothing we really want to do... Christophe Milard (3): validation: errno: adding missing include test: removing current dir from -I validation: errno: populate lib test/Makefile.inc | 5 ++--- test/validation/Makefile.inc | 2

[lng-odp] [RFC-PATCH 1/3] validation: errno: adding missing include

2015-07-23 Thread Christophe Milard
Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- test/validation/errno/errno.c | 1 + 1 file changed, 1 insertion(+) diff --git a/test/validation/errno/errno.c b/test/validation/errno/errno.c index ac1d93c..65da40f 100644 --- a/test/validation/errno/errno.c +++ b/test/validation

Re: [lng-odp] [API-NEXT PATCHv4 06/10] validation: split out cpumask tests into separate file

2015-07-23 Thread Christophe Milard
On 2015-07-22 16:46, Stuart Haslam wrote: Split the majority of the test cases out into a separate file in preparation for them being reused to perform thrmask tests. Signed-off-by: Stuart Haslam stuart.has...@linaro.org Reviewed-by: Christophe Milard christophe.mil...@linaro.org

Re: [lng-odp] [API-NEXT PATCHv4 08/10] validation: thrmask: introduce thrmask tests

2015-07-23 Thread Christophe Milard
On 2015-07-22 16:46, Stuart Haslam wrote: Signed-off-by: Stuart Haslam stuart.has...@linaro.org Reviewed-by: Christophe Milard christophe.mil...@linaro.org --- test/validation/Makefile.inc | 1 + test/validation/common/.gitignore| 1 + test/validation/common/Makefile.am

[lng-odp] [PATCHv2 12/20] validation: cpumask: lib populated

2015-07-24 Thread Christophe Milard
test symbols are now exported in the module lib. Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- test/validation/cpumask/cpumask.c | 32 test/validation/cpumask/cpumask.h | 25 + 2 files changed, 41 insertions(+), 16

[lng-odp] [PATCHv2 15/20] validation: scheduler: lib populated

2015-07-24 Thread Christophe Milard
test symbols are now exported in the module lib. Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- test/validation/scheduler/scheduler.c | 68 +-- test/validation/scheduler/scheduler.h | 45 +++ 2 files changed, 79 insertions

[lng-odp] [PATCHv2 16/20] validation: timer: lib populated

2015-07-24 Thread Christophe Milard
test symbols are now exported in the module lib. Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- test/validation/timer/timer.c | 12 ++-- test/validation/timer/timer.h | 15 +++ 2 files changed, 21 insertions(+), 6 deletions(-) diff --git a/test/validation

[lng-odp] [PATCHv2 17/20] validation: time: lib populated

2015-07-24 Thread Christophe Milard
test symbols are now exported in the module lib. Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- test/validation/time/time.c | 8 test/validation/time/time.h | 14 ++ 2 files changed, 18 insertions(+), 4 deletions(-) diff --git a/test/validation/time

[lng-odp] [PATCHv2 18/20] validation: queue: lib populated

2015-07-24 Thread Christophe Milard
test symbols are now exported in the module lib. Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- test/validation/queue/queue.c | 10 +- test/validation/queue/queue.h | 16 2 files changed, 21 insertions(+), 5 deletions(-) diff --git a/test/validation

[lng-odp] [PATCHv2 14/20] validation: thread: lib populated

2015-07-24 Thread Christophe Milard
test symbols are now exported in the module lib. Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- test/validation/thread/thread.c | 10 +- test/validation/thread/thread.h | 14 ++ 2 files changed, 19 insertions(+), 5 deletions(-) diff --git a/test/validation

Re: [lng-odp] [PATCH] validation: add missing include guards

2015-07-21 Thread Christophe Milard
On 2015-07-17 17:25, Stuart Haslam wrote: Signed-off-by: Stuart Haslam stuart.has...@linaro.org Reviewed-by: Christophe Milard christophe.mil...@linaro.org --- test/validation/buffer/buffer.h | 5 + test/validation/classification/classification.h | 5 + test

Re: [lng-odp] [PATCH 00/17] validation: populating libraries

2015-07-21 Thread Christophe Milard
ping: shall we populatte the test libs? I though there was a need for dpdk... I guess these patches makes sense if we want this... Christophe. On 10 July 2015 at 02:44, Christophe Milard christophe.mil...@linaro.org wrote: This series of patch populate the module test libraries

Re: [lng-odp] Will be on vacation fr 22/07 - 27/07

2015-07-21 Thread Christophe Milard
Krishna, you should update the linaro leave calendar (if not already done) Christophe. On 21 July 2015 at 15:09, Krishna Garapati balakrishna.garap...@linaro.org wrote: Hi, I will be on vacation this week starting from tomorrow. Will be back on Monday 27th. /Krishna

Re: [lng-odp] [API-NEXT PATCH v3 3/7] api: doc: re-organize doxygen doc for synchronizer

2015-10-23 Thread Christophe Milard
October 23, 2015 3:43 PM > *To:* Christophe Milard > *Cc:* Savolainen, Petri (Nokia - FI/Espoo); lng-odp > *Subject:* Re: [lng-odp] [API-NEXT PATCH v3 3/7] api: doc: re-organize > doxygen doc for synchronizer > > > > > > > > On 23 October 2015 at 08:40, Christophe

Re: [lng-odp] [API-NEXT PATCH v3 3/7] api: doc: re-organize doxygen doc for synchronizer

2015-10-23 Thread Christophe Milard
Well, I agree that the test environment should not prevent API improvement: The test environment should simply follow the API as it moves. I see no problem with this change as long as the test environment follows it. I do see a problem, though, if it does not... What structure should the test

[lng-odp] pktio selection

2015-11-12 Thread Christophe Milard
Hi, Today, as far as I can see (at least for linux generic), the type of pktio selected at open time depends on: 1) the priority order defined in pktio_io/io_ops.c (currentely: netmap mmap socket and usual sockets) 2)A set of environment variable (ODP_PKTIO_DISABLE_NETMAP,

[lng-odp] new test directory structure?

2015-10-30 Thread Christophe Milard
Maybe time to move things around and embrace more than just the validation tests in the platform-agnostic/platform-dependent split? reason: 1)Mike is now seeing a case where he wants per-platform behaviour for the performance tests 2)Gary may see that as well in its isolation work (not really sure

Re: [lng-odp] new test directory structure?

2015-10-30 Thread Christophe Milard
On 30 October 2015 at 12:46, Stuart Haslam <stuart.has...@linaro.org> wrote: > On Fri, Oct 30, 2015 at 09:38:08AM +0100, Christophe Milard wrote: > > Maybe time to move things around and embrace more than just the > validation > > tests in the platform-agnostic/

Re: [lng-odp] [API-NEXT PATCH] validation: std_clib: added validation tests

2015-11-05 Thread Christophe Milard
This is not a ODP module, is it? should/could it be one? I am not sure either what is the point of testing libC? If the question is to make sure that libc contains such or such function, I thing we should first decide which libC to use. ( see Jira ODP 224 ) But maybe I am missing some point...

Re: [lng-odp] [API-NEXT PATCH] validation: std_clib: added validation tests

2015-11-05 Thread Christophe Milard
suffix is used to distinguish the executables belonging to the same module, if many) simply call(s) the related _main[_*] from the library. On 5 November 2015 at 11:16, Savolainen, Petri (Nokia - FI/Espoo) < petri.savolai...@nokia.com> wrote: > What rules exactly? > > > > *From:*

Re: [lng-odp] [API-NEXT PATCH] validation: std_clib: added validation tests

2015-11-05 Thread Christophe Milard
Also, this patch does not respect the test naming convention rules. /Christophe. On 5 November 2015 at 11:12, Christophe Milard <christophe.mil...@linaro.org > wrote: > but why should ODP tests test the C lib, as it is not part of ODP? > If there is a good reason to do so

Re: [lng-odp] [API-NEXT PATCH] validation: std_clib: added validation tests

2015-11-05 Thread Christophe Milard
wrote: > E.g. odp_memcpy() is not C library function, it is an ODP function that > happens to have exactly the same function prototype than memcpy(). > > > > We test ODP API call odp_memcpy(), not memcpy(). > > > > -Petri > > > > *From:* EXT Christ

Re: [lng-odp] [API-NEXT PATCH] validation: std_clib: added validation tests

2015-11-05 Thread Christophe Milard
Also, I am not sure about why we need to wrap the libC functions in ODP, how much we should do it, and how it relates to the problems experienced by Nicolas, ODP-224 Maybe somehting for the ARCH call on monday? On 5 November 2015 at 11:43, Christophe Milard <christophe.mil...@linaro.org >

Re: [lng-odp] [API-NEXT PATCH] validation: std_clib: added validation tests

2015-11-05 Thread Christophe Milard
> > > > For example, std_clib_test_memcpy(void) tests odp_memcpy() of the module > “std_clib”. > > > > > > -Petri > > > > > > *From:* EXT Christophe Milard [mailto:christophe.mil...@linaro.org] > *Sent:* Thursday, November 05, 2015 12:44 PM > &g

Re: [lng-odp] [API-NEXT PATCH] validation: std_clib: added validation tests

2015-11-05 Thread Christophe Milard
0, 11, 12, 13, 14, 15}; > + uint8_t dst[16]; > + int ret; > + > + memset(dst, 0, sizeof(dst)); > + > + *odp_memcpy*(dst, src, sizeof(dst)); > + > + ret = memcmp(dst, src, sizeof(dst)); > + > + CU_ASSERT(ret == 0); > +} >

Re: [lng-odp] [API-NEXT PATCH] validation: std_clib: added validation tests

2015-11-05 Thread Christophe Milard
00 300 296 296 180 184 176 176 > 176 176 > > 3840 340 320 320 316 316 316 208 188 184 188 > 184 188 > > 4096 348 348 332 332 332 332 204 204 192 196 > 196 192 > > > > > > > > *Fr

Re: [lng-odp] new test directory structure?

2015-10-30 Thread Christophe Milard
On 30 October 2015 at 15:56, Nicolas Morey-Chaisemartin <nmo...@kalray.eu> wrote: > > > On 10/30/2015 12:46 PM, Bill Fischofer wrote: > > A few comments: > > On Fri, Oct 30, 2015 at 3:38 AM, Christophe Milard < > <christophe.mil...@linaro.org>christophe.mi

Re: [lng-odp] [PATCHv2 5/5] validation: update README

2015-10-14 Thread Christophe Milard
On 13 October 2015 at 18:41, Stuart Haslam wrote: > Remove information that's now duplicated in the Implementers' Guide and > instead just refer to it. > > Signed-off-by: Stuart Haslam > --- > test/validation/README | 45

Re: [lng-odp] [PATCHv2 4/5] doc: implementers-guide: update section on skipping tests

2015-10-14 Thread Christophe Milard
On 14 October 2015 at 08:41, Christophe Milard <christophe.mil...@linaro.org > wrote: > > > On 13 October 2015 at 18:41, Stuart Haslam <stuart.has...@linaro.org> > wrote: > >> Update sections describing how a specific platform may skip tests by >> mar

Re: [lng-odp] [PATCHv2 4/5] doc: implementers-guide: update section on skipping tests

2015-10-14 Thread Christophe Milard
On 13 October 2015 at 18:41, Stuart Haslam wrote: > Update sections describing how a specific platform may skip tests by > marking them as inactive. > > Signed-off-by: Stuart Haslam > --- > doc/implementers-guide/implementers-guide.adoc | 104

Re: [lng-odp] [PATCHv3 3/5] doc: implementers-guide: update names of test module libraries

2015-10-14 Thread Christophe Milard
On 2015-10-14 10:28, Stuart Haslam wrote: > The name of the libraries generated by the test modules were recently > changed from lib.la to libtest.la > > Signed-off-by: Stuart Haslam <stuart.has...@linaro.org> Reviewed-by: Christophe Milard <christophe.mil...@lina

Re: [lng-odp] [PATCHv3 1/5] validation: add ability to mark tests inactive

2015-10-14 Thread Christophe Milard
d as inactive. > > Signed-off-by: Stuart Haslam <stuart.has...@linaro.org> Reviewed-by: Christophe Milard <christophe.mil...@linaro.org> > --- > test/validation/buffer/buffer.c| 20 ++-- > test/validation/buffer/buffer.h|

Re: [lng-odp] [PATCHv3 0/5] add ability to mark tests as inactive

2015-10-14 Thread Christophe Milard
I have just reviewed the all series. As a side note though, I thing a MACRO called ODP_TEST_OVERLOAD(test_func, new_test_func) should be defined if a given test function should be "replaced") using the update function. The TEST_INFO macro would not be able to be used as, for this macro, the name

[lng-odp] [CHECK-ODP PATCH 2/2] build: netmap URL changed to maintained repo

2015-10-16 Thread Christophe Milard
Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- helper/platform/generic | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/helper/platform/generic b/helper/platform/generic index 85ddacc..24a6311 100644 --- a/helper/platform/generic +++ b/helper/pl

[lng-odp] [CHECK-ODP PATCH 1/2] build: build linux-generic with netmap support

2015-10-16 Thread Christophe Milard
. Note: It does not use the netmap interface, it's as broken as it is today. Thanks to Stuart, who initiated this work. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- apply-and-build.sh | 1 - build-netmap.sh | 14 ++ build.sh| 1 +

Re: [lng-odp] [PATCHv10 8/8] linux-generic: internal ipc_pktio test

2015-10-19 Thread Christophe Milard
Shouldn't all this go in example rather than test? Christophe On 19 October 2015 at 18:08, Maxim Uvarov wrote: > 2 example ipc pktio applications create ipc pktio to each other and do > packet transfer, validation magic numbers and packets sequence counters > inside

Re: [lng-odp] [PATCHv3 0/5] add ability to mark tests as inactive

2015-10-14 Thread Christophe Milard
That is my point:I think such a macro would make the overloading clearer (more visible). And I agree it can be done later. hence my reviews :-) Christophe. On 14 October 2015 at 14:03, Stuart Haslam <stuart.has...@linaro.org> wrote: > On Wed, Oct 14, 2015 at 01:17:34PM +0200, Christop

Re: [lng-odp] [PATCH 2/3] linux-generic: pktio: add netmap pktio support

2015-10-09 Thread Christophe Milard
Hi Mike. I have read through this patch: I haven't dared marking it as reviewed as I am not sure about some part of it, especialy related to the build environment. The pktio part is very close to what I did when merging and looked fine to me. Also the debate regarding whether this should be

Re: [lng-odp] [PATCH 0/6] validation: module system created

2015-07-08 Thread Christophe Milard
sorry for the spelling errors. Have to check why this was not detected here. Fixed. Will send a Pull request with those. Christophe. On 8 July 2015 at 14:31, Stuart Haslam stuart.has...@linaro.org wrote: On Fri, Jul 03, 2015 at 01:49:33PM +0200, Christophe Milard wrote: To be applied on top

Re: [lng-odp] [PATCH 3/5] validation: cosmetic changes in odp_timer.c

2015-07-08 Thread Christophe Milard
It did warn me. This is the reason why I broke some of these lines when it looked reasonable. What do you wish me to do? Christophe. On 8 July 2015 at 14:31, Stuart Haslam stuart.has...@linaro.org wrote: On Fri, Jul 03, 2015 at 05:45:38PM +0200, Christophe Milard wrote: As preparation before

[lng-odp] [PATCHv2 3/5] validation: cosmetic changes in odp_timer.c

2015-07-08 Thread Christophe Milard
body. Also a few line over 80 char warning remains, but these lines includes strings which should not be split. Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- test/validation/odp_timer.c | 35 ++- 1 file changed, 18 insertions(+), 17 deletions

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

2015-07-08 Thread Christophe Milard
Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- configure.ac | 1 + test/validation/.gitignore | 1 - test/validation/Makefile.am| 12 ++-- test/validation/timer/.gitignore | 2

[lng-odp] [PATCHv2 2/5] validation: renaming in odp_timer.c

2015-07-08 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: Stuart Haslam

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

2015-07-08 Thread Christophe Milard
Signed-off-by: Christophe Milard christophe.mil...@linaro.org Reviewed-by: Stuart Haslam stuart.has...@linaro.org --- test/validation/Makefile.am | 2 ++ test/validation/odp_timer.c | 15 +-- 2 files changed, 15 insertions(+), 2 deletions(-) diff --git a/test/validation/Makefile.am

[lng-odp] [PATCHv2 0/5] tests for timer moved to plt side

2015-07-08 Thread Christophe Milard
from the platform side. Christophe Milard (5): validation: own main in odp_timer.c validation: renaming in odp_timer.c validation: cosmetic changes in odp_timer.c validation: creating own dir and lib for timer validation: call to timer_main from platform configure.ac

[lng-odp] [PATCHv2 5/5] validation: call to timer_main from platform

2015-07-08 Thread Christophe Milard
timer_main is now ran from the platform side. As timer 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 Reviewed-by: Stuart Haslam stuart.has

[lng-odp] Pull request: validation last modules from platform side

2015-07-08 Thread Christophe Milard
test_module_timer for you to fetch changes up to 46fc2ab3225159d2bababe0d7f020009ebbb69f0: validation: call to timer_main from platform (2015-07-08 16:18:41 +0200) Christophe Milard (20): validation: removed errno from

[lng-odp] [PATCH 06/17] validation: classification: lib populated

2015-07-09 Thread Christophe Milard
test symbols are now exported in the module lib. Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- test/validation/classification/classification.c| 2 +- test/validation/classification/classification.h| 31 ++ .../classification

[lng-odp] [PATCH 07/17] validation: pktio: lib populated

2015-07-09 Thread Christophe Milard
test symbols are now exported in the module lib. Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- test/validation/pktio/pktio.c | 32 test/validation/pktio/pktio.h | 27 +++ 2 files changed, 43 insertions(+), 16 deletions

[lng-odp] [PATCH 11/17] validation: thread: lib populated

2015-07-09 Thread Christophe Milard
test symbols are now exported in the module lib. Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- test/validation/thread/thread.c | 10 +- test/validation/thread/thread.h | 14 ++ 2 files changed, 19 insertions(+), 5 deletions(-) diff --git a/test/validation

[lng-odp] [PATCH 02/17] validation: crypto: lib populated

2015-07-09 Thread Christophe Milard
test symbols are now exported in the module lib. Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- test/validation/crypto/crypto.c | 6 +++--- test/validation/crypto/crypto.h | 24 test/validation/crypto/odp_crypto_test_inp.c

[lng-odp] [PATCH 04/17] validation: init: lib populated

2015-07-09 Thread Christophe Milard
test symbols are now exported in the module lib. Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- test/validation/init/init.c | 18 +- test/validation/init/init.h | 18 ++ 2 files changed, 27 insertions(+), 9 deletions(-) diff --git a/test

[lng-odp] [PATCH 05/17] validation: buffer: lib populated

2015-07-09 Thread Christophe Milard
test symbols are now exported in the module lib. Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- test/validation/buffer/buffer.c | 14 +++--- test/validation/buffer/buffer.h | 18 ++ 2 files changed, 25 insertions(+), 7 deletions(-) diff --git a/test

[lng-odp] [PATCH 03/17] validation: shmem: lib populated

2015-07-09 Thread Christophe Milard
test symbols are now exported in the module lib. Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- test/validation/shmem/shmem.c | 6 +++--- test/validation/shmem/shmem.h | 12 2 files changed, 15 insertions(+), 3 deletions(-) diff --git a/test/validation/shmem

[lng-odp] [PATCH 14/17] validation: time: lib populated

2015-07-09 Thread Christophe Milard
test symbols are now exported in the module lib. Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- test/validation/time/time.c | 8 test/validation/time/time.h | 14 ++ 2 files changed, 18 insertions(+), 4 deletions(-) diff --git a/test/validation/time

[lng-odp] [PATCH 13/17] validation: timer: lib populated

2015-07-09 Thread Christophe Milard
test symbols are now exported in the module lib. Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- test/validation/timer/timer.c | 12 ++-- test/validation/timer/timer.h | 15 +++ 2 files changed, 21 insertions(+), 6 deletions(-) diff --git a/test/validation

[lng-odp] [PATCH 15/17] validation: queue: lib populated

2015-07-09 Thread Christophe Milard
test symbols are now exported in the module lib. Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- test/validation/queue/queue.c | 10 +- test/validation/queue/queue.h | 16 2 files changed, 21 insertions(+), 5 deletions(-) diff --git a/test/validation

[lng-odp] [PATCH 08/17] validation: packet: lib populated

2015-07-09 Thread Christophe Milard
test symbols are now exported in the module lib. Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- test/validation/packet/packet.c | 44 - test/validation/packet/packet.h | 33 +++ 2 files changed, 55 insertions

[lng-odp] [PATCH 10/17] validation: pool: lib populated

2015-07-09 Thread Christophe Milard
test symbols are now exported in the module lib. Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- test/validation/pool/pool.c | 14 +++--- test/validation/pool/pool.h | 16 2 files changed, 23 insertions(+), 7 deletions(-) diff --git a/test/validation

[lng-odp] [PATCH 09/17] validation: cpumask: lib populated

2015-07-09 Thread Christophe Milard
test symbols are now exported in the module lib. Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- test/validation/cpumask/cpumask.c | 32 test/validation/cpumask/cpumask.h | 25 + 2 files changed, 41 insertions(+), 16

[lng-odp] [PATCH 01/17] validation: synchronizers: lib populated

2015-07-09 Thread Christophe Milard
test symbols are now exported in the module lib. Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- test/validation/synchronizers/synchronizers.c | 44 +-- test/validation/synchronizers/synchronizers.h | 35 + 2 files changed, 57

[lng-odp] [PATCH 16/17] validation: system: lib populated

2015-07-09 Thread Christophe Milard
test symbols are now exported in the module lib. Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- test/validation/system/system.c | 18 +- test/validation/system/system.h | 18 ++ 2 files changed, 27 insertions(+), 9 deletions(-) diff --git

[lng-odp] [PATCH 12/17] validation: scheduler: lib populated

2015-07-09 Thread Christophe Milard
test symbols are now exported in the module lib. Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- test/validation/scheduler/scheduler.c | 68 +-- test/validation/scheduler/scheduler.h | 45 +++ 2 files changed, 79 insertions

[lng-odp] [PATCH 17/17] validation: random: lib populated

2015-07-09 Thread Christophe Milard
test symbols are now exported in the module lib. Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- test/validation/random/random.c | 6 +++--- test/validation/random/random.h | 12 2 files changed, 15 insertions(+), 3 deletions(-) diff --git a/test/validation

[lng-odp] [PATCH 00/17] validation: populating libraries

2015-07-09 Thread Christophe Milard
This series of patch populate the module test libraries with the related test symbols. Christophe Milard (17): validation: synchronizers: lib populated validation: crypto: lib populated validation: shmem: lib populated validation: init: lib populated validation: buffer: lib populated

Re: [lng-odp] [PATCHv2 0/3] EXEEXT Fixes

2015-07-10 Thread Christophe Milard
it ... but forgot to reply: Note also that platform specific stuff should not go under linux-generic, But I feel this EXE extension will be needed by others, so I assume it makes sense to have it here. for the series: Reviewed-by: Christophe Milard christophe.mil...@linaro.org Nicolas Morey-Chaisemartin

[lng-odp] [PATCHv2] validation: cleaning up Makefile

2015-07-09 Thread Christophe Milard
The validation/Makefile.am is now simply calling the Makefile of each module and many older things can be cleaned-up from it. Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- since v1: rebased. To be applied once all modules have been converted to be called from the platform

Re: [lng-odp] [API-NEXT PATCH 2/2] validation: thread: test thrmask

2015-07-09 Thread Christophe Milard
On 8 July 2015 at 23:20, Mike Holmes mike.hol...@linaro.org wrote: Signed-off-by: Mike Holmes mike.hol...@linaro.org --- test/validation/thread/thread.c | 34 ++ 1 file changed, 34 insertions(+) diff --git a/test/validation/thread/thread.c

[lng-odp] [PATCH 1/3] validation: remove WEAK def of global_test_init

2015-07-09 Thread Christophe Milard
: odp_cunit_register_global_init(), and odp_cunit_register_global_term(); Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- test/validation/common/odp_cunit_common.c | 8 test/validation/common/odp_cunit_common.h | 6 +- test/validation/crypto/crypto.c | 6 -- test

[lng-odp] [PATCH 0/3] validation: cleaning up in cunit_common

2015-07-09 Thread Christophe Milard
: https://lists.linaro.org/pipermail/lng-odp/2015-July/013121.html Christophe Milard (3): validation: remove WEAK def of global_test_init validation: removed main function from cunit_common validation: removal of odp_testsuites test/validation/common/Makefile.am| 8 ++-- test

[lng-odp] [PATCH 3/3] validation: removal of odp_testsuites

2015-07-09 Thread Christophe Milard
The obsolete symbol odp_testsuites is no longer used and therefore removed. Suite arrays are now called Module_suites[_*], and are part of each module lib. Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- test/validation/common/odp_cunit_common.h | 7 --- 1 file changed, 7

[lng-odp] [PATCH 2/3] validation: removed main function from cunit_common

2015-07-09 Thread Christophe Milard
Now that all modules defines their own main(), the dual library builds for cunit_common (one with main(), the other without) is no longer needed. It is therefore removed. Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- test/validation/common/Makefile.am| 8

Re: [lng-odp] [PATCHv2] validation: cleaning up Makefile

2015-07-09 Thread Christophe Milard
Stuart: Can you review that obvious one as well? It belongs to the clean-up stuff as well... Thanks /Christophe. On 9 July 2015 at 09:30, Christophe Milard christophe.mil...@linaro.org wrote: The validation/Makefile.am is now simply calling the Makefile of each module and many older things

Re: [lng-odp] [PATCH 2/2] doc: userguide: add baseline overview to document

2015-11-16 Thread Christophe Milard
On 17 November 2015 at 00:42, Bill Fischofer wrote: > Add a basic ODP overview to the User's Guide, providing > an overview of ODP concepts, components, etc. Included > are a number of diagrams covering component structure > as well as packet RX, event scheduling, and

Re: [lng-odp] [PATCH] validation: init: fix test when debug-print are disabled

2015-08-31 Thread Christophe Milard
On 2015-08-27 18:47, Nicolas Morey-Chaisemartin wrote: > log_fn is never called when --disable-debug-print was passed to > configure which causes replacement_logging_used to stay at 0 > > Signed-off-by: Nicolas Morey-Chaisemartin <nmo...@kalray.eu> Reviewed-by: Christophe Mila

[lng-odp] [PATCH] validation: renaming module libs

2015-09-03 Thread Christophe Milard
Renaming module libs in /test/validation/. Module test libs are renamed from lib.la to libodpmodule.la. This is done to avoid libname clashes, such as libcrypto, which seems to confuse libtool, and should therefore fix bug ID 1787. Signed-off-by: Christophe Milard <christophe.mil...@linaro.

Re: [lng-odp] [PATCH] test: api_test: remove un-used directory

2015-09-07 Thread Christophe Milard
I agree with this Patch: I don't see any reason to keep non working code in the repos'head. This code is still in git, if we want to get it later. /Christophe. On 7 September 2015 at 09:52, Maxim Uvarov wrote: > Did you review [PATCHv8 00/12] IPC (pktio) patches as we

Re: [lng-odp] [PATCH] validation: renaming module libs

2015-09-07 Thread Christophe Milard
libraries? >>> Let's say libodcrypro.a will be in platform implementation and >>> validation tests. I think >>> there will be again problem with autotools. >>> >>> Because all this libraries are temporary and will not be packaged it's >>> reasonable

[lng-odp] [PATCHv2] validation: renaming module libs

2015-09-07 Thread Christophe Milard
Renaming module libs in /test/validation/. Module test libs are renamed from lib.la to libtestmodule.la. This is done to avoid libname clashes, such as libcrypto, which seems to confuse libtool, and should therefore fix bug ID 1787. Signed-off-by: Christophe Milard <christophe.mil...@linaro.

Re: [lng-odp] [PATCH] validation: update README files

2015-09-04 Thread Christophe Milard
ping... On 5 August 2015 at 15:20, Stuart Haslam <stuart.has...@linaro.org> wrote: > On Fri, Jul 31, 2015 at 09:56:44PM +0200, Christophe Milard wrote: > > Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> > > Reviewed-by: Stuart Hasla

Re: [lng-odp] [PATCH] validation: renaming module libs

2015-09-07 Thread Christophe Milard
xim Uvarov <maxim.uva...@linaro.org> wrote: > On 09/07/15 12:10, Christophe Milard wrote: > >> These libs are test only libs. And hopefully they will be merged in one >> global test superlib, so they will probably not be referenced much. >> In my eyes, we are just

<    1   2   3   4   5   6   7   8   9   10   >