Re: [lng-odp] [API-NEXT PATCH] api: pktio: Clarify doc on default pool usage on odp_pktio_open

2015-04-08 Thread Ciprian Barbu
Looks good, thanks. On Mon, Apr 6, 2015 at 6:56 PM, Bill Fischofer bill.fischo...@linaro.org wrote: Signed-off-by: Bill Fischofer bill.fischo...@linaro.org --- include/odp/api/packet_io.h | 15 +++ 1 file changed, 11 insertions(+), 4 deletions(-) diff --git

[lng-odp] [PATCHv12 1/5] configure.ac: include configure.m4 if platform explicitly selected

2015-04-08 Thread Maxim Uvarov
configure script selects linux-generic as default platform and includes platfrom specific configure scripts: ./platform/linux-generic/m4/configure.m4 In case if --with-platfrom=linux-generic was provided to configure platform specific file is not included and that branch of code does not run. This

[lng-odp] [PATCHv12 5/5] validation: add new test l2fwd_run

2015-04-08 Thread Maxim Uvarov
Signed-off-by: Maxim Uvarov maxim.uva...@linaro.org Signed-off-by: Ciprian Barbu ciprian.ba...@linaro.org --- test/performance/Makefile.am | 2 +- test/performance/odp_l2fwd_run | 80 ++ 2 files changed, 81 insertions(+), 1 deletion(-) create mode

[lng-odp] [PATCHv12 2/5] validation: implement platform env

2015-04-08 Thread Maxim Uvarov
Different platforms need different steps to set up pktio for testing. That might be veth devices for linux-generic, kernel modules and extended set up for dpdk and simple set pktio testing names for other platforms. This patch implements platform/test/pktio_env file which sets up global envs for

Re: [lng-odp] [API-NEXT PATCH] api: pktio: Clarify doc on default pool usage on odp_pktio_open

2015-04-08 Thread Maxim Uvarov
But why that can not go to current version? It's not API change, it's only text clarifying? Maxim. On 04/08/15 10:21, Ciprian Barbu wrote: Looks good, thanks. On Mon, Apr 6, 2015 at 6:56 PM, Bill Fischofer bill.fischo...@linaro.org wrote: Signed-off-by: Bill Fischofer

Re: [lng-odp] odp_packet_has_error return value

2015-04-08 Thread Bill Fischofer
These are booleans. However we decided that ODP considers any non-zero value to be true, hence the doc change. The linux-generic implementation always returns 1 for true and since 1 is a non-zero value that's completely conforming. On Wed, Apr 8, 2015 at 12:07 PM, Mike Holmes

Re: [lng-odp] odp_packet_has_error return value

2015-04-08 Thread Mike Holmes
On 8 April 2015 at 13:10, Bill Fischofer bill.fischo...@linaro.org wrote: These are booleans. However we decided that ODP considers any non-zero value to be true, hence the doc change. The linux-generic implementation always returns 1 for true and since 1 is a non-zero value that's completely

Re: [lng-odp] [PATCHv2] example: ODP classifier example

2015-04-08 Thread Maxim Uvarov
On 04/08/15 20:01, bala.manoha...@linaro.org wrote: + /* Reserve memory for args from shared mem */ + shm = odp_shm_reserve(cls_shm_args, sizeof(appl_args_t), + ODP_CACHE_LINE_SIZE, 0); + args = odp_shm_addr(shm); + + if (args == NULL) { +

Re: [lng-odp] pktio optional parsing

2015-04-08 Thread Mike Holmes
On 8 April 2015 at 13:02, Zoltan Kiss zoltan.k...@linaro.org wrote: Hi, OVS has a major performance issue with pktio at the moment: pktio always does parsing, but OVS does it for itself as well, and it is quite deeply woven into its code, so we can't easily modify it to use the ODP parsed

Re: [lng-odp] [PATCHv12 0/5] validation: implement platform env and add performance/odp_example_l2fwd_run

2015-04-08 Thread Ciprian Barbu
For the entire series: Reviewed-and-tested-by: Ciprian Barbu ciprian.ba...@linaro.org On Wed, Apr 8, 2015 at 3:00 PM, Maxim Uvarov maxim.uva...@linaro.org wrote: v12: worked with Ciprian to test different scenarios of l2fwd and pktio scripts run. Extend search path in that scrips to

[lng-odp] [PATCHv13 2/5] validation: implement platform env

2015-04-08 Thread Maxim Uvarov
Different platforms need different steps to set up pktio for testing. That might be veth devices for linux-generic, kernel modules and extended set up for dpdk and simple set pktio testing names for other platforms. This patch implements platform/test/pktio_env file which sets up global envs for

[lng-odp] pktio optional parsing

2015-04-08 Thread Zoltan Kiss
Hi, OVS has a major performance issue with pktio at the moment: pktio always does parsing, but OVS does it for itself as well, and it is quite deeply woven into its code, so we can't easily modify it to use the ODP parsed data. Also, not every platform accelerate that (e.g. DPDK), at the

[lng-odp] [PATCHv2] example: ODP classifier example

2015-04-08 Thread bala . manoharan
From: Balasubramanian Manoharan bala.manoha...@linaro.org ODP Classifier example This programs gets pmr rules as command-line parameter and configures the classification engine in the system. This initial version supports the following * ODP_PMR_SIP_ADDR pmr term * PMR term MATCH and RANGE

Re: [lng-odp] [PATCH v2 1/2] configure: add ARCH optimisation support by directory

2015-04-08 Thread Anders Roxell
On 6 April 2015 at 21:34, Mike Holmes mike.hol...@linaro.org wrote: Signed-off-by: Mike Holmes mike.hol...@linaro.org Reviewed-by: Anders Roxell anders.rox...@linaro.org --- configure.ac | 11 +++ 1 file changed, 11 insertions(+) diff --git a/configure.ac b/configure.ac index