Re: [PATCH v4 1/7] mm/khugepaged: refactor per-scan state clearing into collapse_control_init_scan()

On 8/12/26 3:23 AM, Pedro Falcato wrote:
> On Tue, Aug 11, 2026 at 06:48:33AM -0600, Nico Pache (Red Hat) wrote:
>> Extract the repeated clearing of node_load, alloc_nmask, and
>> mthp_present_ptes into a helper to reduce duplication in
>> collapse_scan_pmd() and collapse_scan_file(). Althought file scans do not
>> current use the bitmap, they will in the future, and clearing it now is
> 
> Will they? (To be clear, not opposed to clearing this regardless).

That is the hope at least. Baolin has a series for this:
https://lore.kernel.org/lkml/[email protected]/

The plan is to do these cleanups first then Baolin will base on that work.

> 
>> harmless.
>>
>> Reviewed-by: Baolin Wang <[email protected]>
>> Acked-by: Usama Arif <[email protected]>
>> Acked-by: David Hildenbrand (Arm) <[email protected]>
>> Reviewed-by: Lorenzo Stoakes (ARM) <[email protected]>
>> Reviewed-by: Zi Yan <[email protected]>
>> Signed-off-by: Nico Pache (Red Hat) <[email protected]>
> 
> Reviewed-by: Pedro Falcato <[email protected]>

Thank you for reviewing :)

-- Nico

>