inet: Don't lookup the socket if there's a socket attached to the skb
authorKOVACS Krisztian <hidden@sch.bme.hu>
Tue, 7 Oct 2008 19:41:01 +0000 (12:41 -0700)
committerDavid S. Miller <davem@davemloft.net>
Tue, 7 Oct 2008 19:41:01 +0000 (12:41 -0700)
Use the socket cached in the skb if it's present.

Signed-off-by: KOVACS Krisztian <hidden@sch.bme.hu>
Acked-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
include/net/inet6_hashtables.h
include/net/inet_hashtables.h
include/net/sock.h
net/ipv4/udp.c
net/ipv6/udp.c

index 995efbb031ab1676ade9f5e560916843737bfc26..f74665d7bea88374170c644d167167cab229b892 100644 (file)
@@ -96,10 +96,14 @@ static inline struct sock *__inet6_lookup_skb(struct inet_hashinfo *hashinfo,
                                              const __be16 sport,
                                              const __be16 dport)
 {
-       return __inet6_lookup(dev_net(skb->dst->dev), hashinfo,
-                             &ipv6_hdr(skb)->saddr, sport,
-                             &ipv6_hdr(skb)->daddr, ntohs(dport),
-                             inet6_iif(skb));
+       struct sock *sk;
+
+       if (unlikely(sk = skb_steal_sock(skb)))
+               return sk;
+       else return __inet6_lookup(dev_net(skb->dst->dev), hashinfo,
+                                  &ipv6_hdr(skb)->saddr, sport,
+                                  &ipv6_hdr(skb)->daddr, ntohs(dport),
+                                  inet6_iif(skb));
 }
 
 extern struct sock *inet6_lookup(struct net *net, struct inet_hashinfo *hashinfo,
index 3522bbcd546d8aae3be44fbba48a5f315750212d..5cc182f9ecae58154b9d0a0d175887822d81dc2b 100644 (file)
@@ -378,11 +378,15 @@ static inline struct sock *__inet_lookup_skb(struct inet_hashinfo *hashinfo,
                                             const __be16 sport,
                                             const __be16 dport)
 {
+       struct sock *sk;
        const struct iphdr *iph = ip_hdr(skb);
 
-       return __inet_lookup(dev_net(skb->dst->dev), hashinfo,
-                            iph->saddr, sport,
-                            iph->daddr, dport, inet_iif(skb));
+       if (unlikely(sk = skb_steal_sock(skb)))
+               return sk;
+       else
+               return __inet_lookup(dev_net(skb->dst->dev), hashinfo,
+                                    iph->saddr, sport,
+                                    iph->daddr, dport, inet_iif(skb));
 }
 
 extern int __inet_hash_connect(struct inet_timewait_death_row *death_row,
index 75a312d3888a7a8de3578fe318615c03a20c6d1b..18f96708f3a6db257e742e6809e38f7c73570a60 100644 (file)
@@ -1324,6 +1324,18 @@ static inline void sk_change_net(struct sock *sk, struct net *net)
        sock_net_set(sk, hold_net(net));
 }
 
+static inline struct sock *skb_steal_sock(struct sk_buff *skb)
+{
+       if (unlikely(skb->sk)) {
+               struct sock *sk = skb->sk;
+
+               skb->destructor = NULL;
+               skb->sk = NULL;
+               return sk;
+       }
+       return NULL;
+}
+
 extern void sock_enable_timestamp(struct sock *sk);
 extern int sock_get_timestamp(struct sock *, struct timeval __user *);
 extern int sock_get_timestampns(struct sock *, struct timespec __user *);
index c7a90b546b21b9277a7fb5ee5fd7c3e5f3348cf2..822c9deac83b2519ce3a7b5d1809c9b0be45baed 100644 (file)
@@ -306,11 +306,15 @@ static inline struct sock *__udp4_lib_lookup_skb(struct sk_buff *skb,
                                                 __be16 sport, __be16 dport,
                                                 struct hlist_head udptable[])
 {
+       struct sock *sk;
        const struct iphdr *iph = ip_hdr(skb);
 
-       return __udp4_lib_lookup(dev_net(skb->dst->dev), iph->saddr, sport,
-                                iph->daddr, dport, inet_iif(skb),
-                                udptable);
+       if (unlikely(sk = skb_steal_sock(skb)))
+               return sk;
+       else
+               return __udp4_lib_lookup(dev_net(skb->dst->dev), iph->saddr, sport,
+                                        iph->daddr, dport, inet_iif(skb),
+                                        udptable);
 }
 
 struct sock *udp4_lib_lookup(struct net *net, __be32 saddr, __be16 sport,
index ce26c41d157d80c21c44807f4aaf13950dfe2d32..e51da8c092faf66aa7bcfa8476bdc2babbe3f96c 100644 (file)
@@ -111,11 +111,15 @@ static struct sock *__udp6_lib_lookup_skb(struct sk_buff *skb,
                                          __be16 sport, __be16 dport,
                                          struct hlist_head udptable[])
 {
+       struct sock *sk;
        struct ipv6hdr *iph = ipv6_hdr(skb);
 
-       return __udp6_lib_lookup(dev_net(skb->dst->dev), &iph->saddr, sport,
-                                &iph->daddr, dport, inet6_iif(skb),
-                                udptable);
+       if (unlikely(sk = skb_steal_sock(skb)))
+               return sk;
+       else
+               return __udp6_lib_lookup(dev_net(skb->dst->dev), &iph->saddr, sport,
+                                        &iph->daddr, dport, inet6_iif(skb),
+                                        udptable);
 }
 
 /*