Commit 575c57e9 authored Aug 23, 2022 by Kuniyuki Iwashima Committed by Greg Kroah-Hartman Sep 05, 2022

net: Fix a data-race around sysctl_net_busy_read.



[ Upstream commit e59ef36f0795696ab229569c153936bfd068d21c ]

While reading sysctl_net_busy_read, it can be changed concurrently.
Thus, we need to add READ_ONCE() to its reader.

Fixes: 2d48d67f ("net: poll/select low latency socket support")
Signed-off-by: Kuniyuki Iwashima <kuniyu@amazon.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Signed-off-by: Sasha Levin <sashal@kernel.org>

parent 6e8f9df6

net/core/sock.c

+1 −1

Original line number	Diff line number	Diff line
		@@ -2946,7 +2946,7 @@ void sock_init_data(struct socket sock, struct sock sk)

		#ifdef CONFIG_NET_RX_BUSY_POLL
		sk->sk_napi_id = 0;
		sk->sk_ll_usec = sysctl_net_busy_read;
		sk->sk_ll_usec = READ_ONCE(sysctl_net_busy_read);
		#endif

		sk->sk_max_pacing_rate = ~0UL;