Commit bf9a8a06 authored by Stefano Brivio's avatar Stefano Brivio Committed by David S. Miller

ipv6/route: Change return code of rt6_dump_route() for partial node dumps

In the next patch, we are going to add optional dump of exceptions to
rt6_dump_route().

Change the return code of rt6_dump_route() to accomodate partial node
dumps: we might dump multiple routes per node, and might be able to dump
only a given number of them, so fib6_dump_node() will need to know how
many routes have been dumped on partial dump, to restart the dump from the
point where it was interrupted.

Note that fib6_dump_node() is the only caller and already handles all
non-negative return codes as success: those become -1 to signal that we're
done with the node. If we fail, return 0, as we were unable to dump the
single route in the node, but we're not done with it.
Signed-off-by: default avatarStefano Brivio <sbrivio@redhat.com>
Reviewed-by: default avatarDavid Ahern <dsahern@gmail.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 3401bfb1
...@@ -465,7 +465,7 @@ static int fib6_dump_node(struct fib6_walker *w) ...@@ -465,7 +465,7 @@ static int fib6_dump_node(struct fib6_walker *w)
for_each_fib6_walker_rt(w) { for_each_fib6_walker_rt(w) {
res = rt6_dump_route(rt, w->args); res = rt6_dump_route(rt, w->args);
if (res < 0) { if (res >= 0) {
/* Frame is full, suspend walking */ /* Frame is full, suspend walking */
w->leaf = rt; w->leaf = rt;
return 1; return 1;
......
...@@ -5522,6 +5522,7 @@ static bool fib6_info_uses_dev(const struct fib6_info *f6i, ...@@ -5522,6 +5522,7 @@ static bool fib6_info_uses_dev(const struct fib6_info *f6i,
return false; return false;
} }
/* Return -1 if done with node, number of handled routes on partial dump */
int rt6_dump_route(struct fib6_info *rt, void *p_arg) int rt6_dump_route(struct fib6_info *rt, void *p_arg)
{ {
struct rt6_rtnl_dump_arg *arg = (struct rt6_rtnl_dump_arg *) p_arg; struct rt6_rtnl_dump_arg *arg = (struct rt6_rtnl_dump_arg *) p_arg;
...@@ -5530,25 +5531,28 @@ int rt6_dump_route(struct fib6_info *rt, void *p_arg) ...@@ -5530,25 +5531,28 @@ int rt6_dump_route(struct fib6_info *rt, void *p_arg)
struct net *net = arg->net; struct net *net = arg->net;
if (rt == net->ipv6.fib6_null_entry) if (rt == net->ipv6.fib6_null_entry)
return 0; return -1;
if ((filter->flags & RTM_F_PREFIX) && if ((filter->flags & RTM_F_PREFIX) &&
!(rt->fib6_flags & RTF_PREFIX_RT)) { !(rt->fib6_flags & RTF_PREFIX_RT)) {
/* success since this is not a prefix route */ /* success since this is not a prefix route */
return 1; return -1;
} }
if (filter->filter_set) { if (filter->filter_set) {
if ((filter->rt_type && rt->fib6_type != filter->rt_type) || if ((filter->rt_type && rt->fib6_type != filter->rt_type) ||
(filter->dev && !fib6_info_uses_dev(rt, filter->dev)) || (filter->dev && !fib6_info_uses_dev(rt, filter->dev)) ||
(filter->protocol && rt->fib6_protocol != filter->protocol)) { (filter->protocol && rt->fib6_protocol != filter->protocol)) {
return 1; return -1;
} }
flags |= NLM_F_DUMP_FILTERED; flags |= NLM_F_DUMP_FILTERED;
} }
return rt6_fill_node(net, arg->skb, rt, NULL, NULL, NULL, 0, if (rt6_fill_node(net, arg->skb, rt, NULL, NULL, NULL, 0, RTM_NEWROUTE,
RTM_NEWROUTE, NETLINK_CB(arg->cb->skb).portid, NETLINK_CB(arg->cb->skb).portid,
arg->cb->nlh->nlmsg_seq, flags); arg->cb->nlh->nlmsg_seq, flags))
return 0;
return -1;
} }
static int inet6_rtm_valid_getroute_req(struct sk_buff *skb, static int inet6_rtm_valid_getroute_req(struct sk_buff *skb,
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment