diff options
| author | Eric Dumazet <edumazet@google.com> | 2015-10-05 06:08:11 +0200 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2015-10-05 11:45:27 +0200 |
| commit | a1a5344ddbe8fd3e080013b317ac9a664490cfdf (patch) | |
| tree | 75c793a05dd64db3df05f804e429bae45ead20d1 /net/ipv6/syncookies.c | |
| parent | net: use sk_fullsock() in __netdev_pick_tx() (diff) | |
| download | linux-a1a5344ddbe8fd3e080013b317ac9a664490cfdf.tar.xz linux-a1a5344ddbe8fd3e080013b317ac9a664490cfdf.zip | |
tcp: avoid two atomic ops for syncookies
inet_reqsk_alloc() is used to allocate a temporary request
in order to generate a SYNACK with a cookie. Then later,
syncookie validation also uses a temporary request.
These paths already took a reference on listener refcount,
we can avoid a couple of atomic operations.
Signed-off-by: Eric Dumazet <edumazet@google.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to '')
| -rw-r--r-- | net/ipv6/syncookies.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/ipv6/syncookies.c b/net/ipv6/syncookies.c index 7606eba83e7b..f610b5310b17 100644 --- a/net/ipv6/syncookies.c +++ b/net/ipv6/syncookies.c @@ -170,7 +170,7 @@ struct sock *cookie_v6_check(struct sock *sk, struct sk_buff *skb) goto out; ret = NULL; - req = inet_reqsk_alloc(&tcp6_request_sock_ops, sk); + req = inet_reqsk_alloc(&tcp6_request_sock_ops, sk, false); if (!req) goto out; |