Re: [B.A.T.M.A.N.] [PATCH v7 04/19] batman-adv: Rename batadv_neigh_ifinfo *_free_ref function to *_put

On Sunday, January 17, 2016 11:01:12 Sven Eckelmann wrote:

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

v7:

• removed patches which are now applied in the branch master
• split *_get -> *_put rename patches into smaller patches only renaming

single functions v6:

• removed patches which are now applied in the branch next
• rebased remaining patches on the patch "batman-adv: Avoid recursive call_rcu for batadv_nc_node" which was modified by Marek while he applied the patches (this unfortunately made some of the remaining patches slightly harder to apply)

v5:

• add hack which allows to compile against stable kernel like 3.2.44 which also added the kref_get_unless_zero function

v4:

• fix function names in commit messages
• fix double whitespace in batadv_tt_orig_list_entry_release kerneldoc
• add extra patch for batadv_claim_free_ref kerneldoc fix
• change the phrase "free it" in all *_free_ref/*_put functions to "release

it" v3:

• update copyright year

v2:

• split patchset into fixes and kref migration to make it easier when the decision is made where each patch will be applied

net/batman-adv/bat_iv_ogm.c | 24 ++++++++++++------------ net/batman-adv/gateway_client.c | 14 +++++++------- net/batman-adv/network-coding.c | 4 ++-- net/batman-adv/originator.c | 8 ++++---- net/batman-adv/originator.h | 2 +- 5 files changed, 26 insertions(+), 26 deletions(-)

Applied in revision 4c9ec87.

Thanks, Marek