net: ipv6: remove nowait arg to rt6_fill_node
All callers of rt6_fill_node pass 0 for nowait arg. Remove the arg and simplify rt6_fill_node accordingly. rt6_fill_node passes the nowait of 0 to ip6mr_get_route. Remove the nowait arg from it as well. Signed-off-by: David Ahern <dsa@cumulusnetworks.com> Signed-off-by: David S. Miller <davem@davemloft.net>
This commit is contained in:
parent
1ce463dd75
commit
fd61c6ba31
|
@ -116,7 +116,7 @@ struct mfc6_cache {
|
|||
|
||||
struct rtmsg;
|
||||
extern int ip6mr_get_route(struct net *net, struct sk_buff *skb,
|
||||
struct rtmsg *rtm, int nowait, u32 portid);
|
||||
struct rtmsg *rtm, u32 portid);
|
||||
|
||||
#ifdef CONFIG_IPV6_MROUTE
|
||||
extern struct sock *mroute6_socket(struct net *net, struct sk_buff *skb);
|
||||
|
|
|
@ -2288,7 +2288,7 @@ static int __ip6mr_fill_mroute(struct mr6_table *mrt, struct sk_buff *skb,
|
|||
}
|
||||
|
||||
int ip6mr_get_route(struct net *net, struct sk_buff *skb, struct rtmsg *rtm,
|
||||
int nowait, u32 portid)
|
||||
u32 portid)
|
||||
{
|
||||
int err;
|
||||
struct mr6_table *mrt;
|
||||
|
@ -2315,11 +2315,6 @@ int ip6mr_get_route(struct net *net, struct sk_buff *skb, struct rtmsg *rtm,
|
|||
struct net_device *dev;
|
||||
int vif;
|
||||
|
||||
if (nowait) {
|
||||
read_unlock(&mrt_lock);
|
||||
return -EAGAIN;
|
||||
}
|
||||
|
||||
dev = skb->dev;
|
||||
if (!dev || (vif = ip6mr_find_vif(mrt, dev)) < 0) {
|
||||
read_unlock(&mrt_lock);
|
||||
|
@ -2357,7 +2352,7 @@ int ip6mr_get_route(struct net *net, struct sk_buff *skb, struct rtmsg *rtm,
|
|||
return err;
|
||||
}
|
||||
|
||||
if (!nowait && (rtm->rtm_flags&RTM_F_NOTIFY))
|
||||
if (rtm->rtm_flags & RTM_F_NOTIFY)
|
||||
cache->mfc_flags |= MFC_NOTIFY;
|
||||
|
||||
err = __ip6mr_fill_mroute(mrt, skb, cache, rtm);
|
||||
|
|
|
@ -3169,7 +3169,7 @@ static int rt6_fill_node(struct net *net,
|
|||
struct sk_buff *skb, struct rt6_info *rt,
|
||||
struct in6_addr *dst, struct in6_addr *src,
|
||||
int iif, int type, u32 portid, u32 seq,
|
||||
int prefix, int nowait, unsigned int flags)
|
||||
int prefix, unsigned int flags)
|
||||
{
|
||||
u32 metrics[RTAX_MAX];
|
||||
struct rtmsg *rtm;
|
||||
|
@ -3261,19 +3261,12 @@ static int rt6_fill_node(struct net *net,
|
|||
if (iif) {
|
||||
#ifdef CONFIG_IPV6_MROUTE
|
||||
if (ipv6_addr_is_multicast(&rt->rt6i_dst.addr)) {
|
||||
int err = ip6mr_get_route(net, skb, rtm, nowait,
|
||||
portid);
|
||||
int err = ip6mr_get_route(net, skb, rtm, portid);
|
||||
|
||||
if (err <= 0) {
|
||||
if (!nowait) {
|
||||
if (err == 0)
|
||||
return 0;
|
||||
goto nla_put_failure;
|
||||
} else {
|
||||
if (err == -EMSGSIZE)
|
||||
goto nla_put_failure;
|
||||
}
|
||||
}
|
||||
if (err == 0)
|
||||
return 0;
|
||||
if (err < 0)
|
||||
goto nla_put_failure;
|
||||
} else
|
||||
#endif
|
||||
if (nla_put_u32(skb, RTA_IIF, iif))
|
||||
|
@ -3342,7 +3335,7 @@ int rt6_dump_route(struct rt6_info *rt, void *p_arg)
|
|||
return rt6_fill_node(arg->net,
|
||||
arg->skb, rt, NULL, NULL, 0, RTM_NEWROUTE,
|
||||
NETLINK_CB(arg->cb->skb).portid, arg->cb->nlh->nlmsg_seq,
|
||||
prefix, 0, NLM_F_MULTI);
|
||||
prefix, NLM_F_MULTI);
|
||||
}
|
||||
|
||||
static int inet6_rtm_getroute(struct sk_buff *in_skb, struct nlmsghdr *nlh)
|
||||
|
@ -3433,7 +3426,7 @@ static int inet6_rtm_getroute(struct sk_buff *in_skb, struct nlmsghdr *nlh)
|
|||
|
||||
err = rt6_fill_node(net, skb, rt, &fl6.daddr, &fl6.saddr, iif,
|
||||
RTM_NEWROUTE, NETLINK_CB(in_skb).portid,
|
||||
nlh->nlmsg_seq, 0, 0, 0);
|
||||
nlh->nlmsg_seq, 0, 0);
|
||||
if (err < 0) {
|
||||
kfree_skb(skb);
|
||||
goto errout;
|
||||
|
@ -3460,7 +3453,7 @@ void inet6_rt_notify(int event, struct rt6_info *rt, struct nl_info *info,
|
|||
goto errout;
|
||||
|
||||
err = rt6_fill_node(net, skb, rt, NULL, NULL, 0,
|
||||
event, info->portid, seq, 0, 0, nlm_flags);
|
||||
event, info->portid, seq, 0, nlm_flags);
|
||||
if (err < 0) {
|
||||
/* -EMSGSIZE implies BUG in rt6_nlmsg_size() */
|
||||
WARN_ON(err == -EMSGSIZE);
|
||||
|
|
Loading…
Reference in New Issue