Re: [External] Re: [PATCH 01/29] mm: shrinker: add shrinker::private_data field

[Qi Zheng]

Hi Vlastimil,

On 2023/6/22 22:47, Vlastimil Babka wrote:
> On 6/22/23 10:53, Qi Zheng wrote:
> > To prepare for the dynamic allocation of shrinker instances
> > embedded in other structures, add a private_data field to
> > struct shrinker, so that we can use shrinker::private_data
> > to record and get the original embedded structure.
> >
> > Signed-off-by: Qi Zheng <zhengqi.arch@xxxxxxxxxxxxx>
>
> I would fold this to 02/29, less churn.

OK, I will fold this to 02/29 in the v2.

Thanks,
Qi

> > ---
> >   include/linux/shrinker.h | 2 ++
> >   1 file changed, 2 insertions(+)
> >
> > diff --git a/include/linux/shrinker.h b/include/linux/shrinker.h
> > index 224293b2dd06..43e6fcabbf51 100644
> > --- a/include/linux/shrinker.h
> > +++ b/include/linux/shrinker.h
> > @@ -70,6 +70,8 @@ struct shrinker {
> >   	int seeks;	/* seeks to recreate an obj */
> >   	unsigned flags;
> >   
> > +	void *private_data;
> > +
> >   	/* These are for internal use */
> >   	struct list_head list;
> >   #ifdef CONFIG_MEMCG