[dpdk-dev] [PATCH v2] mempool: reduce rte_mempool structure size

2016-02-12 Thread Panu Matilainen
On 02/10/2016 08:02 PM, Thomas Monjalon wrote: > 2016-02-10 18:01, Wiles, Keith: > --- a/config/defconfig_x86_64-native-linuxapp-gcc > +++ b/config/defconfig_x86_64-native-linuxapp-gcc > @@ -40,3 +40,8 @@ CONFIG_RTE_ARCH_64=y > > CONFIG_RTE_TOOLCHAIN="gcc" > CONFIG_RTE_T

[dpdk-dev] [PATCH v2] mempool: reduce rte_mempool structure size

2016-02-10 Thread Olivier MATZ
Hi Keith, On 02/10/2016 07:35 PM, Wiles, Keith wrote: >>> @@ -672,6 +704,24 @@ rte_mempool_count(const struct rte_mempool *mp) >>> static unsigned >>> rte_mempool_dump_cache(FILE *f, const struct rte_mempool *mp) >>> { >>> +#ifdef RTE_NEXT_ABI >>> + unsigned lcore_id; >>> + unsigned count =

[dpdk-dev] [PATCH v2] mempool: reduce rte_mempool structure size

2016-02-10 Thread Thomas Monjalon
2016-02-10 18:01, Wiles, Keith: > >>> --- a/config/defconfig_x86_64-native-linuxapp-gcc > >>> +++ b/config/defconfig_x86_64-native-linuxapp-gcc > >>> @@ -40,3 +40,8 @@ CONFIG_RTE_ARCH_64=y > >>> > >>> CONFIG_RTE_TOOLCHAIN="gcc" > >>> CONFIG_RTE_TOOLCHAIN_GCC=y > >>> +CONFIG_RTE_BUILD_SHARED_LIB

[dpdk-dev] [PATCH v2] mempool: reduce rte_mempool structure size

2016-02-10 Thread Wiles, Keith
>Hi Keith, > >Thank you for adding the RTE_NEXT_ABI. I think this is the way >described in the process. Your changes will be available in next >version (16.4) for people compiling with RTE_NEXT_ABI=y, and in >16.7 without option (I'm just surprised that RTE_NEXT_ABI=y in >default configs...). > >I

[dpdk-dev] [PATCH v2] mempool: reduce rte_mempool structure size

2016-02-10 Thread Wiles, Keith
>>Hi Keith, >> >>Thank you for adding the RTE_NEXT_ABI. I think this is the way >>described in the process. Your changes will be available in next >>version (16.4) for people compiling with RTE_NEXT_ABI=y, and in >>16.7 without option (I'm just surprised that RTE_NEXT_ABI=y in >>default configs...)

[dpdk-dev] [PATCH v2] mempool: reduce rte_mempool structure size

2016-02-10 Thread Olivier MATZ
Hi Keith, Thank you for adding the RTE_NEXT_ABI. I think this is the way described in the process. Your changes will be available in next version (16.4) for people compiling with RTE_NEXT_ABI=y, and in 16.7 without option (I'm just surprised that RTE_NEXT_ABI=y in default configs...). I think a d

[dpdk-dev] [PATCH v2] mempool: reduce rte_mempool structure size

2016-02-10 Thread Wiles, Keith
>Hi Keith, > >Thank you for adding the RTE_NEXT_ABI. I think this is the way >described in the process. Your changes will be available in next >version (16.4) for people compiling with RTE_NEXT_ABI=y, and in >16.7 without option (I'm just surprised that RTE_NEXT_ABI=y in >default configs...). > >I

[dpdk-dev] [PATCH v2] mempool: reduce rte_mempool structure size

2016-02-09 Thread Keith Wiles
Patch v2 to add some comments and setup for RTE_NEXT_ABI changes. The rte_mempool structure is changed, which will cause an ABI change for this structure. Providing backward compat is not reasonable here as this structure is used in multiple defines/inlines. Allow mempool cache support to be dyna