Skip to content

Commit

Permalink
batman-adv: Rename batadv_nc_node *_free_ref function to *_put
Browse files Browse the repository at this point in the history
The batman-adv source code is the only place in the kernel which uses the
*_free_ref naming scheme for the *_put functions. Changing it to *_put
makes it more consistent and makes it easier to understand the connection
to the *_get functions.

Signed-off-by: Sven Eckelmann <sven@narfation.org>
Signed-off-by: Marek Lindner <mareklindner@neomailbox.ch>
Signed-off-by: Antonio Quartulli <a@unstable.cc>
  • Loading branch information
Sven Eckelmann authored and Antonio Quartulli committed Feb 23, 2016
1 parent 9c3bf08 commit 27ad754
Showing 1 changed file with 5 additions and 5 deletions.
10 changes: 5 additions & 5 deletions net/batman-adv/network-coding.c
Original file line number Diff line number Diff line change
Expand Up @@ -223,11 +223,11 @@ static void batadv_nc_node_release(struct kref *ref)
}

/**
* batadv_nc_node_free_ref - decrement the nc_node refcounter and possibly
* batadv_nc_node_put - decrement the nc_node refcounter and possibly
* release it
* @nc_node: nc_node to be free'd
*/
static void batadv_nc_node_free_ref(struct batadv_nc_node *nc_node)
static void batadv_nc_node_put(struct batadv_nc_node *nc_node)
{
kref_put(&nc_node->refcount, batadv_nc_node_release);
}
Expand Down Expand Up @@ -356,7 +356,7 @@ batadv_nc_purge_orig_nc_nodes(struct batadv_priv *bat_priv,
"Removing nc_node %pM -> %pM\n",
nc_node->addr, nc_node->orig_node->orig);
list_del_rcu(&nc_node->list);
batadv_nc_node_free_ref(nc_node);
batadv_nc_node_put(nc_node);
}
spin_unlock_bh(lock);
}
Expand Down Expand Up @@ -942,9 +942,9 @@ void batadv_nc_update_nc_node(struct batadv_priv *bat_priv,

out:
if (in_nc_node)
batadv_nc_node_free_ref(in_nc_node);
batadv_nc_node_put(in_nc_node);
if (out_nc_node)
batadv_nc_node_free_ref(out_nc_node);
batadv_nc_node_put(out_nc_node);
}

/**
Expand Down

0 comments on commit 27ad754

Please sign in to comment.