Re: [dpdk-dev] [PATCH v3 2/6] mempool: add namespace prefix to flags

[David Marchand]

On Tue, Oct 19, 2021 at 7:40 PM Andrew Rybchenko
<andrew.rybche...@oktetlabs.ru> wrote:
> @@ -752,7 +752,7 @@ test_mempool_flag_non_io_set_when_no_iova_contig_set(void)
>         ret = rte_mempool_populate_default(mp);
>         RTE_TEST_ASSERT(ret > 0, "Failed to populate mempool: %s",
>                         rte_strerror(-ret));
> -       RTE_TEST_ASSERT(mp->flags & MEMPOOL_F_NON_IO,
> +       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:

There is one more flag, hunk fixed adding missing:

@@ -745,14 +745,14 @@ test_mempool_flag_non_io_set_when_no_iova_contig_set(void)

     mp = rte_mempool_create_empty("empty", MEMPOOL_SIZE,
                       MEMPOOL_ELT_SIZE, 0, 0,
-                      SOCKET_ID_ANY, MEMPOOL_F_NO_IOVA_CONTIG);
+                      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);
     ret = rte_mempool_populate_default(mp);
     RTE_TEST_ASSERT(ret > 0, "Failed to populate mempool: %s",
             rte_strerror(-ret));
-    RTE_TEST_ASSERT(mp->flags & MEMPOOL_F_NON_IO,
+    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:


-- 
David Marchand