Re: [lng-odp] [MONARCH_LTS PATCH PATCH 2/2] configure.ac update version

2016-08-08 Thread Bill Fischofer
The ODP API has changed between v1.10.1.0 and v1.11.0.0 so we do need to change the .so indicators. On Mon, Aug 8, 2016 at 2:01 AM, Maxim Uvarov wrote: > On 08/05/16 21:27, Anders Roxell wrote: > >> On 2016-08-03 11:40, Maxim Uvarov wrote: >> >>> Signed-off-by: Maxim

Re: [lng-odp] [PATCH] doc: release-guide: elaborate release naming scheme

2016-08-08 Thread Bill Fischofer
On Mon, Aug 8, 2016 at 10:53 AM, Mike Holmes wrote: > Looks good, one comment below > > On 8 August 2016 at 10:52, Bill Fischofer > wrote: > >> Add additional detail and examples to the description of the release >> naming scheme used by ODP,

Re: [lng-odp] [PATCH] doc: release-guide: add LTS details

2016-08-08 Thread Bill Fischofer
As an aside, this patch and my patch to this file collide, so we need to figure out who goes first and who gets to rebase. On Mon, Aug 8, 2016 at 10:34 AM, Mike Holmes wrote: > Signed-off-by: Mike Holmes > --- >

Re: [lng-odp] [PATCH] linux-gen: do not init scheduler context on termination

2016-08-08 Thread Bill Fischofer
I can't say whether this fixes Bug 2457, but this certainly seems like a bug worth correcting to me. Would like Petri's opinion on this. But in the meantime... On Mon, Aug 8, 2016 at 3:40 AM, Maxim Uvarov wrote: > We should not try to redifine current odp thread id on >

Re: [lng-odp] [PATCH v2 2/2] linux-generic: tm: made tm thread creation more flexible

2016-08-08 Thread Bill Fischofer
Sorry for not testing this earlier, but while this works fine on 64-bit systems, it fails to compile on 32-bit systems: CC odp_traffic_mngr.lo odp_traffic_mngr.c: In function ‘_odp_tm_group_create’: odp_traffic_mngr.c:2646:9: error: cast from pointer to integer of different size

Re: [lng-odp] [PATCH] doc: release-guide: elaborate release naming scheme

2016-08-08 Thread Mike Holmes
Looks good, one comment below On 8 August 2016 at 10:52, Bill Fischofer wrote: > Add additional detail and examples to the description of the release > naming scheme used by ODP, detailing the expected impact to ODP > applications and/or other ODP implementations for

[lng-odp] [PATCH] doc: release-guide: add LTS details

2016-08-08 Thread Mike Holmes
Signed-off-by: Mike Holmes --- doc/process-guide/release-guide.adoc | 53 1 file changed, 53 insertions(+) diff --git a/doc/process-guide/release-guide.adoc b/doc/process-guide/release-guide.adoc index 8816c03..c9b8484 100644 ---

[lng-odp] [PATCH] doc: release-guide: elaborate release naming scheme

2016-08-08 Thread Bill Fischofer
Add additional detail and examples to the description of the release naming scheme used by ODP, detailing the expected impact to ODP applications and/or other ODP implementations for each change. Signed-off-by: Bill Fischofer --- doc/process-guide/release-guide.adoc |

[lng-odp] [Bug 2457] odp_scheduling fails under load.

2016-08-08 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=2457 --- Comment #2 from Mike Holmes --- Maxim posted a patch that may fix this -- You are receiving this mail because: You are on the CC list for the bug.

Re: [lng-odp] [API-NEXT PATCH] linux-gen: drv: byteorder: fixed macro typo

2016-08-08 Thread Maxim Uvarov
Merged, Maxim. On 08/04/16 16:40, Mike Holmes wrote: On 4 August 2016 at 04:51, Christophe Milard wrote: Signed-off-by: Christophe Milard Reviewed-by: Mike Holmes ---

Re: [lng-odp] [PATCH v2 1/2] linux-generic: tm: fixed regression when egress changed to use pktio

2016-08-08 Thread Maxim Uvarov
On 08/08/16 13:50, Bill Fischofer wrote: I posted http://patches.opendataplane.org/patch/6784/ last night that I believe is the correct fix to Bug 2458. Part 2 of this series is the fix for Bugs 2444, and perhaps 2449 as well. yes, Bills fix is more clear with locks. Barry, do you want to

Re: [lng-odp] [PATCH v2 1/2] linux-generic: tm: fixed regression when egress changed to use pktio

2016-08-08 Thread Bill Fischofer
I posted http://patches.opendataplane.org/patch/6784/ last night that I believe is the correct fix to Bug 2458. Part 2 of this series is the fix for Bugs 2444, and perhaps 2449 as well. On Mon, Aug 8, 2016 at 1:47 AM, Maxim Uvarov wrote: > On 08/08/16 05:45, Barry

[lng-odp] [PATCH] linux-gen: do not init scheduler context on termination

2016-08-08 Thread Maxim Uvarov
We should not try to redifine current odp thread id on termination as well as set poll queues to invalid values, which has to be cleared later in schedule_term_global(). Signed-off-by: Maxim Uvarov --- platform/linux-generic/odp_schedule.c | 2 -- 1 file changed, 2

Re: [lng-odp] [MONARCH_LTS PATCH PATCH 1/2] scripts/git_hash.sh: to support monarch_lts tags

2016-08-08 Thread Maxim Uvarov
On 08/05/16 21:33, Anders Roxell wrote: On 2016-08-03 11:40, Maxim Uvarov wrote: Signed-off-by: Maxim Uvarov --- scripts/git_hash.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/scripts/git_hash.sh b/scripts/git_hash.sh index 336eb01..83123f4

Re: [lng-odp] [MONARCH_LTS PATCH PATCH 2/2] configure.ac update version

2016-08-08 Thread Maxim Uvarov
On 08/05/16 21:27, Anders Roxell wrote: On 2016-08-03 11:40, Maxim Uvarov wrote: Signed-off-by: Maxim Uvarov --- configure.ac | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/configure.ac b/configure.ac index c0eb207..bd16370 100644 ---

Re: [lng-odp] [PATCHv2] test: linux-gen: add pcap playback test

2016-08-08 Thread Maxim Uvarov
Hello Anders, thanks for review, there are 2 comments bellow. Will send v2. Maxim. On 08/06/16 22:30, Anders Roxell wrote: On 2016-08-02 19:08, Maxim Uvarov wrote: add pcap play back test which takes 2 arguments: 1 - pcap file, 2 - packet mask to match. Intend is to test odp with different

Re: [lng-odp] [PATCH v2 1/2] linux-generic: tm: fixed regression when egress changed to use pktio

2016-08-08 Thread Maxim Uvarov
On 08/08/16 05:45, Barry Spinney wrote: Fixes a crash in the traffic_mgr example caused by the odp_tm_create() call failing because that code uses the ODP_TM_EGRESS_FN form which was not being checked for. Signed-off-by: Barry Spinney ---