Re: [PATCH v3] mm: add thp_utilization metrics to /proc/thp_utilization

From: Rik van Riel
Date: Mon Aug 08 2022 - 14:35:17 EST


On Mon, 2022-08-08 at 10:55 -0700, Yang Shi wrote:
>
> On Fri, Aug 5, 2022 at 12:52 PM Alex Zhu (Kernel) <alexlzhu@xxxxxx>
> wrote:
> >
> > Sounds good, I’ll move this to debugfs then. Will follow up with
> > the shrinker code
> > in another patch. The shrinker relies on this scanning thread to
> > figure out which
> > THPs to reclaim.
>
> I'm wondering whether you could reuse the THP deferred split shrinker
> or not. It is already memcg aware.
>
I'm not convinced that will buy much, since there is
very little code duplication between the two.

Merging the two might also bring about another bit of
extra complexity, due to the deferred split shrinker
wanting to shrink every single THP on its "to split"
list, while for Alex's shrinker we probably want to
split just one (or a few) THPs at a time, depending on
memory pressure.