Skip to content

Commit

Permalink
netlink: cleanup rntl_af_register
Browse files Browse the repository at this point in the history
The function __rtnl_af_register is never called outside this
code, and the return value is always 0.

Signed-off-by: David S. Miller <davem@davemloft.net>
  • Loading branch information
stephen hemminger authored and David S. Miller committed Jan 2, 2014
1 parent 47d1f71 commit 3678a9d
Show file tree
Hide file tree
Showing 4 changed files with 5 additions and 30 deletions.
3 changes: 1 addition & 2 deletions include/net/rtnetlink.h
Original file line number Diff line number Diff line change
Expand Up @@ -115,10 +115,9 @@ struct rtnl_af_ops {
const struct nlattr *attr);
};

int __rtnl_af_register(struct rtnl_af_ops *ops);
void __rtnl_af_unregister(struct rtnl_af_ops *ops);

int rtnl_af_register(struct rtnl_af_ops *ops);
void rtnl_af_register(struct rtnl_af_ops *ops);
void rtnl_af_unregister(struct rtnl_af_ops *ops);

struct net *rtnl_link_get_net(struct net *src_net, struct nlattr *tb[]);
Expand Down
5 changes: 1 addition & 4 deletions net/bridge/br_netlink.c
Original file line number Diff line number Diff line change
Expand Up @@ -481,9 +481,7 @@ int __init br_netlink_init(void)
int err;

br_mdb_init();
err = rtnl_af_register(&br_af_ops);
if (err)
goto out;
rtnl_af_register(&br_af_ops);

err = rtnl_link_register(&br_link_ops);
if (err)
Expand All @@ -493,7 +491,6 @@ int __init br_netlink_init(void)

out_af:
rtnl_af_unregister(&br_af_ops);
out:
br_mdb_uninit();
return err;
}
Expand Down
22 changes: 2 additions & 20 deletions net/core/rtnetlink.c
Original file line number Diff line number Diff line change
Expand Up @@ -402,35 +402,17 @@ static const struct rtnl_af_ops *rtnl_af_lookup(const int family)
return NULL;
}

/**
* __rtnl_af_register - Register rtnl_af_ops with rtnetlink.
* @ops: struct rtnl_af_ops * to register
*
* The caller must hold the rtnl_mutex.
*
* Returns 0 on success or a negative error code.
*/
int __rtnl_af_register(struct rtnl_af_ops *ops)
{
list_add_tail(&ops->list, &rtnl_af_ops);
return 0;
}
EXPORT_SYMBOL_GPL(__rtnl_af_register);

/**
* rtnl_af_register - Register rtnl_af_ops with rtnetlink.
* @ops: struct rtnl_af_ops * to register
*
* Returns 0 on success or a negative error code.
*/
int rtnl_af_register(struct rtnl_af_ops *ops)
void rtnl_af_register(struct rtnl_af_ops *ops)
{
int err;

rtnl_lock();
err = __rtnl_af_register(ops);
list_add_tail(&ops->list, &rtnl_af_ops);
rtnl_unlock();
return err;
}
EXPORT_SYMBOL_GPL(rtnl_af_register);

Expand Down
5 changes: 1 addition & 4 deletions net/ipv6/addrconf.c
Original file line number Diff line number Diff line change
Expand Up @@ -5207,9 +5207,7 @@ int __init addrconf_init(void)

addrconf_verify(0);

err = rtnl_af_register(&inet6_ops);
if (err < 0)
goto errout_af;
rtnl_af_register(&inet6_ops);

err = __rtnl_register(PF_INET6, RTM_GETLINK, NULL, inet6_dump_ifinfo,
NULL);
Expand All @@ -5233,7 +5231,6 @@ int __init addrconf_init(void)
return 0;
errout:
rtnl_af_unregister(&inet6_ops);
errout_af:
unregister_netdevice_notifier(&ipv6_dev_notf);
errlo:
unregister_pernet_subsys(&addrconf_ops);
Expand Down

0 comments on commit 3678a9d

Please sign in to comment.