On Fri, Jan 26, 2024 at 02:28:59PM +0100, Christoph Hellwig wrote: > + /* > + * Mark the folio dirty so that it won't be reclaimed once we drop the > + * (potentially last) reference in xfile_put_folio. > + */ > + if (flags & XFILE_ALLOC) > + folio_set_dirty(folio); What I can't tell from skimming the code is whether we ever get the folio and don't modify it. If we do, it might make sense to not set dirty here, but instead pass a bool to xfile_put_folio(). Or have the caller dirty the folio if they actually modify it. But perhaps that never happens in practice and this is simple and works every time.
