[dpdk-dev] [PATCH] mempool: Reduce rte_mempool structure size

2016-02-08 Thread Wiles, Keith
>Hi Keith, > >Looks good, thanks. Please find some comments below. > >> [PATCH] mempool: Reduce rte_mempool structure size > >nit: we usually avoid uppercase letters in title Will make that change for v2. Why no uppercase letters in the title, seems a bit odd to me in this case?? > >On 02/03/201

[dpdk-dev] [PATCH] mempool: Reduce rte_mempool structure size

2016-02-08 Thread Olivier MATZ
Hi Keith, Looks good, thanks. Please find some comments below. > [PATCH] mempool: Reduce rte_mempool structure size nit: we usually avoid uppercase letters in title On 02/03/2016 12:02 AM, Keith Wiles wrote: > diff --git a/lib/librte_mempool/rte_mempool.c > b/lib/librte_mempool/rte_mempool.c >

[dpdk-dev] [PATCH] mempool: Reduce rte_mempool structure size

2016-02-03 Thread Ananyev, Konstantin
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Keith Wiles > Sent: Tuesday, February 02, 2016 11:03 PM > To: dev at dpdk.org > Subject: [dpdk-dev] [PATCH] mempool: Reduce rte_mempool structure size > > The rte_mempool structure i

[dpdk-dev] [PATCH] mempool: Reduce rte_mempool structure size

2016-02-02 Thread Keith Wiles
The rte_mempool structure is changed, which will cause an ABI change for this structure. Allow mempool cache support to be dynamic depending on if the mempool being created needs cache support. Saves about 1.5M of memory used by the rte_mempool structure. Performance does not seem to be effected r