On 29 October 2015 at 21:25, Maxim Uvarov wrote:
> On 10/29/2015 13:51, Hongbo Zhang wrote:
>>
>> This patch is for https://bugs.linaro.org/show_bug.cgi?id=1870, and
>> has been tested on Juno.
>>
>> On 29 October 2015 at 18:45, wrote:
>>>
>>> From: Hongbo Zhang
>>>
>>> In the default dummy fun
Add tests for validating odp_spinlock_recursive and odp_rwlock_recursive
lock types.
Signed-off-by: Bill Fischofer
---
test/validation/synchronizers/synchronizers.c | 386 ++
test/validation/synchronizers/synchronizers.h | 6 +
2 files changed, 392 insertions(+)
diff -
rwlock functional tests test for write locks, however they should also
test for read locks.
Signed-off-by: Bill Fischofer
---
test/validation/synchronizers/synchronizers.c | 13 +
1 file changed, 13 insertions(+)
diff --git a/test/validation/synchronizers/synchronizers.c
b/test/val
Test of Rx on a stopped pktio was broken due to a missing:
+ pkt = odp_packet_from_event(ev);
pkt value was the last return for odp_packet_alloc so
in the setup ODP_PACKET_INVALID which made it work. However changing
the size of the pool to > 100 changed pkt value so
https://bugs.linaro.org/show_bug.cgi?id=1365
--- Comment #7 from Mike Holmes ---
latest version v5 http://patches.opendataplane.org/patch/3468/
--
You are receiving this mail because:
You are on the CC list for the bug.___
lng-odp mailing list
lng-odp
https://bugs.linaro.org/show_bug.cgi?id=1594
--- Comment #1 from Mike Holmes ---
This has not been reproduced, a detailed description on how to expose the issue
with the current release is needed.
--
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list f
https://bugs.linaro.org/show_bug.cgi?id=1615
--- Comment #19 from Mike Holmes ---
CI is not showing this bug currently, will close next week unless there is
feedback.
--
You are receiving this mail because:
You are on the CC list for the bug.___
lng-o
https://bugs.linaro.org/show_bug.cgi?id=1864
Mike Holmes changed:
What|Removed |Added
Status|UNCONFIRMED |RESOLVED
Resolution|---
https://bugs.linaro.org/show_bug.cgi?id=1866
Mike Holmes changed:
What|Removed |Added
Assignee|lng-odp@lists.linaro.org|bill.fischo...@linaro.org
CC|
On Thu, Oct 29, 2015 at 9:52 AM, Petri Savolainen <
petri.savolai...@nokia.com> wrote:
> Added validation tests for ODP std C library API.
>
> Signed-off-by: Petri Savolainen
>
Reviewed-and-Tested-by: Bill Fischofer
> ---
> configure.ac | 1 +
> platform/linux-ge
Added validation tests for ODP std C library API.
Signed-off-by: Petri Savolainen
---
configure.ac | 1 +
platform/linux-generic/test/Makefile.am | 1 +
test/validation/Makefile.am | 1 +
test/validation/std_clib/.gitignore | 1 +
test/validatio
Its wrong to associate the maximum number of threads to the number of
CPUs available.
Signed-off-by: Anders Roxell
---
platform/linux-generic/odp_thrmask.c | 5 -
1 file changed, 5 deletions(-)
diff --git a/platform/linux-generic/odp_thrmask.c
b/platform/linux-generic/odp_thrmask.c
index a
The API definition of a CPU mask must not be incompatible with the
platform it is implemented on
Signed-off-by: Anders Roxell
---
platform/linux-generic/odp_cpumask.c | 4
1 file changed, 4 insertions(+)
diff --git a/platform/linux-generic/odp_cpumask.c
b/platform/linux-generic/odp_cpumas
On 10/29/2015 13:51, Hongbo Zhang wrote:
This patch is for https://bugs.linaro.org/show_bug.cgi?id=1870, and
has been tested on Juno.
On 29 October 2015 at 18:45, wrote:
From: Hongbo Zhang
In the default dummy function systemcpu(), only cpu_hz[0] and model_str[0]
are set to dummy values, th
Allocate cacheline aligned shared memory for per thread
statistics and simplify stats pointer references.
Signed-off-by: Petri Savolainen
---
test/performance/odp_l2fwd.c | 63
1 file changed, 34 insertions(+), 29 deletions(-)
diff --git a/test/perfo
Separated receive and transmit drop counters to
help debugging the cause of dropped packets.
Signed-off-by: Petri Savolainen
---
test/performance/odp_l2fwd.c | 43 +++
1 file changed, 27 insertions(+), 16 deletions(-)
diff --git a/test/performance/odp_l2f
Increased burst size to 32. This improves packet rate about
15% with netmap, since system call rate is halved.
No harm to any implementation, since implementation decides
how many packets are returned. This is comparable to DPDK l2fwd
burst size.
Signed-off-by: Petri Savolainen
---
test/performa
Queue mode works the same way as direct mode, but uses
odp_schedule_multi for packet input. This improves packet
rate about 40% (with netmap). Only scheduler overhead is now
added which makes it easy to compare direct vs. scheduler inputs
options.
Output through queues or TM can be added as a thir
Added command line option to disable packet error check. Error
check requires full packet parse. Max packet rate measurements
should be done with minimal feature set.
This change gives +10% packet rate increase in direct recv mode
with netmap.
Signed-off-by: Petri Savolainen
---
test/performanc
On 10/28/2015 22:29, Stuart Haslam wrote:
Replace the nm_inject() helper with our own version so that
odp_packet_copydata_out() can be used as this understands segmented
packets.
Signed-off-by: Stuart Haslam
---
platform/linux-generic/pktio/netmap.c | 59 ---
If you add prefix, then getenv() has to be removed. And previous patch
looks like obsolete.
Maxim.
On 10/28/2015 22:29, Stuart Haslam wrote:
The netmap API requires interface names to be prefixed with "netmap:",
the pktio code will do that automatically, e.g. odp_pktio_open("eth0")
becomes nm_
https://bugs.linaro.org/show_bug.cgi?id=1873
Maxim Uvarov changed:
What|Removed |Added
CC||maxim.uva...@linaro.org
--- Comment #1 from Max
Hello Matias,
this test does not test anything. In many validation tests
create_pktio() helper exist.
I think it's reasonable to just add odp_pktio_print() just after pktio
creation in all tests.
So that from log it will be clear which pktio was used for testing.
Thanks,
Maxim.
On 10/28/2015
Reviewed-by: Balasubramanian Manoharan
On 29 October 2015 at 16:38, Maxim Uvarov wrote:
> Looks like merge to api-next, cherry-pick from api-next
> and merge master to api-next produced duplicate patch lines.
> Found that with inspecting branches with 'git diff master api-next'
>
> Signed-off-by
Looks like merge to api-next, cherry-pick from api-next
and merge master to api-next produced duplicate patch lines.
Found that with inspecting branches with 'git diff master api-next'
Signed-off-by: Maxim Uvarov
---
example/classifier/odp_classifier.c | 5 -
1 file changed, 5 deletions(-)
Please _DO_NOT_ merge this one;
I have sent out an updated version today: https://patches.linaro.org/55776/
On 20 October 2015 at 20:38, wrote:
> From: Hongbo Zhang
>
> In the default dummy function systemcpu(), only cpu_hz[0] and model_str[0]
> are set to dummy values, then in the validation
This patch is for https://bugs.linaro.org/show_bug.cgi?id=1870, and
has been tested on Juno.
On 29 October 2015 at 18:45, wrote:
> From: Hongbo Zhang
>
> In the default dummy function systemcpu(), only cpu_hz[0] and model_str[0]
> are set to dummy values, then in the validation code if iterate
From: Hongbo Zhang
In the default dummy function systemcpu(), only cpu_hz[0] and model_str[0]
are set to dummy values, then in the validation code if iterate each CPU,
cores other than core 0 report failure, this patchs pad all the arrays to
default values to pass validation.
Signed-off-by: Hong
Note that my Bug fix patch for Bug 1824 also touches this module. Not sure
if there are merge conflicts between these two.
On Thu, Oct 29, 2015 at 5:24 AM, Maxim Uvarov
wrote:
> odp_queue_lock_count() return int not uint32_t where -1
> is assigned for error.
> Bug: https://bugs.linaro.org/show_
OK. I'll send it shortly.
-Petri
> -Original Message-
> From: EXT Nicolas Morey-Chaisemartin [mailto:nmo...@kalray.eu]
> Sent: Thursday, October 29, 2015 12:18 PM
> To: Savolainen, Petri (Nokia - FI/Espoo); lng-odp@lists.linaro.org
> Subject: Re: [lng-odp] [PATCHv2] test: l2fwd: add extra
Hi Petri,
Let's clarify some aspects that can be taken into account if common time type
is used
for global and local times. Mainly it reflected in operation/conversion
functions.
Mostly that is why I want to have separate functions for global and local times.
Please say what do you think about
https://bugs.linaro.org/show_bug.cgi?id=1867
Maxim Uvarov changed:
What|Removed |Added
CC||maxim.uva...@linaro.org
--- Comment #5 from Max
odp_queue_lock_count() return int not uint32_t where -1
is assigned for error.
Bug: https://bugs.linaro.org/show_bug.cgi?id=1867
Signed-off-by: Maxim Uvarov
---
test/validation/scheduler/scheduler.c | 22 +++---
1 file changed, 15 insertions(+), 7 deletions(-)
diff --git a/test/
It's fine by me.
Yes that's the point. Depending on the traffic source, (ie DPDPK on some HW),
there tend to be a lot of Rx errors.
This make it easier to know if Tx saturates, or the packet generator has issues.
Nicolas
On 10/29/2015 11:15 AM, Savolainen, Petri (Nokia - FI/Espoo) wrote:
> I'm
I'm just optimizing/cleaning l2fwd and this patch would conflict.
I can add it to my series. So, you basically want to separate and print rx_ and
tx_errors?
-Petri
> -Original Message-
> From: lng-odp [mailto:lng-odp-boun...@lists.linaro.org] On Behalf Of EXT
> Nicolas Morey-Chaisemart
Drop packets counts both packets dropped by calls to pktio_send()
and packet which were flagged with errors during parsing.
Add an extra counter to differ these two types of packets.
Signed-off-by: Nicolas Morey-Chaisemartin
---
v2:
Rebased on latest master
test/performance/odp_l2fwd.c | 17
Merged,
Maxim.
On 10/23/2015 11:02, Matias Elo wrote:
Netmap interface takes a few seconds to become active after
calling nm_open(). This caused several test applications to
fail. Check link status at the end of netmap_open() to fix
this.
Signed-off-by: Matias Elo
---
v3:
- Rebased to master
Recursive locks are per thread. CPU ids are per CPU. So, recursive lock
implementation needs still max thread count. But that's implementation, not an
API change.
-Petri
> -Original Message-
> From: lng-odp [mailto:lng-odp-boun...@lists.linaro.org] On Behalf Of EXT
> Maxim Uvarov
> Sent
Merged to api-next. Needed to do the same for rwlock_recursive_types.h.
Will merge patches to master soon.
Maxim.
On 10/29/2015 10:32, Savolainen, Petri (Nokia - FI/Espoo) wrote:
Both patches:
Reviewed-by: Petri Savolainen
-Original Message-
From: EXT Anders Roxell [mailto:anders
Both patches:
Reviewed-by: Petri Savolainen
> -Original Message-
> From: EXT Anders Roxell [mailto:anders.rox...@linaro.org]
> Sent: Wednesday, October 28, 2015 4:33 PM
> To: Orpana, Pasi (Nokia - FI/Espoo); Wallen, Carl (Nokia - FI/Espoo)
> Cc: lng-odp@lists.linaro.org; Savolainen, Pet
On 10/29/2015 10:02, huanggaoyang wrote:
Signed-off-by: huanggaoyang
---
helper/include/odp/helper/table.h | 222 ++
1 file changed, 222 insertions(+)
create mode 100644 helper/include/odp/helper/table.h
diff --git a/helper/include/odp/helper/table.h
b/
Stuart,
That is quite big changes. I merged them to api-next to run CI tests
first, if everything will
be ok, I will merge it to master.
Maxim.
On 10/27/2015 19:42, Stuart Haslam wrote:
Fixes bug: https://bugs.linaro.org/show_bug.cgi?id=1365
Changes since v4:
- Fix seg fault due to unhand
Signed-off-by: huanggaoyang
---
helper/Makefile.am | 5 -
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/helper/Makefile.am b/helper/Makefile.am
index 1a74e8e..196d887 100644
--- a/helper/Makefile.am
+++ b/helper/Makefile.am
@@ -17,6 +17,7 @@ helperinclude_HEADERS = \
v1: helper: added base api of Table
added two impls of Table:hash table & linear table
v2: 1. fix mistakes in Copyright Info of files
2. change the name of an useless param of function odph_linear_table_create
to ODP_IGNORED
v3: update comments, make the description more c
Signed-off-by: huanggaoyang
---
helper/include/odp/helper/table.h | 222 ++
1 file changed, 222 insertions(+)
create mode 100644 helper/include/odp/helper/table.h
diff --git a/helper/include/odp/helper/table.h
b/helper/include/odp/helper/table.h
new file mod
Signed-off-by: huanggaoyang
---
helper/test/Makefile.am | 5 +-
helper/test/odp_table.c | 134
2 files changed, 138 insertions(+), 1 deletion(-)
create mode 100644 helper/test/odp_table.c
diff --git a/helper/test/Makefile.am b/helper/test/Makef
Signed-off-by: huanggaoyang
---
helper/lineartable.c | 211 ++
helper/odph_lineartable.h | 41 +
2 files changed, 252 insertions(+)
create mode 100644 helper/lineartable.c
create mode 100644 helper/odph_lineartable.h
diff --git a/helper
Signed-off-by: huanggaoyang
---
helper/hashtable.c | 346
helper/odph_hashtable.h | 42 ++
helper/odph_list_internal.h | 85 +++
3 files changed, 473 insertions(+)
create mode 100644 helper/hashtable.c
create mode 100644 he
48 matches
Mail list logo