ipv4: Call fib_select_default() only when actually necessary.
authorDavid S. Miller <davem@davemloft.net>
Thu, 14 Apr 2011 21:49:37 +0000 (14:49 -0700)
committerDavid S. Miller <davem@davemloft.net>
Thu, 14 Apr 2011 22:05:22 +0000 (15:05 -0700)
fib_select_default() is a complete NOP, and completely pointless
to invoke, when we have no more than 1 default route installed.

And this is far and away the common case.

So remember how many prefixlen==0 routes we have in the routing
table, and elide the call when we have no more than one of those.

This cuts output route creation time by 157 cycles on Niagara2+.

In order to add the new int to fib_table, we have to correct the type
of ->tb_data[] to unsigned long, otherwise the private area will be
unaligned on 64-bit systems.

Signed-off-by: David S. Miller <davem@davemloft.net>
Reviewed-by: Eric Dumazet <eric.dumazet@gmail.com>
include/net/ip_fib.h
net/ipv4/fib_trie.c
net/ipv4/route.c

index 514627f56339a59e1ff9e26d8a808dbe2c944135..10422ef14e28dcf914264da0c4365401ae018574 100644 (file)
@@ -160,7 +160,8 @@ struct fib_table {
        struct hlist_node tb_hlist;
        u32             tb_id;
        int             tb_default;
-       unsigned char   tb_data[0];
+       int             tb_num_default;
+       unsigned long   tb_data[0];
 };
 
 extern int fib_table_lookup(struct fib_table *tb, const struct flowi4 *flp,
index bde80c450b52cc3e000e69468e27ea921488d70c..9ac481a10d37dbff04910af538ab0a624fd2808c 100644 (file)
@@ -1332,6 +1332,9 @@ int fib_table_insert(struct fib_table *tb, struct fib_config *cfg)
                }
        }
 
+       if (!plen)
+               tb->tb_num_default++;
+
        list_add_tail_rcu(&new_fa->fa_list,
                          (fa ? &fa->fa_list : fa_head));
 
@@ -1697,6 +1700,9 @@ int fib_table_delete(struct fib_table *tb, struct fib_config *cfg)
 
        list_del_rcu(&fa->fa_list);
 
+       if (!plen)
+               tb->tb_num_default--;
+
        if (list_empty(fa_head)) {
                hlist_del_rcu(&li->hlist);
                free_leaf_info(li);
@@ -1987,6 +1993,7 @@ struct fib_table *fib_trie_table(u32 id)
 
        tb->tb_id = id;
        tb->tb_default = -1;
+       tb->tb_num_default = 0;
 
        t = (struct trie *) tb->tb_data;
        memset(t, 0, sizeof(*t));
index 0e7430c327a76fc06de65952b5276b3c80876dd5..e9aee81de3e32caf05ae812c856d31aa41745eed 100644 (file)
@@ -2615,7 +2615,9 @@ static struct rtable *ip_route_output_slow(struct net *net,
                fib_select_multipath(&res);
        else
 #endif
-       if (!res.prefixlen && res.type == RTN_UNICAST && !fl4.flowi4_oif)
+       if (!res.prefixlen &&
+           res.table->tb_num_default > 1 &&
+           res.type == RTN_UNICAST && !fl4.flowi4_oif)
                fib_select_default(&res);
 
        if (!fl4.saddr)