[dpdk-dev] [PATCH] mbuf: replace c memcpy code semantics with optimized rte_memcpy
Olivier Matz
olivier.matz at 6wind.com
Tue May 24 16:59:47 CEST 2016
Hi Jerin,
On 05/24/2016 04:50 PM, Jerin Jacob wrote:
> Signed-off-by: Jerin Jacob <jerin.jacob at caviumnetworks.com>
> ---
> lib/librte_mempool/rte_mempool.h | 5 ++---
> 1 file changed, 2 insertions(+), 3 deletions(-)
>
> diff --git a/lib/librte_mempool/rte_mempool.h b/lib/librte_mempool/rte_mempool.h
> index ed2c110..ebe399a 100644
> --- a/lib/librte_mempool/rte_mempool.h
> +++ b/lib/librte_mempool/rte_mempool.h
> @@ -74,6 +74,7 @@
> #include <rte_memory.h>
> #include <rte_branch_prediction.h>
> #include <rte_ring.h>
> +#include <rte_memcpy.h>
>
> #ifdef __cplusplus
> extern "C" {
> @@ -917,7 +918,6 @@ __mempool_put_bulk(struct rte_mempool *mp, void * const *obj_table,
> unsigned n, __rte_unused int is_mp)
> {
> struct rte_mempool_cache *cache;
> - uint32_t index;
> void **cache_objs;
> unsigned lcore_id = rte_lcore_id();
> uint32_t cache_size = mp->cache_size;
> @@ -946,8 +946,7 @@ __mempool_put_bulk(struct rte_mempool *mp, void * const *obj_table,
> */
>
> /* Add elements back into the cache */
> - for (index = 0; index < n; ++index, obj_table++)
> - cache_objs[index] = *obj_table;
> + rte_memcpy(&cache_objs[0], obj_table, sizeof(void *) * n);
>
> cache->len += n;
>
>
The commit title should be "mempool" instead of "mbuf".
Are you seeing some performance improvement by using rte_memcpy()?
Regards
Olivier
More information about the dev
mailing list