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
next prev parent 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 [RFC PATCH] memory,memory_hotplug: allow restricting memory blocks to zone movable 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 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.