Re: [lng-odp] [PATCH] examples: generator: UDP ports configuration

2017-04-18 Thread Bogdan Pricope
ping. On 12 April 2017 at 15:54, Bogdan Pricope wrote: > Signed-off-by: Bogdan Pricope > --- > example/generator/odp_generator.c | 20 +--- > 1 file changed, 17 insertions(+), 3 deletions(-) > > diff --git a/example/generator/odp_generator.c > b/example/generator/odp_generator.

Re: [lng-odp] [PATCH 0/2] examples: generator: various fixes

2017-04-18 Thread Bogdan Pricope
ping. On 11 April 2017 at 14:25, Bogdan Pricope wrote: > Various fixes for odp_generator. > > Bogdan Pricope (2): > example: generator: fix stop criteria - number of packets sent > example: generator: flush output messages > > example/generator/odp_generator.c | 7 +-- > 1 file changed,

Re: [lng-odp] [API-NEXT PATCH 0/4] add maximum packet counts to pool info

2017-04-18 Thread Savolainen, Petri (Nokia - FI/Espoo)
> -Original Message- > From: lng-odp [mailto:lng-odp-boun...@lists.linaro.org] On Behalf Of Bill > Fischofer > Sent: Thursday, April 13, 2017 3:59 PM > To: Elo, Matias (Nokia - FI/Espoo) > Cc: lng-odp-forward > Subject: Re: [lng-odp] [API-NEXT PATCH 0/4] add maximum packet counts to > p

[lng-odp] [Bug 2945] New: Potentially high (unbound) latency when threads are grouped into multiple schedule groups

2017-04-18 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=2945 Bug ID: 2945 Summary: Potentially high (unbound) latency when threads are grouped into multiple schedule groups Product: OpenDataPlane - linux- generic reference Version: master

Re: [lng-odp] [API-NEXT v4] api: ipsec: factor out definitions for feature support levels

2017-04-18 Thread Savolainen, Petri (Nokia - FI/Espoo)
> -Original Message- > From: lng-odp [mailto:lng-odp-boun...@lists.linaro.org] On Behalf Of > Dmitry Eremin-Solenikov > Sent: Tuesday, April 18, 2017 12:03 AM > To: lng-odp@lists.linaro.org > Subject: [lng-odp] [API-NEXT v4] api: ipsec: factor out definitions for > feature support levels

Re: [lng-odp] [API-NEXT PATCH 0/4] add maximum packet counts to pool info

2017-04-18 Thread Bill Fischofer
On Tue, Apr 18, 2017 at 4:05 AM, Savolainen, Petri (Nokia - FI/Espoo) < petri.savolai...@nokia-bell-labs.com> wrote: > > > > -Original Message- > > From: lng-odp [mailto:lng-odp-boun...@lists.linaro.org] On Behalf Of > Bill > > Fischofer > > Sent: Thursday, April 13, 2017 3:59 PM > > To: E

Re: [lng-odp] [API-NEXT PATCH 1/4] api: pool: add maximum packet counts to pool info

2017-04-18 Thread Bala Manoharan
Regards, Bala On 12 April 2017 at 18:28, Matias Elo wrote: > Add fields to odp_pool_info_t for maximum number of packets of any length, > maximum number of minimum length packets, and maximum number of maximum > length packets. > > odp_pool_param_t documentation is updated to reflect these new f

[lng-odp] [Bug 2852] ODP_STATIC_ASSERT() fails when used by C++

2017-04-18 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=2852 --- Comment #6 from Maxim Uvarov --- https://github.com/muvarov/odp/commit/3e5a07edf190614e739c8dba76cf165330e1b035 2017-04-17T19:37:43+03:00 Bill Fischofer bill.fischo...@linaro.org linux-generic: debug: enable helper use from c++ programs The ODP_ST

Re: [lng-odp] [API-NEXT][RFC][rebased] linux-gen: ipsec: draft IPsec implementation

2017-04-18 Thread Peltonen, Janne (Nokia - FI/Espoo)
Hi, Some comments on the patch: > >> - No tunnel support > >> - No header modification according to RFCs There is some tunneling and header modification code in the patch. Now it is not easy to tell which part of the code is supposed to work and which not. Other limitations include: There is n

[lng-odp] [Linaro/odp] d02590: api: ipsec: factor out definitions for feature sup...

2017-04-18 Thread GitHub
Branch: refs/heads/api-next Home: https://github.com/Linaro/odp Commit: d025907602c5cae0f46b8b5219bf392bdba8dc21 https://github.com/Linaro/odp/commit/d025907602c5cae0f46b8b5219bf392bdba8dc21 Author: Dmitry Eremin-Solenikov Date: 2017-04-18 (Tue, 18 Apr 2017) Changed paths:

[lng-odp] [Bug 2852] ODP_STATIC_ASSERT() fails when used by C++

2017-04-18 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=2852 --- Comment #7 from Maxim Uvarov --- https://github.com/muvarov/odp/commit/3e5a07edf190614e739c8dba76cf165330e1b035 2017-04-17T19:37:43+03:00 Bill Fischofer bill.fischo...@linaro.org linux-generic: debug: enable helper use from c++ programs The ODP_ST

Re: [lng-odp] [API-NEXT][RFC][rebased] linux-gen: ipsec: draft IPsec implementation

2017-04-18 Thread Peltonen, Janne (Nokia - FI/Espoo)
Hi, > >> + /* Setup parameters and call crypto library to create session */ > >> + crypto_param.op = (ODP_IPSEC_DIR_INBOUND == param->dir) ? > >> + ODP_CRYPTO_OP_DECODE : > >> + ODP_CRYPTO_OP_ENCODE; > >> + crypto_param.auth_cipher_text

Re: [lng-odp] [API-NEXT][RFC][rebased] linux-gen: ipsec: draft IPsec implementation

2017-04-18 Thread Dmitry Eremin-Solenikov
On 18.04.2017 19:08, Peltonen, Janne (Nokia - FI/Espoo) wrote: > Hi, Thank you for your review! > + /* Setup parameters and call crypto library to create session */ + crypto_param.op = (ODP_IPSEC_DIR_INBOUND == param->dir) ? + ODP_CRYPTO_OP_DECODE

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

2017-04-18 Thread Maxim Uvarov
ping. If people do not want to see any entries for CI env in test suite code then I can make this patch be applied on each Travis run. If it's good I need a review. Some times this test fails. On 6 April 2017 at 17:39, Maxim Uvarov wrote: > Please review this patch. TM very often fails in Travis

[lng-odp] [Bug 2852] ODP_STATIC_ASSERT() fails when used by C++

2017-04-18 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=2852 --- Comment #8 from Maxim Uvarov --- https://github.com/muvarov/odp/commit/3e5a07edf190614e739c8dba76cf165330e1b035 2017-04-17T19:37:43+03:00 Bill Fischofer bill.fischo...@linaro.org linux-generic: debug: enable helper use from c++ programs The ODP_ST

Re: [lng-odp] [PATCH] examples: generator: UDP ports configuration

2017-04-18 Thread Maxim Uvarov
On 04/12/17 15:54, Bogdan Pricope wrote: > Signed-off-by: Bogdan Pricope > --- > example/generator/odp_generator.c | 20 +--- > 1 file changed, 17 insertions(+), 3 deletions(-) > > diff --git a/example/generator/odp_generator.c > b/example/generator/odp_generator.c > index 95fb5

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

2017-04-18 Thread Bill Fischofer
Do I understand this patch correctly that it attempts to run the test and if it fails for any reason other than a segfault and we are running in Travis that it is then just ignored? That seems reasonable. On Tue, Apr 18, 2017 at 11:27 AM, Maxim Uvarov wrote: > ping. If people do not want to see

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

2017-04-18 Thread Maxim Uvarov
On 04/18/17 22:26, Bill Fischofer wrote: > Do I understand this patch correctly that it attempts to run the test > and if it fails for any reason other than a segfault and we are running > in Travis that it is then just ignored? That seems reasonable. > yes. that is the goal. Run test to make cod

Re: [lng-odp] [API-NEXT PATCH 1/4] api: pool: add maximum packet counts to pool info

2017-04-18 Thread Bill Fischofer
See comments below based on today's discussions. On Wed, Apr 12, 2017 at 7:58 AM, Matias Elo wrote: > Add fields to odp_pool_info_t for maximum number of packets of any length, > maximum number of minimum length packets, and maximum number of maximum > length packets. > > odp_pool_param_t docume

[lng-odp] Patches to merge from V2 of scalable scheduler

2017-04-18 Thread Honnappa Nagarahalli
Hi Bill and Maxim, It is my understanding that discussions on the following patches from V2 series have been concluded and they can be merged to API-NEXT. http://patches.opendataplane.org/patch/8524/ - Clang fix (1/16) http://patches.opendataplane.org/patch/8496/ - Queue size fix for cuckoo ta

[lng-odp] [PATCH v3] build: fix native Clang build on ARMv8

2017-04-18 Thread Brian Brooks
The build is broken when using clang on ARM. -mcx16 is being passed to clang when building natively on ARM. This combined with -Werror causes the breakage. Fix it by skipping anything related to -mcx16 when not building for x86-based architectures. See [1] for details. [1] https://lists.linaro.org

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

2017-04-18 Thread Bill Fischofer
On Fri, Mar 31, 2017 at 3:39 PM, Maxim Uvarov wrote: > 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. Li

Re: [lng-odp] Patches to merge from V2 of scalable scheduler

2017-04-18 Thread Bill Fischofer
On Tue, Apr 18, 2017 at 4:00 PM, Honnappa Nagarahalli < honnappa.nagaraha...@linaro.org> wrote: > Hi Bill and Maxim, > It is my understanding that discussions on the following patches > from V2 series have been concluded and they can be merged to API-NEXT. > > http://patches.opendataplane.org/

Re: [lng-odp] Patches to merge from V2 of scalable scheduler

2017-04-18 Thread Honnappa Nagarahalli
Thanks Bill. Another topic that needs to be discussed is the conditional compilation in the makefile.am file, Maxim had comments on this. Thank you, Honnappa On 18 April 2017 at 17:24, Bill Fischofer wrote: > > > On Tue, Apr 18, 2017 at 4:00 PM, Honnappa Nagarahalli > wrote: >> >> Hi Bill and

Re: [lng-odp] [PATCH] test: generator: replace gettimeofday() with ODP API calls

2017-04-18 Thread Yi He
Hi, Bogdan Consider endianness for this uint64_t time value carried in ICMP packets, need htonl and ntohl here. Best Regards, Yi On 11 April 2017 at 19:31, Bogdan Pricope wrote: > Ping! > > > On 3 April 2017 at 13:00, Bogdan Pricope > wrote: > > Signed-off-by: Bogdan Pricope > > --- > > e

Re: [lng-odp] [PATCH 1/2] example: generator: fix stop criteria - number of packets sent

2017-04-18 Thread Yi He
For this patch series Reviewed-and-tested-by: Yi He On 11 April 2017 at 19:25, Bogdan Pricope wrote: > Signed-off-by: Bogdan Pricope > --- > example/generator/odp_generator.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/example/generator/odp_generator.c b/e