batman-adv: Prefix hash static inline functions with batadv_
authorSven Eckelmann <sven@narfation.org>
Sat, 12 May 2012 11:48:55 +0000 (13:48 +0200)
committerAntonio Quartulli <ordex@autistici.org>
Sun, 24 Jun 2012 16:41:41 +0000 (18:41 +0200)
All non-static symbols of batman-adv were prefixed with batadv_ to avoid
collisions with other symbols of the kernel. Other symbols of batman-adv
should use the same prefix to keep the naming scheme consistent.

Signed-off-by: Sven Eckelmann <sven@narfation.org>
net/batman-adv/bridge_loop_avoidance.c
net/batman-adv/hash.h
net/batman-adv/originator.c
net/batman-adv/translation-table.c
net/batman-adv/vis.c

index 72ff8b90e22255d33ec55216d951e8cda518333d..7a2dfd41d5c8f96dbff669a94d57d0e87f34711a 100644 (file)
@@ -379,8 +379,9 @@ static struct backbone_gw *bla_get_backbone_gw(struct bat_priv *bat_priv,
        /* one for the hash, one for returning */
        atomic_set(&entry->refcount, 2);
 
-       hash_added = hash_add(bat_priv->backbone_hash, compare_backbone_gw,
-                             choose_backbone_gw, entry, &entry->hash_entry);
+       hash_added = batadv_hash_add(bat_priv->backbone_hash,
+                                    compare_backbone_gw, choose_backbone_gw,
+                                    entry, &entry->hash_entry);
 
        if (unlikely(hash_added != 0)) {
                /* hash failed, free the structure */
@@ -540,8 +541,9 @@ static void bla_add_claim(struct bat_priv *bat_priv, const uint8_t *mac,
                bat_dbg(DBG_BLA, bat_priv,
                        "bla_add_claim(): adding new entry %pM, vid %d to hash ...\n",
                        mac, vid);
-               hash_added = hash_add(bat_priv->claim_hash, compare_claim,
-                                     choose_claim, claim, &claim->hash_entry);
+               hash_added = batadv_hash_add(bat_priv->claim_hash,
+                                            compare_claim, choose_claim,
+                                            claim, &claim->hash_entry);
 
                if (unlikely(hash_added != 0)) {
                        /* only local changes happened. */
@@ -590,7 +592,8 @@ static void bla_del_claim(struct bat_priv *bat_priv, const uint8_t *mac,
 
        bat_dbg(DBG_BLA, bat_priv, "bla_del_claim(): %pM, vid %d\n", mac, vid);
 
-       hash_remove(bat_priv->claim_hash, compare_claim, choose_claim, claim);
+       batadv_hash_remove(bat_priv->claim_hash, compare_claim, choose_claim,
+                          claim);
        claim_free_ref(claim); /* reference from the hash is gone */
 
        claim->backbone_gw->crc ^= crc16(0, claim->addr, ETH_ALEN);
index eba8f2a55ccc5287b4cd8f48a904bea5dc8e96e7..7ec4e5b0bd413a2a753446293e5ac86319e2ea6a 100644 (file)
@@ -54,8 +54,8 @@ void batadv_hash_destroy(struct hashtable_t *hash);
  * called to remove the elements inside of the hash.  if you don't remove the
  * elements, memory might be leaked.
  */
-static inline void hash_delete(struct hashtable_t *hash,
-                              hashdata_free_cb free_cb, void *arg)
+static inline void batadv_hash_delete(struct hashtable_t *hash,
+                                     hashdata_free_cb free_cb, void *arg)
 {
        struct hlist_head *head;
        struct hlist_node *node, *node_tmp;
@@ -89,10 +89,11 @@ static inline void hash_delete(struct hashtable_t *hash,
  *     Returns 0 on success, 1 if the element already is in the hash
  *     and -1 on error.
  */
-static inline int hash_add(struct hashtable_t *hash,
-                          hashdata_compare_cb compare,
-                          hashdata_choose_cb choose,
-                          const void *data, struct hlist_node *data_node)
+static inline int batadv_hash_add(struct hashtable_t *hash,
+                                 hashdata_compare_cb compare,
+                                 hashdata_choose_cb choose,
+                                 const void *data,
+                                 struct hlist_node *data_node)
 {
        uint32_t index;
        int ret = -1;
@@ -133,9 +134,9 @@ out:
  * structure you use with just the key filled, we just need the key for
  * comparing.
  */
-static inline void *hash_remove(struct hashtable_t *hash,
-                               hashdata_compare_cb compare,
-                               hashdata_choose_cb choose, void *data)
+static inline void *batadv_hash_remove(struct hashtable_t *hash,
+                                      hashdata_compare_cb compare,
+                                      hashdata_choose_cb choose, void *data)
 {
        uint32_t index;
        struct hlist_node *node;
index 90d24fccb9cf27e12042305151c5642012f283b2..623c23c6ec36b8abf9bd6b8a34d87b125f7ed0c4 100644 (file)
@@ -248,8 +248,9 @@ struct orig_node *batadv_get_orig_node(struct bat_priv *bat_priv,
        if (!orig_node->bcast_own_sum)
                goto free_bcast_own;
 
-       hash_added = hash_add(bat_priv->orig_hash, compare_orig,
-                             choose_orig, orig_node, &orig_node->hash_entry);
+       hash_added = batadv_hash_add(bat_priv->orig_hash, compare_orig,
+                                    choose_orig, orig_node,
+                                    &orig_node->hash_entry);
        if (hash_added != 0)
                goto free_bcast_own_sum;
 
index bc06af4781b3bb5b8f145c4710a7ddcd36a6c323..72a8548515ae00bb5232cddfaf13f8bccd1f315b 100644 (file)
@@ -234,9 +234,9 @@ void batadv_tt_local_add(struct net_device *soft_iface, const uint8_t *addr,
         */
        tt_local_entry->common.flags |= TT_CLIENT_NEW;
 
-       hash_added = hash_add(bat_priv->tt_local_hash, compare_tt, choose_orig,
-                        &tt_local_entry->common,
-                        &tt_local_entry->common.hash_entry);
+       hash_added = batadv_hash_add(bat_priv->tt_local_hash, compare_tt,
+                                    choose_orig, &tt_local_entry->common,
+                                    &tt_local_entry->common.hash_entry);
 
        if (unlikely(hash_added != 0)) {
                /* remove the reference for the hash */
@@ -618,6 +618,7 @@ int batadv_tt_global_add(struct bat_priv *bat_priv, struct orig_node *orig_node,
        struct tt_global_entry *tt_global_entry = NULL;
        int ret = 0;
        int hash_added;
+       struct tt_common_entry *common;
 
        tt_global_entry = tt_global_hash_find(bat_priv, tt_addr);
 
@@ -627,18 +628,19 @@ int batadv_tt_global_add(struct bat_priv *bat_priv, struct orig_node *orig_node,
                if (!tt_global_entry)
                        goto out;
 
-               memcpy(tt_global_entry->common.addr, tt_addr, ETH_ALEN);
+               common = &tt_global_entry->common;
+               memcpy(common->addr, tt_addr, ETH_ALEN);
 
-               tt_global_entry->common.flags = NO_FLAGS;
+               common->flags = NO_FLAGS;
                tt_global_entry->roam_at = 0;
-               atomic_set(&tt_global_entry->common.refcount, 2);
+               atomic_set(&common->refcount, 2);
 
                INIT_HLIST_HEAD(&tt_global_entry->orig_list);
                spin_lock_init(&tt_global_entry->list_lock);
 
-               hash_added = hash_add(bat_priv->tt_global_hash, compare_tt,
-                                choose_orig, &tt_global_entry->common,
-                                &tt_global_entry->common.hash_entry);
+               hash_added = batadv_hash_add(bat_priv->tt_global_hash,
+                                            compare_tt, choose_orig,
+                                            common, &common->hash_entry);
 
                if (unlikely(hash_added != 0)) {
                        /* remove the reference for the hash */
@@ -816,8 +818,8 @@ static void tt_global_del_struct(struct bat_priv *bat_priv,
                "Deleting global tt entry %pM: %s\n",
                tt_global_entry->common.addr, message);
 
-       hash_remove(bat_priv->tt_global_hash, compare_tt, choose_orig,
-                   tt_global_entry->common.addr);
+       batadv_hash_remove(bat_priv->tt_global_hash, compare_tt, choose_orig,
+                          tt_global_entry->common.addr);
        tt_global_entry_free_ref(tt_global_entry);
 
 }
index 619f0a5a84844df5eb4edb12986822594527ff08..e0a90570d66799a8580e81f0ba8a215aec48c26f 100644 (file)
@@ -433,8 +433,8 @@ static struct vis_info *add_packet(struct bat_priv *bat_priv,
                        }
                }
                /* remove old entry */
-               hash_remove(bat_priv->vis_hash, vis_info_cmp, vis_info_choose,
-                           old_info);
+               batadv_hash_remove(bat_priv->vis_hash, vis_info_cmp,
+                                  vis_info_choose, old_info);
                send_list_del(old_info);
                kref_put(&old_info->refcount, free_info);
        }
@@ -474,8 +474,8 @@ static struct vis_info *add_packet(struct bat_priv *bat_priv,
        recv_list_add(bat_priv, &info->recv_list, packet->sender_orig);
 
        /* try to add it */
-       hash_added = hash_add(bat_priv->vis_hash, vis_info_cmp, vis_info_choose,
-                             info, &info->hash_entry);
+       hash_added = batadv_hash_add(bat_priv->vis_hash, vis_info_cmp,
+                                    vis_info_choose, info, &info->hash_entry);
        if (hash_added != 0) {
                /* did not work (for some reason) */
                kref_put(&info->refcount, free_info);
@@ -934,9 +934,9 @@ int batadv_vis_init(struct bat_priv *bat_priv)
 
        INIT_LIST_HEAD(&bat_priv->vis_send_list);
 
-       hash_added = hash_add(bat_priv->vis_hash, vis_info_cmp, vis_info_choose,
-                             bat_priv->my_vis_info,
-                             &bat_priv->my_vis_info->hash_entry);
+       hash_added = batadv_hash_add(bat_priv->vis_hash, vis_info_cmp,
+                                    vis_info_choose, bat_priv->my_vis_info,
+                                    &bat_priv->my_vis_info->hash_entry);
        if (hash_added != 0) {
                pr_err("Can't add own vis packet into hash\n");
                /* not in hash, need to remove it manually. */
@@ -977,7 +977,7 @@ void batadv_vis_quit(struct bat_priv *bat_priv)
 
        spin_lock_bh(&bat_priv->vis_hash_lock);
        /* properly remove, kill timers ... */
-       hash_delete(bat_priv->vis_hash, free_info_ref, NULL);
+       batadv_hash_delete(bat_priv->vis_hash, free_info_ref, NULL);
        bat_priv->vis_hash = NULL;
        bat_priv->my_vis_info = NULL;
        spin_unlock_bh(&bat_priv->vis_hash_lock);