diff options
| author |   Eric Dumazet <edumazet@google.com> | 2015-03-19 19:04:19 -0700 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2015-03-20 12:40:25 -0400 |
| commit | 52452c542559ac980b48dbf22a30ee7fa0af507c (patch) | |
| tree | a212dbe95694ea11e86b9d9aca8a2eba0b06a2d1 /net/dccp/minisocks.c | |
| parent | rhashtable: Round up/down min/max_size to ensure we respect limit (diff) | |
| download | linux-dev-52452c542559ac980b48dbf22a30ee7fa0af507c.tar.xz linux-dev-52452c542559ac980b48dbf22a30ee7fa0af507c.zip | |
inet: drop prev pointer handling in request sock
When request sock are put in ehash table, the whole notion
of having a previous request to update dl_next is pointless.
Also, following patch will get rid of big purge timer,
so we want to delete a request sock without holding listener lock.
Signed-off-by: Eric Dumazet <edumazet@google.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to '')
| -rw-r--r-- | net/dccp/minisocks.c | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/net/dccp/minisocks.c b/net/dccp/minisocks.c index b50dc436db1f..332f7d6d9942 100644 --- a/net/dccp/minisocks.c +++ b/net/dccp/minisocks.c @@ -152,8 +152,7 @@ EXPORT_SYMBOL_GPL(dccp_create_openreq_child); * as an request_sock. */ struct sock *dccp_check_req(struct sock *sk, struct sk_buff *skb, - struct request_sock *req, - struct request_sock **prev) + struct request_sock *req) { struct sock *child = NULL; struct dccp_request_sock *dreq = dccp_rsk(req); @@ -200,7 +199,7 @@ struct sock *dccp_check_req(struct sock *sk, struct sk_buff *skb, if (child == NULL) goto listen_overflow; - inet_csk_reqsk_queue_unlink(sk, req, prev); + inet_csk_reqsk_queue_unlink(sk, req); inet_csk_reqsk_queue_removed(sk, req); inet_csk_reqsk_queue_add(sk, req, child); out: @@ -212,7 +211,7 @@ drop: if (dccp_hdr(skb)->dccph_type != DCCP_PKT_RESET) req->rsk_ops->send_reset(sk, skb); - inet_csk_reqsk_queue_drop(sk, req, prev); + inet_csk_reqsk_queue_drop(sk, req); goto out; } |