Re: [PATCH 0/5] Clean up __folio_put()

[Zi Yan <ziy@nvidia.com>]

[-- Attachment #1: Type: text/plain, Size: 1086 bytes --]

On 5 Apr 2024, at 11:32, Matthew Wilcox (Oracle) wrote:

> With all the changes over the last few years, __folio_put_small and
> __folio_put_large have become almost identical to each other ... except
> you can't tell because they're spread over two files.  Rearrange it all
> so that you can tell, and then inline them both into __folio_put().
>
> Matthew Wilcox (Oracle) (5):
>   mm: Free non-hugetlb large folios in a batch
>   mm: Combine free_the_page() and free_unref_page()
>   mm: Inline destroy_large_folio() into __folio_put_large()
>   mm: Combine __folio_put_small, __folio_put_large and __folio_put
>   mm: Convert free_zone_device_page to free_zone_device_folio
>
>  include/linux/mm.h |  2 --
>  mm/internal.h      |  2 +-
>  mm/memremap.c      | 30 ++++++++++++++++--------------
>  mm/page_alloc.c    | 37 ++++++++++---------------------------
>  mm/swap.c          | 44 ++++++++++++++++----------------------------
>  5 files changed, 43 insertions(+), 72 deletions(-)

The whole series looks good to me. Reviewed-by: Zi Yan <ziy@nvidia.com>

--
Best Regards,
Yan, Zi

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 854 bytes --]