net/mlx5: remove unused hash list operations
authorSuanming Mou <suanmingm@nvidia.com>
Wed, 28 Oct 2020 09:33:50 +0000 (17:33 +0800)
committerFerruh Yigit <ferruh.yigit@intel.com>
Tue, 3 Nov 2020 22:35:05 +0000 (23:35 +0100)
In previous commits the hash list objects have been converted
to new thread safe hash list. The legacy hash list code can be
removed now.

Signed-off-by: Suanming Mou <suanmingm@nvidia.com>
Acked-by: Matan Azrad <matan@nvidia.com>
drivers/net/mlx5/mlx5_utils.c
drivers/net/mlx5/mlx5_utils.h

index df15f63..13590dd 100644 (file)
@@ -170,44 +170,6 @@ done:
        return entry;
 }
 
-struct mlx5_hlist_entry *
-mlx5_hlist_lookup_ex(struct mlx5_hlist *h, uint64_t key,
-                    mlx5_hlist_match_callback_fn cb, void *ctx)
-{
-       uint32_t idx;
-       struct mlx5_hlist_head *first;
-       struct mlx5_hlist_entry *node;
-
-       MLX5_ASSERT(h && cb && ctx);
-       idx = rte_hash_crc_8byte(key, 0) & h->mask;
-       first = &h->heads[idx];
-       LIST_FOREACH(node, first, next) {
-               if (!cb(node, ctx))
-                       return node;
-       }
-       return NULL;
-}
-
-int
-mlx5_hlist_insert_ex(struct mlx5_hlist *h, struct mlx5_hlist_entry *entry,
-                    mlx5_hlist_match_callback_fn cb, void *ctx)
-{
-       uint32_t idx;
-       struct mlx5_hlist_head *first;
-       struct mlx5_hlist_entry *node;
-
-       MLX5_ASSERT(h && entry && cb && ctx);
-       idx = rte_hash_crc_8byte(entry->key, 0) & h->mask;
-       first = &h->heads[idx];
-       /* No need to reuse the lookup function. */
-       LIST_FOREACH(node, first, next) {
-               if (!cb(node, ctx))
-                       return -EEXIST;
-       }
-       LIST_INSERT_HEAD(first, entry, next);
-       return 0;
-}
-
 int
 mlx5_hlist_unregister(struct mlx5_hlist *h, struct mlx5_hlist_entry *entry)
 {
index b00789c..be6e5f6 100644 (file)
 
 #include "mlx5_defs.h"
 
-#define mlx5_hlist_remove(h, e) \
-       mlx5_hlist_unregister(h, e)
-
-#define mlx5_hlist_insert(h, e) \
-       mlx5_hlist_register(h, 0, e)
-
 /* Convert a bit number to the corresponding 64-bit mask */
 #define MLX5_BITSHIFT(v) (UINT64_C(1) << (v))
 
@@ -287,23 +281,6 @@ struct mlx5_hlist_entry {
 /** Structure for hash head. */
 LIST_HEAD(mlx5_hlist_head, mlx5_hlist_entry);
 
-/** Type of function that is used to handle the data before freeing. */
-typedef void (*mlx5_hlist_destroy_callback_fn)(void *p, void *ctx);
-
-/**
- * Type of function for user defined matching.
- *
- * @param entry
- *   The entry in the list.
- * @param ctx
- *   The pointer to new entry context.
- *
- * @return
- *   0 if matching, -1 otherwise.
- */
-typedef int (*mlx5_hlist_match_callback_fn)(struct mlx5_hlist_entry *entry,
-                                            void *ctx);
-
 /**
  * Type of callback function for entry removal.
  *
@@ -428,49 +405,6 @@ struct mlx5_hlist *mlx5_hlist_create(const char *name, uint32_t size,
 struct mlx5_hlist_entry *mlx5_hlist_lookup(struct mlx5_hlist *h, uint64_t key,
                                           void *ctx);
 
-/**
- * Extended routine to search an entry matching the context with
- * user defined match function.
- *
- * @param h
- *   Pointer to the hast list table.
- * @param key
- *   Key for the searching entry.
- * @param cb
- *   Callback function to match the node with context.
- * @param ctx
- *   Common context parameter used by callback function.
- *
- * @return
- *   Pointer of the hlist entry if found, NULL otherwise.
- */
-struct mlx5_hlist_entry *mlx5_hlist_lookup_ex(struct mlx5_hlist *h,
-                                             uint64_t key,
-                                             mlx5_hlist_match_callback_fn cb,
-                                             void *ctx);
-
-/**
- * Extended routine to insert an entry to the list with key collisions.
- *
- * For the list have key collision, the extra user defined match function
- * allows node with same key will be inserted.
- *
- * @param h
- *   Pointer to the hast list table.
- * @param entry
- *   Entry to be inserted into the hash list table.
- * @param cb
- *   Callback function to match the node with context.
- * @param ctx
- *   Common context parameter used by callback function.
- *
- * @return
- *   - zero for success.
- *   - -EEXIST if the entry is already inserted.
- */
-int mlx5_hlist_insert_ex(struct mlx5_hlist *h, struct mlx5_hlist_entry *entry,
-                        mlx5_hlist_match_callback_fn cb, void *ctx);
-
 /**
  * Insert an entry to the hash list table, the entry is only part of whole data
  * element and a 64B key is used for matching. User should construct the key or