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

net: Fix a data-race around sysctl_net_busy_poll.



[ Upstream commit c42b7cddea47503411bfb5f2f93a4154aaffa2d9 ]

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

Fixes: 06021292 ("net: add low latency socket poll")
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 5da0632c

include/net/busy_poll.h

+1 −1

Original line number	Diff line number	Diff line
		@@ -31,7 +31,7 @@ extern unsigned int sysctl_net_busy_poll __read_mostly;

		static inline bool net_busy_loop_on(void)
		{
		return sysctl_net_busy_poll;
		return READ_ONCE(sysctl_net_busy_poll);
		}

		static inline bool sk_can_busy_loop(const struct sock *sk)