diff options
| author |   Eric Dumazet <edumazet@google.com> | 2021-11-19 07:43:32 -0800 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2021-11-22 12:49:42 +0000 |
| commit | 6d872df3e3b91532b142de9044e5b4984017a55f (patch) | |
| tree | 65f08dd7fd68e0584c00a3fb74077be873070a33 /net/core/sock.c | |
| parent | net: annotate accesses to dev->gso_max_size (diff) | |
| download | linux-dev-6d872df3e3b91532b142de9044e5b4984017a55f.tar.xz linux-dev-6d872df3e3b91532b142de9044e5b4984017a55f.zip | |
net: annotate accesses to dev->gso_max_segs
dev->gso_max_segs is written under RTNL protection, or when the device is
not yet visible, but is read locklessly.
Add netif_set_gso_max_segs() helper.
Add the READ_ONCE()/WRITE_ONCE() pairs, and use netif_set_gso_max_segs()
where we can to better document what is going on.
Signed-off-by: Eric Dumazet <edumazet@google.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/core/sock.c')
| -rw-r--r-- | net/core/sock.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/net/core/sock.c b/net/core/sock.c index 4418e2a07c34..31a2b79c9b38 100644 --- a/net/core/sock.c +++ b/net/core/sock.c @@ -2259,7 +2259,8 @@ void sk_setup_caps(struct sock *sk, struct dst_entry *dst) sk->sk_route_caps |= NETIF_F_SG | NETIF_F_HW_CSUM; /* pairs with the WRITE_ONCE() in netif_set_gso_max_size() */ sk->sk_gso_max_size = READ_ONCE(dst->dev->gso_max_size); - max_segs = max_t(u32, dst->dev->gso_max_segs, 1); + /* pairs with the WRITE_ONCE() in netif_set_gso_max_segs() */ + max_segs = max_t(u32, READ_ONCE(dst->dev->gso_max_segs), 1); } } sk->sk_gso_max_segs = max_segs; |