inet: Kill FLOWI_FLAG_PRECOW_METRICS.
authorDavid S. Miller <davem@davemloft.net>
Tue, 10 Jul 2012 11:01:57 +0000 (04:01 -0700)
committerDavid S. Miller <davem@davemloft.net>
Wed, 11 Jul 2012 05:40:12 +0000 (22:40 -0700)
No longer needed.  TCP writes metrics, but now in it's own special
cache that does not dirty the route metrics.  Therefore there is no
longer any reason to pre-cow metrics in this way.

Signed-off-by: David S. Miller <davem@davemloft.net>
include/net/flow.h
include/net/inet_sock.h
include/net/route.h
net/ipv4/inet_connection_sock.c
net/ipv4/route.c
net/ipv6/route.c

index bd524f59856117b913c3d1b43960de109d20edeb..ce9cb7656b47494fc35e21e08195ba124b2d692a 100644 (file)
@@ -20,9 +20,8 @@ struct flowi_common {
        __u8    flowic_proto;
        __u8    flowic_flags;
 #define FLOWI_FLAG_ANYSRC              0x01
-#define FLOWI_FLAG_PRECOW_METRICS      0x02
-#define FLOWI_FLAG_CAN_SLEEP           0x04
-#define FLOWI_FLAG_RT_NOCACHE          0x08
+#define FLOWI_FLAG_CAN_SLEEP           0x02
+#define FLOWI_FLAG_RT_NOCACHE          0x04
        __u32   flowic_secid;
 };
 
index ae17e1352d7eb1c93f9c1b1ba5a67832d6e4c0c6..924d7b98ab60573690fa60d46c5b251230e32654 100644 (file)
@@ -245,8 +245,6 @@ static inline __u8 inet_sk_flowi_flags(const struct sock *sk)
 
        if (inet_sk(sk)->transparent || inet_sk(sk)->hdrincl)
                flags |= FLOWI_FLAG_ANYSRC;
-       if (sk->sk_protocol == IPPROTO_TCP)
-               flags |= FLOWI_FLAG_PRECOW_METRICS;
        return flags;
 }
 
index 211e2665139b1d684b446b59cb430a2ae709f658..635d7a99d1998d3989a18280cb0ff4256b77c0ef 100644 (file)
@@ -278,8 +278,6 @@ static inline void ip_route_connect_init(struct flowi4 *fl4, __be32 dst, __be32
 
        if (inet_sk(sk)->transparent)
                flow_flags |= FLOWI_FLAG_ANYSRC;
-       if (protocol == IPPROTO_TCP)
-               flow_flags |= FLOWI_FLAG_PRECOW_METRICS;
        if (can_sleep)
                flow_flags |= FLOWI_FLAG_CAN_SLEEP;
 
index 034ddbe42adf9fb72b0db0439fb87e8ff0511970..76825be3b64341635fd57a63efae5f40445fc646 100644 (file)
@@ -375,7 +375,7 @@ struct dst_entry *inet_csk_route_req(struct sock *sk,
        const struct inet_request_sock *ireq = inet_rsk(req);
        struct ip_options_rcu *opt = inet_rsk(req)->opt;
        struct net *net = sock_net(sk);
-       int flags = inet_sk_flowi_flags(sk) & ~FLOWI_FLAG_PRECOW_METRICS;
+       int flags = inet_sk_flowi_flags(sk);
 
        if (nocache)
                flags |= FLOWI_FLAG_RT_NOCACHE;
index e376354dcb65745da578890c6e5090af3620b5b9..d4834e2914a0a746fc54bec6dd611b5c3af2ba68 100644 (file)
@@ -1658,7 +1658,7 @@ void ipv4_update_pmtu(struct sk_buff *skb, struct net *net, u32 mtu,
        struct rtable *rt;
 
        flowi4_init_output(&fl4, oif, mark, RT_TOS(iph->tos), RT_SCOPE_UNIVERSE,
-                          protocol, flow_flags | FLOWI_FLAG_PRECOW_METRICS,
+                          protocol, flow_flags,
                           iph->daddr, iph->saddr, 0, 0);
        rt = __ip_route_output_key(net, &fl4);
        if (!IS_ERR(rt)) {
@@ -1836,18 +1836,11 @@ static void rt_init_metrics(struct rtable *rt, const struct flowi4 *fl4,
 {
        struct inet_peer_base *base;
        struct inet_peer *peer;
-       int create = 0;
-
-       /* If a peer entry exists for this destination, we must hook
-        * it up in order to get at cached metrics.
-        */
-       if (fl4 && (fl4->flowi4_flags & FLOWI_FLAG_PRECOW_METRICS))
-               create = 1;
 
        base = inetpeer_base_ptr(rt->_peer);
        BUG_ON(!base);
 
-       peer = inet_getpeer_v4(base, rt->rt_dst, create);
+       peer = inet_getpeer_v4(base, rt->rt_dst, 0);
        if (peer) {
                __rt_set_peer(rt, peer);
                rt->rt_peer_genid = rt_peer_genid();
index 0c06847537818383b8b6f43e5c3d5b5adca200ee..b7eb51e1a0e1c88ea447354056c1a5de4f3aed47 100644 (file)
@@ -1093,7 +1093,7 @@ void ip6_update_pmtu(struct sk_buff *skb, struct net *net, __be32 mtu,
        memset(&fl6, 0, sizeof(fl6));
        fl6.flowi6_oif = oif;
        fl6.flowi6_mark = mark;
-       fl6.flowi6_flags = FLOWI_FLAG_PRECOW_METRICS;
+       fl6.flowi6_flags = 0;
        fl6.daddr = iph->daddr;
        fl6.saddr = iph->saddr;
        fl6.flowlabel = (*(__be32 *) iph) & IPV6_FLOWINFO_MASK;