diff options
| author |   Eric Dumazet <edumazet@google.com> | 2019-11-05 14:11:52 -0800 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2019-11-06 16:14:48 -0800 |
| commit | 7976a11b30929871a4c84c3c406d7681a3dbcc10 (patch) | |
| tree | bdbef4ca196215ba83037b47bbcc5820e614e3e9 /net/sctp/associola.c | |
| parent | net: avoid potential false sharing in neighbor related code (diff) | |
| download | linux-dev-7976a11b30929871a4c84c3c406d7681a3dbcc10.tar.xz linux-dev-7976a11b30929871a4c84c3c406d7681a3dbcc10.zip | |
net: use helpers to change sk_ack_backlog
Writers are holding a lock, but many readers do not.
Following patch will add appropriate barriers in
sk_acceptq_removed() and sk_acceptq_added().
Signed-off-by: Eric Dumazet <edumazet@google.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to '')
| -rw-r--r-- | net/sctp/associola.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/net/sctp/associola.c b/net/sctp/associola.c index 1ba893b85dad..1b9809ad7725 100644 --- a/net/sctp/associola.c +++ b/net/sctp/associola.c @@ -324,7 +324,7 @@ void sctp_association_free(struct sctp_association *asoc) * socket. */ if (sctp_style(sk, TCP) && sctp_sstate(sk, LISTENING)) - sk->sk_ack_backlog--; + sk_acceptq_removed(sk); } /* Mark as dead, so other users can know this structure is @@ -1073,7 +1073,7 @@ void sctp_assoc_migrate(struct sctp_association *assoc, struct sock *newsk) /* Decrement the backlog value for a TCP-style socket. */ if (sctp_style(oldsk, TCP)) - oldsk->sk_ack_backlog--; + sk_acceptq_removed(oldsk); /* Release references to the old endpoint and the sock. */ sctp_endpoint_put(assoc->ep); |