lpm: implement RCU rule reclamation
[dpdk.git] / lib / librte_lpm / rte_lpm.c
index 106916d..126fc5a 100644 (file)
@@ -1,5 +1,6 @@
 /* SPDX-License-Identifier: BSD-3-Clause
  * Copyright(c) 2010-2014 Intel Corporation
+ * Copyright(c) 2020 Arm Limited
  */
 
 #include <string.h>
@@ -22,7 +23,6 @@
 #include <rte_rwlock.h>
 #include <rte_spinlock.h>
 #include <rte_tailq.h>
-#include <rte_function_versioning.h>
 
 #include "rte_lpm.h"
 
@@ -40,6 +40,17 @@ enum valid_flag {
        VALID
 };
 
+/** @internal LPM structure. */
+struct __rte_lpm {
+       /* LPM metadata. */
+       struct rte_lpm lpm;
+
+       /* RCU config. */
+       struct rte_rcu_qsbr *v;         /* RCU QSBR variable. */
+       enum rte_lpm_qsbr_mode rcu_mode;/* Blocking, defer queue. */
+       struct rte_rcu_qsbr_dq *dq;     /* RCU QSBR defer queue. */
+};
+
 /* Macro to enable/disable run-time checks. */
 #if defined(RTE_LIBRTE_LPM_DEBUG)
 #include <rte_debug.h>
@@ -90,34 +101,8 @@ depth_to_range(uint8_t depth)
 /*
  * Find an existing lpm table and return a pointer to it.
  */
-struct rte_lpm_v20 * __vsym
-rte_lpm_find_existing_v20(const char *name)
-{
-       struct rte_lpm_v20 *l = NULL;
-       struct rte_tailq_entry *te;
-       struct rte_lpm_list *lpm_list;
-
-       lpm_list = RTE_TAILQ_CAST(rte_lpm_tailq.head, rte_lpm_list);
-
-       rte_mcfg_tailq_read_lock();
-       TAILQ_FOREACH(te, lpm_list, next) {
-               l = te->data;
-               if (strncmp(name, l->name, RTE_LPM_NAMESIZE) == 0)
-                       break;
-       }
-       rte_mcfg_tailq_read_unlock();
-
-       if (te == NULL) {
-               rte_errno = ENOENT;
-               return NULL;
-       }
-
-       return l;
-}
-VERSION_SYMBOL(rte_lpm_find_existing, _v20, 2.0);
-
-struct rte_lpm * __vsym
-rte_lpm_find_existing_v1604(const char *name)
+struct rte_lpm *
+rte_lpm_find_existing(const char *name)
 {
        struct rte_lpm *l = NULL;
        struct rte_tailq_entry *te;
@@ -140,91 +125,16 @@ rte_lpm_find_existing_v1604(const char *name)
 
        return l;
 }
-BIND_DEFAULT_SYMBOL(rte_lpm_find_existing, _v1604, 16.04);
-MAP_STATIC_SYMBOL(struct rte_lpm *rte_lpm_find_existing(const char *name),
-               rte_lpm_find_existing_v1604);
 
 /*
  * Allocates memory for LPM object
  */
-struct rte_lpm_v20 * __vsym
-rte_lpm_create_v20(const char *name, int socket_id, int max_rules,
-               __rte_unused int flags)
-{
-       char mem_name[RTE_LPM_NAMESIZE];
-       struct rte_lpm_v20 *lpm = NULL;
-       struct rte_tailq_entry *te;
-       uint32_t mem_size;
-       struct rte_lpm_list *lpm_list;
-
-       lpm_list = RTE_TAILQ_CAST(rte_lpm_tailq.head, rte_lpm_list);
-
-       RTE_BUILD_BUG_ON(sizeof(struct rte_lpm_tbl_entry_v20) != 2);
-
-       /* Check user arguments. */
-       if ((name == NULL) || (socket_id < -1) || (max_rules == 0)) {
-               rte_errno = EINVAL;
-               return NULL;
-       }
-
-       snprintf(mem_name, sizeof(mem_name), "LPM_%s", name);
-
-       /* Determine the amount of memory to allocate. */
-       mem_size = sizeof(*lpm) + (sizeof(lpm->rules_tbl[0]) * max_rules);
-
-       rte_mcfg_tailq_write_lock();
-
-       /* guarantee there's no existing */
-       TAILQ_FOREACH(te, lpm_list, next) {
-               lpm = te->data;
-               if (strncmp(name, lpm->name, RTE_LPM_NAMESIZE) == 0)
-                       break;
-       }
-
-       if (te != NULL) {
-               lpm = NULL;
-               rte_errno = EEXIST;
-               goto exit;
-       }
-
-       /* allocate tailq entry */
-       te = rte_zmalloc("LPM_TAILQ_ENTRY", sizeof(*te), 0);
-       if (te == NULL) {
-               RTE_LOG(ERR, LPM, "Failed to allocate tailq entry\n");
-               rte_errno = ENOMEM;
-               goto exit;
-       }
-
-       /* Allocate memory to store the LPM data structures. */
-       lpm = rte_zmalloc_socket(mem_name, mem_size,
-                       RTE_CACHE_LINE_SIZE, socket_id);
-       if (lpm == NULL) {
-               RTE_LOG(ERR, LPM, "LPM memory allocation failed\n");
-               rte_free(te);
-               rte_errno = ENOMEM;
-               goto exit;
-       }
-
-       /* Save user arguments. */
-       lpm->max_rules = max_rules;
-       strlcpy(lpm->name, name, sizeof(lpm->name));
-
-       te->data = lpm;
-
-       TAILQ_INSERT_TAIL(lpm_list, te, next);
-
-exit:
-       rte_mcfg_tailq_write_unlock();
-
-       return lpm;
-}
-VERSION_SYMBOL(rte_lpm_create, _v20, 2.0);
-
-struct rte_lpm * __vsym
-rte_lpm_create_v1604(const char *name, int socket_id,
+struct rte_lpm *
+rte_lpm_create(const char *name, int socket_id,
                const struct rte_lpm_config *config)
 {
        char mem_name[RTE_LPM_NAMESIZE];
+       struct __rte_lpm *internal_lpm;
        struct rte_lpm *lpm = NULL;
        struct rte_tailq_entry *te;
        uint32_t mem_size, rules_size, tbl8s_size;
@@ -243,12 +153,6 @@ rte_lpm_create_v1604(const char *name, int socket_id,
 
        snprintf(mem_name, sizeof(mem_name), "LPM_%s", name);
 
-       /* Determine the amount of memory to allocate. */
-       mem_size = sizeof(*lpm);
-       rules_size = sizeof(struct rte_lpm_rule) * config->max_rules;
-       tbl8s_size = (sizeof(struct rte_lpm_tbl_entry) *
-                       RTE_LPM_TBL8_GROUP_NUM_ENTRIES * config->number_tbl8s);
-
        rte_mcfg_tailq_write_lock();
 
        /* guarantee there's no existing */
@@ -264,6 +168,12 @@ rte_lpm_create_v1604(const char *name, int socket_id,
                goto exit;
        }
 
+       /* Determine the amount of memory to allocate. */
+       mem_size = sizeof(*internal_lpm);
+       rules_size = sizeof(struct rte_lpm_rule) * config->max_rules;
+       tbl8s_size = sizeof(struct rte_lpm_tbl_entry) *
+                       RTE_LPM_TBL8_GROUP_NUM_ENTRIES * config->number_tbl8s;
+
        /* allocate tailq entry */
        te = rte_zmalloc("LPM_TAILQ_ENTRY", sizeof(*te), 0);
        if (te == NULL) {
@@ -273,21 +183,23 @@ rte_lpm_create_v1604(const char *name, int socket_id,
        }
 
        /* Allocate memory to store the LPM data structures. */
-       lpm = rte_zmalloc_socket(mem_name, mem_size,
+       internal_lpm = rte_zmalloc_socket(mem_name, mem_size,
                        RTE_CACHE_LINE_SIZE, socket_id);
-       if (lpm == NULL) {
+       if (internal_lpm == NULL) {
                RTE_LOG(ERR, LPM, "LPM memory allocation failed\n");
                rte_free(te);
                rte_errno = ENOMEM;
                goto exit;
        }
 
+       lpm = &internal_lpm->lpm;
        lpm->rules_tbl = rte_zmalloc_socket(NULL,
                        (size_t)rules_size, RTE_CACHE_LINE_SIZE, socket_id);
 
        if (lpm->rules_tbl == NULL) {
                RTE_LOG(ERR, LPM, "LPM rules_tbl memory allocation failed\n");
-               rte_free(lpm);
+               rte_free(internal_lpm);
+               internal_lpm = NULL;
                lpm = NULL;
                rte_free(te);
                rte_errno = ENOMEM;
@@ -300,7 +212,8 @@ rte_lpm_create_v1604(const char *name, int socket_id,
        if (lpm->tbl8 == NULL) {
                RTE_LOG(ERR, LPM, "LPM tbl8 memory allocation failed\n");
                rte_free(lpm->rules_tbl);
-               rte_free(lpm);
+               rte_free(internal_lpm);
+               internal_lpm = NULL;
                lpm = NULL;
                rte_free(te);
                rte_errno = ENOMEM;
@@ -321,17 +234,14 @@ exit:
 
        return lpm;
 }
-BIND_DEFAULT_SYMBOL(rte_lpm_create, _v1604, 16.04);
-MAP_STATIC_SYMBOL(
-       struct rte_lpm *rte_lpm_create(const char *name, int socket_id,
-                       const struct rte_lpm_config *config), rte_lpm_create_v1604);
 
 /*
  * Deallocates memory for given LPM table.
  */
-void __vsym
-rte_lpm_free_v20(struct rte_lpm_v20 *lpm)
+void
+rte_lpm_free(struct rte_lpm *lpm)
 {
+       struct __rte_lpm *internal_lpm;
        struct rte_lpm_list *lpm_list;
        struct rte_tailq_entry *te;
 
@@ -353,43 +263,83 @@ rte_lpm_free_v20(struct rte_lpm_v20 *lpm)
 
        rte_mcfg_tailq_write_unlock();
 
+       internal_lpm = container_of(lpm, struct __rte_lpm, lpm);
+       if (internal_lpm->dq != NULL)
+               rte_rcu_qsbr_dq_delete(internal_lpm->dq);
+       rte_free(lpm->tbl8);
+       rte_free(lpm->rules_tbl);
        rte_free(lpm);
        rte_free(te);
 }
-VERSION_SYMBOL(rte_lpm_free, _v20, 2.0);
 
-void __vsym
-rte_lpm_free_v1604(struct rte_lpm *lpm)
+static void
+__lpm_rcu_qsbr_free_resource(void *p, void *data, unsigned int n)
 {
-       struct rte_lpm_list *lpm_list;
-       struct rte_tailq_entry *te;
+       struct rte_lpm_tbl_entry *tbl8 = ((struct rte_lpm *)p)->tbl8;
+       struct rte_lpm_tbl_entry zero_tbl8_entry = {0};
+       uint32_t tbl8_group_index = *(uint32_t *)data;
 
-       /* Check user arguments. */
-       if (lpm == NULL)
-               return;
+       RTE_SET_USED(n);
+       /* Set tbl8 group invalid */
+       __atomic_store(&tbl8[tbl8_group_index], &zero_tbl8_entry,
+               __ATOMIC_RELAXED);
+}
 
-       lpm_list = RTE_TAILQ_CAST(rte_lpm_tailq.head, rte_lpm_list);
+/* Associate QSBR variable with an LPM object.
+ */
+int
+rte_lpm_rcu_qsbr_add(struct rte_lpm *lpm, struct rte_lpm_rcu_config *cfg,
+       struct rte_rcu_qsbr_dq **dq)
+{
+       struct rte_rcu_qsbr_dq_parameters params = {0};
+       char rcu_dq_name[RTE_RCU_QSBR_DQ_NAMESIZE];
+       struct __rte_lpm *internal_lpm;
 
-       rte_mcfg_tailq_write_lock();
+       if (lpm == NULL || cfg == NULL) {
+               rte_errno = EINVAL;
+               return 1;
+       }
 
-       /* find our tailq entry */
-       TAILQ_FOREACH(te, lpm_list, next) {
-               if (te->data == (void *) lpm)
-                       break;
+       internal_lpm = container_of(lpm, struct __rte_lpm, lpm);
+       if (internal_lpm->v != NULL) {
+               rte_errno = EEXIST;
+               return 1;
        }
-       if (te != NULL)
-               TAILQ_REMOVE(lpm_list, te, next);
 
-       rte_mcfg_tailq_write_unlock();
+       if (cfg->mode == RTE_LPM_QSBR_MODE_SYNC) {
+               /* No other things to do. */
+       } else if (cfg->mode == RTE_LPM_QSBR_MODE_DQ) {
+               /* Init QSBR defer queue. */
+               snprintf(rcu_dq_name, sizeof(rcu_dq_name),
+                               "LPM_RCU_%s", lpm->name);
+               params.name = rcu_dq_name;
+               params.size = cfg->dq_size;
+               if (params.size == 0)
+                       params.size = lpm->number_tbl8s;
+               params.trigger_reclaim_limit = cfg->reclaim_thd;
+               params.max_reclaim_size = cfg->reclaim_max;
+               if (params.max_reclaim_size == 0)
+                       params.max_reclaim_size = RTE_LPM_RCU_DQ_RECLAIM_MAX;
+               params.esize = sizeof(uint32_t);        /* tbl8 group index */
+               params.free_fn = __lpm_rcu_qsbr_free_resource;
+               params.p = lpm;
+               params.v = cfg->v;
+               internal_lpm->dq = rte_rcu_qsbr_dq_create(&params);
+               if (internal_lpm->dq == NULL) {
+                       RTE_LOG(ERR, LPM, "LPM defer queue creation failed\n");
+                       return 1;
+               }
+               if (dq != NULL)
+                       *dq = internal_lpm->dq;
+       } else {
+               rte_errno = EINVAL;
+               return 1;
+       }
+       internal_lpm->rcu_mode = cfg->mode;
+       internal_lpm->v = cfg->v;
 
-       rte_free(lpm->tbl8);
-       rte_free(lpm->rules_tbl);
-       rte_free(lpm);
-       rte_free(te);
+       return 0;
 }
-BIND_DEFAULT_SYMBOL(rte_lpm_free, _v1604, 16.04);
-MAP_STATIC_SYMBOL(void rte_lpm_free(struct rte_lpm *lpm),
-               rte_lpm_free_v1604);
 
 /*
  * Adds a rule to the rule table.
@@ -402,79 +352,7 @@ MAP_STATIC_SYMBOL(void rte_lpm_free(struct rte_lpm *lpm),
  * NOTE: Valid range for depth parameter is 1 .. 32 inclusive.
  */
 static int32_t
-rule_add_v20(struct rte_lpm_v20 *lpm, uint32_t ip_masked, uint8_t depth,
-       uint8_t next_hop)
-{
-       uint32_t rule_gindex, rule_index, last_rule;
-       int i;
-
-       VERIFY_DEPTH(depth);
-
-       /* Scan through rule group to see if rule already exists. */
-       if (lpm->rule_info[depth - 1].used_rules > 0) {
-
-               /* rule_gindex stands for rule group index. */
-               rule_gindex = lpm->rule_info[depth - 1].first_rule;
-               /* Initialise rule_index to point to start of rule group. */
-               rule_index = rule_gindex;
-               /* Last rule = Last used rule in this rule group. */
-               last_rule = rule_gindex + lpm->rule_info[depth - 1].used_rules;
-
-               for (; rule_index < last_rule; rule_index++) {
-
-                       /* If rule already exists update its next_hop and return. */
-                       if (lpm->rules_tbl[rule_index].ip == ip_masked) {
-                               lpm->rules_tbl[rule_index].next_hop = next_hop;
-
-                               return rule_index;
-                       }
-               }
-
-               if (rule_index == lpm->max_rules)
-                       return -ENOSPC;
-       } else {
-               /* Calculate the position in which the rule will be stored. */
-               rule_index = 0;
-
-               for (i = depth - 1; i > 0; i--) {
-                       if (lpm->rule_info[i - 1].used_rules > 0) {
-                               rule_index = lpm->rule_info[i - 1].first_rule
-                                               + lpm->rule_info[i - 1].used_rules;
-                               break;
-                       }
-               }
-               if (rule_index == lpm->max_rules)
-                       return -ENOSPC;
-
-               lpm->rule_info[depth - 1].first_rule = rule_index;
-       }
-
-       /* Make room for the new rule in the array. */
-       for (i = RTE_LPM_MAX_DEPTH; i > depth; i--) {
-               if (lpm->rule_info[i - 1].first_rule
-                               + lpm->rule_info[i - 1].used_rules == lpm->max_rules)
-                       return -ENOSPC;
-
-               if (lpm->rule_info[i - 1].used_rules > 0) {
-                       lpm->rules_tbl[lpm->rule_info[i - 1].first_rule
-                               + lpm->rule_info[i - 1].used_rules]
-                                       = lpm->rules_tbl[lpm->rule_info[i - 1].first_rule];
-                       lpm->rule_info[i - 1].first_rule++;
-               }
-       }
-
-       /* Add the new rule. */
-       lpm->rules_tbl[rule_index].ip = ip_masked;
-       lpm->rules_tbl[rule_index].next_hop = next_hop;
-
-       /* Increment the used rules counter for this rule group. */
-       lpm->rule_info[depth - 1].used_rules++;
-
-       return rule_index;
-}
-
-static int32_t
-rule_add_v1604(struct rte_lpm *lpm, uint32_t ip_masked, uint8_t depth,
+rule_add(struct rte_lpm *lpm, uint32_t ip_masked, uint8_t depth,
        uint32_t next_hop)
 {
        uint32_t rule_gindex, rule_index, last_rule;
@@ -494,8 +372,12 @@ rule_add_v1604(struct rte_lpm *lpm, uint32_t ip_masked, uint8_t depth,
 
                for (; rule_index < last_rule; rule_index++) {
 
-                       /* If rule already exists update its next_hop and return. */
+                       /* If rule already exists update next hop and return. */
                        if (lpm->rules_tbl[rule_index].ip == ip_masked) {
+
+                               if (lpm->rules_tbl[rule_index].next_hop
+                                               == next_hop)
+                                       return -EEXIST;
                                lpm->rules_tbl[rule_index].next_hop = next_hop;
 
                                return rule_index;
@@ -550,30 +432,7 @@ rule_add_v1604(struct rte_lpm *lpm, uint32_t ip_masked, uint8_t depth,
  * NOTE: Valid range for depth parameter is 1 .. 32 inclusive.
  */
 static void
-rule_delete_v20(struct rte_lpm_v20 *lpm, int32_t rule_index, uint8_t depth)
-{
-       int i;
-
-       VERIFY_DEPTH(depth);
-
-       lpm->rules_tbl[rule_index] =
-                       lpm->rules_tbl[lpm->rule_info[depth - 1].first_rule
-                               + lpm->rule_info[depth - 1].used_rules - 1];
-
-       for (i = depth; i < RTE_LPM_MAX_DEPTH; i++) {
-               if (lpm->rule_info[i].used_rules > 0) {
-                       lpm->rules_tbl[lpm->rule_info[i].first_rule - 1] =
-                               lpm->rules_tbl[lpm->rule_info[i].first_rule
-                                       + lpm->rule_info[i].used_rules - 1];
-                       lpm->rule_info[i].first_rule--;
-               }
-       }
-
-       lpm->rule_info[depth - 1].used_rules--;
-}
-
-static void
-rule_delete_v1604(struct rte_lpm *lpm, int32_t rule_index, uint8_t depth)
+rule_delete(struct rte_lpm *lpm, int32_t rule_index, uint8_t depth)
 {
        int i;
 
@@ -600,28 +459,7 @@ rule_delete_v1604(struct rte_lpm *lpm, int32_t rule_index, uint8_t depth)
  * NOTE: Valid range for depth parameter is 1 .. 32 inclusive.
  */
 static int32_t
-rule_find_v20(struct rte_lpm_v20 *lpm, uint32_t ip_masked, uint8_t depth)
-{
-       uint32_t rule_gindex, last_rule, rule_index;
-
-       VERIFY_DEPTH(depth);
-
-       rule_gindex = lpm->rule_info[depth - 1].first_rule;
-       last_rule = rule_gindex + lpm->rule_info[depth - 1].used_rules;
-
-       /* Scan used rules at given depth to find rule. */
-       for (rule_index = rule_gindex; rule_index < last_rule; rule_index++) {
-               /* If rule is found return the rule index. */
-               if (lpm->rules_tbl[rule_index].ip == ip_masked)
-                       return rule_index;
-       }
-
-       /* If rule is not found return -EINVAL. */
-       return -EINVAL;
-}
-
-static int32_t
-rule_find_v1604(struct rte_lpm *lpm, uint32_t ip_masked, uint8_t depth)
+rule_find(struct rte_lpm *lpm, uint32_t ip_masked, uint8_t depth)
 {
        uint32_t rule_gindex, last_rule, rule_index;
 
@@ -645,23 +483,23 @@ rule_find_v1604(struct rte_lpm *lpm, uint32_t ip_masked, uint8_t depth)
  * Find, clean and allocate a tbl8.
  */
 static int32_t
-tbl8_alloc_v20(struct rte_lpm_tbl_entry_v20 *tbl8)
+_tbl8_alloc(struct rte_lpm *lpm)
 {
        uint32_t group_idx; /* tbl8 group index. */
-       struct rte_lpm_tbl_entry_v20 *tbl8_entry;
+       struct rte_lpm_tbl_entry *tbl8_entry;
 
        /* Scan through tbl8 to find a free (i.e. INVALID) tbl8 group. */
-       for (group_idx = 0; group_idx < RTE_LPM_TBL8_NUM_GROUPS;
-                       group_idx++) {
-               tbl8_entry = &tbl8[group_idx * RTE_LPM_TBL8_GROUP_NUM_ENTRIES];
+       for (group_idx = 0; group_idx < lpm->number_tbl8s; group_idx++) {
+               tbl8_entry = &lpm->tbl8[group_idx *
+                                       RTE_LPM_TBL8_GROUP_NUM_ENTRIES];
                /* If a free tbl8 group is found clean it and set as VALID. */
                if (!tbl8_entry->valid_group) {
-                       struct rte_lpm_tbl_entry_v20 new_tbl8_entry = {
+                       struct rte_lpm_tbl_entry new_tbl8_entry = {
+                               .next_hop = 0,
                                .valid = INVALID,
                                .depth = 0,
                                .valid_group = VALID,
                        };
-                       new_tbl8_entry.next_hop = 0;
 
                        memset(&tbl8_entry[0], 0,
                                        RTE_LPM_TBL8_GROUP_NUM_ENTRIES *
@@ -680,137 +518,50 @@ tbl8_alloc_v20(struct rte_lpm_tbl_entry_v20 *tbl8)
 }
 
 static int32_t
-tbl8_alloc_v1604(struct rte_lpm_tbl_entry *tbl8, uint32_t number_tbl8s)
+tbl8_alloc(struct rte_lpm *lpm)
 {
-       uint32_t group_idx; /* tbl8 group index. */
-       struct rte_lpm_tbl_entry *tbl8_entry;
+       int32_t group_idx; /* tbl8 group index. */
+       struct __rte_lpm *internal_lpm;
+
+       internal_lpm = container_of(lpm, struct __rte_lpm, lpm);
+       group_idx = _tbl8_alloc(lpm);
+       if (group_idx == -ENOSPC && internal_lpm->dq != NULL) {
+               /* If there are no tbl8 groups try to reclaim one. */
+               if (rte_rcu_qsbr_dq_reclaim(internal_lpm->dq, 1,
+                               NULL, NULL, NULL) == 0)
+                       group_idx = _tbl8_alloc(lpm);
+       }
 
-       /* Scan through tbl8 to find a free (i.e. INVALID) tbl8 group. */
-       for (group_idx = 0; group_idx < number_tbl8s; group_idx++) {
-               tbl8_entry = &tbl8[group_idx * RTE_LPM_TBL8_GROUP_NUM_ENTRIES];
-               /* If a free tbl8 group is found clean it and set as VALID. */
-               if (!tbl8_entry->valid_group) {
-                       struct rte_lpm_tbl_entry new_tbl8_entry = {
-                               .next_hop = 0,
-                               .valid = INVALID,
-                               .depth = 0,
-                               .valid_group = VALID,
-                       };
-
-                       memset(&tbl8_entry[0], 0,
-                                       RTE_LPM_TBL8_GROUP_NUM_ENTRIES *
-                                       sizeof(tbl8_entry[0]));
-
-                       __atomic_store(tbl8_entry, &new_tbl8_entry,
-                                       __ATOMIC_RELAXED);
-
-                       /* Return group index for allocated tbl8 group. */
-                       return group_idx;
-               }
-       }
-
-       /* If there are no tbl8 groups free then return error. */
-       return -ENOSPC;
-}
-
-static void
-tbl8_free_v20(struct rte_lpm_tbl_entry_v20 *tbl8, uint32_t tbl8_group_start)
-{
-       /* Set tbl8 group invalid*/
-       struct rte_lpm_tbl_entry_v20 zero_tbl8_entry = {
-               .valid = INVALID,
-               .depth = 0,
-               .valid_group = INVALID,
-       };
-       zero_tbl8_entry.next_hop = 0;
-
-       __atomic_store(&tbl8[tbl8_group_start], &zero_tbl8_entry,
-                       __ATOMIC_RELAXED);
-}
+       return group_idx;
+}
 
 static void
-tbl8_free_v1604(struct rte_lpm_tbl_entry *tbl8, uint32_t tbl8_group_start)
+tbl8_free(struct rte_lpm *lpm, uint32_t tbl8_group_start)
 {
-       /* Set tbl8 group invalid*/
        struct rte_lpm_tbl_entry zero_tbl8_entry = {0};
+       struct __rte_lpm *internal_lpm;
 
-       __atomic_store(&tbl8[tbl8_group_start], &zero_tbl8_entry,
-                       __ATOMIC_RELAXED);
-}
-
-static __rte_noinline int32_t
-add_depth_small_v20(struct rte_lpm_v20 *lpm, uint32_t ip, uint8_t depth,
-               uint8_t next_hop)
-{
-       uint32_t tbl24_index, tbl24_range, tbl8_index, tbl8_group_end, i, j;
-
-       /* Calculate the index into Table24. */
-       tbl24_index = ip >> 8;
-       tbl24_range = depth_to_range(depth);
-
-       for (i = tbl24_index; i < (tbl24_index + tbl24_range); i++) {
-               /*
-                * For invalid OR valid and non-extended tbl 24 entries set
-                * entry.
-                */
-               if (!lpm->tbl24[i].valid || (lpm->tbl24[i].valid_group == 0 &&
-                               lpm->tbl24[i].depth <= depth)) {
-
-                       struct rte_lpm_tbl_entry_v20 new_tbl24_entry = {
-                               .valid = VALID,
-                               .valid_group = 0,
-                               .depth = depth,
-                       };
-                       new_tbl24_entry.next_hop = next_hop;
-
-                       /* Setting tbl24 entry in one go to avoid race
-                        * conditions
-                        */
-                       __atomic_store(&lpm->tbl24[i], &new_tbl24_entry,
-                                       __ATOMIC_RELEASE);
-
-                       continue;
-               }
-
-               if (lpm->tbl24[i].valid_group == 1) {
-                       /* If tbl24 entry is valid and extended calculate the
-                        *  index into tbl8.
-                        */
-                       tbl8_index = lpm->tbl24[i].group_idx *
-                                       RTE_LPM_TBL8_GROUP_NUM_ENTRIES;
-                       tbl8_group_end = tbl8_index +
-                                       RTE_LPM_TBL8_GROUP_NUM_ENTRIES;
-
-                       for (j = tbl8_index; j < tbl8_group_end; j++) {
-                               if (!lpm->tbl8[j].valid ||
-                                               lpm->tbl8[j].depth <= depth) {
-                                       struct rte_lpm_tbl_entry_v20
-                                               new_tbl8_entry = {
-                                               .valid = VALID,
-                                               .valid_group = VALID,
-                                               .depth = depth,
-                                       };
-                                       new_tbl8_entry.next_hop = next_hop;
-
-                                       /*
-                                        * Setting tbl8 entry in one go to avoid
-                                        * race conditions
-                                        */
-                                       __atomic_store(&lpm->tbl8[j],
-                                               &new_tbl8_entry,
-                                               __ATOMIC_RELAXED);
-
-                                       continue;
-                               }
-                       }
-               }
+       internal_lpm = container_of(lpm, struct __rte_lpm, lpm);
+       if (internal_lpm->v == NULL) {
+               /* Set tbl8 group invalid*/
+               __atomic_store(&lpm->tbl8[tbl8_group_start], &zero_tbl8_entry,
+                               __ATOMIC_RELAXED);
+       } else if (internal_lpm->rcu_mode == RTE_LPM_QSBR_MODE_SYNC) {
+               /* Wait for quiescent state change. */
+               rte_rcu_qsbr_synchronize(internal_lpm->v,
+                       RTE_QSBR_THRID_INVALID);
+               /* Set tbl8 group invalid*/
+               __atomic_store(&lpm->tbl8[tbl8_group_start], &zero_tbl8_entry,
+                               __ATOMIC_RELAXED);
+       } else if (internal_lpm->rcu_mode == RTE_LPM_QSBR_MODE_DQ) {
+               /* Push into QSBR defer queue. */
+               rte_rcu_qsbr_dq_enqueue(internal_lpm->dq,
+                               (void *)&tbl8_group_start);
        }
-
-       return 0;
 }
 
 static __rte_noinline int32_t
-add_depth_small_v1604(struct rte_lpm *lpm, uint32_t ip, uint8_t depth,
+add_depth_small(struct rte_lpm *lpm, uint32_t ip, uint8_t depth,
                uint32_t next_hop)
 {
 #define group_idx next_hop
@@ -882,150 +633,7 @@ add_depth_small_v1604(struct rte_lpm *lpm, uint32_t ip, uint8_t depth,
 }
 
 static __rte_noinline int32_t
-add_depth_big_v20(struct rte_lpm_v20 *lpm, uint32_t ip_masked, uint8_t depth,
-               uint8_t next_hop)
-{
-       uint32_t tbl24_index;
-       int32_t tbl8_group_index, tbl8_group_start, tbl8_group_end, tbl8_index,
-               tbl8_range, i;
-
-       tbl24_index = (ip_masked >> 8);
-       tbl8_range = depth_to_range(depth);
-
-       if (!lpm->tbl24[tbl24_index].valid) {
-               /* Search for a free tbl8 group. */
-               tbl8_group_index = tbl8_alloc_v20(lpm->tbl8);
-
-               /* Check tbl8 allocation was successful. */
-               if (tbl8_group_index < 0) {
-                       return tbl8_group_index;
-               }
-
-               /* Find index into tbl8 and range. */
-               tbl8_index = (tbl8_group_index *
-                               RTE_LPM_TBL8_GROUP_NUM_ENTRIES) +
-                               (ip_masked & 0xFF);
-
-               /* Set tbl8 entry. */
-               for (i = tbl8_index; i < (tbl8_index + tbl8_range); i++) {
-                       struct rte_lpm_tbl_entry_v20 new_tbl8_entry = {
-                               .valid = VALID,
-                               .depth = depth,
-                               .valid_group = lpm->tbl8[i].valid_group,
-                       };
-                       new_tbl8_entry.next_hop = next_hop;
-                       __atomic_store(&lpm->tbl8[i], &new_tbl8_entry,
-                                       __ATOMIC_RELAXED);
-               }
-
-               /*
-                * Update tbl24 entry to point to new tbl8 entry. Note: The
-                * ext_flag and tbl8_index need to be updated simultaneously,
-                * so assign whole structure in one go
-                */
-
-               struct rte_lpm_tbl_entry_v20 new_tbl24_entry = {
-                       .group_idx = (uint8_t)tbl8_group_index,
-                       .valid = VALID,
-                       .valid_group = 1,
-                       .depth = 0,
-               };
-
-               __atomic_store(&lpm->tbl24[tbl24_index], &new_tbl24_entry,
-                               __ATOMIC_RELEASE);
-
-       } /* If valid entry but not extended calculate the index into Table8. */
-       else if (lpm->tbl24[tbl24_index].valid_group == 0) {
-               /* Search for free tbl8 group. */
-               tbl8_group_index = tbl8_alloc_v20(lpm->tbl8);
-
-               if (tbl8_group_index < 0) {
-                       return tbl8_group_index;
-               }
-
-               tbl8_group_start = tbl8_group_index *
-                               RTE_LPM_TBL8_GROUP_NUM_ENTRIES;
-               tbl8_group_end = tbl8_group_start +
-                               RTE_LPM_TBL8_GROUP_NUM_ENTRIES;
-
-               /* Populate new tbl8 with tbl24 value. */
-               for (i = tbl8_group_start; i < tbl8_group_end; i++) {
-                       struct rte_lpm_tbl_entry_v20 new_tbl8_entry = {
-                               .valid = VALID,
-                               .depth = lpm->tbl24[tbl24_index].depth,
-                               .valid_group = lpm->tbl8[i].valid_group,
-                       };
-                       new_tbl8_entry.next_hop =
-                               lpm->tbl24[tbl24_index].next_hop;
-                       __atomic_store(&lpm->tbl8[i], &new_tbl8_entry,
-                                       __ATOMIC_RELAXED);
-               }
-
-               tbl8_index = tbl8_group_start + (ip_masked & 0xFF);
-
-               /* Insert new rule into the tbl8 entry. */
-               for (i = tbl8_index; i < tbl8_index + tbl8_range; i++) {
-                       struct rte_lpm_tbl_entry_v20 new_tbl8_entry = {
-                               .valid = VALID,
-                               .depth = depth,
-                               .valid_group = lpm->tbl8[i].valid_group,
-                       };
-                       new_tbl8_entry.next_hop = next_hop;
-                       __atomic_store(&lpm->tbl8[i], &new_tbl8_entry,
-                                       __ATOMIC_RELAXED);
-               }
-
-               /*
-                * Update tbl24 entry to point to new tbl8 entry. Note: The
-                * ext_flag and tbl8_index need to be updated simultaneously,
-                * so assign whole structure in one go.
-                */
-
-               struct rte_lpm_tbl_entry_v20 new_tbl24_entry = {
-                               .group_idx = (uint8_t)tbl8_group_index,
-                               .valid = VALID,
-                               .valid_group = 1,
-                               .depth = 0,
-               };
-
-               __atomic_store(&lpm->tbl24[tbl24_index], &new_tbl24_entry,
-                               __ATOMIC_RELEASE);
-
-       } else { /*
-               * If it is valid, extended entry calculate the index into tbl8.
-               */
-               tbl8_group_index = lpm->tbl24[tbl24_index].group_idx;
-               tbl8_group_start = tbl8_group_index *
-                               RTE_LPM_TBL8_GROUP_NUM_ENTRIES;
-               tbl8_index = tbl8_group_start + (ip_masked & 0xFF);
-
-               for (i = tbl8_index; i < (tbl8_index + tbl8_range); i++) {
-
-                       if (!lpm->tbl8[i].valid ||
-                                       lpm->tbl8[i].depth <= depth) {
-                               struct rte_lpm_tbl_entry_v20 new_tbl8_entry = {
-                                       .valid = VALID,
-                                       .depth = depth,
-                                       .valid_group = lpm->tbl8[i].valid_group,
-                               };
-                               new_tbl8_entry.next_hop = next_hop;
-                               /*
-                                * Setting tbl8 entry in one go to avoid race
-                                * condition
-                                */
-                               __atomic_store(&lpm->tbl8[i], &new_tbl8_entry,
-                                               __ATOMIC_RELAXED);
-
-                               continue;
-                       }
-               }
-       }
-
-       return 0;
-}
-
-static __rte_noinline int32_t
-add_depth_big_v1604(struct rte_lpm *lpm, uint32_t ip_masked, uint8_t depth,
+add_depth_big(struct rte_lpm *lpm, uint32_t ip_masked, uint8_t depth,
                uint32_t next_hop)
 {
 #define group_idx next_hop
@@ -1038,7 +646,7 @@ add_depth_big_v1604(struct rte_lpm *lpm, uint32_t ip_masked, uint8_t depth,
 
        if (!lpm->tbl24[tbl24_index].valid) {
                /* Search for a free tbl8 group. */
-               tbl8_group_index = tbl8_alloc_v1604(lpm->tbl8, lpm->number_tbl8s);
+               tbl8_group_index = tbl8_alloc(lpm);
 
                /* Check tbl8 allocation was successful. */
                if (tbl8_group_index < 0) {
@@ -1084,7 +692,7 @@ add_depth_big_v1604(struct rte_lpm *lpm, uint32_t ip_masked, uint8_t depth,
        } /* If valid entry but not extended calculate the index into Table8. */
        else if (lpm->tbl24[tbl24_index].valid_group == 0) {
                /* Search for free tbl8 group. */
-               tbl8_group_index = tbl8_alloc_v1604(lpm->tbl8, lpm->number_tbl8s);
+               tbl8_group_index = tbl8_alloc(lpm);
 
                if (tbl8_group_index < 0) {
                        return tbl8_group_index;
@@ -1177,49 +785,8 @@ add_depth_big_v1604(struct rte_lpm *lpm, uint32_t ip_masked, uint8_t depth,
 /*
  * Add a route
  */
-int __vsym
-rte_lpm_add_v20(struct rte_lpm_v20 *lpm, uint32_t ip, uint8_t depth,
-               uint8_t next_hop)
-{
-       int32_t rule_index, status = 0;
-       uint32_t ip_masked;
-
-       /* Check user arguments. */
-       if ((lpm == NULL) || (depth < 1) || (depth > RTE_LPM_MAX_DEPTH))
-               return -EINVAL;
-
-       ip_masked = ip & depth_to_mask(depth);
-
-       /* Add the rule to the rule table. */
-       rule_index = rule_add_v20(lpm, ip_masked, depth, next_hop);
-
-       /* If the is no space available for new rule return error. */
-       if (rule_index < 0) {
-               return rule_index;
-       }
-
-       if (depth <= MAX_DEPTH_TBL24) {
-               status = add_depth_small_v20(lpm, ip_masked, depth, next_hop);
-       } else { /* If depth > RTE_LPM_MAX_DEPTH_TBL24 */
-               status = add_depth_big_v20(lpm, ip_masked, depth, next_hop);
-
-               /*
-                * If add fails due to exhaustion of tbl8 extensions delete
-                * rule that was added to rule table.
-                */
-               if (status < 0) {
-                       rule_delete_v20(lpm, rule_index, depth);
-
-                       return status;
-               }
-       }
-
-       return 0;
-}
-VERSION_SYMBOL(rte_lpm_add, _v20, 2.0);
-
-int __vsym
-rte_lpm_add_v1604(struct rte_lpm *lpm, uint32_t ip, uint8_t depth,
+int
+rte_lpm_add(struct rte_lpm *lpm, uint32_t ip, uint8_t depth,
                uint32_t next_hop)
 {
        int32_t rule_index, status = 0;
@@ -1232,7 +799,13 @@ rte_lpm_add_v1604(struct rte_lpm *lpm, uint32_t ip, uint8_t depth,
        ip_masked = ip & depth_to_mask(depth);
 
        /* Add the rule to the rule table. */
-       rule_index = rule_add_v1604(lpm, ip_masked, depth, next_hop);
+       rule_index = rule_add(lpm, ip_masked, depth, next_hop);
+
+       /* Skip table entries update if The rule is the same as
+        * the rule in the rules table.
+        */
+       if (rule_index == -EEXIST)
+               return 0;
 
        /* If the is no space available for new rule return error. */
        if (rule_index < 0) {
@@ -1240,16 +813,16 @@ rte_lpm_add_v1604(struct rte_lpm *lpm, uint32_t ip, uint8_t depth,
        }
 
        if (depth <= MAX_DEPTH_TBL24) {
-               status = add_depth_small_v1604(lpm, ip_masked, depth, next_hop);
+               status = add_depth_small(lpm, ip_masked, depth, next_hop);
        } else { /* If depth > RTE_LPM_MAX_DEPTH_TBL24 */
-               status = add_depth_big_v1604(lpm, ip_masked, depth, next_hop);
+               status = add_depth_big(lpm, ip_masked, depth, next_hop);
 
                /*
                 * If add fails due to exhaustion of tbl8 extensions delete
                 * rule that was added to rule table.
                 */
                if (status < 0) {
-                       rule_delete_v1604(lpm, rule_index, depth);
+                       rule_delete(lpm, rule_index, depth);
 
                        return status;
                }
@@ -1257,219 +830,60 @@ rte_lpm_add_v1604(struct rte_lpm *lpm, uint32_t ip, uint8_t depth,
 
        return 0;
 }
-BIND_DEFAULT_SYMBOL(rte_lpm_add, _v1604, 16.04);
-MAP_STATIC_SYMBOL(int rte_lpm_add(struct rte_lpm *lpm, uint32_t ip,
-               uint8_t depth, uint32_t next_hop), rte_lpm_add_v1604);
 
 /*
  * Look for a rule in the high-level rules table
- */
-int __vsym
-rte_lpm_is_rule_present_v20(struct rte_lpm_v20 *lpm, uint32_t ip, uint8_t depth,
-uint8_t *next_hop)
-{
-       uint32_t ip_masked;
-       int32_t rule_index;
-
-       /* Check user arguments. */
-       if ((lpm == NULL) ||
-               (next_hop == NULL) ||
-               (depth < 1) || (depth > RTE_LPM_MAX_DEPTH))
-               return -EINVAL;
-
-       /* Look for the rule using rule_find. */
-       ip_masked = ip & depth_to_mask(depth);
-       rule_index = rule_find_v20(lpm, ip_masked, depth);
-
-       if (rule_index >= 0) {
-               *next_hop = lpm->rules_tbl[rule_index].next_hop;
-               return 1;
-       }
-
-       /* If rule is not found return 0. */
-       return 0;
-}
-VERSION_SYMBOL(rte_lpm_is_rule_present, _v20, 2.0);
-
-int __vsym
-rte_lpm_is_rule_present_v1604(struct rte_lpm *lpm, uint32_t ip, uint8_t depth,
-uint32_t *next_hop)
-{
-       uint32_t ip_masked;
-       int32_t rule_index;
-
-       /* Check user arguments. */
-       if ((lpm == NULL) ||
-               (next_hop == NULL) ||
-               (depth < 1) || (depth > RTE_LPM_MAX_DEPTH))
-               return -EINVAL;
-
-       /* Look for the rule using rule_find. */
-       ip_masked = ip & depth_to_mask(depth);
-       rule_index = rule_find_v1604(lpm, ip_masked, depth);
-
-       if (rule_index >= 0) {
-               *next_hop = lpm->rules_tbl[rule_index].next_hop;
-               return 1;
-       }
-
-       /* If rule is not found return 0. */
-       return 0;
-}
-BIND_DEFAULT_SYMBOL(rte_lpm_is_rule_present, _v1604, 16.04);
-MAP_STATIC_SYMBOL(int rte_lpm_is_rule_present(struct rte_lpm *lpm, uint32_t ip,
-               uint8_t depth, uint32_t *next_hop), rte_lpm_is_rule_present_v1604);
-
-static int32_t
-find_previous_rule_v20(struct rte_lpm_v20 *lpm, uint32_t ip, uint8_t depth,
-               uint8_t *sub_rule_depth)
-{
-       int32_t rule_index;
-       uint32_t ip_masked;
-       uint8_t prev_depth;
-
-       for (prev_depth = (uint8_t)(depth - 1); prev_depth > 0; prev_depth--) {
-               ip_masked = ip & depth_to_mask(prev_depth);
-
-               rule_index = rule_find_v20(lpm, ip_masked, prev_depth);
-
-               if (rule_index >= 0) {
-                       *sub_rule_depth = prev_depth;
-                       return rule_index;
-               }
-       }
-
-       return -1;
-}
-
-static int32_t
-find_previous_rule_v1604(struct rte_lpm *lpm, uint32_t ip, uint8_t depth,
-               uint8_t *sub_rule_depth)
-{
-       int32_t rule_index;
-       uint32_t ip_masked;
-       uint8_t prev_depth;
-
-       for (prev_depth = (uint8_t)(depth - 1); prev_depth > 0; prev_depth--) {
-               ip_masked = ip & depth_to_mask(prev_depth);
-
-               rule_index = rule_find_v1604(lpm, ip_masked, prev_depth);
-
-               if (rule_index >= 0) {
-                       *sub_rule_depth = prev_depth;
-                       return rule_index;
-               }
-       }
-
-       return -1;
-}
-
-static int32_t
-delete_depth_small_v20(struct rte_lpm_v20 *lpm, uint32_t ip_masked,
-       uint8_t depth, int32_t sub_rule_index, uint8_t sub_rule_depth)
-{
-       uint32_t tbl24_range, tbl24_index, tbl8_group_index, tbl8_index, i, j;
-
-       /* Calculate the range and index into Table24. */
-       tbl24_range = depth_to_range(depth);
-       tbl24_index = (ip_masked >> 8);
-
-       /*
-        * Firstly check the sub_rule_index. A -1 indicates no replacement rule
-        * and a positive number indicates a sub_rule_index.
-        */
-       if (sub_rule_index < 0) {
-               /*
-                * If no replacement rule exists then invalidate entries
-                * associated with this rule.
-                */
-               for (i = tbl24_index; i < (tbl24_index + tbl24_range); i++) {
-
-                       if (lpm->tbl24[i].valid_group == 0 &&
-                                       lpm->tbl24[i].depth <= depth) {
-                               struct rte_lpm_tbl_entry_v20
-                                       zero_tbl24_entry = {
-                                               .valid = INVALID,
-                                               .depth = 0,
-                                               .valid_group = 0,
-                                       };
-                                       zero_tbl24_entry.next_hop = 0;
-                               __atomic_store(&lpm->tbl24[i],
-                                       &zero_tbl24_entry, __ATOMIC_RELEASE);
-                       } else if (lpm->tbl24[i].valid_group == 1) {
-                               /*
-                                * If TBL24 entry is extended, then there has
-                                * to be a rule with depth >= 25 in the
-                                * associated TBL8 group.
-                                */
-
-                               tbl8_group_index = lpm->tbl24[i].group_idx;
-                               tbl8_index = tbl8_group_index *
-                                               RTE_LPM_TBL8_GROUP_NUM_ENTRIES;
-
-                               for (j = tbl8_index; j < (tbl8_index +
-                                       RTE_LPM_TBL8_GROUP_NUM_ENTRIES); j++) {
+ */
+int
+rte_lpm_is_rule_present(struct rte_lpm *lpm, uint32_t ip, uint8_t depth,
+uint32_t *next_hop)
+{
+       uint32_t ip_masked;
+       int32_t rule_index;
 
-                                       if (lpm->tbl8[j].depth <= depth)
-                                               lpm->tbl8[j].valid = INVALID;
-                               }
-                       }
-               }
-       } else {
-               /*
-                * If a replacement rule exists then modify entries
-                * associated with this rule.
-                */
+       /* Check user arguments. */
+       if ((lpm == NULL) ||
+               (next_hop == NULL) ||
+               (depth < 1) || (depth > RTE_LPM_MAX_DEPTH))
+               return -EINVAL;
 
-               struct rte_lpm_tbl_entry_v20 new_tbl24_entry = {
-                       .next_hop = lpm->rules_tbl[sub_rule_index].next_hop,
-                       .valid = VALID,
-                       .valid_group = 0,
-                       .depth = sub_rule_depth,
-               };
+       /* Look for the rule using rule_find. */
+       ip_masked = ip & depth_to_mask(depth);
+       rule_index = rule_find(lpm, ip_masked, depth);
 
-               struct rte_lpm_tbl_entry_v20 new_tbl8_entry = {
-                       .valid = VALID,
-                       .valid_group = VALID,
-                       .depth = sub_rule_depth,
-               };
-               new_tbl8_entry.next_hop =
-                               lpm->rules_tbl[sub_rule_index].next_hop;
+       if (rule_index >= 0) {
+               *next_hop = lpm->rules_tbl[rule_index].next_hop;
+               return 1;
+       }
 
-               for (i = tbl24_index; i < (tbl24_index + tbl24_range); i++) {
+       /* If rule is not found return 0. */
+       return 0;
+}
 
-                       if (lpm->tbl24[i].valid_group == 0 &&
-                                       lpm->tbl24[i].depth <= depth) {
-                               __atomic_store(&lpm->tbl24[i], &new_tbl24_entry,
-                                               __ATOMIC_RELEASE);
-                       } else  if (lpm->tbl24[i].valid_group == 1) {
-                               /*
-                                * If TBL24 entry is extended, then there has
-                                * to be a rule with depth >= 25 in the
-                                * associated TBL8 group.
-                                */
+static int32_t
+find_previous_rule(struct rte_lpm *lpm, uint32_t ip, uint8_t depth,
+               uint8_t *sub_rule_depth)
+{
+       int32_t rule_index;
+       uint32_t ip_masked;
+       uint8_t prev_depth;
 
-                               tbl8_group_index = lpm->tbl24[i].group_idx;
-                               tbl8_index = tbl8_group_index *
-                                               RTE_LPM_TBL8_GROUP_NUM_ENTRIES;
+       for (prev_depth = (uint8_t)(depth - 1); prev_depth > 0; prev_depth--) {
+               ip_masked = ip & depth_to_mask(prev_depth);
 
-                               for (j = tbl8_index; j < (tbl8_index +
-                                       RTE_LPM_TBL8_GROUP_NUM_ENTRIES); j++) {
+               rule_index = rule_find(lpm, ip_masked, prev_depth);
 
-                                       if (lpm->tbl8[j].depth <= depth)
-                                               __atomic_store(&lpm->tbl8[j],
-                                                       &new_tbl8_entry,
-                                                       __ATOMIC_RELAXED);
-                               }
-                       }
+               if (rule_index >= 0) {
+                       *sub_rule_depth = prev_depth;
+                       return rule_index;
                }
        }
 
-       return 0;
+       return -1;
 }
 
 static int32_t
-delete_depth_small_v1604(struct rte_lpm *lpm, uint32_t ip_masked,
+delete_depth_small(struct rte_lpm *lpm, uint32_t ip_masked,
        uint8_t depth, int32_t sub_rule_index, uint8_t sub_rule_depth)
 {
 #define group_idx next_hop
@@ -1576,54 +990,7 @@ delete_depth_small_v1604(struct rte_lpm *lpm, uint32_t ip_masked,
  * thus can be recycled
  */
 static int32_t
-tbl8_recycle_check_v20(struct rte_lpm_tbl_entry_v20 *tbl8,
-               uint32_t tbl8_group_start)
-{
-       uint32_t tbl8_group_end, i;
-       tbl8_group_end = tbl8_group_start + RTE_LPM_TBL8_GROUP_NUM_ENTRIES;
-
-       /*
-        * Check the first entry of the given tbl8. If it is invalid we know
-        * this tbl8 does not contain any rule with a depth < RTE_LPM_MAX_DEPTH
-        *  (As they would affect all entries in a tbl8) and thus this table
-        *  can not be recycled.
-        */
-       if (tbl8[tbl8_group_start].valid) {
-               /*
-                * If first entry is valid check if the depth is less than 24
-                * and if so check the rest of the entries to verify that they
-                * are all of this depth.
-                */
-               if (tbl8[tbl8_group_start].depth <= MAX_DEPTH_TBL24) {
-                       for (i = (tbl8_group_start + 1); i < tbl8_group_end;
-                                       i++) {
-
-                               if (tbl8[i].depth !=
-                                               tbl8[tbl8_group_start].depth) {
-
-                                       return -EEXIST;
-                               }
-                       }
-                       /* If all entries are the same return the tb8 index */
-                       return tbl8_group_start;
-               }
-
-               return -EEXIST;
-       }
-       /*
-        * If the first entry is invalid check if the rest of the entries in
-        * the tbl8 are invalid.
-        */
-       for (i = (tbl8_group_start + 1); i < tbl8_group_end; i++) {
-               if (tbl8[i].valid)
-                       return -EEXIST;
-       }
-       /* If no valid entries are found then return -EINVAL. */
-       return -EINVAL;
-}
-
-static int32_t
-tbl8_recycle_check_v1604(struct rte_lpm_tbl_entry *tbl8,
+tbl8_recycle_check(struct rte_lpm_tbl_entry *tbl8,
                uint32_t tbl8_group_start)
 {
        uint32_t tbl8_group_end, i;
@@ -1670,93 +1037,7 @@ tbl8_recycle_check_v1604(struct rte_lpm_tbl_entry *tbl8,
 }
 
 static int32_t
-delete_depth_big_v20(struct rte_lpm_v20 *lpm, uint32_t ip_masked,
-       uint8_t depth, int32_t sub_rule_index, uint8_t sub_rule_depth)
-{
-       uint32_t tbl24_index, tbl8_group_index, tbl8_group_start, tbl8_index,
-                       tbl8_range, i;
-       int32_t tbl8_recycle_index;
-
-       /*
-        * Calculate the index into tbl24 and range. Note: All depths larger
-        * than MAX_DEPTH_TBL24 are associated with only one tbl24 entry.
-        */
-       tbl24_index = ip_masked >> 8;
-
-       /* Calculate the index into tbl8 and range. */
-       tbl8_group_index = lpm->tbl24[tbl24_index].group_idx;
-       tbl8_group_start = tbl8_group_index * RTE_LPM_TBL8_GROUP_NUM_ENTRIES;
-       tbl8_index = tbl8_group_start + (ip_masked & 0xFF);
-       tbl8_range = depth_to_range(depth);
-
-       if (sub_rule_index < 0) {
-               /*
-                * Loop through the range of entries on tbl8 for which the
-                * rule_to_delete must be removed or modified.
-                */
-               for (i = tbl8_index; i < (tbl8_index + tbl8_range); i++) {
-                       if (lpm->tbl8[i].depth <= depth)
-                               lpm->tbl8[i].valid = INVALID;
-               }
-       } else {
-               /* Set new tbl8 entry. */
-               struct rte_lpm_tbl_entry_v20 new_tbl8_entry = {
-                       .valid = VALID,
-                       .depth = sub_rule_depth,
-                       .valid_group = lpm->tbl8[tbl8_group_start].valid_group,
-               };
-
-               new_tbl8_entry.next_hop =
-                               lpm->rules_tbl[sub_rule_index].next_hop;
-               /*
-                * Loop through the range of entries on tbl8 for which the
-                * rule_to_delete must be modified.
-                */
-               for (i = tbl8_index; i < (tbl8_index + tbl8_range); i++) {
-                       if (lpm->tbl8[i].depth <= depth)
-                               __atomic_store(&lpm->tbl8[i], &new_tbl8_entry,
-                                               __ATOMIC_RELAXED);
-               }
-       }
-
-       /*
-        * Check if there are any valid entries in this tbl8 group. If all
-        * tbl8 entries are invalid we can free the tbl8 and invalidate the
-        * associated tbl24 entry.
-        */
-
-       tbl8_recycle_index = tbl8_recycle_check_v20(lpm->tbl8, tbl8_group_start);
-
-       if (tbl8_recycle_index == -EINVAL) {
-               /* Set tbl24 before freeing tbl8 to avoid race condition.
-                * Prevent the free of the tbl8 group from hoisting.
-                */
-               lpm->tbl24[tbl24_index].valid = 0;
-               __atomic_thread_fence(__ATOMIC_RELEASE);
-               tbl8_free_v20(lpm->tbl8, tbl8_group_start);
-       } else if (tbl8_recycle_index > -1) {
-               /* Update tbl24 entry. */
-               struct rte_lpm_tbl_entry_v20 new_tbl24_entry = {
-                       .next_hop = lpm->tbl8[tbl8_recycle_index].next_hop,
-                       .valid = VALID,
-                       .valid_group = 0,
-                       .depth = lpm->tbl8[tbl8_recycle_index].depth,
-               };
-
-               /* Set tbl24 before freeing tbl8 to avoid race condition.
-                * Prevent the free of the tbl8 group from hoisting.
-                */
-               __atomic_store(&lpm->tbl24[tbl24_index], &new_tbl24_entry,
-                               __ATOMIC_RELAXED);
-               __atomic_thread_fence(__ATOMIC_RELEASE);
-               tbl8_free_v20(lpm->tbl8, tbl8_group_start);
-       }
-
-       return 0;
-}
-
-static int32_t
-delete_depth_big_v1604(struct rte_lpm *lpm, uint32_t ip_masked,
+delete_depth_big(struct rte_lpm *lpm, uint32_t ip_masked,
        uint8_t depth, int32_t sub_rule_index, uint8_t sub_rule_depth)
 {
 #define group_idx next_hop
@@ -1811,7 +1092,7 @@ delete_depth_big_v1604(struct rte_lpm *lpm, uint32_t ip_masked,
         * associated tbl24 entry.
         */
 
-       tbl8_recycle_index = tbl8_recycle_check_v1604(lpm->tbl8, tbl8_group_start);
+       tbl8_recycle_index = tbl8_recycle_check(lpm->tbl8, tbl8_group_start);
 
        if (tbl8_recycle_index == -EINVAL) {
                /* Set tbl24 before freeing tbl8 to avoid race condition.
@@ -1819,7 +1100,7 @@ delete_depth_big_v1604(struct rte_lpm *lpm, uint32_t ip_masked,
                 */
                lpm->tbl24[tbl24_index].valid = 0;
                __atomic_thread_fence(__ATOMIC_RELEASE);
-               tbl8_free_v1604(lpm->tbl8, tbl8_group_start);
+               tbl8_free(lpm, tbl8_group_start);
        } else if (tbl8_recycle_index > -1) {
                /* Update tbl24 entry. */
                struct rte_lpm_tbl_entry new_tbl24_entry = {
@@ -1835,7 +1116,7 @@ delete_depth_big_v1604(struct rte_lpm *lpm, uint32_t ip_masked,
                __atomic_store(&lpm->tbl24[tbl24_index], &new_tbl24_entry,
                                __ATOMIC_RELAXED);
                __atomic_thread_fence(__ATOMIC_RELEASE);
-               tbl8_free_v1604(lpm->tbl8, tbl8_group_start);
+               tbl8_free(lpm, tbl8_group_start);
        }
 #undef group_idx
        return 0;
@@ -1844,62 +1125,8 @@ delete_depth_big_v1604(struct rte_lpm *lpm, uint32_t ip_masked,
 /*
  * Deletes a rule
  */
-int __vsym
-rte_lpm_delete_v20(struct rte_lpm_v20 *lpm, uint32_t ip, uint8_t depth)
-{
-       int32_t rule_to_delete_index, sub_rule_index;
-       uint32_t ip_masked;
-       uint8_t sub_rule_depth;
-       /*
-        * Check input arguments. Note: IP must be a positive integer of 32
-        * bits in length therefore it need not be checked.
-        */
-       if ((lpm == NULL) || (depth < 1) || (depth > RTE_LPM_MAX_DEPTH)) {
-               return -EINVAL;
-       }
-
-       ip_masked = ip & depth_to_mask(depth);
-
-       /*
-        * Find the index of the input rule, that needs to be deleted, in the
-        * rule table.
-        */
-       rule_to_delete_index = rule_find_v20(lpm, ip_masked, depth);
-
-       /*
-        * Check if rule_to_delete_index was found. If no rule was found the
-        * function rule_find returns -EINVAL.
-        */
-       if (rule_to_delete_index < 0)
-               return -EINVAL;
-
-       /* Delete the rule from the rule table. */
-       rule_delete_v20(lpm, rule_to_delete_index, depth);
-
-       /*
-        * Find rule to replace the rule_to_delete. If there is no rule to
-        * replace the rule_to_delete we return -1 and invalidate the table
-        * entries associated with this rule.
-        */
-       sub_rule_depth = 0;
-       sub_rule_index = find_previous_rule_v20(lpm, ip, depth, &sub_rule_depth);
-
-       /*
-        * If the input depth value is less than 25 use function
-        * delete_depth_small otherwise use delete_depth_big.
-        */
-       if (depth <= MAX_DEPTH_TBL24) {
-               return delete_depth_small_v20(lpm, ip_masked, depth,
-                               sub_rule_index, sub_rule_depth);
-       } else { /* If depth > MAX_DEPTH_TBL24 */
-               return delete_depth_big_v20(lpm, ip_masked, depth, sub_rule_index,
-                               sub_rule_depth);
-       }
-}
-VERSION_SYMBOL(rte_lpm_delete, _v20, 2.0);
-
-int __vsym
-rte_lpm_delete_v1604(struct rte_lpm *lpm, uint32_t ip, uint8_t depth)
+int
+rte_lpm_delete(struct rte_lpm *lpm, uint32_t ip, uint8_t depth)
 {
        int32_t rule_to_delete_index, sub_rule_index;
        uint32_t ip_masked;
@@ -1918,7 +1145,7 @@ rte_lpm_delete_v1604(struct rte_lpm *lpm, uint32_t ip, uint8_t depth)
         * Find the index of the input rule, that needs to be deleted, in the
         * rule table.
         */
-       rule_to_delete_index = rule_find_v1604(lpm, ip_masked, depth);
+       rule_to_delete_index = rule_find(lpm, ip_masked, depth);
 
        /*
         * Check if rule_to_delete_index was found. If no rule was found the
@@ -1928,7 +1155,7 @@ rte_lpm_delete_v1604(struct rte_lpm *lpm, uint32_t ip, uint8_t depth)
                return -EINVAL;
 
        /* Delete the rule from the rule table. */
-       rule_delete_v1604(lpm, rule_to_delete_index, depth);
+       rule_delete(lpm, rule_to_delete_index, depth);
 
        /*
         * Find rule to replace the rule_to_delete. If there is no rule to
@@ -1936,46 +1163,26 @@ rte_lpm_delete_v1604(struct rte_lpm *lpm, uint32_t ip, uint8_t depth)
         * entries associated with this rule.
         */
        sub_rule_depth = 0;
-       sub_rule_index = find_previous_rule_v1604(lpm, ip, depth, &sub_rule_depth);
+       sub_rule_index = find_previous_rule(lpm, ip, depth, &sub_rule_depth);
 
        /*
         * If the input depth value is less than 25 use function
         * delete_depth_small otherwise use delete_depth_big.
         */
        if (depth <= MAX_DEPTH_TBL24) {
-               return delete_depth_small_v1604(lpm, ip_masked, depth,
+               return delete_depth_small(lpm, ip_masked, depth,
                                sub_rule_index, sub_rule_depth);
        } else { /* If depth > MAX_DEPTH_TBL24 */
-               return delete_depth_big_v1604(lpm, ip_masked, depth, sub_rule_index,
+               return delete_depth_big(lpm, ip_masked, depth, sub_rule_index,
                                sub_rule_depth);
        }
 }
-BIND_DEFAULT_SYMBOL(rte_lpm_delete, _v1604, 16.04);
-MAP_STATIC_SYMBOL(int rte_lpm_delete(struct rte_lpm *lpm, uint32_t ip,
-               uint8_t depth), rte_lpm_delete_v1604);
 
 /*
  * Delete all rules from the LPM table.
  */
-void __vsym
-rte_lpm_delete_all_v20(struct rte_lpm_v20 *lpm)
-{
-       /* Zero rule information. */
-       memset(lpm->rule_info, 0, sizeof(lpm->rule_info));
-
-       /* Zero tbl24. */
-       memset(lpm->tbl24, 0, sizeof(lpm->tbl24));
-
-       /* Zero tbl8. */
-       memset(lpm->tbl8, 0, sizeof(lpm->tbl8));
-
-       /* Delete all rules form the rules table. */
-       memset(lpm->rules_tbl, 0, sizeof(lpm->rules_tbl[0]) * lpm->max_rules);
-}
-VERSION_SYMBOL(rte_lpm_delete_all, _v20, 2.0);
-
-void __vsym
-rte_lpm_delete_all_v1604(struct rte_lpm *lpm)
+void
+rte_lpm_delete_all(struct rte_lpm *lpm)
 {
        /* Zero rule information. */
        memset(lpm->rule_info, 0, sizeof(lpm->rule_info));
@@ -1990,6 +1197,3 @@ rte_lpm_delete_all_v1604(struct rte_lpm *lpm)
        /* Delete all rules form the rules table. */
        memset(lpm->rules_tbl, 0, sizeof(lpm->rules_tbl[0]) * lpm->max_rules);
 }
-BIND_DEFAULT_SYMBOL(rte_lpm_delete_all, _v1604, 16.04);
-MAP_STATIC_SYMBOL(void rte_lpm_delete_all(struct rte_lpm *lpm),
-               rte_lpm_delete_all_v1604);