linux-mm.kvack.org archive mirror
 help / color / mirror / Atom feed
From: Andrew Morton <akpm@linux-foundation.org>
To: Barry Song <21cnbao@gmail.com>
Cc: Kanchana P Sridhar <kanchana.p.sridhar@intel.com>,
	Johannes Weiner <hannes@cmpxchg.org>,
	linux-mm@kvack.org, linux-kernel@vger.kernel.org,
	Barry Song <v-songbaohua@oppo.com>, Nhat Pham <nphamcs@gmail.com>,
	Usama Arif <usamaarif642@gmail.com>,
	Chengming Zhou <chengming.zhou@linux.dev>,
	Yosry Ahmed <yosryahmed@google.com>,
	Hailong Liu <hailong.liu@oppo.com>,
	David Hildenbrand <david@redhat.com>,
	Hugh Dickins <hughd@google.com>,
	Matthew Wilcox <willy@infradead.org>,
	Shakeel Butt <shakeel.butt@linux.dev>,
	Andi Kleen <ak@linux.intel.com>,
	Baolin Wang <baolin.wang@linux.alibaba.com>,
	Chris Li <chrisl@kernel.org>,
	"Huang, Ying" <ying.huang@intel.com>,
	Kairui Song <kasong@tencent.com>,
	Ryan Roberts <ryan.roberts@arm.com>
Subject: Re: [PATCH v3] mm: count zeromap read and set for swapout and swapin
Date: Wed, 6 Nov 2024 14:02:11 -0800	[thread overview]
Message-ID: <20241106140211.a1272fe0e23d99d7c37b7751@linux-foundation.org> (raw)
In-Reply-To: <CAGsJ_4zLv=HpPL3g085vUaMo8tZZnPZBGT_SfLVCV-10zn+D3Q@mail.gmail.com>

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



      reply	other threads:[~2024-11-06 22:02 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-11-05 21:19 Barry Song
2024-11-06  2:52 ` Chengming Zhou
2024-11-06 15:06 ` Johannes Weiner
2024-11-06 20:01   ` Barry Song
2024-11-06 20:42     ` Andrew Morton
2024-11-06 21:00       ` Barry Song
2024-11-06 21:44         ` Andrew Morton
2024-11-06 21:53           ` Barry Song
2024-11-06 22:02             ` Andrew Morton [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20241106140211.a1272fe0e23d99d7c37b7751@linux-foundation.org \
    --to=akpm@linux-foundation.org \
    --cc=21cnbao@gmail.com \
    --cc=ak@linux.intel.com \
    --cc=baolin.wang@linux.alibaba.com \
    --cc=chengming.zhou@linux.dev \
    --cc=chrisl@kernel.org \
    --cc=david@redhat.com \
    --cc=hailong.liu@oppo.com \
    --cc=hannes@cmpxchg.org \
    --cc=hughd@google.com \
    --cc=kanchana.p.sridhar@intel.com \
    --cc=kasong@tencent.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    --cc=nphamcs@gmail.com \
    --cc=ryan.roberts@arm.com \
    --cc=shakeel.butt@linux.dev \
    --cc=usamaarif642@gmail.com \
    --cc=v-songbaohua@oppo.com \
    --cc=willy@infradead.org \
    --cc=ying.huang@intel.com \
    --cc=yosryahmed@google.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox