On Mon, Dec 07 2020 at 14:38, Paul E. McKenney wrote: > On Mon, Dec 07, 2020 at 10:46:33PM +0100, Thomas Gleixner wrote: >> On Mon, Dec 07 2020 at 11:44, Paul E. McKenney wrote: >> > On Mon, Dec 07, 2020 at 07:19:51PM +0100, Marco Elver wrote: >> >> On Mon, 7 Dec 2020 at 18:46, Thomas Gleixner <t...@linutronix.de> wrote: >> >> I currently don't know what the rule for Peter's preferred variant >> >> would be, without running the risk of some accidentally data_race()'d >> >> accesses. >> >> >> >> Thoughts? >> > >> > I am also concerned about inadvertently covering code with data_race(). >> > >> > Also, in this particular case, why data_race() rather than READ_ONCE()? >> > Do we really expect the compiler to be able to optimize this case >> > significantly without READ_ONCE()? >> >> That was your suggestion a week or so ago :) > > You expected my suggestion to change? ;-)
Your suggestion was data_race() IIRC but I might have lost track in that conversation.