Re: [PATCH v3 24/30] mm: memcontrol: prepare for reparenting LRU pages for lruvec lock
From: Shakeel Butt
Date: Tue Jan 20 2026 - 10:55:01 EST
On Mon, Jan 19, 2026 at 11:44:09AM +0800, Qi Zheng wrote:
>
>
[...]
> > >
> > > If no one else objects, I will drop __acquires/__releases in the next
> > > version.
> > >
> >
> > If you drop these annotations from header file and keep in the C file,
> > do you still get the compilation error?
>
> I did test it this way, and it does fix the compilation error, but
> Muchun thinks these functions are very simple and there's no need to put
> them in a C file.
>
I think we should prefer correctness at this stage. We can always change
these functions later if we see performance impact of these. So, I would
recommend to keep them in C file along with these annotations.