Linux Documentation
 help / color / mirror / Atom feed
From: Tim Chen <tim.c.chen@linux.intel.com>
To: Yafang Shao <laoar.shao@gmail.com>, Chen Yu <yu.c.chen@intel.com>,
	 "David Hildenbrand (Red Hat)" <david@kernel.org>,
	ljs@kernel.org
Cc: Li Zhe <lizhe.67@bytedance.com>,
	mingo@redhat.com, peterz@infradead.org, juri.lelli@redhat.com,
	vincent.guittot@linaro.org, dietmar.eggemann@arm.com,
		rostedt@goodmis.org, bsegall@google.com, mgorman@suse.de,
	vschneid@redhat.com, 	kprateek.nayak@amd.com,
	linux-kernel@vger.kernel.org, 	linux-fsdevel@vger.kernel.org,
	linux-doc@vger.kernel.org, tim.c.chen@intel.com,
		mkoutny@suse.com, tj@kernel.org
Subject: Re: [PATCH 0/4] sched/numa: Add per-process automatic NUMA balancing control
Date: Thu, 15 Oct 2026 12:58:20 -0700	[thread overview]
Message-ID: <807eb0e80c76c3ccb85024b360bd47ace201931b.camel@linux.intel.com> (raw)
In-Reply-To: <CALOAHbBssx9ZHGwebGZCvzT4d_soyjNfZRj4L6gnxHRXtc7eMA@mail.gmail.com>

On Tue, 2026-10-13 at 17:31 +0800, Yafang Shao wrote:
> On Tue, Oct 13, 2026 at 3:52 PM Chen Yu <yu.c.chen@intel.com> wrote:
> > 
> > On Tue, Sep 08, 2026 at 08:24:42PM +0800, Li Zhe wrote:
> > > Automatic NUMA balancing is controlled globally through the
> > > kernel.numa_balancing sysctl and can be influenced indirectly through
> > > memory policy. That works well as a system default, but it is too coarse
> > > for workloads where a launcher wants most processes to use the default
> > > policy while a selected process opts out because it already manages NUMA
> > > placement or cannot afford the sampling overhead.
> > > 
> > > A recent per-cgroup proposal tried to address this by adding a
> > > cgroup-local enable knob plus a NUMA_BALANCING_CGROUP mode where
> > > balancing was disabled by default and then enabled for selected cgroups
> > > [1]. The discussion also noted that automatic NUMA balancing already
> > > operates at task/process granularity, that the knob is not a resource
> > > control attribute, and that task/process controls such as sched_setattr()
> > > or prctl() may be a better fit while cgroups and cpusets continue to
> > > describe workload grouping and placement domains.
> > > 
> > > There is also prior art from 2023 prctl() proposals [2][3]. Those
> > > versions used a per-mm mode with disabled/enabled/default states, added
> > > NumaB_mode to /proc/<pid>/status, and changed the global static key
> > > handling so a per-process enable could override the global sysctl.
> > > 
> > > This series takes a more conservative variant to preserve existing
> > > administrator and workload expectations. Keeping the global sysctl as a
> > > hard off switch means an administrator can still disable all automatic
> > > NUMA balancing activity with one knob. Using a weak two-state
> > > per-process opt-out/allow ABI avoids exposing a "default" state whose
> > > meaning depends on global policy and is therefore easy to misinterpret.
> > > Storing the configured mode in signal_struct gives the control explicit
> > > thread-group semantics and avoids making unrelated non-thread CLONE_VM
> > > users share the policy merely because they share an mm.
> > > 
> > > The series implements a per-process interface: a thread group may use
> > > prctl() to set or get its configured automatic NUMA balancing mode. A
> > > process can opt out of new periodic NUMA scan scheduling and NUMA
> > > scheduler accounting, or opt back in. Existing hinting faults and queued
> > > work are allowed to drain naturally as the disabled state takes effect.
> > > When kernel.numa_balancing is 0, a process-local enable request does not
> > > override it.
> > > 
> > > A separate task_struct scheduler snapshot is used by enqueue/dequeue
> > > accounting so the scheduler can consistently account the state that was
> > > in effect when a task entered the runqueue. This avoids changing
> > > historical task accounting while still allowing a process mode change to
> > > take effect through the normal dequeue/update/enqueue path.
> > > 
> > > The series also exposes the process NUMA balancing mode in
> > > /proc/<pid>/status and documents the new prctl() ABI and its interaction
> > > with the global sysctl.
> > > 
> > 
> > Can the global key be extended to tri-state, like the THP settings?
> 
> IIUC, a BPF-based THP policy is yet to be developed as prctl() doesn't
> fit some use cases, especially mTHP.
> 
> > To keep it backward-compatible, echoing 0 and 1 remains the same effect.
> > The user can also echo always, advise, or never to it.
> > We have a similar proposal here for cache-aware scheduling[1]
> 
> We are also experimenting with the LLC-aware scheduler on production
> servers, and we want to enable it per cgroup rather than globally
> there as well.  Given the lessons we have learned from THP, what
> about implementing it as a BPF-based feature from the beginning, so
> that the semantics can still change when new use cases show up?

Some questions about per-cgroup based cache aware scheduling:

1) What should be done if you want to group specific processes in different cgroups?
It may not be desirable to put them together in the same cgroup for other reasons
like IO or memory resource policies.

2) What is the inheritance rule for cache grouping of tasks in a cgroup
with child cgroups?
There may be too many tasks in parent to try to fit them all in a LLC when
there are only certain subset of tasks sharing data that should be grouped
together cache wise.

With prctl, we can be precise in grouping only tasks that we truly
want to be together.  If we want to bunch tasks in a cgroup cache wise,
it is easy to write a daemon to put tasks in a cgroup into the same
cache group.

Thanks.

Tim


> 
> > In this way, the user does not have to disable NUMA balancing one by one
> > across the whole system.
> > 
> > Yafang Shao has a bpf-cgroup based NUMA balancing proposal[2].
> > Overall I support fine-grained control of NUMA balancing, and I'm curious
> > what the possible setting would be in the production environment, would
> > something like Kubernetes be responsible for setting the NUMA
> > balancing attribute via prctl, or as Yafang proposed, leverage cgroup
> > interface?
> > 
> > [1] https://lore.kernel.org/lkml/fd341106e7e6cbd2f047656650f43ee87c6633c1.1787955777.git.tim.c.chen@linux.intel.com/
> > [2] https://lore.kernel.org/lkml/20261012120438.52849-1-laoar.shao@gmail.com/
> 

  parent reply	other threads:[~2026-10-15 19:58 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-09-08 12:24 [PATCH 0/4] sched/numa: Add per-process automatic NUMA balancing control Li Zhe
2026-09-08 12:24 ` [PATCH 1/4] sched/numa: Track per-process automatic NUMA balancing mode Li Zhe
2026-10-13  9:54   ` Chen Yu
2026-09-08 12:24 ` [PATCH 2/4] sched/numa: Add prctl controls for process mode Li Zhe
2026-09-08 12:24 ` [PATCH 3/4] proc: Report process NUMA balancing mode Li Zhe
2026-09-08 12:24 ` [PATCH 4/4] Documentation: Describe per-process NUMA balancing control Li Zhe
2026-10-13  7:39 ` [PATCH 0/4] sched/numa: Add per-process automatic " Chen Yu
2026-10-13  9:31   ` Yafang Shao
2026-10-14  7:16     ` Li Zhe
2026-10-15 19:58     ` Tim Chen [this message]
2026-10-14  7:05   ` Li Zhe

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=807eb0e80c76c3ccb85024b360bd47ace201931b.camel@linux.intel.com \
    --to=tim.c.chen@linux.intel.com \
    --cc=bsegall@google.com \
    --cc=david@kernel.org \
    --cc=dietmar.eggemann@arm.com \
    --cc=juri.lelli@redhat.com \
    --cc=kprateek.nayak@amd.com \
    --cc=laoar.shao@gmail.com \
    --cc=linux-doc@vger.kernel.org \
    --cc=linux-fsdevel@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=lizhe.67@bytedance.com \
    --cc=ljs@kernel.org \
    --cc=mgorman@suse.de \
    --cc=mingo@redhat.com \
    --cc=mkoutny@suse.com \
    --cc=peterz@infradead.org \
    --cc=rostedt@goodmis.org \
    --cc=tim.c.chen@intel.com \
    --cc=tj@kernel.org \
    --cc=vincent.guittot@linaro.org \
    --cc=vschneid@redhat.com \
    --cc=yu.c.chen@intel.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