[lng-odp] [Bug 1693] CPU frequency should be locked when it is used to calculate time

https://bugs.linaro.org/show_bug.cgi?id=1693 --- Comment #2 from Hongbo Zhang --- As discussed with Stuart, he suggested this can be solved outside ODP, eg lock frequency before launch it, if so no patch is needed for this. -- You are receiving this mail because: You are on the CC list for the

[lng-odp] [API-NEXT PATCHv3 6/6] linux-generic: schedule: implement odp_schedule_release_ordered()

Signed-off-by: Bill Fischofer --- platform/linux-generic/odp_queue.c | 169 + 1 file changed, 152 insertions(+), 17 deletions(-) diff --git a/platform/linux-generic/odp_queue.c b/platform/linux-generic/odp_queue.c index 4d0e1b4..a2460e7 100644 --- a/platform/

[lng-odp] [API-NEXT PATCHv3 5/6] api: schedule: revised definition of odp_schedule_release_ordered

Signed-off-by: Bill Fischofer --- include/odp/api/schedule.h | 38 -- 1 file changed, 24 insertions(+), 14 deletions(-) diff --git a/include/odp/api/schedule.h b/include/odp/api/schedule.h index 95fc8df..0ab91e4 100644 --- a/include/odp/api/schedule.h +++ b/in

[lng-odp] [API-NEXT PATCHv3 3/6] linux-generic: schedule: implement scheduler groups

Signed-off-by: Bill Fischofer --- include/odp/api/config.h | 5 + .../include/odp/plat/schedule_types.h | 4 + platform/linux-generic/odp_schedule.c | 158 + platform/linux-generic/odp_thread.c| 25 +++-

[lng-odp] [API-NEXT PATCHv3 4/6] linux-generic: queue: implement ordered queues

Signed-off-by: Bill Fischofer --- .../linux-generic/include/odp_buffer_internal.h| 5 + .../linux-generic/include/odp_queue_internal.h | 4 + .../linux-generic/include/odp_schedule_internal.h | 2 +- platform/linux-generic/odp_pool.c | 3 + platform/linux-generic

[lng-odp] [API-NEXT PATCHv3 2/6] validation: schedule: initialize queue and pool parameters

Signed-off-by: Bill Fischofer --- test/validation/scheduler/scheduler.c | 4 1 file changed, 4 insertions(+) diff --git a/test/validation/scheduler/scheduler.c b/test/validation/scheduler/scheduler.c index fe03ab7..bbe829e 100644 --- a/test/validation/scheduler/scheduler.c +++ b/test/valid

[lng-odp] [API-NEXT PATCHv3 1/6] api: schedule: additional scheduler group functions

Signed-off-by: Bill Fischofer --- include/odp/api/schedule.h | 55 ++ 1 file changed, 55 insertions(+) diff --git a/include/odp/api/schedule.h b/include/odp/api/schedule.h index 36e52cd..95fc8df 100644 --- a/include/odp/api/schedule.h +++ b/include/odp

[lng-odp] [API-NEXT PATCHv3 0/6] Scheduler API Enhancements

Changes in v3 - Refine API for odp_schedule_release_ordered() and implement it - Numerous bug fixes and cleanup -- Changes in v2 - Incorporate Stuart's review comments - Add odp_schedule_group_count() API to return the count of threads belonging to a schedule group -- This patch series implem

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

Hi Christophe, Could you please clarify how to build validation tests from platform side. As I see all tests stop to build only because i do: --- a/test/Makefile.am +++ b/test/Makefile.am @@ -1,5 +1,5 @@ SUBDIRS = api_test performance miscellaneous if cunit_support -SUBDIRS += validation

[lng-odp] [PATCH] doc: publish contributing text

Signed-off-by: Mike Holmes --- This drops the DEPENDENCIES so I have submitted it as a new patch and changed the subject. doc/doxygen.cfg | 2 +- doc/odp.dox | 4 2 files changed, 5 insertions(+), 1 deletion(-) diff --git a/doc/doxygen.cfg b/doc/doxygen.cfg index 213a903..f28ec24 10064

[lng-odp] [Bug 1675] CID 98149: CID 98148: CID 98147: Integer handling issues

https://bugs.linaro.org/show_bug.cgi?id=1675 --- Comment #3 from Mike Holmes --- This has not been fixed in Coverty before we close it as resolved. I will set it to that state now -- You are receiving this mail because: You are on the CC list for the bug.___

[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 --- since v

[lng-odp] [Bug 1365] Lock-up following sendmmsg failure

https://bugs.linaro.org/show_bug.cgi?id=1365 Mike Holmes changed: What|Removed |Added Status|UNCONFIRMED |IN_PROGRESS Ever confirmed|0

[lng-odp] [Bug 1547] Untested API copy_md_to_packet

https://bugs.linaro.org/show_bug.cgi?id=1547 Mike Holmes changed: What|Removed |Added Assignee|lng-odp@lists.linaro.org|alexandru.badicioiu@linaro. |

[lng-odp] [Bug 1615] odp_timer fails in CI with Segmentation fault

https://bugs.linaro.org/show_bug.cgi?id=1615 Maxim Uvarov changed: What|Removed |Added CC||maxim.uva...@linaro.org --- Comment #6 from Max

[lng-odp] [Bug 1722] CID 99100: BUFFER_SIZE_WARNING: /example/classifier/odp_classifier.c

https://bugs.linaro.org/show_bug.cgi?id=1722 Mike Holmes changed: What|Removed |Added Status|UNCONFIRMED |IN_PROGRESS Ever confirmed|0

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

On 07/29/15 17:28, Christophe Milard wrote: 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.

[lng-odp] [Bug 1541] Untested API verify_pmr_packet_len

https://bugs.linaro.org/show_bug.cgi?id=1541 --- Comment #1 from Bala Manoharan --- patch submitted: https://patches.linaro.org/51699/ -- You are receiving this mail because: You are on the CC list for the bug.___ lng-odp mailing list lng-odp@lists.li

[lng-odp] [Bug 1546] Untested API verify_pmr_udp_dport

https://bugs.linaro.org/show_bug.cgi?id=1546 --- Comment #1 from Bala Manoharan --- patch submitted: https://patches.linaro.org/51699/ -- You are receiving this mail because: You are on the CC list for the bug.___ lng-odp mailing list lng-odp@lists.li

[lng-odp] [Bug 1545] Untested API verify_pmr_tcp_dport

https://bugs.linaro.org/show_bug.cgi?id=1545 --- Comment #1 from Bala Manoharan --- patch submitted: https://patches.linaro.org/51699/ -- You are receiving this mail because: You are on the CC list for the bug.___ lng-odp mailing list lng-odp@lists.li

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

Additional test suite is added to classification validation suite to test individual PMRs. This suite will test the defined PMRs by configuring pktio separately for every test case. Signed-off-by: Balasubramanian Manoharan --- helper/include/odp/helper/tcp.h| 4 + test/vali

Re: [lng-odp] [PATCH 3/3] linux-generic: pktio: handle transmit errors correctly

Sturart, test case looks like valuable. Do you plan to update that patch? I think it's better to have test case in separate commit. Maxim. On 04/02/15 14:07, Stuart Haslam wrote: The linux-generic pktio implementations don't correctly handle socket errors which occur while sending packets via

[lng-odp] [Bug 1544] Untested API verify_pmr_tcp_sport

https://bugs.linaro.org/show_bug.cgi?id=1544 --- Comment #1 from Bala Manoharan --- patch submitted: https://patches.linaro.org/51699/ -- You are receiving this mail because: You are on the CC list for the bug.___ lng-odp mailing list lng-odp@lists.li

[lng-odp] [Bug 1542] Untested API verify_pmr_ip_proto

https://bugs.linaro.org/show_bug.cgi?id=1542 --- Comment #1 from Bala Manoharan --- patch submitted: https://patches.linaro.org/51699/ -- You are receiving this mail because: You are on the CC list for the bug.___ lng-odp mailing list lng-odp@lists.li

[lng-odp] [Bug 1722] CID 99100: BUFFER_SIZE_WARNING: /example/classifier/odp_classifier.c

https://bugs.linaro.org/show_bug.cgi?id=1722 --- Comment #1 from Bala Manoharan --- patch submitted: https://patches.linaro.org/51700/ -- You are receiving this mail because: You are on the CC list for the bug.___ lng-odp mailing list lng-odp@lists.li

[lng-odp] [PATCH] example: classifier: fix buffer size warning

Fixes: https://bugs.linaro.org/show_bug.cgi?id=1722 Signed-off-by: Balasubramanian Manoharan --- example/classifier/odp_classifier.c | 8 +--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/example/classifier/odp_classifier.c b/example/classifier/odp_classifier.c index 53c7fd

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

Merged, Maxim. On 07/28/15 14:46, Stuart Haslam wrote: On Tue, Jul 28, 2015 at 01:41:54PM +0200, Christophe Milard wrote: Signed-off-by: Christophe Milard Reviewed-by: Stuart Haslam --- test/validation/cpumask/cpumask.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a

Re: [lng-odp] [PATCHv3] example:generator:printing verbose output

Merged, Maxim. On 07/30/15 13:55, Stuart Haslam wrote: On Wed, Jul 29, 2015 at 04:25:41PM +0200, Balakrishna.Garapati wrote: Signed-off-by: Balakrishna.Garapati Reviewed-by: Stuart Haslam --- Added fflush example/generator/odp_generator.c | 14 +- 1 file changed, 13 insert

Re: [lng-odp] [API-NEXT PATCH] api: crypto: add crypto IPSec extension

On Thu, Jul 30, 2015 at 02:42:08PM +0300, Alexandru Badicioiu wrote: > On 30 July 2015 at 13:50, Stuart Haslam wrote: > > > On Wed, Jul 22, 2015 at 11:26:03AM +0300, alexandru.badici...@linaro.org > > wrote: > > > From: Alexandru Badicioiu > > > > > > This patch adds IPSec protocol processing ca

[lng-odp] [Bug 1675] CID 98149: CID 98148: CID 98147: Integer handling issues

https://bugs.linaro.org/show_bug.cgi?id=1675 Bala Manoharan changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED Resolution|---

[lng-odp] [Bug 1706] odp_term_local() not being called from main thread

https://bugs.linaro.org/show_bug.cgi?id=1706 --- Comment #1 from Stuart Haslam --- I had a quick look at this and it's not just a matter of adding the missing calls. The odp_term_ functions report errors if things haven't been cleaned up properly before calling them, for example if there are some

Re: [lng-odp] RFC - Application counters helpers

Hi, Yes. The above use-case makes sense. Also we need to document explicit limitation that these variables should be created before starting the worker threads and that dynamic creation of variables should be avoided. Regards, Bala On 30 July 2015 at 13:29, Alexandru Badicioiu < alexandru.badici

Re: [lng-odp] [API-NEXT PATCH] api: crypto: add crypto IPSec extension

From: Alexandru Badicioiu Sent: Thursday, July 30, 2015 6:44 PM To: Jacob, Jerin Cc: LNG ODP Mailman List Subject: Re: [lng-odp] [API-NEXT PATCH] api: crypto: add crypto IPSec extension On 30 July 2015 at 16:00, Jerin Jacob mailto:jerin.ja...@caviumnetworks.

Re: [lng-odp] [API-NEXT PATCH] api: crypto: add crypto IPSec extension

On Wed, Jul 22, 2015 at 11:26:03AM +0300, alexandru.badici...@linaro.org wrote: > From: Alexandru Badicioiu > > This patch adds IPSec protocol processing capabilities to crypto > sesssions. Implementations which have these capabilities in hardware > crypto engines can use the extension to offload

Re: [lng-odp] [API-NEXT PATCH] api: crypto: add crypto IPSec extension

On 30 July 2015 at 13:50, Stuart Haslam wrote: > On Wed, Jul 22, 2015 at 11:26:03AM +0300, alexandru.badici...@linaro.org > wrote: > > From: Alexandru Badicioiu > > > > This patch adds IPSec protocol processing capabilities to crypto > > sesssions. Implementations which have these capabilities i

[lng-odp] [PATCHv3] example:generator:option to supply core mask

Signed-off-by: Balakrishna.Garapati --- changed the command line argument conventions. example/generator/odp_generator.c | 57 ++- 1 file changed, 39 insertions(+), 18 deletions(-) diff --git a/example/generator/odp_generator.c b/example/generator/odp_genera

Re: [lng-odp] [PATCH] example:generator:option to supply core mask

I agree. I can make the changes. On 30 July 2015 at 12:58, Stuart Haslam wrote: > On Thu, Jul 30, 2015 at 10:02:46AM +0200, Balakrishna.Garapati wrote: > > Signed-off-by: Balakrishna.Garapati > > --- > > This patch also includes changes to enable the option to support number > of workers. > >

[lng-odp] [PATCHv2] example:generator:option to supply core mask

Signed-off-by: Balakrishna.Garapati --- Fixed cpu_mask for PING mode. The mask is set to what is actually supplied. example/generator/odp_generator.c | 51 ++- 1 file changed, 39 insertions(+), 12 deletions(-) diff --git a/example/generator/odp_generator.c

Re: [lng-odp] [PATCH] example:generator:option to supply core mask

On Thu, Jul 30, 2015 at 10:02:46AM +0200, Balakrishna.Garapati wrote: > Signed-off-by: Balakrishna.Garapati > --- > This patch also includes changes to enable the option to support number of > workers. > example/generator/odp_generator.c | 35 +-- > 1 file change

Re: [lng-odp] [PATCHv1] example:generator:option to supply core mask

Ignore this patch. Pushing another one with PATCHv2. On 30 July 2015 at 12:54, Balakrishna.Garapati < balakrishna.garap...@linaro.org> wrote: > Signed-off-by: Balakrishna.Garapati > --- > > Fixed cpu_mask for PING mode. The mask is set to what is actually > supplied. > > example/generator/odp_

Re: [lng-odp] [PATCHv3] example:generator:printing verbose output

On Wed, Jul 29, 2015 at 04:25:41PM +0200, Balakrishna.Garapati wrote: > Signed-off-by: Balakrishna.Garapati Reviewed-by: Stuart Haslam > --- > Added fflush > example/generator/odp_generator.c | 14 +- > 1 file changed, 13 insertions(+), 1 deletion(-) > > diff --git a/example/gene

[lng-odp] [PATCHv1] example:generator:option to supply core mask

Signed-off-by: Balakrishna.Garapati --- Fixed cpu_mask for PING mode. The mask is set to what is actually supplied. example/generator/odp_generator.c | 51 ++- 1 file changed, 39 insertions(+), 12 deletions(-) diff --git a/example/generator/odp_generator.c

Re: [lng-odp] [API-NEXT PATCH 2/5] linux-generic: schedule: implement scheduler groups

I've fixed these in v2. Also added is the new API odp_schedule_group_count() that gives the number of threads belonging to a schedule group. On Thu, Jul 30, 2015 at 1:12 AM, Mario Torrecillas Rodriguez < mario.torrecillasrodrig...@arm.com> wrote: > The problem with using count as a way of determ

[lng-odp] [API-NEXT PATCHv2 5/5] api: schedule: revised definition of odp_schedule_release_ordered

Signed-off-by: Bill Fischofer --- include/odp/api/schedule.h | 24 +--- 1 file changed, 13 insertions(+), 11 deletions(-) diff --git a/include/odp/api/schedule.h b/include/odp/api/schedule.h index 95fc8df..0c3d8ba 100644 --- a/include/odp/api/schedule.h +++ b/include/odp/api/

[lng-odp] [API-NEXT PATCHv2 4/5] linux-generic: queue: implement ordered queues

Signed-off-by: Bill Fischofer --- .../linux-generic/include/odp_buffer_internal.h| 5 + .../linux-generic/include/odp_queue_internal.h | 4 + .../linux-generic/include/odp_schedule_internal.h | 7 + platform/linux-generic/odp_pool.c | 3 + platform/linux-generic/

[lng-odp] [API-NEXT PATCHv2 3/5] linux-generic: schedule: implement scheduler groups

Signed-off-by: Bill Fischofer --- include/odp/api/config.h | 5 + .../include/odp/plat/schedule_types.h | 4 + platform/linux-generic/odp_schedule.c | 158 + platform/linux-generic/odp_thread.c| 25 +++-

[lng-odp] [API-NEXT PATCHv2 2/5] validation: schedule: initialize queue and pool parameters

Signed-off-by: Bill Fischofer --- test/validation/scheduler/scheduler.c | 4 1 file changed, 4 insertions(+) diff --git a/test/validation/scheduler/scheduler.c b/test/validation/scheduler/scheduler.c index fe03ab7..bbe829e 100644 --- a/test/validation/scheduler/scheduler.c +++ b/test/valid

[lng-odp] [API-NEXT PATCHv2 1/5] api: schedule: additional scheduler group functions

Signed-off-by: Bill Fischofer --- include/odp/api/schedule.h | 55 ++ 1 file changed, 55 insertions(+) diff --git a/include/odp/api/schedule.h b/include/odp/api/schedule.h index 36e52cd..95fc8df 100644 --- a/include/odp/api/schedule.h +++ b/include/odp

[lng-odp] [API-NEXT PATCHv2 0/5] Scheduler API Enhancements

Changes in v2 - Incorporate Stuart's review comments - Add odp_schedule_group_count() API to return the count of threads belonging to a schedule group -- This patch series implements (most of) Petri's latest scheduler changes. Major changes include: 1. Implement scheduler groups, including ad

Re: [lng-odp] [PATCHv2] example:generator:printing verbose output

Forget that, I didn't notice he had sent a v3. Stuart. On 30 July 2015 at 11:51, Stuart Haslam wrote: > On Thu, Jul 30, 2015 at 11:56:42AM +0300, Maxim Uvarov wrote: > > On 07/29/15 17:40, Stuart Haslam wrote: > > >I mentioned that (in an off-list mail) to Krishna but said I thought it > > >cou

Re: [lng-odp] [PATCHv2] example:generator:printing verbose output

On Thu, Jul 30, 2015 at 11:56:42AM +0300, Maxim Uvarov wrote: > On 07/29/15 17:40, Stuart Haslam wrote: > >I mentioned that (in an off-list mail) to Krishna but said I thought it > >could be changed later, as it's complicated slightly by the different > >modes you can be running in and I figured it

Re: [lng-odp] [API-NEXT PATCH] api: crypto: add crypto IPSec extension

On Wed, Jul 22, 2015 at 11:26:03AM +0300, alexandru.badici...@linaro.org wrote: > From: Alexandru Badicioiu > > This patch adds IPSec protocol processing capabilities to crypto > sesssions. Implementations which have these capabilities in hardware > crypto engines can use the extension to offload

Re: [lng-odp] [API-NEXT PATCH] api: crypto: add crypto IPSec extension

Hi Maxim, I noticed that but I wanted to know if there are some other comments more oriented to the functionality this patch proposes. Thanks, Alex On 30 July 2015 at 12:21, Maxim Uvarov wrote: > On 07/30/15 12:04, Alexandru Badicioiu wrote: > >> Ping. >> > > Alex, Mike replayed to you to fix m

Re: [lng-odp] [API-NEXT PATCH] api: crypto: add crypto IPSec extension

On 07/30/15 12:04, Alexandru Badicioiu wrote: Ping. Alex, Mike replayed to you to fix minor fixes. Maxim. On 22 July 2015 at 11:26, > wrote: From: Alexandru Badicioiu mailto:alexandru.badici...@linaro.org>> This patch adds IPSec protocol proc

Re: [lng-odp] [API-NEXT PATCH] api: crypto: add crypto IPSec extension

Ping. On 22 July 2015 at 11:26, wrote: > From: Alexandru Badicioiu > > This patch adds IPSec protocol processing capabilities to crypto > sesssions. Implementations which have these capabilities in hardware > crypto engines can use the extension to offload the application from > IPSec protocol

Re: [lng-odp] [PATCHv2] example:generator:printing verbose output

On 07/29/15 17:40, Stuart Haslam wrote: I mentioned that (in an off-list mail) to Krishna but said I thought it could be changed later, as it's complicated slightly by the different modes you can be running in and I figured it's not worth delaying this change over. It's questionable though. Ok,

Re: [lng-odp] [PATCH] example:generator:option to supply core mask

just for the info, This patch only enables and sets the cpu masks correctly. Still working on the patch to fix the PING mode to run on the supplied cpu mask instead of hard coded cpu0. On 30 July 2015 at 10:02, Balakrishna.Garapati < balakrishna.garap...@linaro.org> wrote: > Signed-off-by: Balakr

Re: [lng-odp] [API-NEXT PATCH 2/5] linux-generic: schedule: implement scheduler groups

The problem with using count as a way of determining whether the slot is unallocated or not is that count keeps the current number of threads in the group. If a group is created with no threads in it (which is allowed by the implementation as far as I can see), then no thread will be able to joi

[lng-odp] [PATCH] example:generator:option to supply core mask

Signed-off-by: Balakrishna.Garapati --- This patch also includes changes to enable the option to support number of workers. example/generator/odp_generator.c | 35 +-- 1 file changed, 29 insertions(+), 6 deletions(-) diff --git a/example/generator/odp_generator.

Re: [lng-odp] RFC - Application counters helpers

Bala, I agree that functions to subtract from a counter or reset it to 0 or other value might be required for the completeness. I don't see these functions as APIs (since the implementation is the same for all HW) but only as an application helper and only in case the application uses threads as wo