Re: [PATCH 2/2] cgroup: explain the race between updater and flusher

From: Paul E. McKenney
Date: Fri Jul 04 2025 - 13:58:24 EST


On Fri, Jul 04, 2025 at 10:45:25AM -0700, Shakeel Butt wrote:
> On Thu, Jul 03, 2025 at 09:44:58PM -0700, Paul E. McKenney wrote:
> [...]
> > >
> > > Thanks a lot Paul for the awesome explanation. Do you think keeping
> > > data_race() here would be harmful in a sense that it might cause
> > > confusion in future?
> >
> > Yes, plus it might incorrectly suppress a KCSAN warning for a very
> > real bug. So I strongly recommend removing the data_race() in this case.
>
> I will remove data_race() tags but keep the comments and squash into the
> first one. I will keep your reviewed-by tag unless you disagree.

Works for me!

Thanx, Paul