Re: [PATCH v8 6/6] zswap: shrinks zswap pool based on memory pressure

From: Yosry Ahmed
Date: Wed Dec 06 2023 - 16:14:17 EST


On Wed, Dec 6, 2023 at 11:47 AM Nhat Pham <nphamcs@xxxxxxxxx> wrote:
>
> [...]
> >
> > Hmm so how should we proceed from here? How about this:
> >
> > a) I can send a fixlet to move the enablement check above the stats
> > flushing + use mem_cgroup_flush_stats
> > b) Then maybe, you can send a fixlet to update this new callsite?
> >
> > Does that sound reasonable?
>
> I just sent out the fixlet. Yosry and Chengming, let me know if that
> looks good. Thank you both for detecting this issue and proposing the
> fix!

The fixlet looks good, and Andrew already took care of (b) before I
could send a followup fixlet out :)