[lng-odp] Maintaner change

2017-03-02 Thread Nicolas Morey-Chaisemartin
Hi everyone, Just to let you know that this is my last week at Kalray. Jérome Reybert (I think he's subscribing to this ML already) will be in charge of the ODP MPPA port. If you have any questions on Kalray's port or need some feedback, please contact him from now on. Regards Nicolas

Re: [lng-odp] [PATCH 1/5] test: generator: compose sending packets from reference packet plus differences

2017-02-14 Thread Nicolas Morey-Chaisemartin
Le 02/13/2017 à 01:49 PM, Bogdan Pricope a écrit : > Signed-off-by: Bogdan Pricope > --- > example/generator/odp_generator.c | 131 > +- > 1 file changed, 114 insertions(+), 17 deletions(-) > > diff --git

[lng-odp] [MONARCH PATCH] example: generator: fix udp checksum computation

2017-02-02 Thread Nicolas Morey-Chaisemartin
odph_ipv4_udp_chksum checks for IPv4 and UDP in packet flags which were not set as the packet parse is triggered when setting the l3 offset Signed-off-by: Nicolas Morey-Chaisemartin <nmo...@kalray.eu> --- example/generator/odp_generator.c | 2 ++ 1 file changed, 2 insertions(+) diff

Re: [lng-odp] pktio: enable_loop

2017-01-24 Thread Nicolas Morey-Chaisemartin
Le 01/24/2017 à 06:11 PM, Bala Manoharan a écrit : > On 24 January 2017 at 22:25, Nicolas Morey-Chaisemartin > <nmo...@kalray.eu> wrote: >> Just a quick question about the API. >> >> Bala introduced a patch to check/enable loopback mode on interface. >>

[lng-odp] pktio: enable_loop

2017-01-24 Thread Nicolas Morey-Chaisemartin
Just a quick question about the API. Bala introduced a patch to check/enable loopback mode on interface. But the patch adds an "enable_loop" in odp_pktio_config_t and a "loop_supported" in odp_pktio_capability_t. As there is a odp_pktio_config_t in odp_pktio_capability_t which the API

Re: [lng-odp] odp struct sizes (x86_64)

2017-01-13 Thread Nicolas Morey-Chaisemartin
It's called "pahole". Should come with dwarves packages on most standard disto. Nicolas Le 01/13/2017 à 04:57 PM, Mike Holmes a écrit : > how about a link to the tool ? > > On 13 January 2017 at 10:28, Bill Fischofer > wrote: > >> On Fri, Jan 13, 2017 at 9:09 AM,

Re: [lng-odp] [PATCHv2] linux-generic: packet: fix buggy compiler error

2016-12-22 Thread Nicolas Morey-Chaisemartin
Le 12/22/2016 à 02:49 PM, Maxim Uvarov a écrit : > On debian jessie gcc unable to compile current code > with error: > odp_packet.c: In function 'odp_packet_trunc_head': > odp_packet.c:314:46: error: array subscript is above array bounds > [-Werror=array-bounds] > to->buf_hdr.seg[i].len =

Re: [lng-odp] [PATCH v2] example: traffic_mgmt: use PRIu32 instead of %u

2016-12-15 Thread Nicolas Morey-Chaisemartin
>>> On 12/13/16 19:55, Nicolas Morey-Chaisemartin wrote: >>>> Signed-off-by: Nicolas Morey-Chaisemartin <nmo...@kalray.eu> >>>> --- >>>> v2: add missing include. >>>> Should be applied to monarch_lts too >>>> >>>>

Re: [lng-odp] [PATCH 0/6] Remove Linux specifics from odp/helpers

2016-12-14 Thread Nicolas Morey-Chaisemartin
Le 12/14/2016 à 11:14 AM, Savolainen, Petri (Nokia - FI/Espoo) a écrit : > > From: Mike Holmes [mailto:mike.hol...@linaro.org] > Sent: Tuesday, December 13, 2016 2:48 PM > To: Savolainen, Petri (Nokia - FI/Espoo) > > Cc: lng-odp@lists.linaro.org >

[lng-odp] [PATCH v2] example: traffic_mgmt: use PRIu32 instead of %u

2016-12-13 Thread Nicolas Morey-Chaisemartin
Signed-off-by: Nicolas Morey-Chaisemartin <nmo...@kalray.eu> --- v2: add missing include. Should be applied to monarch_lts too example/traffic_mgmt/odp_traffic_mgmt.c | 20 1 file changed, 12 insertions(+), 8 deletions(-) diff --git a/example/traffic_mgmt/odp_traffic_

Re: [lng-odp] [PATCH] example: traffic_mgmt: use PRIu32 instead of %u

2016-12-13 Thread Nicolas Morey-Chaisemartin
hen it compiles and runs fine. Nicolas, can > you submit a v2? > > On Tue, Dec 13, 2016 at 8:51 AM, Maxim Uvarov <maxim.uva...@linaro.org> wrote: >> On 12/13/16 06:41, Bill Fischofer wrote: >>> On Tue, Dec 6, 2016 at 9:25 AM, Nicolas Morey-Chaisemartin >>> <n

Re: [lng-odp] [PATCH 6/6] helper: take default os from odp platform

2016-12-13 Thread Nicolas Morey-Chaisemartin
Le 12/13/2016 à 02:25 PM, Mike Holmes a écrit : > > > On 13 December 2016 at 03:05, Nicolas Morey-Chaisemartin <nmo...@kalray.eu > <mailto:nmo...@kalray.eu>> wrote: > > > > Le 12/12/2016 à 03:52 PM, Mike Holmes a écrit : > > Signed-o

Re: [lng-odp] [PATCH 4/6] helper: select os in configure

2016-12-13 Thread Nicolas Morey-Chaisemartin
reflecting these 2 > concepts, but hopefully this patchset is a step towards it. (at least, it > will show the problems that follows) > > Chrstophe > mppa is not an OS but I agree ;) Nicolas > > > On 13 December 2016 at 09:01, Nicolas Morey-Chaisemartin <nmo...@kalray.

Re: [lng-odp] [PATCH 6/6] helper: take default os from odp platform

2016-12-13 Thread Nicolas Morey-Chaisemartin
Le 12/12/2016 à 03:52 PM, Mike Holmes a écrit : > Signed-off-by: Mike Holmes > --- > configure.ac | 4 ++-- > platform/linux-generic/m4/configure.m4 | 2 ++ > 2 files changed, 4 insertions(+), 2 deletions(-) > > diff --git a/configure.ac

Re: [lng-odp] [PATCH 4/6] helper: select os in configure

2016-12-13 Thread Nicolas Morey-Chaisemartin
Le 12/13/2016 à 08:46 AM, Christophe Milard a écrit : > On 12 December 2016 at 15:52, Mike Holmes wrote: > >> Signed-off-by: Mike Holmes >> --- >> configure.ac | 14 ++ >> helper/Makefile.am | 2 +- >> 2 files changed, 15

Re: [lng-odp] [PATCH 1/6] helper: rename linux.h to threads.h

2016-12-12 Thread Nicolas Morey-Chaisemartin
Not really an issue for me but this patch breaks the build. You did not change the include in the helper test (at least thread.c), but they are removed in the next patch. I know you guys like your patch to build/work individually :) Nicolas Le 12/12/2016 à 03:52 PM, Mike Holmes a écrit : >

Re: [lng-odp] [PATCH v2 4/6] helper: use ODP_LIB_FLAVOR to generate libodp name

2016-12-12 Thread Nicolas Morey-Chaisemartin
from the platfrom in use via its > configure.m4 > > On 9 December 2016 at 08:45, Nicolas Morey-Chaisemartin <nmo...@kalray.eu > <mailto:nmo...@kalray.eu>> wrote: > > > > Le 12/09/2016 à 02:41 PM, Mike Holmes a écrit : >> >> >> On 9

Re: [lng-odp] [PATCH v2 4/6] helper: use ODP_LIB_FLAVOR to generate libodp name

2016-12-09 Thread Nicolas Morey-Chaisemartin
Le 12/09/2016 à 02:41 PM, Mike Holmes a écrit : > > > On 9 December 2016 at 08:33, Nicolas Morey-Chaisemartin <nmo...@kalray.eu > <mailto:nmo...@kalray.eu>> wrote: > > > > Le 12/09/2016 à 02:18 PM, Mike Holmes a écrit : >> >> >> O

Re: [lng-odp] [PATCH v2 4/6] helper: use ODP_LIB_FLAVOR to generate libodp name

2016-12-09 Thread Nicolas Morey-Chaisemartin
Le 12/09/2016 à 02:18 PM, Mike Holmes a écrit : > > > On 9 December 2016 at 04:41, Nicolas Morey-Chaisemartin <nmo...@kalray.eu > <mailto:nmo...@kalray.eu>> wrote: > > I'm OK with this. The only issue I have with this is that it moves back > platform/OS sel

Re: [lng-odp] odp_rwlock_read_trylock

2016-12-09 Thread Nicolas Morey-Chaisemartin
Le 12/09/2016 à 10:45 AM, Savolainen, Petri (Nokia - FI/Espoo) a écrit : > >> -Original Message- >> From: Nicolas Morey-Chaisemartin [mailto:nmo...@kalray.eu] >> Sent: Thursday, December 08, 2016 5:30 PM >> To: Savolainen, Petri (Nokia - FI/Espoo) &

Re: [lng-odp] [PATCH v2 4/6] helper: use ODP_LIB_FLAVOR to generate libodp name

2016-12-09 Thread Nicolas Morey-Chaisemartin
6 at 13:44, Mike Holmes <mike.hol...@linaro.org > <mailto:mike.hol...@linaro.org>> wrote: > > > > On 7 December 2016 at 01:54, Nicolas Morey-Chaisemartin <nmo...@kalray.eu > <mailto:nmo...@kalray.eu>> wrote: > > > > Le 12/06/2016 à

Re: [lng-odp] odp_rwlock_read_trylock

2016-12-08 Thread Nicolas Morey-Chaisemartin
Le 12/08/2016 à 03:50 PM, Savolainen, Petri (Nokia - FI/Espoo) a écrit : > >> -Original Message- >> From: lng-odp [mailto:lng-odp-boun...@lists.linaro.org] On Behalf Of >> Nicolas Morey-Chaisemartin >> Sent: Wednesday, December 07, 2016 10:12 AM >>

Re: [lng-odp] [PATCH] validation: packet: fix concat test

2016-12-07 Thread Nicolas Morey-Chaisemartin
meanwhile. > > -Petri > >> -Original Message- >> From: lng-odp [mailto:lng-odp-boun...@lists.linaro.org] On Behalf Of >> Nicolas Morey-Chaisemartin >> Sent: Wednesday, December 07, 2016 11:01 AM >> To: lng-odp@lists.linaro.org >> Subject: [lng-odp] [PA

[lng-odp] [PATCH] validation: classification: increase SHM_PKT_BUF_SIZE

2016-12-07 Thread Nicolas Morey-Chaisemartin
classification_test_pmr_term_packet_len allocs a packet with 1024 of payload + headers which is greater than the segsize used to allocate the pool and breaks on implementation without segmentation support Signed-off-by: Nicolas Morey-Chaisemartin <nmo...@kalray.eu> --- Should be

[lng-odp] [PATCH] validation: packet: fix concat test

2016-12-07 Thread Nicolas Morey-Chaisemartin
concat test assumes that segmentation is supported. Add an extra concat_test_packet that can fit twice into seglen if segmentation is not supported. Signed-off-by: Nicolas Morey-Chaisemartin <nmo...@kalray.eu> --- The patch should be applied on monarch_lts too. apart from the path

Re: [lng-odp] [PATCH v2 4/6] helper: use ODP_LIB_FLAVOR to generate libodp name

2016-12-07 Thread Nicolas Morey-Chaisemartin
les or > helper code? > > On 7 December 2016 at 10:43, Nicolas Morey-Chaisemartin <nmo...@kalray.eu > <mailto:nmo...@kalray.eu>> wrote: > > I usually force somme access to be uncached. A whole barrier would work, > but the performance cost is way too big. > >

[lng-odp] odp_rwlock_read_trylock

2016-12-07 Thread Nicolas Morey-Chaisemartin
HI, While looking into the test/code, I noticed something weird in the implementation of the rwlock read trylock on monarch. int odp_rwlock_read_trylock(odp_rwlock_t *rwlock) { uint32_t zero = 0; return odp_atomic_cas_acq_u32(>cnt, , (uint32_t)1); } This mean the trylock only

Re: [lng-odp] [PATCH v2 4/6] helper: use ODP_LIB_FLAVOR to generate libodp name

2016-12-06 Thread Nicolas Morey-Chaisemartin
what do you do to solve coherency problem? I think that maybe we are > missing some api in odp, like read/write barriers. > > Maxim. > > On 7 December 2016 at 09:54, Nicolas Morey-Chaisemartin <nmo...@kalray.eu > <mailto:nmo...@kalray.eu>> wrote: > > &g

Re: [lng-odp] [PATCH v2 4/6] helper: use ODP_LIB_FLAVOR to generate libodp name

2016-12-06 Thread Nicolas Morey-Chaisemartin
Le 12/06/2016 à 09:37 PM, Mike Holmes a écrit : > On 6 December 2016 at 12:08, Nicolas Morey-Chaisemartin > <nmo...@kalray.eu> wrote: >> Signed-off-by: Nicolas Morey-Chaisemartin <nmo...@kalray.eu> >> --- >> helper/Makefile.am | 6 +++--- >> help

Re: [lng-odp] [PATCH v2 1/6] m4: add platform configurable library suffixes

2016-12-06 Thread Nicolas Morey-Chaisemartin
Le 12/06/2016 à 09:25 PM, Mike Holmes a écrit : > On 6 December 2016 at 12:05, Nicolas Morey-Chaisemartin > <nmo...@kalray.eu> wrote: >> This is the suffix to be used to compute the name of ODP >> and ODP helper libraries. >> platform can set a ODP_LIB

Re: [lng-odp] [PATCH v2 4/6] helper: use ODP_LIB_FLAVOR to generate libodp name

2016-12-06 Thread Nicolas Morey-Chaisemartin
Le 12/06/2016 à 10:17 PM, Maxim Uvarov a écrit : > On 12/06/16 23:37, Mike Holmes wrote: >> On 6 December 2016 at 12:08, Nicolas Morey-Chaisemartin >> <nmo...@kalray.eu> wrote: >>> Signed-off-by: Nicolas Morey-Chaisemartin <nmo...@kalray.eu> >

[lng-odp] [PATCH v2 5/6] example: use ODP_LIB_FLAVOR to generate libodp name

2016-12-06 Thread Nicolas Morey-Chaisemartin
Signed-off-by: Nicolas Morey-Chaisemartin <nmo...@kalray.eu> --- example/Makefile.inc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/example/Makefile.inc b/example/Makefile.inc index 19d3994..62ead02 100644 --- a/example/Makefile.inc +++ b/example/Makefile.inc @@ -1,

[lng-odp] [PATCH v2 6/6] platform: use ODP_LIB_FLAVOR to generate libodp name

2016-12-06 Thread Nicolas Morey-Chaisemartin
Signed-off-by: Nicolas Morey-Chaisemartin <nmo...@kalray.eu> --- platform/Makefile.inc | 6 +++--- platform/linux-generic/Makefile.am | 4 ++-- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/platform/Makefile.inc b/platform/Makefile.inc index 2722946..a37fd10

[lng-odp] [PATCH v2 3/6] test: use ODP_LIB_FLAVOR to generate libodp name

2016-12-06 Thread Nicolas Morey-Chaisemartin
Signed-off-by: Nicolas Morey-Chaisemartin <nmo...@kalray.eu> --- test/Makefile.inc| 2 +- test/common_plat/validation/api/Makefile.inc | 2 +- test/linux-generic/Makefile.inc | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git

[lng-odp] [PATCH v2 4/6] helper: use ODP_LIB_FLAVOR to generate libodp name

2016-12-06 Thread Nicolas Morey-Chaisemartin
Signed-off-by: Nicolas Morey-Chaisemartin <nmo...@kalray.eu> --- helper/Makefile.am | 6 +++--- helper/test/Makefile.am | 6 +++--- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/helper/Makefile.am b/helper/Makefile.am index d09d900..c75db00 100644 --- a/helper/Makef

[lng-odp] [PATCH v2 2/6] platform: move pkgconfig include to linux-generic

2016-12-06 Thread Nicolas Morey-Chaisemartin
Signed-off-by: Nicolas Morey-Chaisemartin <nmo...@kalray.eu> --- configure.ac | 2 -- platform/linux-generic/m4/configure.m4 | 4 +++- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/configure.ac b/configure.ac index 01d6448..0e1513f

[lng-odp] [PATCH v2 1/6] m4: add platform configurable library suffixes

2016-12-06 Thread Nicolas Morey-Chaisemartin
This is the suffix to be used to compute the name of ODP and ODP helper libraries. platform can set a ODP_LIB_SUFFIX which is then exported as ODP_LIB_FLAVOR and ODP_LIB_FLAVOR_AM (for automake rules) Signed-off-by: Nicolas Morey-Chaisemartin <nmo...@kalray.eu> --- config

[lng-odp] [PATCH v2 0/6] Make lidodp/libodp helper name platform specific

2016-12-06 Thread Nicolas Morey-Chaisemartin
2: - Moved to master (instead of monarch_lts) - rename ODP_LIB_NAME to ODP_LIB_FLAVOR - Supports empty ODP_LIB_FLAVOR (libs will be libodp.a libodphelper.a - Fix mising replace (helpers, linux-generic specific tests) Nicolas Morey-Chaisemartin (6): m4: add platform configurable library suffixes

Re: [lng-odp] [PATCH] configure: add option to enable/disable helper debug prints

2016-12-06 Thread Nicolas Morey-Chaisemartin
Le 12/06/2016 à 05:35 PM, Christophe Milard a écrit : > On 6 December 2016 at 15:28, Nicolas Morey-Chaisemartin <nmo...@kalray.eu> > wrote: > >> As a side effect, debug print are turned off by default >> >> Signed-off-by: Nicolas Morey-Chaisemartin <nmo..

[lng-odp] [PATCH] example: traffic_mgmt: use PRIu32 instead of %u

2016-12-06 Thread Nicolas Morey-Chaisemartin
Signed-off-by: Nicolas Morey-Chaisemartin <nmo...@kalray.eu> --- This should be backported to monarch_lts too. However it requires an extra #include example/traffic_mgmt/odp_traffic_mgmt.c | 19 +++ 1 file changed, 11 insertions(+), 8 deletions(-) diff --git a/e

[lng-odp] [PATCH] configure: add option to enable/disable helper debug prints

2016-12-06 Thread Nicolas Morey-Chaisemartin
As a side effect, debug print are turned off by default Signed-off-by: Nicolas Morey-Chaisemartin <nmo...@kalray.eu> --- configure.ac | 14 ++ helper/Makefile.am | 3 ++- 2 files changed, 16 insertions(+), 1 deletion(-) diff --git a/configure.ac b/configure.ac index 3

Re: [lng-odp] Traffic Manager

2016-12-06 Thread Nicolas Morey-Chaisemartin
Le 12/06/2016 à 01:27 PM, Mike Holmes a écrit : > > > On 6 December 2016 at 03:59, Nicolas Morey-Chaisemartin <nmo...@kalray.eu > <mailto:nmo...@kalray.eu>> wrote: > > Hi all, > > > I'm currently moving odp-mppa to monarch_lts (we ware based on 1

[lng-odp] [PATCH 4/4] example: hello: only call sched_setaffinity if available

2016-12-06 Thread Nicolas Morey-Chaisemartin
Signed-off-by: Nicolas Morey-Chaisemartin <nmo...@kalray.eu> --- example/hello/odp_hello.c | 21 + 1 file changed, 13 insertions(+), 8 deletions(-) diff --git a/example/hello/odp_hello.c b/example/hello/odp_hello.c index 6d114ee..9cb3498 100644 --- a/example/hello/odp_h

[lng-odp] [PATCH 3/4] example: traffic_mgmt: only call glibc specific functions if available

2016-12-06 Thread Nicolas Morey-Chaisemartin
Signed-off-by: Nicolas Morey-Chaisemartin <nmo...@kalray.eu> --- example/traffic_mgmt/odp_traffic_mgmt.c | 38 + 1 file changed, 29 insertions(+), 9 deletions(-) diff --git a/example/traffic_mgmt/odp_traffic_mgmt.c b/example/traffic_mgmt/odp_traffic_mgmt.c

[lng-odp] [PATCH 1/4] configure: check for non standard function availability

2016-12-06 Thread Nicolas Morey-Chaisemartin
Add check for sync, getrusage, getrlimit, setrlimit, sigaction, sched_setaffinity Signed-off-by: Nicolas Morey-Chaisemartin <nmo...@kalray.eu> --- configure.ac | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/configure.ac b/configure.ac index 3e89b0a..6e5125e

[lng-odp] [PATCH 2/4] performance: crypto: only call getrusage if available

2016-12-06 Thread Nicolas Morey-Chaisemartin
Signed-off-by: Nicolas Morey-Chaisemartin <nmo...@kalray.eu> --- test/common_plat/performance/odp_crypto.c | 5 + 1 file changed, 5 insertions(+) diff --git a/test/common_plat/performance/odp_crypto.c b/test/common_plat/performance/odp_crypto.c index 49a9f4b..5d74072 100644 ---

[lng-odp] [PATCH 0/4] Support non glibc in examples

2016-12-06 Thread Nicolas Morey-Chaisemartin
are not available This series should apply (and be applied) on the monarch_lts branch too. Nicolas Morey-Chaisemartin (4): configure: check for non standard function availability performance: crypto: only call getrusage if available example: traffic_mgmt: only call glibc specific functions

[lng-odp] Traffic Manager

2016-12-06 Thread Nicolas Morey-Chaisemartin
Hi all, I'm currently moving odp-mppa to monarch_lts (we ware based on 1.6 until now). I was wondering if the implementation MUST support the TM API. The traffic_mngr.h is a bit hazy on that point. It looks to me as if odp_tm_capabilities returns 0, it announces that the TM API is not

Re: [lng-odp] [MONARCH PATCH 1/2] validation: pktio: use PRIu32 to print uint32_t values

2016-12-05 Thread Nicolas Morey-Chaisemartin
h 3 way, it should go there first I > think > > On 5 December 2016 at 11:33, Nicolas Morey-Chaisemartin <nmo...@kalray.eu> > wrote: > >> Signed-off-by: Nicolas Morey-Chaisemartin <nmo...@kalray.eu> >> > Reviewed-by: Mike Holmes <mike.hol...@linaro.org>

Re: [lng-odp] [MONARCH PATCH 0/5] Make lidodp/libodp helper name platform specific

2016-12-05 Thread Nicolas Morey-Chaisemartin
CH 0/4] Make configure.ac generic" series? Although it changes different things, the goal is one and the same: make it easier to add/build platform reusing all the build/test infrastructure of linux-generic. Nicolas > On 2 December 2016 at 09:09, Nicolas Morey-Chaisemartin <nmo...@kalra

[lng-odp] [MONARCH PATCH 2/2] validation: traffic_mngr: use PRI macro to print uint*

2016-12-05 Thread Nicolas Morey-Chaisemartin
Signed-off-by: Nicolas Morey-Chaisemartin <nmo...@kalray.eu> --- test/validation/traffic_mngr/traffic_mngr.c | 116 +++- 1 file changed, 64 insertions(+), 52 deletions(-) diff --git a/test/validation/traffic_mngr/traffic_mngr.c b/test/validation/traffi

[lng-odp] [MONARCH PATCH 1/2] validation: pktio: use PRIu32 to print uint32_t values

2016-12-05 Thread Nicolas Morey-Chaisemartin
Signed-off-by: Nicolas Morey-Chaisemartin <nmo...@kalray.eu> --- test/validation/pktio/pktio.c | 10 +++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/test/validation/pktio/pktio.c b/test/validation/pktio/pktio.c index d53a419..4e93e9b 100644 --- a/test/validation

Re: [lng-odp] [API-NEXT 1/2] api: init: add initializer function for odp_platform_init_t

2016-12-02 Thread Nicolas Morey-Chaisemartin
lts Nicolas > > On 12/02/16 13:25, Nicolas Morey-Chaisemartin wrote: >> Signed-off-by: Nicolas Morey-Chaisemartin <nmo...@kalray.eu> >> --- >> include/odp/api/spec/init.h | 8 >> platform/linux-generic/odp_init.c | 5 + >> 2 files changed, 13

Re: [lng-odp] [MONARCH PATCH 0/5] Make lidodp/libodp helper name platform specific

2016-12-02 Thread Nicolas Morey-Chaisemartin
Le 12/02/2016 à 02:52 PM, Mike Holmes a écrit : > > > On 2 December 2016 at 08:50, Nicolas Morey-Chaisemartin <nmo...@kalray.eu > <mailto:nmo...@kalray.eu>> wrote: > > > > Le 12/02/2016 à 02:45 PM, Maxim Uvarov a écrit : > > On 12/02/

Re: [lng-odp] [MONARCH PATCH 0/5] Make lidodp/libodp helper name platform specific

2016-12-02 Thread Nicolas Morey-Chaisemartin
Le 12/02/2016 à 02:45 PM, Maxim Uvarov a écrit : > On 12/02/16 16:34, Nicolas Morey-Chaisemartin wrote: >> >> Le 12/02/2016 à 02:30 PM, Maxim Uvarov a écrit : >>> That needs to go to master first, than if needed back-ported to Monarch. >> Ok. >> >>>

Re: [lng-odp] [MONARCH PATCH 0/5] Make lidodp/libodp helper name platform specific

2016-12-02 Thread Nicolas Morey-Chaisemartin
t; > Maxim. > > On 12/02/16 15:16, Nicolas Morey-Chaisemartin wrote: >> The lib suffix "linux" (libodp-linux, libodphelper-linux) is harcoded pretty >> much everywhere in the build system. >> The goal of this series is to allow each platform to define its

[lng-odp] [MONARCH PATCH 5/5] example: use ODP_LIB_NAME to generate libodp name

2016-12-02 Thread Nicolas Morey-Chaisemartin
Signed-off-by: Nicolas Morey-Chaisemartin <nmo...@kalray.eu> --- example/Makefile.inc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/example/Makefile.inc b/example/Makefile.inc index 70ba2c0..b14cd69 100644 --- a/example/Makefile.inc +++ b/example/Makefile.inc @@ -1,

[lng-odp] [MONARCH PATCH 4/5] helper: use ODP_LIB_NAME to generate libodp name

2016-12-02 Thread Nicolas Morey-Chaisemartin
Signed-off-by: Nicolas Morey-Chaisemartin <nmo...@kalray.eu> --- helper/test/Makefile.am | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/helper/test/Makefile.am b/helper/test/Makefile.am index 545db73..8073283 100644 --- a/helper/test/Makefile.am +++ b/helpe

[lng-odp] [MONARCH PATCH 3/5] test: use ODP_LIB_NAME to generate libodp name

2016-12-02 Thread Nicolas Morey-Chaisemartin
Signed-off-by: Nicolas Morey-Chaisemartin <nmo...@kalray.eu> --- test/Makefile.inc| 2 +- test/validation/Makefile.inc | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/test/Makefile.inc b/test/Makefile.inc index 5def923..bb1f40f 100644 --- a/test/Makefi

[lng-odp] [MONARCH PATCH 2/5] platform: move odp-linux specific rules to linux-generic

2016-12-02 Thread Nicolas Morey-Chaisemartin
Signed-off-by: Nicolas Morey-Chaisemartin <nmo...@kalray.eu> --- configure.ac | 2 -- platform/Makefile.inc | 4 platform/linux-generic/Makefile.am | 6 ++ platform/linux-generic/m4/configure.m4 | 4 +++- 4 files changed, 9 insertions

[lng-odp] [MONARCH PATCH 1/5] m4: platform exports ODP_LIB_NAME

2016-12-02 Thread Nicolas Morey-Chaisemartin
This is the suffix to be used to compute the name of ODP and ODP helper libraries. Signed-off-by: Nicolas Morey-Chaisemartin <nmo...@kalray.eu> --- configure.ac | 1 + platform/linux-generic/m4/configure.m4 | 2 ++ 2 files changed, 3 insertions(+) diff

[lng-odp] [MONARCH PATCH 0/5] Make lidodp/libodp helper name platform specific

2016-12-02 Thread Nicolas Morey-Chaisemartin
The lib suffix "linux" (libodp-linux, libodphelper-linux) is harcoded pretty much everywhere in the build system. The goal of this series is to allow each platform to define its library suffix so they don't have to touch the generic files but simply the platform specific files. Nic

[lng-odp] [API-NEXT 2/2] validation: init: add test for init with platform parameters

2016-12-02 Thread Nicolas Morey-Chaisemartin
Signed-off-by: Nicolas Morey-Chaisemartin <nmo...@kalray.eu> --- test/common_plat/validation/api/init/init.c | 17 + test/common_plat/validation/api/init/init.h | 1 + 2 files changed, 18 insertions(+) diff --git a/test/common_plat/validation/api/init/init.c b/test/commo

[lng-odp] [API-NEXT 1/2] api: init: add initializer function for odp_platform_init_t

2016-12-02 Thread Nicolas Morey-Chaisemartin
Signed-off-by: Nicolas Morey-Chaisemartin <nmo...@kalray.eu> --- include/odp/api/spec/init.h | 8 platform/linux-generic/odp_init.c | 5 + 2 files changed, 13 insertions(+) diff --git a/include/odp/api/spec/init.h b/include/odp/api/spec/init.h index 154cdf8..0600691

Re: [lng-odp] [PATCH 1/4] configure.ac: do not set linux-generic as default platform

2016-12-02 Thread Nicolas Morey-Chaisemartin
Just realized I didn't update the help string. One of a side effect of this series is there is NO default platform anymore. I'll wait for some feedback and repost a v2 if needed Le 12/02/2016 à 10:49 AM, Nicolas Morey-Chaisemartin a écrit : > This prepares for the second patch and add supp

[lng-odp] [PATCH 4/4] configure.ac: move linux-generic conditionals to platform side

2016-12-02 Thread Nicolas Morey-Chaisemartin
Signed-off-by: Nicolas Morey-Chaisemartin <nmo...@kalray.eu> --- configure.ac | 4 platform/linux-generic/m4/conditionals.m4 | 4 2 files changed, 4 insertions(+), 4 deletions(-) create mode 100644 platform/linux-generic/m4/conditionals.m4 diff

[lng-odp] [PATCH 2/4] bootstrap: auto generate platform list

2016-12-02 Thread Nicolas Morey-Chaisemartin
This list all the available platforms looking for platform/*/m4/configure.m4 and generate the appropriate m4 file for the configure.ac to include Signed-off-by: Nicolas Morey-Chaisemartin <nmo...@kalray.eu> --- bootstrap| 25 + configure.ac | 11 +-- 2

[lng-odp] [PATCH 3/4] bootstrap: auto-include platform specific conditional definitions

2016-12-02 Thread Nicolas Morey-Chaisemartin
when including multiple platforms in the same repo Signed-off-by: Nicolas Morey-Chaisemartin <nmo...@kalray.eu> --- bootstrap | 12 1 file changed, 12 insertions(+) diff --git a/bootstrap b/bootstrap index 274ce2a..1af732d 100755 --- a/bootstrap +++ b/bootstrap @@ -7,6 +7,7 @@

[lng-odp] [PATCH 1/4] configure.ac: do not set linux-generic as default platform

2016-12-02 Thread Nicolas Morey-Chaisemartin
This prepares for the second patch and add support for repo which do not have a linux-generic platform Signed-off-by: Nicolas Morey-Chaisemartin <nmo...@kalray.eu> --- configure.ac | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/configure.ac b/configure.ac index b

[lng-odp] [PATCH 0/4] Make configure.ac generic

2016-12-02 Thread Nicolas Morey-Chaisemartin
, without modifying configure.ac. This applies to master, I haven't tried it on next yet. I use it on monarch_lts too. It won't apply directly as the platform specific tests moved between the two branches; If interested, I can send one for monarch too. Nicolas Morey-Chaisemartin (4): configure.ac

[lng-odp] [MONARCH PATCH] doc: remove VPATH override

2016-12-02 Thread Nicolas Morey-Chaisemartin
It breaks out-of-tree build of documentation as .adoc file are not in the VPATH and default rules cannot be applied to them. Signed-off-by: Nicolas Morey-Chaisemartin <nmo...@kalray.eu> --- doc/Makefile.inc | 2 -- 1 file changed, 2 deletions(-) diff --git a/doc/Makefile.inc b/doc/Makefi

Re: [lng-odp] [MONARCH PATCH] git_hash: handle git worktree

2016-11-18 Thread Nicolas Morey-Chaisemartin
Hi everyone, Could someone using gmail confirm that my mail didn't end up in your junk folder please :) Nicolas Le 11/18/2016 à 11:18 AM, Nicolas Morey-Chaisemartin a écrit : > In git worktrees .git is a file and not a directory. > This patches replaces [ -d .git ] test by [ -e .git ]

[lng-odp] [MONARCH PATCH] git_hash: handle git worktree

2016-11-18 Thread Nicolas Morey-Chaisemartin
In git worktrees .git is a file and not a directory. This patches replaces [ -d .git ] test by [ -e .git ] so it works in both cases Signed-off-by: Nicolas Morey-Chaisemartin <nmo...@kalray.eu> --- scripts/git_hash.sh | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff

Re: [lng-odp] Mailing test again

2016-07-29 Thread Nicolas Morey-Chaisemartin
http://lists.linaro.org> > (Postfix) to Linaro.org (GMail). > > > Best regards, > Ciaran. > > On 29 July 2016 at 13:26, Nicolas Morey-Chaisemartin <nmo...@kalray.eu > <mailto:nmo...@kalray.eu>> wrote: > > I guess your mail server (or mailman) i

Re: [lng-odp] Mailing test again

2016-07-29 Thread Nicolas Morey-Chaisemartin
ause I was on the To: list explicitly? It does appear in the odp-lng list > archives so it clearly was received. > > On Thu, Jul 28, 2016 at 9:09 AM, Nicolas Morey-Chaisemartin <nmo...@kalray.eu > <mailto:nmo...@kalray.eu>> wrote: > > Another mailing list test fo find o

[lng-odp] Mailing test again

2016-07-28 Thread Nicolas Morey-Chaisemartin
Another mailing list test fo find oud why kalray's email end up in spam for gmail users. Nicolas

[lng-odp] Test mail again

2016-07-08 Thread Nicolas Morey-Chaisemartin
Me again and again. Sorry for the spam. As for the previous ones (that you probably didn't see anyway): If people using GMail, can see this (or find this in their junk folder), could you reply to this mail. In the case where it was marked as Junk, could you also forward Gmail infos about why

[lng-odp] One more test mail

2016-07-06 Thread Nicolas Morey-Chaisemartin
Me again and again. Sorry for the spam. As for the previous ones (that you probably didn't see anyway): If people using GMail, can see this (or find this in their junk folder), could you reply to this mail. In the case where it was marked as Junk, could you also forward Gmail infos about why

[lng-odp] Yet Another Test Mail

2016-07-01 Thread Nicolas Morey-Chaisemartin
Me again. Sorry for the spam. As for the previous one (that yuo probably didn't see anyway): If people using GMail, can see this (or find this in their junk folder), could you reply to this mail. In the case where it was marked as Junk, could you also forward Gmail infos about why the mail was

[lng-odp] Another mailing list test for gmail users

2016-07-01 Thread Nicolas Morey-Chaisemartin
Sorry to do another mail like this, I'm still trying to figure out what is wrong with our mail server setup, mailman et Gmail. If people using GMail, can see this (or find this in their junk folder), could you reply to this mail. In the case where it was marked as Junk, could you also forward

[lng-odp] ML test

2016-06-08 Thread Nicolas Morey-Chaisemartin
Hi, Can ayone using the ML on gmail can see this? IT tweaked some things in our mail server configuration and not sure if we still ned up in Junk or not :) Thanks Nicolas ___ lng-odp mailing list lng-odp@lists.linaro.org

[lng-odp] Mailing List test again

2016-03-20 Thread Nicolas Morey-Chaisemartin (ODP)
Sorry for the spam guys Still trying to get the ODP mailing list to properly let me through :) Maxim could you tell me if you see this one? Nicolas ___ lng-odp mailing list lng-odp@lists.linaro.org https://lists.linaro.org/mailman/listinfo/lng-odp

[lng-odp] Test mail

2016-03-14 Thread Nicolas Morey-Chaisemartin
Making sure if we can send mail to the ML again :) Nicolas ___ lng-odp mailing list lng-odp@lists.linaro.org https://lists.linaro.org/mailman/listinfo/lng-odp

[lng-odp] Error building user guide

2016-02-24 Thread Nicolas Morey-Chaisemartin
Hi, I just pulled 1.7.0 and tried to build the official release (not our port) and I get some error messages during the doc build. asciidoc: WARNING: implementers-guide.adoc: line 43: {sys:"/usr/bin/python2" -u -c "import base64,sys; base64.encode(sys.stdin,sys.stdout)" <

[lng-odp] [PATCH] example: time: use PRIu32 where needed

2016-02-23 Thread Nicolas Morey-Chaisemartin
Signed-off-by: Nicolas Morey-Chaisemartin <nmo...@kalray.eu> --- example/time/time_global_test.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/example/time/time_global_test.c b/example/time/time_global_test.c index 7cfd969..0c08f2c 100644 --- a/exampl

[lng-odp] [PATCH] linux-generic: pktio: do not include net/if.h and linux/if.h

2016-02-23 Thread Nicolas Morey-Chaisemartin
/if.h:71:2: note: previous definition of ‘IFF_UP’ was here IFF_UP= 1<<0, /* sysfs */ ^ and so on.. Signed-off-by: Nicolas Morey-Chaisemartin <nmo...@kalray.eu> --- platform/linux-generic/pktio/ethtool.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/pl

[lng-odp] [PATCH] validation: timer: replace %d by PRIu32

2016-02-23 Thread Nicolas Morey-Chaisemartin
Signed-off-by: Nicolas Morey-Chaisemartin <nmo...@kalray.eu> --- test/validation/timer/timer.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/test/validation/timer/timer.c b/test/validation/timer/timer.c index 004670a..8d3ba47 100644 --- a/test/validation/timer/t

[lng-odp] [PATCH] scripts: git_hash: support gitfiles

2016-02-23 Thread Nicolas Morey-Chaisemartin
Morey-Chaisemartin <nmo...@kalray.eu> --- scripts/git_hash.sh | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/scripts/git_hash.sh b/scripts/git_hash.sh index ccd62ab..e7f43f2 100755 --- a/scripts/git_hash.sh +++ b/scripts/git_hash.sh @@ -6,7 +6,7 @@ if [ -z ${1} ]; th

[lng-odp] [PATCHv2 2/2] validation: init: add test for init with platform parameters

2016-02-23 Thread Nicolas Morey-Chaisemartin
Signed-off-by: Nicolas Morey-Chaisemartin <nmo...@kalray.eu> --- Reposted due to mail issues test/validation/init/init.c | 16 test/validation/init/init.h | 1 + 2 files changed, 17 insertions(+) diff --git a/test/validation/init/init.c b/test/validation/init/init.c

[lng-odp] [PATCHv2 1/2] api: init: add initializer function for odp_platform_init_t

2016-02-23 Thread Nicolas Morey-Chaisemartin
Signed-off-by: Nicolas Morey-Chaisemartin <nmo...@kalray.eu> --- Reposted due to mail issues include/odp/api/spec/init.h | 8 platform/linux-generic/odp_init.c | 4 2 files changed, 12 insertions(+) diff --git a/include/odp/api/spec/init.h b/include/odp/api/spec/

[lng-odp] Mailman Test

2016-02-23 Thread Nicolas Morey-Chaisemartin
Test to make sure Mailman is not breaking DKIM anymore. ___ lng-odp mailing list lng-odp@lists.linaro.org https://lists.linaro.org/mailman/listinfo/lng-odp

Re: [lng-odp] [API-NEXT PATCHv1 0/2] linux-generic:classification:fix the endian problem when verify ODP_PMR_CUSTOM_FRAME

2016-01-31 Thread Nicolas Morey-Chaisemartin
I don't completely agree with your assessment. The fact that it ends up in a uint64_t is linux-generic specific. More precisely because uint64_t was already used in the pmr_term_value_t struct. Nothing prevents another implementation to support any size of ODP_PMR_CUSTOM_FRAME (ie more than 64b)

[lng-odp] [API-NEXT 2/2] validation: init: add test for init with platform parameters

2016-01-26 Thread Nicolas Morey-Chaisemartin
Signed-off-by: Nicolas Morey-Chaisemartin <nmo...@kalray.eu> --- test/validation/init/init.c | 16 test/validation/init/init.h | 1 + 2 files changed, 17 insertions(+) diff --git a/test/validation/init/init.c b/test/validation/init/init.c index 62bd75c..f7424a9

[lng-odp] [API-NEXT 1/2] api: init: add initializer function for odp_platform_init_t

2016-01-26 Thread Nicolas Morey-Chaisemartin
Signed-off-by: Nicolas Morey-Chaisemartin <nmo...@kalray.eu> --- I'm not very happy with a odp_platform_init_init but odp_platform_init would be misleading for users and odp_platform_param_init wouldn't fit ODP semantics. If someone has a better idea, I'm all ears. include/odp/api/

[lng-odp] [PATCH] linux-generic: crypto: fix AES-GCM compatibility with old version of OpenSSL

2016-01-18 Thread Nicolas Morey-Chaisemartin
Old version of OpenSSL require SET_TAG to be called before decrypting the data. New versions are compatible either way Signed-off-by: Nicolas Morey-Chaisemartin <nmo...@kalray.eu> --- Repost due to previous mail tagged as SPAM platform/linux-generic/odp_crypto.c | 3 +-- 1 file chan

Re: [lng-odp] Maintain code copy-pasted from linux-generic in ODP-DPDK

2016-01-14 Thread Nicolas Morey-Chaisemartin
On 01/13/2016 02:46 PM, Maxim Uvarov wrote: > On 01/13/2016 00:15, Nicolas Morey-Chaisemartin wrote: >> Hi, >> >> I have the same issue with our implementation. >> I usually either work from the git diff of linux generic before and post >> update and backport

Re: [lng-odp] ODP 226: need for shmem->refresh()?

2016-01-14 Thread Nicolas Morey-Chaisemartin
to flush that one address -> stale data -> crash). > > > > -Petri > > > > > > *From:*EXT Christophe Milard [mailto:christophe.mil...@linaro.org] > *Sent:* Thursday, January 14, 2016 12:16 PM > *To:* Hongbo Zhang; Mike Holmes; Petri Savolainen; Anders Roxell

Re: [lng-odp] [PATCH] validation: classification: remove double frees

2016-01-13 Thread Nicolas Morey-Chaisemartin
On 01/13/2016 01:42 PM, Stuart Haslam wrote: > On Wed, Jan 13, 2016 at 03:30:03PM +0300, Maxim Uvarov wrote: >> Please resend this patch with Review. Original patch is missing for >> unknown reason. >> >> Maxim. > Mails from Nicolas get flagged as spam by google, looks to be due to a > DKIM

Re: [lng-odp] Maintain code copy-pasted from linux-generic in ODP-DPDK

2016-01-12 Thread Nicolas Morey-Chaisemartin
Hi, I have the same issue with our implementation. I usually either work from the git diff of linux generic before and post update and backport changed or to a diff of the diff of linux-generic vs mppa platform before and after the merge. Theoretically the diff you be clode to void (except line

[lng-odp] [PATCH] linux-generic: crypto: fix AES-GCM compatibility with old version of OpenSSL

2016-01-12 Thread Nicolas Morey-Chaisemartin
Old version of OpenSSL require SET_TAG to be called before decrypting the data. New versions are compatible either way Signed-off-by: Nicolas Morey-Chaisemartin <nmo...@kalray.eu> --- platform/linux-generic/odp_crypto.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff

  1   2   3   4   5   6   >