Re: [PATCH 03/13] iomap: move the io_folios field out of struct iomap_ioend

Ritesh Harjani
2023-11-27T05:33:41Z

Christoph Hellwig <hch@lst.de> writes:

> The io_folios member in struct iomap_ioend counts the number of folios
> added to an ioend. It is only used at submission time and can thus be
> moved to iomap_writepage_ctx instead.
>

No objections there. We indeed used io_folios to limit the ioend bio
chain lengths and we do this only at the submission time, which is where
wpc is also used for.

Looks good to me. Please feel free to add -

Reviewed-by: Ritesh Harjani (IBM) <ritesh.list@gmail.com>