aboutsummaryrefslogtreecommitdiffstats
path: root/net
diff options
context:
space:
mode:
authorDavid S. Miller <davem@davemloft.net>2008-12-18 01:11:38 -0500
committerDavid S. Miller <davem@davemloft.net>2008-12-18 01:11:38 -0500
commit49ad9599d42da4787d5b3a19263440e0fcd4d1fc (patch)
treebbeef6eb9316d30524aa73cf12580441f42e420a /net
parent3b5b35d02264580950d1e1565431b99d97c20ded (diff)
Revert "net: release skb->dst in sock_queue_rcv_skb()"
This reverts commit 70355602879229c6f8bd694ec9c0814222bc4936. As pointed out by Mark McLoughlin IP_PKTINFO cmsg data is one post-queueing user, so this optimization is not valid right now. Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net')
-rw-r--r--net/core/sock.c6
1 files changed, 1 insertions, 5 deletions
diff --git a/net/core/sock.c b/net/core/sock.c
index ac4f0e79226b..f3a0d08cbb48 100644
--- a/net/core/sock.c
+++ b/net/core/sock.c
@@ -289,11 +289,7 @@ int sock_queue_rcv_skb(struct sock *sk, struct sk_buff *skb)
289 289
290 skb->dev = NULL; 290 skb->dev = NULL;
291 skb_set_owner_r(skb, sk); 291 skb_set_owner_r(skb, sk);
292 /* 292
293 * release dst right now while its hot
294 */
295 dst_release(skb->dst);
296 skb->dst = NULL;
297 /* Cache the SKB length before we tack it onto the receive 293 /* Cache the SKB length before we tack it onto the receive
298 * queue. Once it is added it no longer belongs to us and 294 * queue. Once it is added it no longer belongs to us and
299 * may be freed by other threads of control pulling packets 295 * may be freed by other threads of control pulling packets