mempool: fix non-IO flag inference
[dpdk.git] / app / test / test_mempool.c
index 5339a4c..4b0f6b0 100644 (file)
@@ -12,6 +12,7 @@
 #include <sys/queue.h>
 
 #include <rte_common.h>
+#include <rte_eal_paging.h>
 #include <rte_log.h>
 #include <rte_debug.h>
 #include <rte_errno.h>
@@ -112,7 +113,7 @@ test_mempool_basic(struct rte_mempool *mp, int use_external_cache)
 
        printf("get private data\n");
        if (rte_mempool_get_priv(mp) != (char *)mp +
-                       MEMPOOL_HEADER_SIZE(mp, mp->cache_size))
+                       RTE_MEMPOOL_HEADER_SIZE(mp, mp->cache_size))
                GOTO_ERR(ret, out);
 
 #ifndef RTE_EXEC_ENV_FREEBSD /* rte_mem_virt2iova() not supported on bsd */
@@ -206,15 +207,15 @@ static int test_mempool_creation_with_exceeded_cache_size(void)
        return 0;
 }
 
-static int test_mempool_creation_with_unknown_flag(void)
+static int test_mempool_creation_with_invalid_flags(void)
 {
        struct rte_mempool *mp_cov;
 
-       mp_cov = rte_mempool_create("test_mempool_unknown_flag", MEMPOOL_SIZE,
+       mp_cov = rte_mempool_create("test_mempool_invalid_flags", MEMPOOL_SIZE,
                MEMPOOL_ELT_SIZE, 0, 0,
                NULL, NULL,
                NULL, NULL,
-               SOCKET_ID_ANY, MEMPOOL_F_NO_IOVA_CONTIG << 1);
+               SOCKET_ID_ANY, ~RTE_MEMPOOL_VALID_USER_FLAGS);
 
        if (mp_cov != NULL) {
                rte_mempool_free(mp_cov);
@@ -337,8 +338,8 @@ test_mempool_sp_sc(void)
                        my_mp_init, NULL,
                        my_obj_init, NULL,
                        SOCKET_ID_ANY,
-                       MEMPOOL_F_NO_CACHE_ALIGN | MEMPOOL_F_SP_PUT |
-                       MEMPOOL_F_SC_GET);
+                       RTE_MEMPOOL_F_NO_CACHE_ALIGN | RTE_MEMPOOL_F_SP_PUT |
+                       RTE_MEMPOOL_F_SC_GET);
                if (mp_spsc == NULL)
                        RET_ERR();
        }
@@ -729,6 +730,109 @@ exit:
 #pragma pop_macro("RTE_TEST_TRACE_FAILURE")
 }
 
+#pragma push_macro("RTE_TEST_TRACE_FAILURE")
+#undef RTE_TEST_TRACE_FAILURE
+#define RTE_TEST_TRACE_FAILURE(...) do { \
+               ret = TEST_FAILED; \
+               goto exit; \
+       } while (0)
+
+static int
+test_mempool_flag_non_io_set_when_no_iova_contig_set(void)
+{
+       void *virt = NULL;
+       rte_iova_t iova;
+       size_t size = MEMPOOL_ELT_SIZE * 16;
+       struct rte_mempool *mp = NULL;
+       int ret;
+
+       virt = rte_malloc("test_mempool", size, rte_mem_page_size());
+       RTE_TEST_ASSERT_NOT_NULL(virt, "Cannot allocate memory");
+       iova = rte_mem_virt2iova(virt);
+       RTE_TEST_ASSERT_NOT_EQUAL(iova,  RTE_BAD_IOVA, "Cannot get IOVA");
+       mp = rte_mempool_create_empty("empty", MEMPOOL_SIZE,
+                                     MEMPOOL_ELT_SIZE, 0, 0,
+                                     SOCKET_ID_ANY, RTE_MEMPOOL_F_NO_IOVA_CONTIG);
+       RTE_TEST_ASSERT_NOT_NULL(mp, "Cannot create mempool: %s",
+                                rte_strerror(rte_errno));
+       rte_mempool_set_ops_byname(mp, rte_mbuf_best_mempool_ops(), NULL);
+
+       RTE_TEST_ASSERT(mp->flags & RTE_MEMPOOL_F_NON_IO,
+                       "NON_IO flag is not set on an empty mempool");
+
+       /*
+        * Always use valid IOVA so that populate() has no other reason
+        * to infer that the mempool cannot be used for IO.
+        */
+       ret = rte_mempool_populate_iova(mp, virt, iova, size, NULL, NULL);
+       RTE_TEST_ASSERT(ret > 0, "Failed to populate mempool: %s",
+                       rte_strerror(-ret));
+       RTE_TEST_ASSERT(mp->flags & RTE_MEMPOOL_F_NON_IO,
+                       "NON_IO flag is not set when NO_IOVA_CONTIG is set");
+       ret = TEST_SUCCESS;
+exit:
+       rte_mempool_free(mp);
+       rte_free(virt);
+       return ret;
+}
+
+static int
+test_mempool_flag_non_io_unset_when_populated_with_valid_iova(void)
+{
+       void *virt = NULL;
+       rte_iova_t iova;
+       size_t total_size = MEMPOOL_ELT_SIZE * MEMPOOL_SIZE;
+       size_t block_size = total_size / 3;
+       struct rte_mempool *mp = NULL;
+       int ret;
+
+       /*
+        * Since objects from the pool are never used in the test,
+        * we don't care for contiguous IOVA, on the other hand,
+        * reiuring it could cause spurious test failures.
+        */
+       virt = rte_malloc("test_mempool", total_size, rte_mem_page_size());
+       RTE_TEST_ASSERT_NOT_NULL(virt, "Cannot allocate memory");
+       iova = rte_mem_virt2iova(virt);
+       RTE_TEST_ASSERT_NOT_EQUAL(iova,  RTE_BAD_IOVA, "Cannot get IOVA");
+       mp = rte_mempool_create_empty("empty", MEMPOOL_SIZE,
+                                     MEMPOOL_ELT_SIZE, 0, 0,
+                                     SOCKET_ID_ANY, 0);
+       RTE_TEST_ASSERT_NOT_NULL(mp, "Cannot create mempool: %s",
+                                rte_strerror(rte_errno));
+
+       RTE_TEST_ASSERT(mp->flags & RTE_MEMPOOL_F_NON_IO,
+                       "NON_IO flag is not set on an empty mempool");
+
+       ret = rte_mempool_populate_iova(mp, RTE_PTR_ADD(virt, 1 * block_size),
+                                       RTE_BAD_IOVA, block_size, NULL, NULL);
+       RTE_TEST_ASSERT(ret > 0, "Failed to populate mempool: %s",
+                       rte_strerror(-ret));
+       RTE_TEST_ASSERT(mp->flags & RTE_MEMPOOL_F_NON_IO,
+                       "NON_IO flag is not set when mempool is populated with only RTE_BAD_IOVA");
+
+       ret = rte_mempool_populate_iova(mp, virt, iova, block_size, NULL, NULL);
+       RTE_TEST_ASSERT(ret > 0, "Failed to populate mempool: %s",
+                       rte_strerror(-ret));
+       RTE_TEST_ASSERT(!(mp->flags & RTE_MEMPOOL_F_NON_IO),
+                       "NON_IO flag is not unset when mempool is populated with valid IOVA");
+
+       ret = rte_mempool_populate_iova(mp, RTE_PTR_ADD(virt, 2 * block_size),
+                                       RTE_BAD_IOVA, block_size, NULL, NULL);
+       RTE_TEST_ASSERT(ret > 0, "Failed to populate mempool: %s",
+                       rte_strerror(-ret));
+       RTE_TEST_ASSERT(!(mp->flags & RTE_MEMPOOL_F_NON_IO),
+                       "NON_IO flag is set even when some objects have valid IOVA");
+       ret = TEST_SUCCESS;
+
+exit:
+       rte_mempool_free(mp);
+       rte_free(virt);
+       return ret;
+}
+
+#pragma pop_macro("RTE_TEST_TRACE_FAILURE")
+
 static int
 test_mempool(void)
 {
@@ -893,7 +997,7 @@ test_mempool(void)
        if (test_mempool_creation_with_exceeded_cache_size() < 0)
                GOTO_ERR(ret, err);
 
-       if (test_mempool_creation_with_unknown_flag() < 0)
+       if (test_mempool_creation_with_invalid_flags() < 0)
                GOTO_ERR(ret, err);
 
        if (test_mempool_same_name_twice_creation() < 0)
@@ -914,6 +1018,12 @@ test_mempool(void)
        if (test_mempool_events_safety() < 0)
                GOTO_ERR(ret, err);
 
+       /* test NON_IO flag inference */
+       if (test_mempool_flag_non_io_set_when_no_iova_contig_set() < 0)
+               GOTO_ERR(ret, err);
+       if (test_mempool_flag_non_io_unset_when_populated_with_valid_iova() < 0)
+               GOTO_ERR(ret, err);
+
        rte_mempool_list_dump(stdout);
 
        ret = 0;