| Date | Tue, 15 Jun 2021 07:31:38 +0100 | From | Christoph Hellwig <> | Subject | Re: [PATCH v11 33/33] mm: Add folio_mapped() |
| |
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@lst.de>
|