On 9 Feb 2024, at 11:37, Vlastimil Babka wrote:
>>
>> extern void free_unref_page(struct page *page, unsigned int order);
>> @@ -473,6 +475,11 @@ int split_free_page(struct page *free_page,
>> /*
>> * in mm/compaction.c
>> */
>> +
>> +struct page_list {
>> + struct list_head pages;
>> + unsigned long nr_pages;
>
> I've checked and even with patch 3/3 I don't think you actually need the
> counter? The only check of the counter I noticed was to check for
> zero/non-zero, and you could use list_empty() instead.
Should I just use struct list_head instead of a new struct page_list?
list_empty(&cc->freepages[order]) vs list_empty(&cc->freepages[order].pages).
--
Best Regards,
Yan, Zi
Attachment:
signature.asc
Description: OpenPGP digital signature
