diff options
| author | Eric Dumazet <edumazet@google.com> | 2023-03-17 18:55:39 +0300 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2023-03-18 15:23:34 +0300 |
| commit | e9d9da91548b21e189fcd0259a0f2d26d1afc509 (patch) | |
| tree | fd943e26281cfd7a7e0948d1949d6e0009c19510 /net/ipv4/tcp_recovery.c | |
| parent | 403a40f2304d4730a780ab9d6a2b93d1e4ac39d2 (diff) | |
| download | linux-e9d9da91548b21e189fcd0259a0f2d26d1afc509.tar.xz | |
tcp: preserve const qualifier in tcp_sk()
We can change tcp_sk() to propagate its argument const qualifier,
thanks to container_of_const().
We have two places where a const sock pointer has to be upgraded
to a write one. We have been using const qualifier for lockless
listeners to clearly identify points where writes could happen.
Add tcp_sk_rw() helper to better document these.
tcp_inbound_md5_hash(), __tcp_grow_window(), tcp_reset_check()
and tcp_rack_reo_wnd() get an additional const qualififer
for their @tp local variables.
smc_check_reset_syn_req() also needs a similar change.
Signed-off-by: Eric Dumazet <edumazet@google.com>
Reviewed-by: Simon Horman <simon.horman@corigine.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/ipv4/tcp_recovery.c')
| -rw-r--r-- | net/ipv4/tcp_recovery.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/ipv4/tcp_recovery.c b/net/ipv4/tcp_recovery.c index 50abaa941387..acf4869c5d3b 100644 --- a/net/ipv4/tcp_recovery.c +++ b/net/ipv4/tcp_recovery.c @@ -4,7 +4,7 @@ static u32 tcp_rack_reo_wnd(const struct sock *sk) { - struct tcp_sock *tp = tcp_sk(sk); + const struct tcp_sock *tp = tcp_sk(sk); if (!tp->reord_seen) { /* If reordering has not been observed, be aggressive during |