Re: [PATCH 04/11] mm: add PG_dropbehind folio flag

From: Kirill A. Shutemov
Date: Fri Dec 20 2024 - 10:12:17 EST


On Fri, Dec 20, 2024 at 02:38:09PM +0000, Matthew Wilcox wrote:
> On Fri, Dec 20, 2024 at 01:08:39PM +0200, Kirill A. Shutemov wrote:
> > On Fri, Dec 13, 2024 at 08:55:18AM -0700, Jens Axboe wrote:
> > > Add a folio flag that file IO can use to indicate that the cached IO
> > > being done should be dropped from the page cache upon completion.
> > >
> > > Signed-off-by: Jens Axboe <axboe@xxxxxxxxx>
> >
> > Reviewed-by: Kirill A. Shutemov <kirill.shutemov@xxxxxxxxxxxxxxx>
> >
> > + David, Vlastimil.
> >
> > I think we should consider converting existing folio_set_reclaim() /
> > SetPageReclaim() users to the new flag. From a quick scan, all of them
> > would benefit from dropping the page after writeback is complete instead
> > of leaving the folio on the LRU.
>
> Ooh, that would be nice. Removes the overloading of PG_reclaim with
> PG_readahead, right?

Yep.

--
Kiryl Shutsemau / Kirill A. Shutemov