Hi Lazaros,
On 06/16/2016 01:02 PM, Lazaros Koromilas wrote: > Updated version of the user-owned cache patchset. It applies on top of > the latest external mempool manager patches from David Hunt [1]. > > [1] http://dpdk.org/ml/archives/dev/2016-June/041479.html > > v3 changes: > > * Deprecate specific mempool API calls instead of removing them. > * Split deprecation into a separate commit to limit noise. > * Fix cache flush by setting cache->len = 0 and make it inline. > * Remove cache->size == 0 checks and ensure size != 0 at creation. > * Fix tests to check if cache creation succeeded. > * Fix tests to free allocated resources on error. Thanks for the update. The patchset looks good to me. I have some minor comments for patch 2/3 and 3/3. One more thing: would you mind adding some words in doc/guides/prog_guide/mempool_lib.rst ? Thanks, Olivier