[lng-odp] [API-NEXT PATCH 1/2] validation: synchro tests split into 3 groups

No functionnal changes: just code reordering to match the ODP modules. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- configure.ac|3 + platform/linux-generic/test/Makefile.am |3 + test/validation/Makefile.am |5 +-

Re: [lng-odp] Pool DMA mapping

pools, created on > different parts of the application). > > > > -Petri > > > > > > > > *From:* EXT Christophe Milard [mailto:christophe.mil...@linaro.org] > *Sent:* Wednesday, December 16, 2015 5:39 PM > *To:* Petri Savolainen; Ola Liljedahl

Re: [lng-odp] Pool DMA mapping

wrote: > Why the driver must map pools one by one? Is this a (legacy) kernel driver? > The driver can map one or more DMA-ble memory regions at initialization > time and this memory region should be used for all PACKET type pools. > > Alex > > On 17 December 2015 at 15:15, Christoph

[lng-odp] Pool DMA mapping

Hi, Following the discussion at the ARCH call, today, would it be reasonable to require that all packet pools that can be used by a NIC have to be created before the related nic pktio is opened? -This is implicitly required in RX, as the pool handle from which buffer should be allocated in RX is

[lng-odp] [PATCH 1/2] validation: synchro tests split into 3 groups

No functionnal changes: just code reordering to match the ODP modules. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- configure.ac|3 + platform/linux-generic/test/Makefile.am |3 + test/validation/Makefile.am |5 +-

[lng-odp] [PATCH 2/2] validation: removing synchronizers tests

Now redundant with atomic, barrier and lock tests. Signed-off-by: Christophe Milard <christophe.mil...@linaro.org> --- configure.ac |1 - platform/linux-generic/test/Makefile.am|1 - test/validation/Makef

[lng-odp] wrong event after crypto...?

Hi, A colleague at Ericsson is having the following issue. It sounds quite similar to the one I have reported earlyer (https://patches.linaro.org/49661), But I have honestlely not digged deeply into it... Bill, does it ring a bell to you? /Christophe. >Here is a short description of the ODP bug

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

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

[lng-odp] pktio selection

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,

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

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

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

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

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

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

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

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

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

[lng-odp] new test directory structure?

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?

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] new test directory structure?

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] [API-NEXT PATCH v3 3/7] api: doc: re-organize doxygen doc for synchronizer

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

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

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

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

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

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

. 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] [PATCHv2 5/5] validation: update README

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

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

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

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

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

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

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

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

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

[lng-odp] [PATCHv3] validation: creation of the test superlib

libtestodp.la contains all platform agnostic test functions and symbols defined testodp.h (from all modules) It is meant to be used by ODP devloppers to pick up the tests they need (and disgard others) during the devlopment process on their platform. Signed-off-by: Christophe Milard

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

On 2015-09-16 10:42, 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] [PATCH 1/4] validation: add ability to mark tests inactive

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] [PATCH 2/4] validation: add odp_cunit_update() to modify registered tests

On 2015-09-16 10:42, Stuart Haslam wrote: > Add an odp_cunit_update() function to allow some attributes of previously > registered tests to be modified, such as whether it's active or not. > > Previously, registering and running tests was done in a single step; > > odp_cunit_run(testsuites); >

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

On 2015-09-16 10:42, 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 | 58 > +- > 1

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

On 2015-09-16 10:41, Stuart Haslam wrote: > This series adds the ability to skip individual test cases either at > compile time or conditionally at run time. There's no functional change > since none of the existing tests use this facility. > > There are a few checkpatch warnings about camel case

Re: [lng-odp] [PATCH 2/4] validation: add odp_cunit_update() to modify registered tests

On 16 September 2015 at 18:07, Stuart Haslam <stuart.has...@linaro.org> wrote: > On Wed, Sep 16, 2015 at 05:03:38PM +0200, Christophe Milard wrote: > > On 2015-09-16 10:42, Stuart Haslam wrote: > > > Add an odp_cunit_update() function to allow some attributes of >

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

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

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

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: renaming module libs

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

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

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

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

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] validation: init: fix test when debug-print are disabled

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] [PATCHv2 08/13] validation: scheduler: using _CU_TEST_INFO()

Signed-off-by: Christophe Milard christophe.mil...@linaro.org Reviewed-by: Stuart Haslam stuart.has...@linaro.org --- test/validation/scheduler/scheduler.c | 60 +-- 1 file changed, 30 insertions(+), 30 deletions(-) diff --git a/test/validation/scheduler

[lng-odp] [PATCHv2 12/13] validation: time: using _CU_TEST_INFO()

Signed-off-by: Christophe Milard christophe.mil...@linaro.org Reviewed-by: Stuart Haslam stuart.has...@linaro.org --- test/validation/time/time.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/test/validation/time/time.c b/test/validation/time/time.c index 4fa2eaf

Re: [lng-odp] [PATCHv2 00/13] validation: use macro _CU_TEST_INFO() in all tests

since v1, I mean... :-) On 20 August 2015 at 12:09, Christophe Milard christophe.mil...@linaro.org wrote: Just rebased since v2 On 20 August 2015 at 12:05, Christophe Milard christophe.mil...@linaro.org wrote: This is another step in getting the test environment more homogeneous

[lng-odp] [PATCHv2 01/13] validation: factorise _CU_TEST_INFO definition

The macro _CU_TEST_INFO (used ensure a match between test function names and test names (strings)) is now defined in .../common/odp_cunit_common.h as it is common to many API module tests. Local definitions are also removed. Signed-off-by: Christophe Milard christophe.mil...@linaro.org Reviewed

[lng-odp] [PATCHv2 02/13] validation: cpumask: using _CU_TEST_INFO()

Signed-off-by: Christophe Milard christophe.mil...@linaro.org Reviewed-by: Stuart Haslam stuart.has...@linaro.org --- test/validation/cpumask/cpumask.c | 36 ++-- 1 file changed, 18 insertions(+), 18 deletions(-) diff --git a/test/validation/cpumask/cpumask.c b

[lng-odp] [PATCHv2 04/13] validation: errno: using _CU_TEST_INFO()

Signed-off-by: Christophe Milard christophe.mil...@linaro.org Reviewed-by: Stuart Haslam stuart.has...@linaro.org --- test/validation/errno/errno.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/validation/errno/errno.c b/test/validation/errno/errno.c index 9b6b125

[lng-odp] [PATCHv2 00/13] validation: use macro _CU_TEST_INFO() in all tests

This is another step in getting the test environment more homogeneous. The usage of this macro guaranties that the test name (string in C_unit) will always match the function name. This is needed as the C_unit strings will be used towards the C_unit API (to skip a test, for instance). Christophe

[lng-odp] [PATCHv2 05/13] validation: init: using _CU_TEST_INFO()

Signed-off-by: Christophe Milard christophe.mil...@linaro.org Reviewed-by: Stuart Haslam stuart.has...@linaro.org --- test/validation/init/init.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/test/validation/init/init.c b/test/validation/init/init.c index 12aa554

[lng-odp] [PATCHv2 10/13] validation: synchronizers: using _CU_TEST_INFO()

Signed-off-by: Christophe Milard christophe.mil...@linaro.org Reviewed-by: Stuart Haslam stuart.has...@linaro.org --- test/validation/synchronizers/synchronizers.c | 26 +- 1 file changed, 13 insertions(+), 13 deletions(-) diff --git a/test/validation/synchronizers

[lng-odp] [PATCHv2 09/13] validation: shmem: using _CU_TEST_INFO()

Signed-off-by: Christophe Milard christophe.mil...@linaro.org Reviewed-by: Stuart Haslam stuart.has...@linaro.org --- test/validation/shmem/shmem.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/validation/shmem/shmem.c b/test/validation/shmem/shmem.c index 3abba0c

[lng-odp] [PATCHv2 03/13] validation: crypto: using _CU_TEST_INFO()

Signed-off-by: Christophe Milard christophe.mil...@linaro.org Reviewed-by: Stuart Haslam stuart.has...@linaro.org --- test/validation/crypto/odp_crypto_test_inp.c | 18 ++ 1 file changed, 6 insertions(+), 12 deletions(-) diff --git a/test/validation/crypto/odp_crypto_test_inp.c

[lng-odp] [PATCHv2 13/13] validation: timer: using _CU_TEST_INFO()

Signed-off-by: Christophe Milard christophe.mil...@linaro.org Reviewed-by: Stuart Haslam stuart.has...@linaro.org --- test/validation/timer/timer.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/test/validation/timer/timer.c b/test/validation/timer/timer.c index

[lng-odp] [PATCHv2 11/13] validation: system: using _CU_TEST_INFO()

Signed-off-by: Christophe Milard christophe.mil...@linaro.org Reviewed-by: Stuart Haslam stuart.has...@linaro.org --- test/validation/system/system.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/test/validation/system/system.c b/test/validation/system

[lng-odp] [PATCHv2 07/13] validation: queue: using _CU_TEST_INFO()

Signed-off-by: Christophe Milard christophe.mil...@linaro.org Reviewed-by: Stuart Haslam stuart.has...@linaro.org --- test/validation/queue/queue.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/validation/queue/queue.c b/test/validation/queue/queue.c index 5b2a13a

[lng-odp] [PATCHv2 06/13] validation: pktio: using _CU_TEST_INFO()

Signed-off-by: Christophe Milard christophe.mil...@linaro.org Reviewed-by: Stuart Haslam stuart.has...@linaro.org --- test/validation/pktio/pktio.c | 24 1 file changed, 12 insertions(+), 12 deletions(-) diff --git a/test/validation/pktio/pktio.c b/test/validation/pktio

Re: [lng-odp] [PATCH] linux-generic: move cpumask common functions to separate file

Hi, There is another aspect of this which is a bit unfortunate: In the validation, what is regarded as common regarding cpumask is what is common to both cpumasks and threadmask. The file test/validation/common/mask_common.c contains tests that are common to both threadmask and cpumask. In the

Re: [lng-odp] [API-NEXT] validation: classification: added additional suite to test individual PMRs

I am sorry, as this is not well documented yet (I am currentely working on the documentation), but this patch does not respect the naming convention for test function, test-suite... Possibly you started your work on this before my modification went in... Anyway here follows the naming convention

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

Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- test/README| 8 +--- test/validation/README | 50 ++ 2 files changed, 51 insertions(+), 7 deletions(-) create mode 100644 test/validation/README diff --git a/test

[lng-odp] [PATCHv2] validation: creation of the test superlib

libodptests.la contains all platform agnostic test functions and symbols defined odptests.h (from all modules) It is meant to be used by ODP devloppers to pick up the tests they need (and disgard others) during the devlopment process on their platform. Signed-off-by: Christophe Milard

[lng-odp] [PATCH] validation: cpumask: fixed typo

Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- test/validation/cpumask/cpumask.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/validation/cpumask/cpumask.c b/test/validation/cpumask/cpumask.c index 76b7648..90d1150 100644 --- a/test/validation

[lng-odp] [PATCH 07/13] validation: queue: using _CU_TEST_INFO()

Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- test/validation/queue/queue.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/validation/queue/queue.c b/test/validation/queue/queue.c index ef7c8f2..e9a3711 100644 --- a/test/validation/queue/queue.c +++ b

[lng-odp] [PATCH 05/13] validation: init: using _CU_TEST_INFO()

Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- test/validation/init/init.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/test/validation/init/init.c b/test/validation/init/init.c index 169bd5c..afdb316 100644 --- a/test/validation/init/init.c +++ b

[lng-odp] [PATCH 04/13] validation: errno: using _CU_TEST_INFO()

Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- test/validation/errno/errno.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/validation/errno/errno.c b/test/validation/errno/errno.c index ac1d93c..85b800c 100644 --- a/test/validation/errno/errno.c +++ b

[lng-odp] [PATCH 01/13] validation: factorise _CU_TEST_INFO definition

The macro _CU_TEST_INFO (used ensure a match between test function names and test names (strings)) is now defined in .../common/odp_cunit_common.h as it is common to many API module tests. Local definitions are also removed. Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- test

[lng-odp] [PATCH 00/13] validation: use macro _CU_TEST_INFO() in all tests

This is another step in getting the test environment more homogeneous. The usage of this macro guaranties that the test name (string in C_unit) will always match the function name. This is needed as the C_unit strings will be used towards the C_unit API (to skip a test, for instance). Christophe

[lng-odp] [PATCH 09/13] validation: shmem: using _CU_TEST_INFO()

Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- test/validation/shmem/shmem.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/validation/shmem/shmem.c b/test/validation/shmem/shmem.c index 0c713c6..2780f21 100644 --- a/test/validation/shmem/shmem.c +++ b

[lng-odp] [PATCH 08/13] validation: scheduler: using _CU_TEST_INFO()

Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- test/validation/scheduler/scheduler.c | 60 +-- 1 file changed, 30 insertions(+), 30 deletions(-) diff --git a/test/validation/scheduler/scheduler.c b/test/validation/scheduler/scheduler.c index

[lng-odp] [PATCH 06/13] validation: pktio: using _CU_TEST_INFO()

Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- test/validation/pktio/pktio.c | 24 1 file changed, 12 insertions(+), 12 deletions(-) diff --git a/test/validation/pktio/pktio.c b/test/validation/pktio/pktio.c index bfdf525..0b9325b 100644 --- a/test

[lng-odp] [PATCH 10/13] validation: synchronizers: using _CU_TEST_INFO()

Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- test/validation/synchronizers/synchronizers.c | 26 +- 1 file changed, 13 insertions(+), 13 deletions(-) diff --git a/test/validation/synchronizers/synchronizers.c b/test/validation/synchronizers

[lng-odp] [PATCH 11/13] validation: system: using _CU_TEST_INFO()

Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- test/validation/system/system.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/test/validation/system/system.c b/test/validation/system/system.c index b80fd39..ac70762 100644 --- a/test

[lng-odp] [PATCH 12/13] validation: time: using _CU_TEST_INFO()

Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- test/validation/time/time.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/test/validation/time/time.c b/test/validation/time/time.c index 0aac599..f6e2906 100644 --- a/test/validation/time/time.c +++ b

[lng-odp] [PATCH 13/13] validation: timer: using _CU_TEST_INFO()

Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- test/validation/timer/timer.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/test/validation/timer/timer.c b/test/validation/timer/timer.c index ecca785..6e2af24 100644 --- a/test/validation/timer

[lng-odp] [PATCH 02/13] validation: cpumask: using _CU_TEST_INFO()

Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- test/validation/cpumask/cpumask.c | 36 ++-- 1 file changed, 18 insertions(+), 18 deletions(-) diff --git a/test/validation/cpumask/cpumask.c b/test/validation/cpumask/cpumask.c index 76b7648

[lng-odp] [PATCH 03/13] validation: crypto: using _CU_TEST_INFO()

Signed-off-by: Christophe Milard christophe.mil...@linaro.org --- test/validation/crypto/odp_crypto_test_inp.c | 18 ++ 1 file changed, 6 insertions(+), 12 deletions(-) diff --git a/test/validation/crypto/odp_crypto_test_inp.c b/test/validation/crypto/odp_crypto_test_inp.c index

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

-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] [PATCHv3 03/20] validation: errno: lib populated

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] [PATCHv3 09/20] validation: classification: lib populated

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] [PATCHv3 10/20] validation: pktio: lib populated

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] [PATCHv3 17/20] validation: time: lib populated

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] [PATCHv3 16/20] validation: timer: lib populated

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] [PATCHv3 04/20] validation: synchronizers: lib populated

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] [PATCHv3 06/20] validation: shmem: lib populated

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] [PATCHv3 12/20] validation: cpumask: lib populated

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

[lng-odp] [PATCHv3 11/20] validation: packet: lib populated

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] [PATCHv3 14/20] validation: thread: lib populated

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

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

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

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

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

Re: [lng-odp] ODP timer behavior

I wild guess you are using a 32 bit value and 10*ODP_TIME_SEC does not fit. (please confirm) Christophe On 27 July 2015 at 18:46, Krishna Garapati balakrishna.garap...@linaro.org wrote: Hi, As part odp_generator verbose update, I am trying to use 10*ODP_TIME_SEC ( Second in nsec -

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

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] [PATCHv3 08/20] validation: buffer: lib populated

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] [PATCHv3 19/20] validation: system: lib populated

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] [PATCHv3 01/20] validation: errno: adding missing include

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] [PATCHv3 00/20] validation: populating libraries

Since v2: fix spell typo and rebased 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

<    7   8   9   10   11   12   13   14   15   16   >