All of lore.kernel.org
 help / color / mirror / Atom feed
From: "David Hildenbrand (Arm)" <david@kernel.org>
To: Nico Pache <npache@redhat.com>, Usama Arif <usama.arif@linux.dev>,
	Zi Yan <ziy@nvidia.com>,
	ljs@kernel.org
Cc: xu.xin16@zte.com.cn, linux-kernel@vger.kernel.org,
	linux-mm@kvack.org, usamaarif642@gmail.com, yuzhao@google.com,
	aarcange@redhat.com, akpm@linux-foundation.org,
	chengming.zhou@linux.dev, baolin.wang@linux.alibaba.com,
	liam@infradead.org, ryan.roberts@arm.com, dev.jain@arm.com,
	baohua@kernel.org, lance.yang@linux.dev, matthew.brost@intel.com,
	joshua.hahnjy@gmail.com, rakie.kim@sk.com, byungchul@sk.com,
	gourry@gourry.net, ying.huang@linux.alibaba.com,
	apopple@nvidia.com
Subject: Re: [PATCH mm-unstable v1 2/3] mm/migrate.c: Prevent folio splitting from interacting with KSM
Date: Wed, 15 Jul 2026 11:30:27 +0200	[thread overview]
Message-ID: <a464c20c-8b41-4264-b560-497ed91cbaaa@kernel.org> (raw)
In-Reply-To: <CAA1CXcA84BtQZxOpE0sWgb2ed6y3m0NLKkWD+5MfWS9eF5dqTQ@mail.gmail.com>

On 7/15/26 06:05, Nico Pache wrote:
> On Tue, Jun 9, 2026 at 11:27 AM Usama Arif <usama.arif@linux.dev> wrote:
>>
>> On Tue, 09 Jun 2026 10:07:20 -0400 Zi Yan <ziy@nvidia.com> wrote:
>>
>>>
> 
> This would be a good intermediate solution while allowing backward
> compatibility.

And an absolute horrible user-visible toggle :)

> 
>>>
>>> Just think out loud. Or just skip huge pages all the time unless memory pressure
>>> is present. Basically treat KSM as a way of reducing memory pressure by merging
>>> pages.
> 
> Hmm interesting. We should decide on future changes before committing
> to one direction; however, i think the immediate solution would be
> what I currently have prepped for v2.
> 
> My v2 follows my RFC alternative approach: skip zeropage remapping if
> the split comes from KSM. It also contains the patch (3) from this
> version, which skips zeropage remapping if the shrinker is disabled.
> 
>>
>> I do agree with this. I questioned about if it even makes sense to split THP
>> with KSM in RFC (https://lore.kernel.org/all/20260510114001.600681-1-usama.arif@linux.dev/)
> 
> This is indeed a good question but I don't know the right solution.
> 
> Perhaps we can discuss it here.
> 
> So far here are the potential solutions:
> a) skip zeropage remapping if the split comes from KSM
> b) skip zeropage remapping completely if the shrinker is disabled
> c) Disable KSM THP splitting completely
> d) add a skip_huge_page sysfs toggle
> e) only scan THPs when memory pressure is present
> 
> (a)+(b) would be my V2 which I can send out whenever.
> 
> (c) is the nuclear option, but perhaps KSM for THP is rather
> inefficient and pointless
> 
> (d) could be used with (a) and (b) and allows more flexibility than (c)
> 
> (e) is a more dynamic approach, and could be combined with (b) and (d)
> 
> Let me know what you think :)

Where does the current patch set fall into and what's the problem with it?

-- 
Cheers,

David

  parent reply	other threads:[~2026-07-15  9:30 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-06-09 11:46 [PATCH mm-unstable v1 0/3] MM: Tighten control over zero-page remapping Nico Pache
2026-06-09 11:46 ` [PATCH mm-unstable v1 1/3] mm/ksm: export ksm_is_running() to check KSM merge state Nico Pache
2026-06-09 14:13   ` Lorenzo Stoakes
2026-06-09 11:46 ` [PATCH mm-unstable v1 2/3] mm/migrate.c: Prevent folio splitting from interacting with KSM Nico Pache
2026-06-09 12:12   ` xu.xin16
2026-06-09 12:57     ` Nico Pache
2026-06-09 12:59       ` David Hildenbrand (Arm)
2026-06-09 13:47       ` xu.xin16
2026-06-09 14:07         ` Zi Yan
2026-06-09 17:27           ` Usama Arif
2026-07-15  4:05             ` Nico Pache
2026-07-15  8:47               ` xu.xin16
2026-07-15  8:56                 ` xu.xin16
2026-07-15  9:34                   ` David Hildenbrand (Arm)
2026-07-15  9:30               ` David Hildenbrand (Arm) [this message]
2026-07-15 10:54               ` Usama Arif
2026-06-09 13:06     ` Lance Yang
2026-06-09 13:42       ` Nico Pache
2026-06-09 13:49         ` xu.xin16
2026-06-09 14:14           ` Lorenzo Stoakes
2026-06-09 14:26   ` Lorenzo Stoakes
2026-06-09 11:46 ` [PATCH mm-unstable v1 3/3] mm/huge_memory.c: Skip zero-page remapping when underused THP shrinker is disabled Nico Pache

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=a464c20c-8b41-4264-b560-497ed91cbaaa@kernel.org \
    --to=david@kernel.org \
    --cc=aarcange@redhat.com \
    --cc=akpm@linux-foundation.org \
    --cc=apopple@nvidia.com \
    --cc=baohua@kernel.org \
    --cc=baolin.wang@linux.alibaba.com \
    --cc=byungchul@sk.com \
    --cc=chengming.zhou@linux.dev \
    --cc=dev.jain@arm.com \
    --cc=gourry@gourry.net \
    --cc=joshua.hahnjy@gmail.com \
    --cc=lance.yang@linux.dev \
    --cc=liam@infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    --cc=ljs@kernel.org \
    --cc=matthew.brost@intel.com \
    --cc=npache@redhat.com \
    --cc=rakie.kim@sk.com \
    --cc=ryan.roberts@arm.com \
    --cc=usama.arif@linux.dev \
    --cc=usamaarif642@gmail.com \
    --cc=xu.xin16@zte.com.cn \
    --cc=ying.huang@linux.alibaba.com \
    --cc=yuzhao@google.com \
    --cc=ziy@nvidia.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.