Re: [PATCH] mm: rename reclaim_pages() to reclaim_folios()
From: Huang, Ying
Date: Sun Apr 16 2023 - 22:28:42 EST
Matthew Wilcox <willy@xxxxxxxxxxxxx> writes:
> On Sat, Apr 15, 2023 at 05:27:16PM +0800, Kefeng Wang wrote:
>> As commit a83f0551f496 ("mm/vmscan: convert reclaim_pages() to use
>> a folio") changes the arg from page_list to folio_list, but not
>> the defination, let's correct it and rename it to reclaim_folios too.
>
> I didn't bother. It's not inaccurate; we're reclaiming the pages
> in the folios.
What's the general rule about renaming "pages" to "folios" in function
names? I am thinking whether it's necessary to rename migrate_pages()
to migrate_folios(). It's unnecessary?
Best Regards,
Huang, Ying