Re: [PATCH v14 024/138] mm/filemap: Add folio_end_writeback()

[Mike Rapoport]

On Thu, Jul 15, 2021 at 04:35:10AM +0100, Matthew Wilcox (Oracle) wrote:
> Add an end_page_writeback() wrapper function for users that are not yet
> converted to folios.
> 
> folio_end_writeback() is less than half the size of end_page_writeback()
> at just 105 bytes compared to 228 bytes, due to removing all the
> compound_head() calls.  The 30 byte wrapper function makes this a net
> saving of 93 bytes.
> 
> Signed-off-by: Matthew Wilcox (Oracle) <willy@xxxxxxxxxxxxx>
> Reviewed-by: Christoph Hellwig <hch@xxxxxx>
> Acked-by: Jeff Layton <jlayton@xxxxxxxxxx>
> Acked-by: Kirill A. Shutemov <kirill.shutemov@xxxxxxxxxxxxxxx>
> Acked-by: Vlastimil Babka <vbabka@xxxxxxx>
> Reviewed-by: William Kucharski <william.kucharski@xxxxxxxxxx>
> Reviewed-by: David Howells <dhowells@xxxxxxxxxx>
> ---
>  include/linux/pagemap.h |  3 ++-
>  mm/filemap.c            | 43 ++++++++++++++++++++---------------------
>  mm/folio-compat.c       |  6 ++++++
>  3 files changed, 29 insertions(+), 23 deletions(-)

Acked-by: Mike Rapoport <rppt@xxxxxxxxxxxxx>