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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-Id: <20241106140211.a1272fe0e23d99d7c37b7751@linux-foundation.org>
Date: Wed, 6 Nov 2024 14:02:11 -0800
From: Andrew Morton <akpm@...ux-foundation.org>
To: Barry Song <21cnbao@...il.com>
Cc: Kanchana P Sridhar <kanchana.p.sridhar@...el.com>, Johannes Weiner
 <hannes@...xchg.org>, linux-mm@...ck.org, linux-kernel@...r.kernel.org,
 Barry Song <v-songbaohua@...o.com>, Nhat Pham <nphamcs@...il.com>, Usama
 Arif <usamaarif642@...il.com>, Chengming Zhou <chengming.zhou@...ux.dev>,
 Yosry Ahmed <yosryahmed@...gle.com>, Hailong Liu <hailong.liu@...o.com>,
 David Hildenbrand <david@...hat.com>, Hugh Dickins <hughd@...gle.com>,
 Matthew Wilcox <willy@...radead.org>, Shakeel Butt
 <shakeel.butt@...ux.dev>, Andi Kleen <ak@...ux.intel.com>, Baolin Wang
 <baolin.wang@...ux.alibaba.com>, Chris Li <chrisl@...nel.org>,
 "Huang, Ying" <ying.huang@...el.com>, Kairui Song <kasong@...cent.com>,
 Ryan Roberts <ryan.roberts@....com>
Subject: Re: [PATCH v3] mm: count zeromap read and set for swapout and
 swapin

On Thu, 7 Nov 2024 10:53:11 +1300 Barry Song <21cnbao@...il.com> 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.