v6 changes (Andrew Rybchenko): - Fix spelling
v5 changes (Andrew Rybchenko): - Factor out cosmetic fixes into separate patches to make all patches smaller and easier to review - Remove extra check as per review notes - Factor out entire cache flushing into a separate patch. It is nice from logical changes separation point of view, easier to bisect and revert. v4 changes: - Updated patch title to reflect that the scope of the patch is only mempool cache flushing. - Do not replace rte_memcpy() with alternative copying method. This was a pure optimization, not a fix. - Elaborate even more on the bugs fixed by the modifications. - Added 4th bullet item to the patch description, regarding rte_mempool_ops_enqueue_bulk() with RTE_LIBRTE_MEMPOOL_DEBUG. v3 changes: - Actually remove my modifications of the rte_mempool_cache structure. v2 changes: - Not adding the new objects to the mempool cache before flushing it also allows the memory allocated for the mempool cache to be reduced from 3 x to 2 x RTE_MEMPOOL_CACHE_MAX_SIZE. However, such this change would break the ABI, so it was removed in v2. - The mempool cache should be cache line aligned for the benefit of the copying method, which on some CPU architectures performs worse on data crossing a cache boundary. However, such this change would break the ABI, so it was removed in v2; and yet another alternative copying method replaced the rte_memcpy(). Andrew Rybchenko (3): mempool: check driver enqueue result in one place mempool: avoid usage of term ring on put mempool: flush cache completely on overflow Morten Brørup (1): mempool: fix cache flushing algorithm lib/mempool/rte_mempool.c | 5 ++++ lib/mempool/rte_mempool.h | 55 ++++++++++++++++++++------------------- 2 files changed, 33 insertions(+), 27 deletions(-) -- 2.30.2
