]> git.droids-corp.org - dpdk.git/commitdiff
fib: remove maximum type enums
authorVladimir Medvedkin <vladimir.medvedkin@intel.com>
Tue, 27 Oct 2020 15:11:31 +0000 (15:11 +0000)
committerDavid Marchand <david.marchand@redhat.com>
Wed, 28 Oct 2020 20:23:11 +0000 (21:23 +0100)
FIB type RTE_FIB_TYPE_MAX is used only for sanity checks,
remove it to prevent applications start using it.
The same is for FIB6's RTE_FIB6_TYPE_MAX.

Signed-off-by: Vladimir Medvedkin <vladimir.medvedkin@intel.com>
Acked-by: David Marchand <david.marchand@redhat.com>
app/test/test_fib.c
app/test/test_fib6.c
lib/librte_fib/rte_fib.c
lib/librte_fib/rte_fib.h
lib/librte_fib/rte_fib6.c
lib/librte_fib/rte_fib6.h

index ca80a5dd2f7556e348587120e7dd1cbc9d280f54..e46b9934fe700467deb7e0f33f36a6fb60cb7b93 100644 (file)
@@ -61,7 +61,7 @@ test_create_invalid(void)
                "Call succeeded with invalid parameters\n");
        config.max_routes = MAX_ROUTES;
 
-       config.type = RTE_FIB_TYPE_MAX;
+       config.type = RTE_FIB_DIR24_8 + 1;
        fib = rte_fib_create(__func__, SOCKET_ID_ANY, &config);
        RTE_TEST_ASSERT(fib == NULL,
                "Call succeeded with invalid parameters\n");
index af589fe8cb9816297189618e5b67c2a66970bfb6..74abfc7a5d3fa712a7034e17794c4d323b906e9d 100644 (file)
@@ -63,7 +63,7 @@ test_create_invalid(void)
                "Call succeeded with invalid parameters\n");
        config.max_routes = MAX_ROUTES;
 
-       config.type = RTE_FIB6_TYPE_MAX;
+       config.type = RTE_FIB6_TRIE + 1;
        fib = rte_fib6_create(__func__, SOCKET_ID_ANY, &config);
        RTE_TEST_ASSERT(fib == NULL,
                "Call succeeded with invalid parameters\n");
index e0908084fec73e43af8ded684a54be31a45e4ef5..18a422c3def1e06903dea12c3ffb3278fccfb5b0 100644 (file)
@@ -158,7 +158,7 @@ rte_fib_create(const char *name, int socket_id, struct rte_fib_conf *conf)
 
        /* Check user arguments. */
        if ((name == NULL) || (conf == NULL) || (conf->max_routes < 0) ||
-                       (conf->type >= RTE_FIB_TYPE_MAX)) {
+                       (conf->type > RTE_FIB_DIR24_8)) {
                rte_errno = EINVAL;
                return NULL;
        }
index 84ee774d2d0295cdc0207dbc89c31ab920a25e14..a58ac3caea8cdbf1e1166bd6752103188c8ccbef 100644 (file)
@@ -34,8 +34,7 @@ struct rte_rib;
 /** Type of FIB struct */
 enum rte_fib_type {
        RTE_FIB_DUMMY,          /**< RIB tree based FIB */
-       RTE_FIB_DIR24_8,        /**< DIR24_8 based FIB */
-       RTE_FIB_TYPE_MAX
+       RTE_FIB_DIR24_8         /**< DIR24_8 based FIB */
 };
 
 /** Modify FIB function */
index a1f0db844a35e800c3ecfd8d8da7d16d2ead4548..9652ac71d6203b15edee37fdc26fec39b06caec9 100644 (file)
@@ -160,7 +160,7 @@ rte_fib6_create(const char *name, int socket_id, struct rte_fib6_conf *conf)
 
        /* Check user arguments. */
        if ((name == NULL) || (conf == NULL) || (conf->max_routes < 0) ||
-                       (conf->type >= RTE_FIB6_TYPE_MAX)) {
+                       (conf->type > RTE_FIB6_TRIE)) {
                rte_errno = EINVAL;
                return NULL;
        }
index bbfcf23a851be52174b01bb1dd5960fb26d06045..338a416fb804466de90ae4dd3873ca6a792ab3df 100644 (file)
@@ -35,8 +35,7 @@ struct rte_rib6;
 /** Type of FIB struct */
 enum rte_fib6_type {
        RTE_FIB6_DUMMY,         /**< RIB6 tree based FIB */
-       RTE_FIB6_TRIE,          /**< TRIE based fib  */
-       RTE_FIB6_TYPE_MAX
+       RTE_FIB6_TRIE           /**< TRIE based fib  */
 };
 
 /** Modify FIB function */