[lng-odp] [PATCH] test: tm: skip tm result under travis run

2017-03-31 Thread Maxim Uvarov
tm test fails time to time in Travis environment. Because of we can not control that machine we can not do things like taskset and core isolation there. So simple run test and ignore it's result. Threat only segfault as actual error. Linaro CI will still do full test. https://bugs.linaro.org/show_b

Re: [lng-odp] [PATCHv4] linux-generic: debug: enable helper use from c++ programs

2017-03-31 Thread Maxim Uvarov
rechecked this patch and it fails: https://travis-ci.org/muvarov/odp/jobs/217195437 Maxim. On 02/08/17 04:04, Bill Fischofer wrote: > I've done a fair amount of experimentation with using variants of > Brian's suggested macros. This works for C but for C++ it results in > an error message saying

Re: [lng-odp] [API-NEXT PATCH v2 1/2] api: queue: added queue size param

2017-03-31 Thread Bill Fischofer
On Fri, Mar 31, 2017 at 6:46 AM, Petri Savolainen wrote: > Added capability information about maximum number of queues > and queue sizes. Both are defined per queue type, since > plain and scheduled queues may have different implementations > (e.g. one uses HW while the other is SW). > > Added que

Re: [lng-odp] [API-NEXT PATCH v2 2/2] validation: queue: test queue max_num per type

2017-03-31 Thread Bill Fischofer
This patch is incomplete. Since you're adding a max_size, we need to add a test for that. If an implementation says the max size of a queue is n (n > 0) then we have to attempt to allocate a queue of that size and verify that trying to add an n+1st element to it will fail. Otherwise, we shouldn't h

Re: [lng-odp] [PATCH] helper: iplookuptable: fix prefix_entry_t member order

2017-03-31 Thread Elo, Matias (Nokia - FI/Espoo)
> On 31 Mar 2017, at 16:54, Maxim Uvarov wrote: > > On 03/31/17 10:43, Matias Elo wrote: >> Fixes https://bugs.linaro.org/show_bug.cgi?id=2910 >> > > Matias please add some description here. Link to problem is good > but people like to read only git logs. > > Maxim. Sure, fixed in v2. -Mati

[lng-odp] [PATCH v2] helper: iplookuptable: fix prefix_entry_t member order

2017-03-31 Thread Matias Elo
Depending on the alignment/padding odph_iplookup_table_put_value() could end up overwriting the wrong fields in prefix_entry_t. Fix this by reverting the order of prefix_entry_t members. Fixes https://bugs.linaro.org/show_bug.cgi?id=2910 Signed-off-by: Matias Elo --- V2: - Improved commit log (M

Re: [lng-odp] [PATCH] helper: iplookuptable: fix prefix_entry_t member order

2017-03-31 Thread Maxim Uvarov
On 03/31/17 10:43, Matias Elo wrote: > Fixes https://bugs.linaro.org/show_bug.cgi?id=2910 > Matias please add some description here. Link to problem is good but people like to read only git logs. Maxim. > Signed-off-by: Matias Elo > --- > helper/iplookuptable.c | 8 > 1 file changed,

Re: [lng-odp] [API-NEXT 2/4] linux-generic: ring.c: use required memory orderings

2017-03-31 Thread Maxim Uvarov
Merged to master branch. Maxim. On 03/31/17 16:45, Ola Liljedahl wrote: > On 31 March 2017 at 15:21, Maxim Uvarov wrote: >> On 03/28/17 22:23, Brian Brooks wrote: >>> From: Ola Liljedahl >>> >>> Signed-off-by: Ola Liljedahl >>> Reviewed-by: Brian Brooks >>> --- >>> platform/linux-generic/pkt

Re: [lng-odp] My last day at linaro/LNG

2017-03-31 Thread Bill Fischofer
Au revoir! It was great working with you and I hope our paths will cross again. Best wishes for your future success. On Fri, Mar 31, 2017 at 6:20 AM, Mike Holmes wrote: > Sad to see you go Christophe, it really was energizing to work with you. > > Mike > > On Mar 31, 2017 7:13 AM, "Maxim Uvarov"

[lng-odp] [Linaro/odp] 8286cf: linux-generic: ring.c: use required memory orderin...

2017-03-31 Thread GitHub
Branch: refs/heads/master Home: https://github.com/Linaro/odp Commit: 8286cff9c5bb890d169b58857e3ab6cd118a2af4 https://github.com/Linaro/odp/commit/8286cff9c5bb890d169b58857e3ab6cd118a2af4 Author: Ola Liljedahl Date: 2017-03-31 (Fri, 31 Mar 2017) Changed paths: M platfo

Re: [lng-odp] [API-NEXT 2/4] linux-generic: ring.c: use required memory orderings

2017-03-31 Thread Ola Liljedahl
On 31 March 2017 at 15:21, Maxim Uvarov wrote: > On 03/28/17 22:23, Brian Brooks wrote: >> From: Ola Liljedahl >> >> Signed-off-by: Ola Liljedahl >> Reviewed-by: Brian Brooks >> --- >> platform/linux-generic/pktio/ring.c | 30 ++ >> 1 file changed, 14 insertions(+),

Re: [lng-odp] [API-NEXT 2/4] linux-generic: ring.c: use required memory orderings

2017-03-31 Thread Maxim Uvarov
On 03/28/17 22:23, Brian Brooks wrote: > From: Ola Liljedahl > > Signed-off-by: Ola Liljedahl > Reviewed-by: Brian Brooks > --- > platform/linux-generic/pktio/ring.c | 30 ++ > 1 file changed, 14 insertions(+), 16 deletions(-) > mode change 100644 => 100755 platfor

[lng-odp] [PATCH v5 3/3] validation: packet: use common define for test pool sizes

2017-03-31 Thread Matias Elo
Signed-off-by: Matias Elo --- V5: - Added common define test/common_plat/validation/api/packet/packet.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/test/common_plat/validation/api/packet/packet.c b/test/common_plat/validation/api/packet/packet.c index 8c2a7c7.

[lng-odp] [PATCH v5 1/3] validation: packet: increase test pool size

2017-03-31 Thread Matias Elo
Previously packet_test_concatsplit() could fail on some pool implementations as the pool ran out of buffers. Increase default pools size and use capability to make sure the value is valid. Signed-off-by: Matias Elo --- test/common_plat/validation/api/packet/packet.c | 7 ++- 1 file changed,

[lng-odp] [PATCH v5 2/3] validation: packet: remove invalid check from packet_test_alloc_segmented()

2017-03-31 Thread Matias Elo
One can't assume that the packet should be segmented as this test is using a different pool with different parameters than the default test pool. Signed-off-by: Matias Elo --- test/common_plat/validation/api/packet/packet.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/test/common_plat/v

Re: [lng-odp] [PATCHv4] test: generator: Updated global statistics calculation to provide useful/easy-to-parse information

2017-03-31 Thread Bogdan Pricope
Hi Maxim, Yi, Since this part is fresh in our memory, can you please review this patch as well? BR, Bogdan On 15 March 2017 at 09:48, Bogdan Pricope wrote: > Signed-off-by: Bogdan Pricope > --- > example/generator/odp_generator.c | 57 > --- > 1 file chang

Re: [lng-odp] [API-NEXT PATCH 1/2] api: queue: added queue size param

2017-03-31 Thread Savolainen, Petri (Nokia - FI/Espoo)
This has a small doxygen issue. Fixed that on v2.

[lng-odp] [API-NEXT PATCH v2 2/2] validation: queue: test queue max_num per type

2017-03-31 Thread Petri Savolainen
Updated implementation and test with type specific number of queues. Signed-off-by: Petri Savolainen --- platform/linux-generic/odp_queue.c| 2 ++ test/common_plat/validation/api/queue/queue.c | 43 --- 2 files changed, 28 insertions(+), 17 deletions(-) diff

[lng-odp] [API-NEXT PATCH v2 1/2] api: queue: added queue size param

2017-03-31 Thread Petri Savolainen
Added capability information about maximum number of queues and queue sizes. Both are defined per queue type, since plain and scheduled queues may have different implementations (e.g. one uses HW while the other is SW). Added queue size parameter, which specifies how large storage size application

[lng-odp] [API-NEXT PATCH 2/2] validation: queue: test queue max_num per type

2017-03-31 Thread Petri Savolainen
Updated implementation and test with type specific number of queues. Signed-off-by: Petri Savolainen --- platform/linux-generic/odp_queue.c| 2 ++ test/common_plat/validation/api/queue/queue.c | 43 --- 2 files changed, 28 insertions(+), 17 deletions(-) diff

[lng-odp] [API-NEXT PATCH 1/2] api: queue: added queue size param

2017-03-31 Thread Petri Savolainen
Added capability information about maximum number of queues and queue sizes. Both are defined per queue type, since plain and scheduled queues may have different implementations (e.g. one uses HW while the other is SW). Added queue size parameter, which specifies how large storage size application

Re: [lng-odp] My last day at linaro/LNG

2017-03-31 Thread Mike Holmes
Sad to see you go Christophe, it really was energizing to work with you. Mike On Mar 31, 2017 7:13 AM, "Maxim Uvarov" wrote: > Thanks Christophe for all that work which you did! It's really good impact > to the project. And all the best wishes in future. > > Maxim. > > On 31 March 2017 at 11:32

Re: [lng-odp] My last day at linaro/LNG

2017-03-31 Thread Maxim Uvarov
Thanks Christophe for all that work which you did! It's really good impact to the project. And all the best wishes in future. Maxim. On 31 March 2017 at 11:32, Yi He wrote: > Hi, Christophe, > > Wish you all the best and joy in your new endeavors. > > Best Regards, Yi > > On 31 March 2017 at 16

[lng-odp] [Bug 2921] odp_init_global() and odp_pool_create() set errno = 12

2017-03-31 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=2921 --- Comment #4 from Oriol Arcas --- Sorry, I meant that ODP calls a library or system call that sets errno. I dug a little into the problem and I found that errno is set in odp_shm_reserve(), which is called both by odp_init_global() and odp_pool_crea

Re: [lng-odp] My last day at linaro/LNG

2017-03-31 Thread Yi He
Hi, Christophe, Wish you all the best and joy in your new endeavors. Best Regards, Yi On 31 March 2017 at 16:27, Francois Ozog wrote: > Hi Christophe, > > It is a big loss for LNG. > > On the technical side you have both conceptual talent and great > implementation skills. This is not so commo

Re: [lng-odp] My last day at linaro/LNG

2017-03-31 Thread Francois Ozog
Hi Christophe, It is a big loss for LNG. On the technical side you have both conceptual talent and great implementation skills. This is not so common and DDF was a challenge at your "measure" (as a French you will know what I mean as I don't know if it translates well in English). Despite all th

[lng-odp] My last day at linaro/LNG

2017-03-31 Thread Christophe Milard
Hi all, As most of you know, my assignment for linaro/LNG is ending today. I was really willing to join connect in Budapest as this would have given me an opportunity to see you a last time face to face but this did not happen, sadly. I wish to thank all of you that helped me growing in this open s

[lng-odp] [PATCH] helper: iplookuptable: fix prefix_entry_t member order

2017-03-31 Thread Matias Elo
Fixes https://bugs.linaro.org/show_bug.cgi?id=2910 Signed-off-by: Matias Elo --- helper/iplookuptable.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/helper/iplookuptable.c b/helper/iplookuptable.c index aae2199..37d31e3 100644 --- a/helper/iplookuptable.c +++ b/hel

Re: [lng-odp] [API-NEXT PATCH] api: packet: add per packet checksum control

2017-03-31 Thread Savolainen, Petri (Nokia - FI/Espoo)
> -Original Message- > From: Bill Fischofer [mailto:bill.fischo...@linaro.org] > Sent: Wednesday, March 29, 2017 6:01 PM > To: Bogdan Pricope > Cc: Maxim Uvarov ; Savolainen, Petri (Nokia - > FI/Espoo) ; lng-odp-forward o...@lists.linaro.org> > Subject: Re: [lng-odp] [API-NEXT PATCH] ap