Re: [PATCH v11 33/33] mm: Add folio_mapped()

From: Christoph Hellwig
Date: Tue Jun 15 2021 - 02:32:05 EST


On Mon, Jun 14, 2021 at 09:14:35PM +0100, Matthew Wilcox (Oracle) wrote:
> This function is the equivalent of page_mapped(). It is slightly
> shorter as we do not need to handle the PageTail() case. Reimplement
> page_mapped() as a wrapper around folio_mapped(). folio_mapped()
> is 13 bytes smaller than page_mapped(), but the page_mapped() wrapper
> is 30 bytes, for a net increase of 17 bytes of text.

I'd still prefer the code flow I suggested last time, but that really
is just style and nothing substantial, so:

Reviewed-by: Christoph Hellwig <hch@xxxxxx>