Re: [lng-odp] [PATCH] ODP buffer pool restructure for v1.0 APIs

2014-12-02 Thread Taras Kondratiuk
On 12/01/2014 03:47 AM, Bill Fischofer wrote: +int odp_buffer_pool_destroy(odp_buffer_pool_t pool_hdl) +{ + uint32_t pool_id = pool_handle_to_index(pool_hdl); + pool_entry_t *pool = get_pool_entry(pool_id); - if (chunk-chunk.num_bufs == 0) { - /* give the chunk

Re: [lng-odp] [PATCH] ODP buffer pool restructure for v1.0 APIs

2014-12-02 Thread Bill Fischofer
I've posted v2 of this patch, which incorporates the comments from Taras an Anders. It's also now split into 3 parts that can be applied sequentially if desired. Bill On Tue, Dec 2, 2014 at 12:09 PM, Taras Kondratiuk taras.kondrat...@linaro.org wrote: On 12/01/2014 03:47 AM, Bill Fischofer

Re: [lng-odp] [PATCH] ODP buffer pool restructure for v1.0 APIs

2014-12-01 Thread Anders Roxell
Drop for v1.0 APIs You introduced some warnings that we didn't have: odp_config.h:55: warning: Member ODP_CONFIG_BUF_MAX_SIZE (macro definition) of group odp_compiler_optim is not documented. odp_config.h:55: warning: Member ODP_CONFIG_BUF_MAX_SIZE (macro definition) of group odp_compiler_optim

Re: [lng-odp] [PATCH] ODP buffer pool restructure for v1.0 APIs

2014-12-01 Thread Bill Fischofer
When I run the version of checkpatch that's part of odp.git I get this: bill@Ubuntu13:~/linaro/v10bufpool$ ./scripts/checkpatch.pl *.patch total: 0 errors, 0 warnings, 0 checks, 2392 lines checked NOTE: Ignored message types: DEPRECATED_VARIABLE NEW_TYPEDEFS

Re: [lng-odp] [PATCH] ODP buffer pool restructure for v1.0 APIs

2014-12-01 Thread Anders Roxell
On 2 December 2014 at 01:15, Bill Fischofer bill.fischo...@linaro.org wrote: When I run the version of checkpatch that's part of odp.git I get this: bill@Ubuntu13:~/linaro/v10bufpool$ ./scripts/checkpatch.pl *.patch total: 0 errors, 0 warnings, 0 checks, 2392 lines checked NOTE: Ignored

[lng-odp] [PATCH] ODP buffer pool restructure for v1.0 APIs

2014-11-30 Thread Bill Fischofer
ODP buffer pool restricture to enable support for v1.0 APIs. Implements the following revised/new APIs: odp_buffer_pool_create() odp_buffer_pool_destroy() odp_buffer_pool_info() Signed-off-by: Bill Fischofer bill.fischo...@linaro.org --- example/generator/odp_generator.c |