mbox series

[0/5] Clean up __folio_put()

Message ID 20240405153228.2563754-1-willy@infradead.org (mailing list archive)
Headers show
Series Clean up __folio_put() | expand

Message

Matthew Wilcox April 5, 2024, 3:32 p.m. UTC
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(-)

Comments

Zi Yan April 5, 2024, 4:34 p.m. UTC | #1
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