mempool: detect physical contiguous objects
authorSantosh Shukla <santosh.shukla@caviumnetworks.com>
Sun, 1 Oct 2017 09:29:00 +0000 (14:59 +0530)
committerThomas Monjalon <thomas@monjalon.net>
Fri, 6 Oct 2017 19:58:39 +0000 (21:58 +0200)
The memory area containing all the objects must be physically
contiguous.
Introducing MEMPOOL_F_CAPA_PHYS_CONTIG flag for such use-case.

The flag useful to detect whether pool area has sufficient space
to fit all objects. If not then return -ENOSPC.
This way, we make sure that all object within a pool is contiguous.

Signed-off-by: Santosh Shukla <santosh.shukla@caviumnetworks.com>
Signed-off-by: Jerin Jacob <jerin.jacob@caviumnetworks.com>
Acked-by: Olivier Matz <olivier.matz@6wind.com>
lib/librte_mempool/rte_mempool.c
lib/librte_mempool/rte_mempool.h

index 92de395..146e386 100644 (file)
@@ -369,6 +369,16 @@ rte_mempool_populate_phys(struct rte_mempool *mp, char *vaddr,
 
        total_elt_sz = mp->header_size + mp->elt_size + mp->trailer_size;
 
+       /* Detect pool area has sufficient space for elements */
+       if (mp->flags & MEMPOOL_F_CAPA_PHYS_CONTIG) {
+               if (len < total_elt_sz * mp->size) {
+                       RTE_LOG(ERR, MEMPOOL,
+                               "pool area %" PRIx64 " not enough\n",
+                               (uint64_t)len);
+                       return -ENOSPC;
+               }
+       }
+
        memhdr = rte_zmalloc("MEMPOOL_MEMHDR", sizeof(*memhdr), 0);
        if (memhdr == NULL)
                return -ENOMEM;
index d251d42..7343925 100644 (file)
@@ -265,6 +265,12 @@ struct rte_mempool {
 #define MEMPOOL_F_SC_GET         0x0008 /**< Default get is "single-consumer".*/
 #define MEMPOOL_F_POOL_CREATED   0x0010 /**< Internal: pool is created. */
 #define MEMPOOL_F_NO_PHYS_CONTIG 0x0020 /**< Don't need physically contiguous objs. */
+/**
+ * This capability flag is advertised by a mempool handler, if the whole
+ * memory area containing the objects must be physically contiguous.
+ * Note: This flag should not be passed by application.
+ */
+#define MEMPOOL_F_CAPA_PHYS_CONTIG 0x0040
 
 /**
  * @internal When debug is enabled, store some statistics.