mempool: use specific macro for object alignment
authorOlivier Matz <olivier.matz@6wind.com>
Tue, 5 Nov 2019 15:37:06 +0000 (16:37 +0100)
committerThomas Monjalon <thomas@monjalon.net>
Wed, 6 Nov 2019 10:34:19 +0000 (11:34 +0100)
For consistency, RTE_MEMPOOL_ALIGN should be used in place of
RTE_CACHE_LINE_SIZE. They have the same value, because the only arch
that was defining a specific value for it has been removed from DPDK.

Signed-off-by: Olivier Matz <olivier.matz@6wind.com>
Reviewed-by: Andrew Rybchenko <arybchenko@solarflare.com>
Acked-by: Nipun Gupta <nipun.gupta@nxp.com>
lib/librte_mempool/rte_mempool.c
lib/librte_mempool/rte_mempool.h
lib/librte_mempool/rte_mempool_ops_default.c

index d3db927..40cae3e 100644 (file)
@@ -329,7 +329,7 @@ rte_mempool_populate_iova(struct rte_mempool *mp, char *vaddr,
        if (mp->flags & MEMPOOL_F_NO_CACHE_ALIGN)
                off = RTE_PTR_ALIGN_CEIL(vaddr, 8) - vaddr;
        else
-               off = RTE_PTR_ALIGN_CEIL(vaddr, RTE_CACHE_LINE_SIZE) - vaddr;
+               off = RTE_PTR_ALIGN_CEIL(vaddr, RTE_MEMPOOL_ALIGN) - vaddr;
 
        if (off > len) {
                ret = -EINVAL;
index 225bf9f..f81152a 100644 (file)
@@ -116,6 +116,9 @@ struct rte_mempool_objsz {
 #define        MEMPOOL_PG_NUM_DEFAULT  1
 
 #ifndef RTE_MEMPOOL_ALIGN
+/**
+ * Alignment of elements inside mempool.
+ */
 #define RTE_MEMPOOL_ALIGN      RTE_CACHE_LINE_SIZE
 #endif
 
index e6be715..22fccf9 100644 (file)
@@ -60,7 +60,7 @@ rte_mempool_op_calc_mem_size_helper(const struct rte_mempool *mp,
        }
 
        *min_chunk_size = total_elt_sz;
-       *align = RTE_CACHE_LINE_SIZE;
+       *align = RTE_MEMPOOL_ALIGN;
 
        return mem_size;
 }