From: David Ahern <dsahern@gmail.com>
To: Stefano Brivio <sbrivio@redhat.com>, David Miller <davem@davemloft.net>
Cc: Jianlin Shi <jishi@redhat.com>, Wei Wang <weiwan@google.com>,
Martin KaFai Lau <kafai@fb.com>,
Eric Dumazet <edumazet@google.com>,
Matti Vaittinen <matti.vaittinen@fi.rohmeurope.com>,
netdev@vger.kernel.org
Subject: Re: [PATCH net v5 3/6] ipv4: Dump route exceptions if requested
Date: Tue, 18 Jun 2019 08:48:23 -0600 [thread overview]
Message-ID: <b85db470-81c2-3abe-a68b-154711147656@gmail.com> (raw)
In-Reply-To: <106687f38b1eaf957f4ff2bad343519231815482.1560827176.git.sbrivio@redhat.com>
On 6/18/19 7:20 AM, Stefano Brivio wrote:
> diff --git a/include/net/route.h b/include/net/route.h
> index 065b47754f05..f0d0086e76ce 100644
> --- a/include/net/route.h
> +++ b/include/net/route.h
> @@ -221,6 +221,9 @@ void ip_rt_get_source(u8 *src, struct sk_buff *skb, struct rtable *rt);
> struct rtable *rt_dst_alloc(struct net_device *dev,
> unsigned int flags, u16 type,
> bool nopolicy, bool noxfrm, bool will_cache);
> +int rt_fill_info(struct net *net, __be32 dst, __be32 src, struct rtable *rt,
> + u32 table_id, struct flowi4 *fl4, struct sk_buff *skb,
> + u32 portid, u32 seq);
>
> struct in_ifaddr;
> void fib_add_ifaddr(struct in_ifaddr *);
> diff --git a/net/ipv4/fib_trie.c b/net/ipv4/fib_trie.c
> index 868c74771fa9..a00408827ae8 100644
> --- a/net/ipv4/fib_trie.c
> +++ b/net/ipv4/fib_trie.c
> @@ -2000,28 +2000,92 @@ void fib_free_table(struct fib_table *tb)
> call_rcu(&tb->rcu, __trie_free_rcu);
> }
>
> +static int fib_dump_fnhe_from_leaf(struct fib_alias *fa, struct sk_buff *skb,
> + struct netlink_callback *cb,
> + int *fa_index, int fa_start)
> +{
> + struct net *net = sock_net(cb->skb->sk);
> + struct fib_info *fi = fa->fa_info;
> + struct fnhe_hash_bucket *bucket;
> + struct fib_nh_common *nhc;
> + int i, genid;
> +
> + if (!fi || fi->fib_flags & RTNH_F_DEAD)
> + return 0;
> +
> + nhc = fib_info_nhc(fi, 0);
This should be a loop over fi->fib_nhs for net:
for (i = 0; i < fi->fib_nhs; i++) {
nhc = fib_info_nhc(fi, 0);
...
and a loop over fib_info_num_path(fi) for net-next:
for (i = 0; i < fib_info_num_path(fi); i++) {
nhc = fib_info_nhc(fi, 0);
...
> + if (nhc->nhc_flags & RTNH_F_DEAD)
> + return 0;
And then the loop over the exception bucket could be a helper in route.c
in which case you don't need to export rt_fill_info and nhc_exceptions
code does not spread to fib_trie.c
> +
> + bucket = rcu_dereference(nhc->nhc_exceptions);
> + if (!bucket)
> + return 0;
> +
> + genid = fnhe_genid(net);
> +
> + for (i = 0; i < FNHE_HASH_SIZE; i++) {
> + struct fib_nh_exception *fnhe;
> +
> + for (fnhe = rcu_dereference(bucket[i].chain); fnhe;
> + fnhe = rcu_dereference(fnhe->fnhe_next)) {
> + struct flowi4 fl4 = {};
rather than pass an empty flow struct, update rt_fill_info to handle a
NULL fl4; it's only a few checks.
> + struct rtable *rt;
> + int err;
> +
> + if (*fa_index < fa_start)
> + goto next;
> +
> + if (fnhe->fnhe_genid != genid)
> + goto next;
> +
> + if (fnhe->fnhe_expires &&
> + time_after(jiffies, fnhe->fnhe_expires))
> + goto next;
> +
> + rt = rcu_dereference(fnhe->fnhe_rth_input);
> + if (!rt)
> + rt = rcu_dereference(fnhe->fnhe_rth_output);
> + if (!rt)
> + goto next;
> +
> + err = rt_fill_info(net, fnhe->fnhe_daddr, 0, rt,
> + fa->tb_id, &fl4, skb,
> + NETLINK_CB(cb->skb).portid,
> + cb->nlh->nlmsg_seq);
> + if (err)
> + return err;
> +next:
> + (*fa_index)++;
> + }
> + }
> +
> + return 0;
> +}
> +
next prev parent reply other threads:[~2019-06-18 14:48 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-06-18 13:20 [PATCH net v5 0/6] Fix listing (IPv4, IPv6) and flushing (IPv6) of cached route exceptions Stefano Brivio
2019-06-18 13:20 ` [PATCH net v5 1/6] fib_frontend, ip6_fib: Select routes or exceptions dump from RTM_F_CLONED Stefano Brivio
2019-06-18 14:49 ` David Ahern
2019-06-18 13:20 ` [PATCH net v5 2/6] ipv4/fib_frontend: Allow RTM_F_CLONED flag to be used for filtering Stefano Brivio
2019-06-18 14:49 ` David Ahern
2019-06-18 13:20 ` [PATCH net v5 3/6] ipv4: Dump route exceptions if requested Stefano Brivio
2019-06-18 14:48 ` David Ahern [this message]
2019-06-19 23:57 ` Stefano Brivio
2019-06-18 13:20 ` [PATCH net v5 4/6] Revert "net/ipv6: Bail early if user only wants cloned entries" Stefano Brivio
2019-06-18 14:51 ` David Ahern
2019-06-18 13:20 ` [PATCH net v5 5/6] ipv6: Dump route exceptions if requested Stefano Brivio
2019-06-18 15:19 ` David Ahern
2019-06-19 23:57 ` Stefano Brivio
2019-06-18 13:20 ` [PATCH net v5 6/6] ip6_fib: Don't discard nodes with valid routing information in fib6_locate_1() Stefano Brivio
2019-06-18 14:51 ` [PATCH net v5 0/6] Fix listing (IPv4, IPv6) and flushing (IPv6) of cached route exceptions David Ahern
2019-06-18 16:25 ` David Miller
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=b85db470-81c2-3abe-a68b-154711147656@gmail.com \
--to=dsahern@gmail.com \
--cc=davem@davemloft.net \
--cc=edumazet@google.com \
--cc=jishi@redhat.com \
--cc=kafai@fb.com \
--cc=matti.vaittinen@fi.rohmeurope.com \
--cc=netdev@vger.kernel.org \
--cc=sbrivio@redhat.com \
--cc=weiwan@google.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).