linux-mm.kvack.org archive mirror
 help / color / mirror / Atom feed
From: Gregory Price <gourry@gourry.net>
To: Hannes Reinecke <hare@suse.de>
Cc: "David Hildenbrand (Red Hat)" <david@kernel.org>,
	linux-mm@kvack.org, linux-kernel@vger.kernel.org,
	kernel-team@meta.com, osalvador@suse.de,
	gregkh@linuxfoundation.org, rafael@kernel.org, dakr@kernel.org,
	akpm@linux-foundation.org, lorenzo.stoakes@oracle.com,
	Liam.Howlett@oracle.com, vbabka@suse.cz, rppt@kernel.org,
	surenb@google.com, mhocko@suse.com
Subject: Re: [RFC PATCH] memory,memory_hotplug: allow restricting memory blocks to zone movable
Date: Mon, 12 Jan 2026 09:23:35 -0500	[thread overview]
Message-ID: <aWUD597aHXFXbwja@gourry-fedora-PF4VCD3F> (raw)
In-Reply-To: <b1b4ce23-d02c-46ec-96fe-ada6ae0948c6@suse.de>

On Mon, Jan 12, 2026 at 08:28:29AM +0100, Hannes Reinecke wrote:
> On 1/9/26 17:41, Gregory Price wrote:
> > On Thu, Jan 08, 2026 at 03:16:24PM +0100, David Hildenbrand (Red Hat) wrote:
> > > On 1/8/26 08:31, Hannes Reinecke wrote:
> > > > On 1/6/26 21:22, David Hildenbrand (Red Hat) wrote:
> > > > > On 1/6/26 20:59, Gregory Price wrote:
> > > 
> > 
> > I'm starting to think this issue is actually the result of bad patterns
> > in the cxl driver - namely using dax as a path to hotplug sysram.
> > 
> > I suppose either we need a `cxl/dax_region/remove` that handles the
> > whole operation in one go, or
> > 
> > we want `cxl/region/commit` to handle hot(un)plug as a single action.
> > 
> > tl;dr:  Split the dax use case from the sysram use case, and make a
> >          cxl sysram driver directly manage hotplug rather than use dax.
> > 
> 
> Well ... not sure.
> We are doing fine even currently during boot up; we can align policies
> and everything to ensure the system comes up with the 'correct' setting
> Things start to get iffy if one is reconfiguring memory to move from
> daxdev to system ram and vice versa.

Hang tight, i'll post an RFC this morning that shows the idea in code.

region0/memctl/hotplug
  - if not already hotplugged, creates the memory blocks
    unless MHP auto-online or new build config set, do not auto-online

region0/memctl/hotunplug
  - remove memory and hotunplug the blocks
    Done in a single action

region0/memctl/state
  - accepts [online, online_normal, offline]
    does the action on the entire memory region under the hotplug lock
    online - ZONE_MOVABLE
    online_normal - ZONE_NORMAL

I also added the memory event callback to prevent someone from onlining
in zone normal if the last online action was 'online'.  I've tested this
on QEMU and it works fine.

~Gregory


  reply	other threads:[~2026-01-12 14:24 UTC|newest]

Thread overview: 23+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-01-05 20:36 Gregory Price
2026-01-06 15:05 ` Michal Hocko
2026-01-06 16:53   ` Gregory Price
2026-01-06 19:49     ` Michal Hocko
2026-01-07 12:47       ` Hannes Reinecke
2026-01-07 17:17         ` Michal Hocko
2026-01-07 15:09       ` David Hildenbrand (Red Hat)
2026-01-07 16:00         ` Gregory Price
2026-01-07 17:19         ` Michal Hocko
2026-01-06 15:24 ` David Hildenbrand (Red Hat)
2026-01-06 16:58   ` Gregory Price
2026-01-06 17:52     ` David Hildenbrand (Red Hat)
2026-01-06 18:06       ` Gregory Price
2026-01-06 18:38         ` David Hildenbrand (Red Hat)
2026-01-06 19:59           ` Gregory Price
2026-01-06 20:22             ` David Hildenbrand (Red Hat)
2026-01-08  7:31               ` Hannes Reinecke
2026-01-08 14:16                 ` David Hildenbrand (Red Hat)
2026-01-09 16:41                   ` Gregory Price
2026-01-12  7:28                     ` Hannes Reinecke
2026-01-12 14:23                       ` Gregory Price [this message]
2026-01-08  7:21         ` Hannes Reinecke
2026-01-08  7:22         ` Hannes Reinecke

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=aWUD597aHXFXbwja@gourry-fedora-PF4VCD3F \
    --to=gourry@gourry.net \
    --cc=Liam.Howlett@oracle.com \
    --cc=akpm@linux-foundation.org \
    --cc=dakr@kernel.org \
    --cc=david@kernel.org \
    --cc=gregkh@linuxfoundation.org \
    --cc=hare@suse.de \
    --cc=kernel-team@meta.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    --cc=lorenzo.stoakes@oracle.com \
    --cc=mhocko@suse.com \
    --cc=osalvador@suse.de \
    --cc=rafael@kernel.org \
    --cc=rppt@kernel.org \
    --cc=surenb@google.com \
    --cc=vbabka@suse.cz \
    /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