Re: [lng-odp] [PATCHv7 1/2] example: introducing l3fwd

2016-08-15 Thread Bill Fischofer
Do we want to discuss this during tomorrow's ODP public call? I'll add it to the agenda. On Mon, Aug 15, 2016 at 6:36 AM, Elo, Matias (Nokia - FI/Espoo) < matias@nokia-bell-labs.com> wrote: > Hi, > > ODP_ASSERT isn’t part of the public API. After removing it the application > builds

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

2016-08-15 Thread Maxim Uvarov
Hello Anders, do you agree with current order? My answer was bellow. Thank you, Maxim. On 08/09/16 15:46, Maxim Uvarov wrote: On 08/06/16 22:30, Anders Roxell wrote: --- a/test/linux-generic/m4/configure.m4 >+++ b/test/linux-generic/m4/configure.m4 >@@ -1,5 +1,6 @@ >

Re: [lng-odp] [PATCH] linux-gen: sched: keep scheduling over empty queues

2016-08-15 Thread Brian Brooks
Tested-by: Brian Brooks On 08/15 15:21:41, Petri Savolainen wrote: > Empty queues are kept in scheduler priority queues for > performance reasons. The scheduler moved to the next priority > queue after seeing an empty queue, which resulted sometimes > scheduler to return

Re: [lng-odp] [PATCH] linux-gen: sched: keep scheduling over empty queues

2016-08-15 Thread Savolainen, Petri (Nokia - FI/Espoo)
> -Original Message- > From: lng-odp [mailto:lng-odp-boun...@lists.linaro.org] On Behalf Of Maxim > Uvarov > Sent: Monday, August 15, 2016 4:46 PM > To: lng-odp@lists.linaro.org > Subject: Re: [lng-odp] [PATCH] linux-gen: sched: keep scheduling over > empty queues > > On 08/15/16 15:21,

Re: [lng-odp] [PATCH] linux-gen: sched: keep scheduling over empty queues

2016-08-15 Thread Maxim Uvarov
On 08/15/16 15:21, Petri Savolainen wrote: if (num < 0) { - /* Destroyed queue */ + /* Destroyed queue. Continue scheduling the same +* priority queue. */

[lng-odp] [Bug 2437] LAVA: FAIL: odp_timer_simple Juno

2016-08-15 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=2437 --- Comment #4 from Mike Holmes --- Need to add a capability structure for the test to queuerie the resolution Need to validate the requested resolution if possible Current patch looks sensible for Monarch -- You are

[lng-odp] [Bug 2437] LAVA: FAIL: odp_timer_simple Juno

2016-08-15 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=2437 Mike Holmes changed: What|Removed |Added Status|UNCONFIRMED |IN_PROGRESS Ever

[lng-odp] [Bug 2456] ODP name string lengths are multiply defined across the API

2016-08-15 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=2456 Mike Holmes changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED

Re: [lng-odp] [PATCH] linux-gen: sched: keep scheduling over empty queues

2016-08-15 Thread Maxim Uvarov
fixes issue in my environment. Maxim. On 08/15/16 15:21, Petri Savolainen wrote: Empty queues are kept in scheduler priority queues for performance reasons. The scheduler moved to the next priority queue after seeing an empty queue, which resulted sometimes scheduler to return zero events,

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

2016-08-15 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=2457 --- Comment #4 from Bill Fischofer --- Patch http://patches.opendataplane.org/patch/6807/ posted -- You are receiving this mail because: You are on the CC list for the bug.

[lng-odp] [PATCH] linux-gen: sched: keep scheduling over empty queues

2016-08-15 Thread Petri Savolainen
Empty queues are kept in scheduler priority queues for performance reasons. The scheduler moved to the next priority queue after seeing an empty queue, which resulted sometimes scheduler to return zero events, although there were events available deeper in the priority queue. This patch keeps

[lng-odp] [Bug 2458] Traffic Manager example crashes

2016-08-15 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=2458 Bill Fischofer changed: What|Removed |Added Resolution|--- |FIXED

Re: [lng-odp] [PATCHv7 1/2] example: introducing l3fwd

2016-08-15 Thread Elo, Matias (Nokia - FI/Espoo)
Hi, ODP_ASSERT isn’t part of the public API. After removing it the application builds correctly. The argument parsing issue is now fixed. However, after the ODP_PKTIO_OP_MT -> ODP_PKTIO_OP_MT_UNSAFE fix the TX side is now broken. Syslog is full of error messages like: nm_txsync_prologue

Re: [lng-odp] [API-NEXT PATCH] test: drv: atomic renamed to drvatomic

2016-08-15 Thread Maxim Uvarov
On 08/12/16 20:30, Mike Holmes wrote: +#ifndef_ODP_TEST_ATOMIC_H_ >+#define_ODP_TEST_ATOMIC_H_ drv prefix here is missing. Maxim.

Re: [lng-odp] [PATCHv2] linux-generic: tm: handle pktout queue check properly

2016-08-15 Thread Maxim Uvarov
Merged, Maxim. On 08/12/16 21:08, Mike Holmes wrote: On 10 August 2016 at 16:41, Bill Fischofer wrote: Resolve bug https://bugs.linaro.org/show_bug.cgi?id=2458 by only checking for a proper pktout_queue count if the egress_kind is ODP_TM_EGRESS_PKT_IO. This check

Re: [lng-odp] [PATCHv7 1/2] example: introducing l3fwd

2016-08-15 Thread Elo, Matias (Nokia - FI/Espoo)
Hi Forrest, No packets are forwarded unless the ‘-d’ option with a value greater than zero is given. I’m using an external Ixia traffic generator. E.g. No packets forwarded: sudo ./example/l3fwd/odp_l3fwd -i p2p1,p1p1 -r 1.1.1.0/24,p2p1 -r 2.2.2.0/24,p1p1 -t 1 Packets forwarded: sudo