Re: [PATCH 5/6] mm/page_io: refer to folios in swap_writeout() comments

[Lorenzo Stoakes (ARM)]

On Thu, Aug 06, 2026 at 06:55:37PM -0400, Tal Zussman wrote:
> swap_writeout() operates on folios, not pages. Update its comments
> accordingly.
>
> Signed-off-by: Tal Zussman <tz2294@xxxxxxxxxxxx>

LGTM so:

Reviewed-by: Lorenzo Stoakes (ARM) <ljs@xxxxxxxxxx>

> ---
>  mm/page_io.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/mm/page_io.c b/mm/page_io.c
> index 8573f7b8967a..b18d3b7d853c 100644
> --- a/mm/page_io.c
> +++ b/mm/page_io.c
> @@ -208,7 +208,7 @@ int swap_writeout(struct swap_io_ctx *ctx, struct folio *folio)
>  		goto out_unlock;
>
>  	/*
> -	 * Arch code may have to preserve more data than just the page
> +	 * Arch code may have to preserve more data than just the folio
>  	 * contents, e.g. memory tags.
>  	 */
>  	ret = arch_prepare_to_swap(folio);
> @@ -219,7 +219,7 @@ int swap_writeout(struct swap_io_ctx *ctx, struct folio *folio)
>
>  	/*
>  	 * Use the swap table zero mark to avoid doing IO for zero-filled
> -	 * pages. The zero mark is protected by the cluster lock, which is
> +	 * folios. The zero mark is protected by the cluster lock, which is
>  	 * acquired internally by swap_zeromap_folio_set/clear.
>  	 */
>  	if (is_folio_zero_filled(folio)) {
>
> --
> 2.39.5
>

--
Cheers, Lorenzo