[v3 RFC PATCH 3/8] ipv4: Set DST_NOCACHE in rt_dst_alloc().
From: David Miller <davem@davemloft.net>
Date: 2011-03-03 01:01:22
Subsystem:
networking [general], networking [ipv4/ipv6], the rest · Maintainers:
"David S. Miller", Eric Dumazet, Jakub Kicinski, Paolo Abeni, David Ahern, Ido Schimmel, Linus Torvalds
Instead of using a read/modify/write in rt_finalize(). Signed-off-by: David S. Miller <davem@davemloft.net> --- net/ipv4/route.c | 11 +++++------ 1 files changed, 5 insertions(+), 6 deletions(-)
diff --git a/net/ipv4/route.c b/net/ipv4/route.c
index dac8adc..efefefa 100644
--- a/net/ipv4/route.c
+++ b/net/ipv4/route.c@@ -492,11 +492,6 @@ void rt_cache_flush(struct net *net, int delay) static struct rtable *rt_finalize(struct rtable *rt, struct sk_buff *skb) { - /* To avoid expensive rcu stuff for this uncached dst, we set - * DST_NOCACHE so that dst_release() can free dst without - * waiting a grace period. - */ - rt->dst.flags |= DST_NOCACHE; if (rt->rt_type == RTN_UNICAST || rt_is_output_route(rt)) { int err = arp_bind_neighbour(&rt->dst); if (err) {
@@ -1103,7 +1098,11 @@ static struct rtable *rt_dst_alloc(bool nopolicy, bool noxfrm) if (rt) { rt->dst.obsolete = -1; - rt->dst.flags = DST_HOST | + /* To avoid expensive rcu stuff for this uncached dst, we set + * DST_NOCACHE so that dst_release() can free dst without + * waiting a grace period. + */ + rt->dst.flags = DST_NOCACHE | DST_HOST | (nopolicy ? DST_NOPOLICY : 0) | (noxfrm ? DST_NOXFRM : 0); }
--
1.7.4.1