Re: [PATCH v2 3/7] iomap: Remove unnecessary test from iomap_release_folio()

Christoph Hellwig
2023-06-05T07:13:51Z

> - * dirty folios to ->release_folio() via shrink_active_list();
> - * skip those here.
> + * If the folio is dirty, we refuse to release our metadata because
> + * it may be partially dirty (FIXME, add a test for that).

I'd prefer not to add this FIXME as we're fine without the per-block
dirty tracking.

Otherwise looks good:

Reviewed-by: Christoph Hellwig <hch@lst.de>