[lng-odp] [PATCHv2] linux-generic: test: shmem: atomic check+open fifo

2016-03-29 Thread Christophe Milard
Fix for bug 2146, CID 159395: The open system call is directely used to check the presence of the fifo and open it at the same time. Signed-off-by: Christophe Milard --- since v1: changed loop to avoid open() line duplication (Maxim) platform/linux-generic/test/shmem/shmem_linux.c | 8 +---

[lng-odp] [PATCH] linux-generic: sysinfo: handle missing @ sign in CPU model

2016-03-29 Thread Juha-Matti Tilli
Previously, the sysinfo cpuinfo parser assumed that every CPU model has an @ sign, after which comes the maximum CPU frequency. However, many environments such as virtualized KVM environments have CPU models not having @ sign such as "Intel Xeon E312xx (Sandy Bridge)". The result of the missing @

[lng-odp] [Bug 2122] CID 158533: Integer handling issues odp_timer_wheel.c

2016-03-29 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=2122 Bala Manoharan changed: What|Removed |Added Resolution|--- |FIXED Status|UNCONFIRMED

Re: [lng-odp] [PATCH] gitignore: add .deb is .tar.gz

2016-03-29 Thread Bill Fischofer
On Tue, Mar 29, 2016 at 9:52 AM, Maxim Uvarov wrote: > Add generated by builddeb script .deb and .tar.gz files. > > Signed-off-by: Maxim Uvarov > Reviewed-by: Bill Fischofer > --- > .gitignore | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/.gitignore b/.gitignore > index b4c146b

[lng-odp] [PATCHv2 06/10] configure: move test_helper to its own m4 file

2016-03-29 Thread Anders Roxell
Signed-off-by: Anders Roxell --- configure.ac | 13 ++--- helper/m4/configure.m4 | 9 + 2 files changed, 11 insertions(+), 11 deletions(-) create mode 100644 helper/m4/configure.m4 diff --git a/configure.ac b/configure.ac index 0a9918b..8ba0538 100644 --- a/configure.

[lng-odp] [PATCHv2 10/10] example: m4: configure option to check examples

2016-03-29 Thread Anders Roxell
Suggested-by: Mike Holmes Signed-off-by: Anders Roxell --- configure.ac | 1 + example/l2fwd_simple/Makefile.am | 2 ++ example/m4/configure.m4 | 12 3 files changed, 15 insertions(+) diff --git a/configure.ac b/configure.ac index b7879eb..c8fb91e 100

[lng-odp] [PATCHv2 09/10] configure: move test_vald to its own m4 file

2016-03-29 Thread Anders Roxell
Signed-off-by: Anders Roxell --- configure.ac | 51 ++- test/m4/configure.m4 | 1 + test/m4/validation.m4 | 44 3 files changed, 47 insertions(+), 49 deletions(-) create mode 100644 test/m4/va

[lng-odp] [PATCHv2 07/10] configure: move docs to its own m4 file

2016-03-29 Thread Anders Roxell
Signed-off-by: Anders Roxell --- configure.ac| 49 - doc/m4/configure.m4 | 38 ++ 2 files changed, 42 insertions(+), 45 deletions(-) diff --git a/configure.ac b/configure.ac index 8ba0538..883f52f 100644

[lng-odp] [PATCHv2 04/10] configure: move test_perf to its own m4 file

2016-03-29 Thread Anders Roxell
Signed-off-by: Anders Roxell --- configure.ac | 19 ++- test/m4/configure.m4 | 2 ++ test/m4/performance.m4 | 9 + 3 files changed, 17 insertions(+), 13 deletions(-) create mode 100644 test/m4/performance.m4 diff --git a/configure.ac b/configure.ac index 75

[lng-odp] [PATCHv2 01/10] linux-generic: m4: netmap: set netmap_support=no

2016-03-29 Thread Anders Roxell
Conform with other m4 files Signed-off-by: Anders Roxell --- configure.ac| 1 - platform/linux-generic/m4/odp_netmap.m4 | 1 + 2 files changed, 1 insertion(+), 1 deletion(-) diff --git a/configure.ac b/configure.ac index 31d4d32..7b92a7f 100644 --- a/configure.ac +++

[lng-odp] [PATCHv2 08/10] configure: move example configure to the others

2016-03-29 Thread Anders Roxell
Signed-off-by: Anders Roxell --- configure.ac | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/configure.ac b/configure.ac index 883f52f..e25c7a4 100644 --- a/configure.ac +++ b/configure.ac @@ -85,6 +85,7 @@ AC_SUBST([platform_with_platform_test], ["platform/${with_platform

[lng-odp] [PATCHv2 05/10] configure: move test_cpp to its own m4 file

2016-03-29 Thread Anders Roxell
Signed-off-by: Anders Roxell --- configure.ac | 16 +--- test/m4/configure.m4 | 1 + test/m4/miscellaneous.m4 | 9 + 3 files changed, 11 insertions(+), 15 deletions(-) create mode 100644 test/m4/miscellaneous.m4 diff --git a/configure.ac b/configure.ac inde

[lng-odp] [PATCHv2 02/10] linux-generic: m4: dpdk: fix to conform with other m4 files

2016-03-29 Thread Anders Roxell
Signed-off-by: Anders Roxell --- configure.ac | 5 - platform/linux-generic/m4/odp_dpdk.m4 | 1 + 2 files changed, 1 insertion(+), 5 deletions(-) diff --git a/configure.ac b/configure.ac index 7b92a7f..5c4332a 100644 --- a/configure.ac +++ b/configure.ac @@ -82,11 +8

[lng-odp] [PATCHv2 03/10] configure: group AM_CONDITIONAL

2016-03-29 Thread Anders Roxell
Signed-off-by: Anders Roxell --- configure.ac | 19 ++- 1 file changed, 10 insertions(+), 9 deletions(-) diff --git a/configure.ac b/configure.ac index 5c4332a..75d846e 100644 --- a/configure.ac +++ b/configure.ac @@ -93,13 +93,8 @@ else fi ###

[lng-odp] [PATCHv2 00/10] cleanup configure.ac

2016-03-29 Thread Anders Roxell
Hi, Did some minor cleanups in configure.ac... v2, rebased patchset Cheers, Anders Anders Roxell (10): linux-generic: m4: netmap: set netmap_support=no linux-generic: m4: dpdk: fix to conform with other m4 files configure: group AM_CONDITIONAL configure: move test_perf to its own m4 fil

Re: [lng-odp] [PATCH] linux-generic: test: shmem: atomic check+open fifo

2016-03-29 Thread Maxim Uvarov
On 03/29/16 20:30, Christophe Milard wrote: Fix for bug 2146, CID 159395: The open system call is directely used to check the presence of the fifo and open it at the same time. Signed-off-by: Christophe Milard --- platform/linux-generic/test/shmem/shmem_linux.c | 5 +++-- 1 file changed, 3 i

[lng-odp] [PATCH] linux-generic: test: shmem: atomic check+open fifo

2016-03-29 Thread Christophe Milard
Fix for bug 2146, CID 159395: The open system call is directely used to check the presence of the fifo and open it at the same time. Signed-off-by: Christophe Milard --- platform/linux-generic/test/shmem/shmem_linux.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/platf

Re: [lng-odp] No "@" in CPU string causes ODP to crash

2016-03-29 Thread Maxim Uvarov
I think that is correct. Can you please send patch with git send-email so that I can apply it? Please consider it for master branch. You can also refer to CONTRIBUTING file for things needed to be done to send a patch. Please as any questions if you have one. Thank you, Maxim. On 29 March 2016 at

[lng-odp] [PATCH] gitignore: add .deb is .tar.gz

2016-03-29 Thread Maxim Uvarov
Add generated by builddeb script .deb and .tar.gz files. Signed-off-by: Maxim Uvarov --- .gitignore | 2 ++ 1 file changed, 2 insertions(+) diff --git a/.gitignore b/.gitignore index b4c146b..2b1da19 100644 --- a/.gitignore +++ b/.gitignore @@ -29,6 +29,8 @@ config.sub configure core cscope.

Re: [lng-odp] [PATCHv5 0/2] scripts: add builddpdk

2016-03-29 Thread Maxim Uvarov
On 03/29/16 15:20, Mike Holmes wrote: Were you able to test this on ARM64 ? The only other thing I can think might be missing is a note to say that "x86_64-native-linuxapp-gcc" is replaced with string "X" to compile on ARM No I did not test it. But I can try Maxim. On 29 March 2016 a

Re: [lng-odp] [PATCH] linux-generic: dpdk: check hugepage availability before running validation tests

2016-03-29 Thread Maxim Uvarov
On 03/29/16 16:22, Matias Elo wrote: Check that the DPDK hugepage directory exists and hugepages are mounted there before running pktio validation tests. Signed-off-by: Matias Elo --- platform/linux-generic/test/pktio/pktio_run_dpdk | 19 ++- 1 file changed, 18 insertions(+),

[lng-odp] [PATCH 1/2] linux-generic: pool: add more debug print for pool_destroy

2016-03-29 Thread Maxim Uvarov
Print cause and pool name on destroy to make debug more easily. Signed-off-by: Maxim Uvarov --- platform/linux-generic/odp_pool.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/platform/linux-generic/odp_pool.c b/platform/linux-generic/odp_pool.c index 588362b..eb545b2 100644 --- a/platf

[lng-odp] [PATCH 2/2] example: classifier: add termination path

2016-03-29 Thread Maxim Uvarov
Cleanly destroy all allocated resources. https://bugs.linaro.org/show_bug.cgi?id=1706 Signed-off-by: Maxim Uvarov --- example/classifier/odp_classifier.c | 45 - 1 file changed, 34 insertions(+), 11 deletions(-) diff --git a/example/classifier/odp_classifier.

[lng-odp] [PATCH 0/2] linux-generic: pool: add more debug print for pool_destroy

2016-03-29 Thread Maxim Uvarov
Maxim Uvarov (2): linux-generic: pool: add more debug print for pool_destroy example: classifier: add termination path example/classifier/odp_classifier.c | 45 - platform/linux-generic/odp_pool.c | 2 ++ 2 files changed, 36 insertions(+), 11 deletions(-

[lng-odp] [PATCH] linux-generic: dpdk: check hugepage availability before running validation tests

2016-03-29 Thread Matias Elo
Check that the DPDK hugepage directory exists and hugepages are mounted there before running pktio validation tests. Signed-off-by: Matias Elo --- platform/linux-generic/test/pktio/pktio_run_dpdk | 19 ++- 1 file changed, 18 insertions(+), 1 deletion(-) diff --git a/platform/lin

[lng-odp] NOTICE about documentation on the ODP webpage

2016-03-29 Thread Mike Holmes
To clear out old documents and ensure that all the data is the current output of the CI testing, the documents at http://docs.opendataplane.org/ are all currently being regenerated. You may see 404 errors for the next 24 hours from the webpage http://www.opendataplane.org/ Mike -- Mike Holmes Te

Re: [lng-odp] [API-NEXT PATCH 5/6] api: pktio: add checksum offload options

2016-03-29 Thread Savolainen, Petri (Nokia - FI/Espoo)
From: EXT Bill Fischofer [mailto:bill.fischo...@linaro.org] Sent: Monday, March 28, 2016 4:45 PM To: Bala Manoharan Cc: Savolainen, Petri (Nokia - FI/Espoo) ; LNG ODP Mailman List Subject: Re: [lng-odp] [API-NEXT PATCH 5/6] api: pktio: add checksum offload options On Mon, Mar 28, 2016 at 5

Re: [lng-odp] [API-NEXT PATCH 4/6] api: pktio: add classifier enable pktin_queue_param

2016-03-29 Thread Savolainen, Petri (Nokia - FI/Espoo)
From: EXT Bala Manoharan [mailto:bala.manoha...@linaro.org] Sent: Monday, March 28, 2016 12:34 PM To: Savolainen, Petri (Nokia - FI/Espoo) Cc: LNG ODP Mailman List Subject: Re: [lng-odp] [API-NEXT PATCH 4/6] api: pktio: add classifier enable pktin_queue_param On 24 March 2016 at 20:19, Petri

Re: [lng-odp] No "@" in CPU string causes ODP to crash

2016-03-29 Thread Tilli, Juha-Matti (Nokia - FI/Espoo)
Hello, I just created a patch for this that seems to fix the crash, but I have no idea if I'm doing it in the correct way. Basically, the idea is to leave cpu_hz_max uninitialized if we can't find the information. So, could somebody comment on this patch: diff --git a/platform/linux-generic/ar

Re: [lng-odp] [PATCHv5 0/2] scripts: add builddpdk

2016-03-29 Thread Mike Holmes
Were you able to test this on ARM64 ? The only other thing I can think might be missing is a note to say that " x86_64-native-linuxapp-gcc" is replaced with string "X" to compile on ARM On 29 March 2016 at 06:55, Maxim Uvarov wrote: > ping. > > On 03/25/16 09:27, Maxim Uvarov wrote: > >> v5: -

Re: [lng-odp] [PATCH v3 0/6] dpdk pktio vdev validation test support

2016-03-29 Thread Elo, Matias (Nokia - FI/Espoo)
Thanks, I'll create another patch for the hugepage check. -Matias > -Original Message- > From: lng-odp [mailto:lng-odp-boun...@lists.linaro.org] On Behalf Of EXT Maxim > Uvarov > Sent: Tuesday, March 29, 2016 12:39 PM > To: lng-odp@lists.linaro.org > Subject: Re: [lng-odp] [PATCH v3 0/6]

Re: [lng-odp] [PATCH 1/2] linux-generic: test: Mimic the test validation directory structure

2016-03-29 Thread Mike Holmes
Thanks On 29 March 2016 at 07:55, Christophe Milard wrote: > On 2016-03-21 14:34, Mike Holmes wrote: > > I have problem applying this patch: > Applying: linux-generic: test: Mimic the test validation directory > structure > error: patch failed: platform/linux-generic/m4/configure.m4:24 > error

[lng-odp] No "@" in CPU string causes ODP to crash

2016-03-29 Thread Tilli, Juha-Matti (Nokia - FI/Espoo)
Hi, My coworker recently had an ODP crash (he used the most recent api-next), apparently related to not having "@" in CPU string. The CPU string is: $ cat /proc/cpuinfo |grep Xeon|uniq -c 8 model name : Intel Xeon E312xx (Sandy Bridge) This is a virtualized environment (KVM running o

Re: [lng-odp] [PATCH 1/2] linux-generic: test: Mimic the test validation directory structure

2016-03-29 Thread Christophe Milard
On 2016-03-21 14:34, Mike Holmes wrote: I have problem applying this patch: Applying: linux-generic: test: Mimic the test validation directory structure error: patch failed: platform/linux-generic/m4/configure.m4:24 error: platform/linux-generic/m4/configure.m4: patch does not apply error: patch f

Re: [lng-odp] [PATCHv5 0/2] scripts: add builddpdk

2016-03-29 Thread Maxim Uvarov
ping. On 03/25/16 09:27, Maxim Uvarov wrote: v5: - in one more place changed x86-... to ${TARGET}. v4: - make it possible to run sript from different locations; - add check for installed pcap; Maxim Uvarov (2): scripts: add builddpdk DEPENDENCIES: take a note about scripts/build-pkti

[lng-odp] [Bug 2122] CID 158533: Integer handling issues odp_timer_wheel.c

2016-03-29 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=2122 Maxim Uvarov changed: What|Removed |Added CC||maxim.uva...@linaro.org --- Comment #2 from Max

Re: [lng-odp] [PATCHv2] linux-generic: traffic-manager: fix integer handling issue

2016-03-29 Thread Maxim Uvarov
Merged, added Bills review from previous version. Maxim. On 03/29/16 12:22, Balasubramanian Manoharan wrote: Fixes: https://bugs.linaro.org/show_bug.cgi?id=2122 Signed-off-by: Balasubramanian Manoharan --- v2: Incorporates comments from Maxim platform/linux-generic/odp_timer_wheel.c | 2 +-

Re: [lng-odp] [PATCH v3 0/6] dpdk pktio vdev validation test support

2016-03-29 Thread Maxim Uvarov
Merged, Maxim. On 03/23/16 16:14, Matias Elo wrote: V3: - Changed ODP_PLATFORM_PARAMS environment variable to ODP_PKTIO_DPDK_PARAMS (Maxim Uvarov) V2: - Removed deprecated recv/send functions - Don't try to receive or send packets when interface is stopped This patch set enables running ODP

[lng-odp] [PATCHv2] linux-generic: traffic-manager: fix integer handling issue

2016-03-29 Thread Balasubramanian Manoharan
Fixes: https://bugs.linaro.org/show_bug.cgi?id=2122 Signed-off-by: Balasubramanian Manoharan --- v2: Incorporates comments from Maxim platform/linux-generic/odp_timer_wheel.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/platform/linux-generic/odp_timer_wheel.c b/platform

Re: [lng-odp] [PATCH v3 6/6] linux-generic: dpdk: run validation tests with virtual pcap interfaces

2016-03-29 Thread Maxim Uvarov
On 03/29/16 09:59, Elo, Matias (Nokia - FI/Espoo) wrote: -Original Message- From: lng-odp [mailto:lng-odp-boun...@lists.linaro.org] On Behalf Of EXT Maxim Uvarov Sent: Friday, March 25, 2016 3:06 PM To: lng-odp@lists.linaro.org Subject: Re: [lng-odp] [PATCH v3 6/6] linux-generic: dpdk: ru

Re: [lng-odp] [PATCH v3 6/6] linux-generic: dpdk: run validation tests with virtual pcap interfaces

2016-03-29 Thread Elo, Matias (Nokia - FI/Espoo)
> -Original Message- > From: lng-odp [mailto:lng-odp-boun...@lists.linaro.org] On Behalf Of EXT Maxim > Uvarov > Sent: Friday, March 25, 2016 3:06 PM > To: lng-odp@lists.linaro.org > Subject: Re: [lng-odp] [PATCH v3 6/6] linux-generic: dpdk: run validation > tests > with virtual pcap inter