diff options
author | David S. Miller <davem@davemloft.net> | 2008-12-18 01:11:38 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2008-12-18 01:11:38 -0500 |
commit | 49ad9599d42da4787d5b3a19263440e0fcd4d1fc (patch) | |
tree | bbeef6eb9316d30524aa73cf12580441f42e420a /net/core | |
parent | 3b5b35d02264580950d1e1565431b99d97c20ded (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/core')
-rw-r--r-- | net/core/sock.c | 6 |
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 |