Re: [PATCH v3] mm: count zeromap read and set for swapout and swapin

[Andrew Morton]

On Thu, 7 Nov 2024 10:53:11 +1300 Barry Song <21cnbao@xxxxxxxxx> wrote:

> 
> Kanchana's commit 'mm: change count_objcg_event() to count_objcg_events()'
> changes count_objcg_event() to count_objcg_events() and supports
> nr_pages more than 1. This is what we need for the minimal patch of
> fixing zeromap
> as zeromap could be nr_pages > 1 for large folios.
> 
> So my question is that, do I combine Kanchana's change into my patch
> and send a single patch, or do I send a patch series with 2 patches:
> 
> 1: Kanchana's mm: change count_objcg_event() to count_objcg_events()
> 2: mm: count zeromap read and set for swapout and swapin

A single self-contained backportable patch is preferable, please.

> If we combine them into a single patch, I'll need to incorporate the changes
> from 1 into 2. I'm also unsure how to acknowledge Kanchana's contribution
> —perhaps mark it as co-developed?

Sure.