[dpdk-dev] [PATCH v1 0/9] mempool: prepare to add bucket driver

2018-03-10 Thread Andrew Rybchenko
The initial patch series [1] is split into two to simplify processing. The second series relies on this one and will add bucket mempool driver and related ops. The patch series has generic enhancements suggested by Olivier. Basically it adds driver callbacks to calculate required memory size and t

Re: [dpdk-dev] [PATCH v1 0/9] mempool: prepare to add bucket driver

2018-03-14 Thread santosh
Hi Andrew, On Saturday 10 March 2018 09:09 PM, Andrew Rybchenko wrote: [...] > RFCv1 -> RFCv2: > - add driver ops to calculate required memory size and populate > mempool objects, remove extra flags which were required before > to control it > - transition of octeontx and dpaa drive

Re: [dpdk-dev] [PATCH v1 0/9] mempool: prepare to add bucket driver

2018-03-14 Thread Andrew Rybchenko
Hi Santosh, On 03/14/2018 06:49 PM, santosh wrote: Hi Andrew, On Saturday 10 March 2018 09:09 PM, Andrew Rybchenko wrote: [...] RFCv1 -> RFCv2: - add driver ops to calculate required memory size and populate mempool objects, remove extra flags which were required before to cont

Re: [dpdk-dev] [PATCH v1 0/9] mempool: prepare to add bucket driver

2018-03-19 Thread Olivier Matz
Hi Andrew, Thank you for this nice rework. Globally, the patchset looks good to me. I'm sending some comments as reply to specific patches. On Sat, Mar 10, 2018 at 03:39:33PM +, Andrew Rybchenko wrote: > The initial patch series [1] is split into two to simplify processing. > The second serie

Re: [dpdk-dev] [PATCH v1 0/9] mempool: prepare to add bucket driver

2018-03-20 Thread Andrew Rybchenko
On 03/19/2018 08:03 PM, Olivier Matz wrote: I've kept rte_mempool_populate_iova_tab() intact since it seems to be not directly related XMEM API functions. The function rte_mempool_populate_iova_tab() (actually, it was rte_mempool_populate_phys_tab()) was introduced to support XMEM API. In my opi

Re: [dpdk-dev] [PATCH v1 0/9] mempool: prepare to add bucket driver

2018-03-20 Thread Thomas Monjalon
20/03/2018 11:09, Andrew Rybchenko: > On 03/19/2018 08:03 PM, Olivier Matz wrote: > >> I've kept rte_mempool_populate_iova_tab() intact since it seems to > >> be not directly related XMEM API functions. > > The function rte_mempool_populate_iova_tab() (actually, it was > > rte_mempool_populate_phys