diff options
| author | Eric Dumazet | 2023-07-28 15:03:15 +0000 |
|---|---|---|
| committer | Greg Kroah-Hartman | 2023-08-11 12:08:14 +0200 |
| commit | b53468041d20177c8b8808d1891c0145718ceadf (patch) | |
| tree | d2af7b8b8f9b9271e738df72a7bfabb3f33461b3 /net/xdp/xsk.c | |
| parent | c7bb6860645f83134842e27a2fd1c1449edc5d80 (diff) | |
net: annotate data-races around sk->sk_mark
[ Upstream commit 3c5b4d69c358a9275a8de98f87caf6eda644b086 ]
sk->sk_mark is often read while another thread could change the value.
Fixes: 4a19ec5800fc ("[NET]: Introducing socket mark socket option.")
Signed-off-by: Eric Dumazet <edumazet@google.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Signed-off-by: Sasha Levin <sashal@kernel.org>
Diffstat (limited to 'net/xdp/xsk.c')
| -rw-r--r-- | net/xdp/xsk.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/xdp/xsk.c b/net/xdp/xsk.c index 371d269d22fa..22bf10ffbf2d 100644 --- a/net/xdp/xsk.c +++ b/net/xdp/xsk.c @@ -504,7 +504,7 @@ static struct sk_buff *xsk_build_skb(struct xdp_sock *xs, skb->dev = dev; skb->priority = xs->sk.sk_priority; - skb->mark = xs->sk.sk_mark; + skb->mark = READ_ONCE(xs->sk.sk_mark); skb_shinfo(skb)->destructor_arg = (void *)(long)desc->addr; skb->destructor = xsk_destruct_skb; |