From: Christian Brauner <[email protected]>
Hey everyone,
This adds the new PR_CORE_SCHED prctl() command PR_SCHED_CORE_SHARE to
allow a third process to pull a core scheduling domain from one task and
push it to another task.
The core scheduling uapi is exposed via the PR_SCHED_CORE option of the
prctl() system call. Two commands can be used to alter the core
scheduling domain of a task:
1. PR_SCHED_CORE_SHARE_TO
This command takes the cookie for the caller's core scheduling domain
and applies it to a target task identified by passing a pid.
2. PR_SCHED_CORE_SHARE_FROM
This command takes the cookie for a task's core scheduling domain and
applies it to the calling task.
While these options cover nearly all use-cases they are rather
inconvient for some common use-cases. A vm/container manager often
supervises a large number of vms/containers:
vm/container manager
vm-supervisor-1 container-supervisor-1 vm-supervisor-2 container-supervisor-2
Where none of the vms/container are its immediate children.
For container managers each container often has a separate supervising
process and the workload is the parent of the container. In the example
below the supervising process is "[lxc monitor]" and the workload is
"/sbin/init" and all descendant processes:
├─[lxc monitor] /var/lib/lxd/containers imp1
│ └─systemd
│ ├─agetty -o -p -- \\u --noclear --keep-baud console 115200,38400,9600 linux
│ ├─cron -f -P
│ ├─dbus-daemon --system --address=systemd: --nofork --nopidfile --systemd-activation --syslog-only
│ ├─networkd-dispat /usr/bin/networkd-dispatcher --run-startup-triggers
│ ├─rsyslogd -n -iNONE
│ │ ├─{rsyslogd}
│ │ └─{rsyslogd}
│ ├─systemd-journal
│ ├─systemd-logind
│ ├─systemd-network
│ ├─systemd-resolve
│ └─systemd-udevd
Similiar in spirit but different in layout a vm often has a supervising
process and multiple threads for each vcpu:
├─qemu-system-x86 -S -name f2-vm [...]
│ ├─{qemu-system-x86}
│ ├─{qemu-system-x86}
│ ├─{qemu-system-x86}
│ ├─{qemu-system-x86}
│ ├─{qemu-system-x86}
│ ├─{qemu-system-x86}
│ ├─{qemu-system-x86}
│ ├─{qemu-system-x86}
│ ├─{qemu-system-x86}
│ ├─{qemu-system-x86}
│ └─{qemu-system-x86}
So ultimately an approximation of that layout would be:
vm/container manager
vm-supervisor-1 container-supervisor-1 vm-supervisor-2 container-supervisor-2
| | | |
vcpus workload vcpus workload
(/sbin/init) (/sbin/init)
For containers a new core scheduling domain is allocated for the init
process. Any descendant processes and threads init spawns will
automatically inherit the correct core scheduling domain.
For vms a new core scheduling domain is allocated and each vcpu thread
will be made to join the new core scheduling domain.
Whenever the tool or library that we use to run containers or vms
exposes an option to automatically create a new core scheduling domain
we will make use of it. However that is not always the case. In such
cases the vm/container manager will need to allocate and set the core
scheduling domain for the relevant processes or threads.
Neither the vm/container mananger nor the indivial vm/container
supervisors are supposed to run in any or the same core scheduling
domain as the respective vcpus/workloads.
So in order to create to create a new core scheduling domain we need to
fork() off a new helper process which allocates a core scheduling domain
and then pushes the cookie for the core scheduling domain to the
relevant vcpus/workloads.
This works but things get rather tricky, especially for containers, when
a new process is supposed to be spawned into a running container.
An important step in creating a new process inside a running container
involves:
- getting a handle on the container's init process (pid or nowadays
often a pidfd)
- getting a handle on the container's namespaces (namespace file
descriptors reachable via /proc/<init-pid>/ns/<ns-typ> or nowadays
often a pidfd)
- calling setns() either on each namespace file descriptor individually
or on the pidfd of the init process
An important sub-step here is to attach to the container's pid namespace
via setns(). After attaching to the container's pid namespace any
process created via a fork()-like system calls will be a full member of
the container's pid namespace.
So attaching often involves two child processes. The first child simply
attaches to the namespaces of the container including the container's
pid namespace. The second child fork()s and ultimately exec()s thereby
guaranteeing that the newly created process is a full member of the
container's pid namespace:
first_child = fork();
if (first_child == 0) {
setns(CLONE_NEWPID);
second_child = fork();
if (second_child == 0) {
execlp();
}
}
As part of this we also need to make sure that the second child - the
one ultimately exec()ing the relevant programm in an already running
container - joins the core scheduling domain of the container. When the
container runs in a new pid namespace this can usually be done by
calling:
first_child = fork();
if (first_child == 0) {
setns(CLONE_NEWPID);
second_child = fork();
if (second_child == 0) {
prctl(PR_SCHED_CORE, PR_SCHED_CORE_SHARE_FROM,
1, PR_SCHED_CORE_SCOPE_THREAD, 0);
execlp();
}
}
from the second child since we know that pid 1 in a container running
inside of a separate pid namespace is the correct process to get the
core scheduling domain from.
However, this doesn't work when the container does not run in a separate
pid namespace or when it shares the pid namespace with another
container. In these scenarios we can't simply call
PR_SCHED_CORE_SHARE_FROM from the second child since we don't know the
correct pid number to call it on in the pid namespace.
(Note it is of course possible to learn the pid of the process in the
relevant pid namespace but it is rather complex involving three separate
processes and an AF_UNIX domain socket over which to send a message
including struct ucred from which to learn the relevant pid. But that
doesn't work in all cases since it requires privileges to translate
arbitrary pids. In any case, this is not an option for performance
reasons alone. However, I do also have a separate patchset in [1]
allowing translation of pids between pid namespaces which will help with
that in the future - something which I had discussed with Joel a while
back but haven't pushed for yet since implementing it early 2020. Both
patches are useful independent of one another.)
Additionally, we ideally always want to manage the core scheduling
domain from the first child since the first child knows the pids for the
relevant processes in its current pid namespace. The first child knows
the pid of the init process in the current pid namespace from which to
pull the core scheduling domain and it knows the pid of the second child
it created to which to apply the core scheduling domain.
The core scheduling domain of the first child needs to be unaffected as
it might run sensitive codepaths that should not be exposed in smt attacks.
The new PR_CORE_SCHED_SHARE command for the PR_SCHED_CORE prctl() option
allows to support this and other use-cases by making it possible to pull
the core scheduling domain from a task identified via its pid and push
it to another task identified via its pid from a third managing task:
prctl(PR_SCHED_CORE, PR_SCHED_CORE_SHARE,
<pid-to-which-to-apply-coresched-domain>,
PR_SCHED_CORE_SCOPE_{THREAD,THREAD_GROUP,PROCESS_GROUP},
<pid-from-which-to-take-coresched-domain>)
In order to use PR_SCHED_CORE_SHARE the caller must have
ptrace_may_access() rights to both the task from which to take the core
scheduling domain and to the task to which to apply the core scheduling
domain. If the caller passes zero as the 5th argument then its own core
scheduling domain is applied to the target making the option adhere to
regular prctl() semantics.
Thanks!
Christian
Christian Brauner (3):
pid: introduce task_by_pid()
sched/prctl: add PR_SCHED_CORE_SHARE command
tests: add new PR_SCHED_CORE_SHARE test
arch/mips/kernel/mips-mt-fpaff.c | 14 +-----
arch/x86/kernel/cpu/resctrl/rdtgroup.c | 19 +++-----
block/ioprio.c | 10 +----
include/linux/sched.h | 9 +++-
include/uapi/linux/prctl.h | 3 +-
kernel/cgroup/cgroup.c | 12 ++---
kernel/events/core.c | 5 +--
kernel/futex/syscalls.c | 20 +++------
kernel/pid.c | 5 +++
kernel/sched/core.c | 27 ++++--------
kernel/sched/core_sched.c | 44 ++++++++++++++-----
kernel/sys.c | 12 ++---
mm/mempolicy.c | 2 +-
tools/testing/selftests/sched/cs_prctl_test.c | 23 ++++++++++
14 files changed, 105 insertions(+), 100 deletions(-)
base-commit: 136057256686de39cc3a07c2e39ef6bc43003ff6
--
2.30.2
From: Christian Brauner <[email protected]>
We have a lot of places that open code
if (who)
p = find_task_by_vpid(who);
else
p = current;
Introduce a simpler helper which can be used instead.
Cc: Thomas Gleixner <[email protected]>
Cc: Ingo Molnar <[email protected]>
Cc: Jens Axboe <[email protected]>
Cc: Peter Zijlstra <[email protected]>
Cc: Juri Lelli <[email protected]>
Cc: Vincent Guittot <[email protected]>
Cc: Steven Rostedt <[email protected]>
Cc: Daniel Bristot de Oliveira <[email protected]>
Cc: Tejun Heo <[email protected]>
Cc: Sebastian Andrzej Siewior <[email protected]>
Cc: [email protected]
Signed-off-by: Christian Brauner <[email protected]>
---
arch/mips/kernel/mips-mt-fpaff.c | 14 ++-----------
arch/x86/kernel/cpu/resctrl/rdtgroup.c | 19 +++++++-----------
block/ioprio.c | 10 ++--------
include/linux/sched.h | 7 +++++++
kernel/cgroup/cgroup.c | 12 ++++--------
kernel/events/core.c | 5 +----
kernel/futex/syscalls.c | 20 ++++++-------------
kernel/pid.c | 5 +++++
kernel/sched/core.c | 27 ++++++++------------------
kernel/sched/core_sched.c | 12 ++++--------
kernel/sys.c | 12 +++---------
mm/mempolicy.c | 2 +-
12 files changed, 50 insertions(+), 95 deletions(-)
diff --git a/arch/mips/kernel/mips-mt-fpaff.c b/arch/mips/kernel/mips-mt-fpaff.c
index 67e130d3f038..53c8a56815ea 100644
--- a/arch/mips/kernel/mips-mt-fpaff.c
+++ b/arch/mips/kernel/mips-mt-fpaff.c
@@ -33,16 +33,6 @@ unsigned long mt_fpemul_threshold;
* updated when kernel/sched/core.c changes.
*/
-/*
- * find_process_by_pid - find a process with a matching PID value.
- * used in sys_sched_set/getaffinity() in kernel/sched/core.c, so
- * cloned here.
- */
-static inline struct task_struct *find_process_by_pid(pid_t pid)
-{
- return pid ? find_task_by_vpid(pid) : current;
-}
-
/*
* check the target process has a UID that matches the current process's
*/
@@ -79,7 +69,7 @@ asmlinkage long mipsmt_sys_sched_setaffinity(pid_t pid, unsigned int len,
cpus_read_lock();
rcu_read_lock();
- p = find_process_by_pid(pid);
+ p = task_by_pid(pid);
if (!p) {
rcu_read_unlock();
cpus_read_unlock();
@@ -170,7 +160,7 @@ asmlinkage long mipsmt_sys_sched_getaffinity(pid_t pid, unsigned int len,
rcu_read_lock();
retval = -ESRCH;
- p = find_process_by_pid(pid);
+ p = task_by_pid(pid);
if (!p)
goto out_unlock;
retval = security_task_getscheduler(p);
diff --git a/arch/x86/kernel/cpu/resctrl/rdtgroup.c b/arch/x86/kernel/cpu/resctrl/rdtgroup.c
index b57b3db9a6a7..577d0ffebb9d 100644
--- a/arch/x86/kernel/cpu/resctrl/rdtgroup.c
+++ b/arch/x86/kernel/cpu/resctrl/rdtgroup.c
@@ -660,19 +660,14 @@ static int rdtgroup_move_task(pid_t pid, struct rdtgroup *rdtgrp,
int ret;
rcu_read_lock();
- if (pid) {
- tsk = find_task_by_vpid(pid);
- if (!tsk) {
- rcu_read_unlock();
- rdt_last_cmd_printf("No task %d\n", pid);
- return -ESRCH;
- }
- } else {
- tsk = current;
- }
-
- get_task_struct(tsk);
+ tsk = task_by_pid(pid);
+ if (tsk)
+ get_task_struct(tsk);
rcu_read_unlock();
+ if (!tsk) {
+ rdt_last_cmd_printf("No task %d\n", pid);
+ return -ESRCH;
+ }
ret = rdtgroup_task_write_permission(tsk, of);
if (!ret)
diff --git a/block/ioprio.c b/block/ioprio.c
index 313c14a70bbd..9a9349048d0d 100644
--- a/block/ioprio.c
+++ b/block/ioprio.c
@@ -113,10 +113,7 @@ SYSCALL_DEFINE3(ioprio_set, int, which, int, who, int, ioprio)
rcu_read_lock();
switch (which) {
case IOPRIO_WHO_PROCESS:
- if (!who)
- p = current;
- else
- p = find_task_by_vpid(who);
+ p = task_by_pid(who);
if (p)
ret = set_task_ioprio(p, ioprio);
break;
@@ -208,10 +205,7 @@ SYSCALL_DEFINE2(ioprio_get, int, which, int, who)
rcu_read_lock();
switch (which) {
case IOPRIO_WHO_PROCESS:
- if (!who)
- p = current;
- else
- p = find_task_by_vpid(who);
+ p = task_by_pid(who);
if (p)
ret = get_task_ioprio(p);
break;
diff --git a/include/linux/sched.h b/include/linux/sched.h
index 78c351e35fec..28ce2fb581f7 100644
--- a/include/linux/sched.h
+++ b/include/linux/sched.h
@@ -1900,6 +1900,13 @@ extern unsigned long init_stack[THREAD_SIZE / sizeof(unsigned long)];
*/
extern struct task_struct *find_task_by_vpid(pid_t nr);
+/**
+ * task_by_pid - find a process with a matching PID value.
+ * @pid: the pid in question.
+ *
+ * The task of @pid, if found. %NULL otherwise.
+ */
+extern struct task_struct *task_by_pid(pid_t nr);
extern struct task_struct *find_task_by_pid_ns(pid_t nr, struct pid_namespace *ns);
/*
diff --git a/kernel/cgroup/cgroup.c b/kernel/cgroup/cgroup.c
index 919194de39c8..a674fa07a291 100644
--- a/kernel/cgroup/cgroup.c
+++ b/kernel/cgroup/cgroup.c
@@ -2838,14 +2838,10 @@ struct task_struct *cgroup_procs_write_start(char *buf, bool threadgroup,
}
rcu_read_lock();
- if (pid) {
- tsk = find_task_by_vpid(pid);
- if (!tsk) {
- tsk = ERR_PTR(-ESRCH);
- goto out_unlock_threadgroup;
- }
- } else {
- tsk = current;
+ tsk = task_by_pid(pid);
+ if (!tsk) {
+ tsk = ERR_PTR(-ESRCH);
+ goto out_unlock_threadgroup;
}
if (threadgroup)
diff --git a/kernel/events/core.c b/kernel/events/core.c
index 523106a506ee..ef417285911d 100644
--- a/kernel/events/core.c
+++ b/kernel/events/core.c
@@ -4604,10 +4604,7 @@ find_lively_task_by_vpid(pid_t vpid)
struct task_struct *task;
rcu_read_lock();
- if (!vpid)
- task = current;
- else
- task = find_task_by_vpid(vpid);
+ task = task_by_pid(vpid);
if (task)
get_task_struct(task);
rcu_read_unlock();
diff --git a/kernel/futex/syscalls.c b/kernel/futex/syscalls.c
index 6f91a07a6a83..3e22f6d51815 100644
--- a/kernel/futex/syscalls.c
+++ b/kernel/futex/syscalls.c
@@ -62,13 +62,9 @@ SYSCALL_DEFINE3(get_robust_list, int, pid,
rcu_read_lock();
ret = -ESRCH;
- if (!pid)
- p = current;
- else {
- p = find_task_by_vpid(pid);
- if (!p)
- goto err_unlock;
- }
+ p = task_by_pid(pid);
+ if (!p)
+ goto err_unlock;
ret = -EPERM;
if (!ptrace_may_access(p, PTRACE_MODE_READ_REALCREDS))
@@ -348,13 +344,9 @@ COMPAT_SYSCALL_DEFINE3(get_robust_list, int, pid,
rcu_read_lock();
ret = -ESRCH;
- if (!pid)
- p = current;
- else {
- p = find_task_by_vpid(pid);
- if (!p)
- goto err_unlock;
- }
+ p = task_by_pid(pid);
+ if (!p)
+ goto err_unlock;
ret = -EPERM;
if (!ptrace_may_access(p, PTRACE_MODE_READ_REALCREDS))
diff --git a/kernel/pid.c b/kernel/pid.c
index 2fc0a16ec77b..1cd82fa58273 100644
--- a/kernel/pid.c
+++ b/kernel/pid.c
@@ -422,6 +422,11 @@ struct task_struct *find_task_by_vpid(pid_t vnr)
return find_task_by_pid_ns(vnr, task_active_pid_ns(current));
}
+struct task_struct *task_by_pid(pid_t nr)
+{
+ return nr ? find_task_by_vpid(nr) : current;
+}
+
struct task_struct *find_get_task_by_vpid(pid_t nr)
{
struct task_struct *task;
diff --git a/kernel/sched/core.c b/kernel/sched/core.c
index 3c9b0fda64ac..01a517cba040 100644
--- a/kernel/sched/core.c
+++ b/kernel/sched/core.c
@@ -7131,17 +7131,6 @@ unsigned long sched_cpu_util(int cpu, unsigned long max)
}
#endif /* CONFIG_SMP */
-/**
- * find_process_by_pid - find a process with a matching PID value.
- * @pid: the pid in question.
- *
- * The task of @pid, if found. %NULL otherwise.
- */
-static struct task_struct *find_process_by_pid(pid_t pid)
-{
- return pid ? find_task_by_vpid(pid) : current;
-}
-
/*
* sched_setparam() passes in -1 for its policy, to let the functions
* it calls know not to change it.
@@ -7584,7 +7573,7 @@ do_sched_setscheduler(pid_t pid, int policy, struct sched_param __user *param)
rcu_read_lock();
retval = -ESRCH;
- p = find_process_by_pid(pid);
+ p = task_by_pid(pid);
if (likely(p))
get_task_struct(p);
rcu_read_unlock();
@@ -7707,7 +7696,7 @@ SYSCALL_DEFINE3(sched_setattr, pid_t, pid, struct sched_attr __user *, uattr,
rcu_read_lock();
retval = -ESRCH;
- p = find_process_by_pid(pid);
+ p = task_by_pid(pid);
if (likely(p))
get_task_struct(p);
rcu_read_unlock();
@@ -7739,7 +7728,7 @@ SYSCALL_DEFINE1(sched_getscheduler, pid_t, pid)
retval = -ESRCH;
rcu_read_lock();
- p = find_process_by_pid(pid);
+ p = task_by_pid(pid);
if (p) {
retval = security_task_getscheduler(p);
if (!retval)
@@ -7768,7 +7757,7 @@ SYSCALL_DEFINE2(sched_getparam, pid_t, pid, struct sched_param __user *, param)
return -EINVAL;
rcu_read_lock();
- p = find_process_by_pid(pid);
+ p = task_by_pid(pid);
retval = -ESRCH;
if (!p)
goto out_unlock;
@@ -7851,7 +7840,7 @@ SYSCALL_DEFINE4(sched_getattr, pid_t, pid, struct sched_attr __user *, uattr,
return -EINVAL;
rcu_read_lock();
- p = find_process_by_pid(pid);
+ p = task_by_pid(pid);
retval = -ESRCH;
if (!p)
goto out_unlock;
@@ -7960,7 +7949,7 @@ long sched_setaffinity(pid_t pid, const struct cpumask *in_mask)
rcu_read_lock();
- p = find_process_by_pid(pid);
+ p = task_by_pid(pid);
if (!p) {
rcu_read_unlock();
return -ESRCH;
@@ -8039,7 +8028,7 @@ long sched_getaffinity(pid_t pid, struct cpumask *mask)
rcu_read_lock();
retval = -ESRCH;
- p = find_process_by_pid(pid);
+ p = task_by_pid(pid);
if (!p)
goto out_unlock;
@@ -8439,7 +8428,7 @@ static int sched_rr_get_interval(pid_t pid, struct timespec64 *t)
retval = -ESRCH;
rcu_read_lock();
- p = find_process_by_pid(pid);
+ p = task_by_pid(pid);
if (!p)
goto out_unlock;
diff --git a/kernel/sched/core_sched.c b/kernel/sched/core_sched.c
index 517f72b008f5..9a1ef7fffc94 100644
--- a/kernel/sched/core_sched.c
+++ b/kernel/sched/core_sched.c
@@ -144,14 +144,10 @@ int sched_core_share_pid(unsigned int cmd, pid_t pid, enum pid_type type,
return -EINVAL;
rcu_read_lock();
- if (pid == 0) {
- task = current;
- } else {
- task = find_task_by_vpid(pid);
- if (!task) {
- rcu_read_unlock();
- return -ESRCH;
- }
+ task = task_by_pid(pid);
+ if (!task) {
+ rcu_read_unlock();
+ return -ESRCH;
}
get_task_struct(task);
rcu_read_unlock();
diff --git a/kernel/sys.c b/kernel/sys.c
index 8fdac0d90504..1e75b0088214 100644
--- a/kernel/sys.c
+++ b/kernel/sys.c
@@ -223,10 +223,7 @@ SYSCALL_DEFINE3(setpriority, int, which, int, who, int, niceval)
read_lock(&tasklist_lock);
switch (which) {
case PRIO_PROCESS:
- if (who)
- p = find_task_by_vpid(who);
- else
- p = current;
+ p = task_by_pid(who);
if (p)
error = set_one_prio(p, niceval, error);
break;
@@ -286,10 +283,7 @@ SYSCALL_DEFINE2(getpriority, int, which, int, who)
read_lock(&tasklist_lock);
switch (which) {
case PRIO_PROCESS:
- if (who)
- p = find_task_by_vpid(who);
- else
- p = current;
+ p = task_by_pid(who);
if (p) {
niceval = nice_to_rlimit(task_nice(p));
if (niceval > retval)
@@ -1659,7 +1653,7 @@ SYSCALL_DEFINE4(prlimit64, pid_t, pid, unsigned int, resource,
}
rcu_read_lock();
- tsk = pid ? find_task_by_vpid(pid) : current;
+ tsk = task_by_pid(pid);
if (!tsk) {
rcu_read_unlock();
return -ESRCH;
diff --git a/mm/mempolicy.c b/mm/mempolicy.c
index 10e9c87260ed..1199b9fc2250 100644
--- a/mm/mempolicy.c
+++ b/mm/mempolicy.c
@@ -1538,7 +1538,7 @@ static int kernel_migrate_pages(pid_t pid, unsigned long maxnode,
/* Find the mm_struct */
rcu_read_lock();
- task = pid ? find_task_by_vpid(pid) : current;
+ task = task_by_pid(pid);
if (!task) {
rcu_read_unlock();
err = -ESRCH;
--
2.30.2
From: Christian Brauner <[email protected]>
This adds the new PR_CORE_SCHED prctl() command PR_SCHED_CORE_SHARE to
allow a third process to pull a core scheduling domain from one task and
push it to another task.
The core scheduling uapi is exposed via the PR_SCHED_CORE option of the
prctl() system call. Two commands can be used to alter the core
scheduling domain of a task:
1. PR_SCHED_CORE_SHARE_TO
This command takes the cookie for the caller's core scheduling domain
and applies it to a target task identified by passing a pid.
2. PR_SCHED_CORE_SHARE_FROM
This command takes the cookie for a task's core scheduling domain and
applies it to the calling task.
While these options cover nearly all use-cases they are rather
inconvient for some common use-cases. A vm/container manager often
supervises a large number of vms/containers:
vm/container manager
vm-supervisor-1 container-supervisor-1 vm-supervisor-2 container-supervisor-2
Where none of the vms/container are its immediate children.
For container managers each container often has a separate supervising
process and the workload is the parent of the container. In the example
below the supervising process is "[lxc monitor]" and the workload is
"/sbin/init" and all descendant processes:
├─[lxc monitor] /var/lib/lxd/containers imp1
│ └─systemd
│ ├─agetty -o -p -- \\u --noclear --keep-baud console 115200,38400,9600 linux
│ ├─cron -f -P
│ ├─dbus-daemon --system --address=systemd: --nofork --nopidfile --systemd-activation --syslog-only
│ ├─networkd-dispat /usr/bin/networkd-dispatcher --run-startup-triggers
│ ├─rsyslogd -n -iNONE
│ │ ├─{rsyslogd}
│ │ └─{rsyslogd}
│ ├─systemd-journal
│ ├─systemd-logind
│ ├─systemd-network
│ ├─systemd-resolve
│ └─systemd-udevd
Similiar in spirit but different in layout a vm often has a supervising
process and multiple threads for each vcpu:
├─qemu-system-x86 -S -name f2-vm [...]
│ ├─{qemu-system-x86}
│ ├─{qemu-system-x86}
│ ├─{qemu-system-x86}
│ ├─{qemu-system-x86}
│ ├─{qemu-system-x86}
│ ├─{qemu-system-x86}
│ ├─{qemu-system-x86}
│ ├─{qemu-system-x86}
│ ├─{qemu-system-x86}
│ ├─{qemu-system-x86}
│ └─{qemu-system-x86}
So ultimately an approximation of that layout would be:
vm/container manager
vm-supervisor-1 container-supervisor-1 vm-supervisor-2 container-supervisor-2
| | | |
vcpus workload vcpus workload
(/sbin/init) (/sbin/init)
For containers a new core scheduling domain is allocated for the init
process. Any descendant processes and threads init spawns will
automatically inherit the correct core scheduling domain.
For vms a new core scheduling domain is allocated and each vcpu thread
will be made to join the new core scheduling domain.
Whenever the tool or library that we use to run containers or vms
exposes an option to automatically create a new core scheduling domain
we will make use of it. However that is not always the case. In such
cases the vm/container manager will need to allocate and set the core
scheduling domain for the relevant processes or threads.
Neither the vm/container mananger nor the indivial vm/container
supervisors are supposed to run in any or the same core scheduling
domain as the respective vcpus/workloads.
So in order to create to create a new core scheduling domain we need to
fork() off a new helper process which allocates a core scheduling domain
and then pushes the cookie for the core scheduling domain to the
relevant vcpus/workloads.
This works but things get rather tricky, especially for containers, when
a new process is supposed to be spawned into a running container.
An important step in creating a new process inside a running container
involves:
- getting a handle on the container's init process (pid or nowadays
often a pidfd)
- getting a handle on the container's namespaces (namespace file
descriptors reachable via /proc/<init-pid>/ns/<ns-typ> or nowadays
often a pidfd)
- calling setns() either on each namespace file descriptor individually
or on the pidfd of the init process
An important sub-step here is to attach to the container's pid namespace
via setns(). After attaching to the container's pid namespace any
process created via a fork()-like system calls will be a full member of
the container's pid namespace.
So attaching often involves two child processes. The first child simply
attaches to the namespaces of the container including the container's
pid namespace. The second child fork()s and ultimately exec()s thereby
guaranteeing that the newly created process is a full member of the
container's pid namespace:
first_child = fork();
if (first_child == 0) {
setns(CLONE_NEWPID);
second_child = fork();
if (second_child == 0) {
execlp();
}
}
As part of this we also need to make sure that the second child - the
one ultimately exec()ing the relevant programm in an already running
container - joins the core scheduling domain of the container. When the
container runs in a new pid namespace this can usually be done by
calling:
first_child = fork();
if (first_child == 0) {
setns(CLONE_NEWPID);
second_child = fork();
if (second_child == 0) {
prctl(PR_SCHED_CORE, PR_SCHED_CORE_SHARE_FROM,
1, PR_SCHED_CORE_SCOPE_THREAD, 0);
execlp();
}
}
from the second child since we know that pid 1 in a container running
inside of a separate pid namespace is the correct process to get the
core scheduling domain from.
However, this doesn't work when the container does not run in a separate
pid namespace or when it shares the pid namespace with another
container. In these scenarios we can't simply call
PR_SCHED_CORE_SHARE_FROM from the second child since we don't know the
correct pid number to call it on in the pid namespace.
(Note it is of course possible to learn the pid of the process in the
relevant pid namespace but it is rather complex involving three separate
processes and an AF_UNIX domain socket over which to send a message
including struct ucred from which to learn the relevant pid. But that
doesn't work in all cases since it requires privileges to translate
arbitrary pids. In any case, this is not an option for performance
reasons alone. However, I do also have a separate patchset in [1]
allowing translation of pids between pid namespaces which will help with
that in the future - something which I had discussed with Joel a while
back but haven't pushed for yet since implementing it early 2020. Both
patches are useful independent of one another.)
Additionally, we ideally always want to manage the core scheduling
domain from the first child since the first child knows the pids for the
relevant processes in its current pid namespace. The first child knows
the pid of the init process in the current pid namespace from which to
pull the core scheduling domain and it knows the pid of the second child
it created to which to apply the core scheduling domain.
The core scheduling domain of the first child needs to be unaffected as
it might run sensitive codepaths that should not be exposed in smt attacks.
The new PR_CORE_SCHED_SHARE command for the PR_SCHED_CORE prctl() option
allows to support this and other use-cases by making it possible to pull
the core scheduling domain from a task identified via its pid and push
it to another task identified via its pid from a third managing task:
prctl(PR_SCHED_CORE, PR_SCHED_CORE_SHARE,
<pid-to-which-to-apply-coresched-domain>,
PR_SCHED_CORE_SCOPE_{THREAD,THREAD_GROUP,PROCESS_GROUP},
<pid-from-which-to-take-coresched-domain>)
In order to use PR_SCHED_CORE_SHARE the caller must have
ptrace_may_access() rights to both the task from which to take the core
scheduling domain and to the task to which to apply the core scheduling
domain. If the caller passes zero as the 5th argument then its own core
scheduling domain is applied to the target making the option adhere to
regular prctl() semantics.
[1]: https://git.kernel.org/brauner/h/ioctl_ns_get_init_pid
https://git.kernel.org/brauner/c/1ad81fd698dd7e6511c3db422eba42dec3ce1b08
Cc: Peter Collingbourne <[email protected]>
Cc: Dietmar Eggemann <[email protected]>
Cc: Joel Fernandes <[email protected]>
Cc: Thomas Gleixner <[email protected]>
Cc: Mel Gorman <[email protected]>
Cc: Vincent Guittot <[email protected]>
Cc: Chris Hyser <[email protected]>
Cc: Juri Lelli <[email protected]>
Cc: Catalin Marinas <[email protected]>
Cc: Ingo Molnar <[email protected]>
Cc: Daniel Bristot de Oliveira <[email protected]>
Cc: Steven Rostedt <[email protected]>
Cc: Ben Segall <[email protected]>
Cc: Balbir Singh <[email protected]>
Cc: Peter Zijlstra <[email protected]>
Cc: [email protected]
Signed-off-by: Christian Brauner <[email protected]>
---
include/linux/sched.h | 2 +-
include/uapi/linux/prctl.h | 3 ++-
kernel/sched/core_sched.c | 32 +++++++++++++++++++++++++++++---
3 files changed, 32 insertions(+), 5 deletions(-)
diff --git a/include/linux/sched.h b/include/linux/sched.h
index 28ce2fb581f7..a139f1db98d4 100644
--- a/include/linux/sched.h
+++ b/include/linux/sched.h
@@ -2320,7 +2320,7 @@ const struct cpumask *sched_trace_rd_span(struct root_domain *rd);
extern void sched_core_free(struct task_struct *tsk);
extern void sched_core_fork(struct task_struct *p);
extern int sched_core_share_pid(unsigned int cmd, pid_t pid, enum pid_type type,
- unsigned long uaddr);
+ unsigned long arg);
#else
static inline void sched_core_free(struct task_struct *tsk) { }
static inline void sched_core_fork(struct task_struct *p) { }
diff --git a/include/uapi/linux/prctl.h b/include/uapi/linux/prctl.h
index bb73e9a0b24f..189041747fe6 100644
--- a/include/uapi/linux/prctl.h
+++ b/include/uapi/linux/prctl.h
@@ -267,7 +267,8 @@ struct prctl_mm_map {
# define PR_SCHED_CORE_CREATE 1 /* create unique core_sched cookie */
# define PR_SCHED_CORE_SHARE_TO 2 /* push core_sched cookie to pid */
# define PR_SCHED_CORE_SHARE_FROM 3 /* pull core_sched cookie to pid */
-# define PR_SCHED_CORE_MAX 4
+# define PR_SCHED_CORE_SHARE 4
+# define PR_SCHED_CORE_MAX 5
# define PR_SCHED_CORE_SCOPE_THREAD 0
# define PR_SCHED_CORE_SCOPE_THREAD_GROUP 1
# define PR_SCHED_CORE_SCOPE_PROCESS_GROUP 2
diff --git a/kernel/sched/core_sched.c b/kernel/sched/core_sched.c
index 9a1ef7fffc94..6faedc979f25 100644
--- a/kernel/sched/core_sched.c
+++ b/kernel/sched/core_sched.c
@@ -125,9 +125,10 @@ static void __sched_core_set(struct task_struct *p, unsigned long cookie)
/* Called from prctl interface: PR_SCHED_CORE */
int sched_core_share_pid(unsigned int cmd, pid_t pid, enum pid_type type,
- unsigned long uaddr)
+ unsigned long arg)
{
- unsigned long cookie = 0, id = 0;
+ unsigned long cookie = 0, id = 0, uaddr = 0;
+ pid_t pid_share = -1;
struct task_struct *task, *p;
struct pid *grp;
int err = 0;
@@ -140,9 +141,20 @@ int sched_core_share_pid(unsigned int cmd, pid_t pid, enum pid_type type,
BUILD_BUG_ON(PR_SCHED_CORE_SCOPE_PROCESS_GROUP != PIDTYPE_PGID);
if (type > PIDTYPE_PGID || cmd >= PR_SCHED_CORE_MAX || pid < 0 ||
- (cmd != PR_SCHED_CORE_GET && uaddr))
+ (cmd != PR_SCHED_CORE_GET && cmd != PR_SCHED_CORE_SHARE && arg))
return -EINVAL;
+ switch (cmd) {
+ case PR_SCHED_CORE_GET:
+ uaddr = arg;
+ break;
+ case PR_SCHED_CORE_SHARE:
+ pid_share = arg;
+ if (pid_share < 0)
+ return -EINVAL;
+ break;
+ }
+
rcu_read_lock();
task = task_by_pid(pid);
if (!task) {
@@ -196,6 +208,20 @@ int sched_core_share_pid(unsigned int cmd, pid_t pid, enum pid_type type,
__sched_core_set(current, cookie);
goto out;
+ case PR_SCHED_CORE_SHARE:
+ rcu_read_lock();
+ p = task_by_pid(pid_share);
+ if (!p)
+ err = -ESRCH;
+ else if (!ptrace_may_access(p, PTRACE_MODE_READ_REALCREDS))
+ err = -EPERM;
+ if (!err)
+ cookie = sched_core_clone_cookie(p);
+ rcu_read_unlock();
+ if (err)
+ goto out;
+ break;
+
default:
err = -EINVAL;
goto out;
--
2.30.2
From: Christian Brauner <[email protected]>
Add tests for the new PR_SCHED_CORE_SHARE command.
Cc: Peter Collingbourne <[email protected]>
Cc: Dietmar Eggemann <[email protected]>
Cc: Joel Fernandes <[email protected]>
Cc: Thomas Gleixner <[email protected]>
Cc: Mel Gorman <[email protected]>
Cc: Vincent Guittot <[email protected]>
Cc: Chris Hyser <[email protected]>
Cc: Shuah Khan <[email protected]>
Cc: Juri Lelli <[email protected]>
Cc: Catalin Marinas <[email protected]>
Cc: Ingo Molnar <[email protected]>
Cc: Daniel Bristot de Oliveira <[email protected]>
Cc: Steven Rostedt <[email protected]>
Cc: Ben Segall <[email protected]>
Cc: Balbir Singh <[email protected]>
Cc: Peter Zijlstra <[email protected]>
Cc: [email protected]
Cc: [email protected]
Cc: [email protected]
Signed-off-by: Christian Brauner <[email protected]>
---
tools/testing/selftests/sched/cs_prctl_test.c | 23 +++++++++++++++++++
1 file changed, 23 insertions(+)
diff --git a/tools/testing/selftests/sched/cs_prctl_test.c b/tools/testing/selftests/sched/cs_prctl_test.c
index 8109b17dc764..985b83fe7221 100644
--- a/tools/testing/selftests/sched/cs_prctl_test.c
+++ b/tools/testing/selftests/sched/cs_prctl_test.c
@@ -229,6 +229,7 @@ int main(int argc, char *argv[])
int pidx;
int pid;
int opt;
+ int i;
while ((opt = getopt(argc, argv, ":hkT:P:d:")) != -1) {
switch (opt) {
@@ -325,6 +326,28 @@ int main(int argc, char *argv[])
validate(get_cs_cookie(pid) != 0);
validate(get_cs_cookie(pid) == get_cs_cookie(procs[pidx].thr_tids[0]));
+ printf("\n## Set a new cookie on a single thread/PR_SCHED_CORE_SCOPE_THREAD [%d]\n", pid);
+ if (_prctl(PR_SCHED_CORE, PR_SCHED_CORE_CREATE, pid, PR_SCHED_CORE_SCOPE_THREAD, 0) < 0)
+ handle_error("core_sched create failed -- PR_SCHED_CORE_SCOPE_THREAD");
+ disp_processes(num_processes, procs);
+
+ validate(get_cs_cookie(pid) != get_cs_cookie(procs[pidx].thr_tids[0]));
+
+ printf("\n## Copy cookie from a thread [%d] to [%d] as PR_SCHED_CORE_SCOPE_THREAD\n", pid, procs[pidx].thr_tids[0]);
+ if (_prctl(PR_SCHED_CORE, PR_SCHED_CORE_SHARE, procs[pidx].thr_tids[0], PR_SCHED_CORE_SCOPE_THREAD, pid) < 0)
+ handle_error("core_sched share cookie from and to thread failed -- PR_SCHED_CORE_SCOPE_THREAD");
+ disp_processes(num_processes, procs);
+
+ validate(get_cs_cookie(pid) == get_cs_cookie(procs[pidx].thr_tids[0]));
+
+ printf("\n## Copy cookie from a thread [%d] to [%d] as PR_SCHED_CORE_SCOPE_THREAD_GROUP\n", pid, pid);
+ if (_prctl(PR_SCHED_CORE, PR_SCHED_CORE_SHARE, pid, PR_SCHED_CORE_SCOPE_THREAD_GROUP, pid) < 0)
+ handle_error("core_sched share cookie from and to thread-group failed -- PR_SCHED_CORE_SCOPE_THREAD_GROUP");
+ disp_processes(num_processes, procs);
+
+ for (i = 0; i < procs[pidx].num_threads; ++i)
+ validate(get_cs_cookie(pid) == get_cs_cookie(procs[pidx].thr_tids[i]));
+
if (errors) {
printf("TESTS FAILED. errors: %d\n", errors);
res = 10;
--
2.30.2
On Tue, Nov 23, 2021 at 02:29:04PM +0100, Christian Brauner wrote:
> From: Christian Brauner <[email protected]>
>
> Hey everyone,
>
> This adds the new PR_CORE_SCHED prctl() command PR_SCHED_CORE_SHARE to
> allow a third process to pull a core scheduling domain from one task and
> push it to another task.
Hey Peter,
Did you get anywhere in finding out whether we can support something
like this here or whether there was a reason we can't?
Thanks!
Christian
>
> The core scheduling uapi is exposed via the PR_SCHED_CORE option of the
> prctl() system call. Two commands can be used to alter the core
> scheduling domain of a task:
>
> 1. PR_SCHED_CORE_SHARE_TO
> This command takes the cookie for the caller's core scheduling domain
> and applies it to a target task identified by passing a pid.
>
> 2. PR_SCHED_CORE_SHARE_FROM
> This command takes the cookie for a task's core scheduling domain and
> applies it to the calling task.
>
> While these options cover nearly all use-cases they are rather
> inconvient for some common use-cases. A vm/container manager often
> supervises a large number of vms/containers:
>
> vm/container manager
>
> vm-supervisor-1 container-supervisor-1 vm-supervisor-2 container-supervisor-2
>
> Where none of the vms/container are its immediate children.
>
> For container managers each container often has a separate supervising
> process and the workload is the parent of the container. In the example
> below the supervising process is "[lxc monitor]" and the workload is
> "/sbin/init" and all descendant processes:
>
> ├─[lxc monitor] /var/lib/lxd/containers imp1
> │ └─systemd
> │ ├─agetty -o -p -- \\u --noclear --keep-baud console 115200,38400,9600 linux
> │ ├─cron -f -P
> │ ├─dbus-daemon --system --address=systemd: --nofork --nopidfile --systemd-activation --syslog-only
> │ ├─networkd-dispat /usr/bin/networkd-dispatcher --run-startup-triggers
> │ ├─rsyslogd -n -iNONE
> │ │ ├─{rsyslogd}
> │ │ └─{rsyslogd}
> │ ├─systemd-journal
> │ ├─systemd-logind
> │ ├─systemd-network
> │ ├─systemd-resolve
> │ └─systemd-udevd
>
> Similiar in spirit but different in layout a vm often has a supervising
> process and multiple threads for each vcpu:
>
> ├─qemu-system-x86 -S -name f2-vm [...]
> │ ├─{qemu-system-x86}
> │ ├─{qemu-system-x86}
> │ ├─{qemu-system-x86}
> │ ├─{qemu-system-x86}
> │ ├─{qemu-system-x86}
> │ ├─{qemu-system-x86}
> │ ├─{qemu-system-x86}
> │ ├─{qemu-system-x86}
> │ ├─{qemu-system-x86}
> │ ├─{qemu-system-x86}
> │ └─{qemu-system-x86}
>
> So ultimately an approximation of that layout would be:
>
> vm/container manager
>
> vm-supervisor-1 container-supervisor-1 vm-supervisor-2 container-supervisor-2
> | | | |
> vcpus workload vcpus workload
> (/sbin/init) (/sbin/init)
>
> For containers a new core scheduling domain is allocated for the init
> process. Any descendant processes and threads init spawns will
> automatically inherit the correct core scheduling domain.
>
> For vms a new core scheduling domain is allocated and each vcpu thread
> will be made to join the new core scheduling domain.
>
> Whenever the tool or library that we use to run containers or vms
> exposes an option to automatically create a new core scheduling domain
> we will make use of it. However that is not always the case. In such
> cases the vm/container manager will need to allocate and set the core
> scheduling domain for the relevant processes or threads.
>
> Neither the vm/container mananger nor the indivial vm/container
> supervisors are supposed to run in any or the same core scheduling
> domain as the respective vcpus/workloads.
>
> So in order to create to create a new core scheduling domain we need to
> fork() off a new helper process which allocates a core scheduling domain
> and then pushes the cookie for the core scheduling domain to the
> relevant vcpus/workloads.
>
> This works but things get rather tricky, especially for containers, when
> a new process is supposed to be spawned into a running container.
> An important step in creating a new process inside a running container
> involves:
>
> - getting a handle on the container's init process (pid or nowadays
> often a pidfd)
> - getting a handle on the container's namespaces (namespace file
> descriptors reachable via /proc/<init-pid>/ns/<ns-typ> or nowadays
> often a pidfd)
> - calling setns() either on each namespace file descriptor individually
> or on the pidfd of the init process
>
> An important sub-step here is to attach to the container's pid namespace
> via setns(). After attaching to the container's pid namespace any
> process created via a fork()-like system calls will be a full member of
> the container's pid namespace.
>
> So attaching often involves two child processes. The first child simply
> attaches to the namespaces of the container including the container's
> pid namespace. The second child fork()s and ultimately exec()s thereby
> guaranteeing that the newly created process is a full member of the
> container's pid namespace:
>
> first_child = fork();
> if (first_child == 0) {
> setns(CLONE_NEWPID);
>
> second_child = fork();
> if (second_child == 0) {
> execlp();
> }
> }
>
> As part of this we also need to make sure that the second child - the
> one ultimately exec()ing the relevant programm in an already running
> container - joins the core scheduling domain of the container. When the
> container runs in a new pid namespace this can usually be done by
> calling:
>
> first_child = fork();
> if (first_child == 0) {
> setns(CLONE_NEWPID);
>
> second_child = fork();
> if (second_child == 0) {
> prctl(PR_SCHED_CORE, PR_SCHED_CORE_SHARE_FROM,
> 1, PR_SCHED_CORE_SCOPE_THREAD, 0);
>
> execlp();
> }
> }
>
> from the second child since we know that pid 1 in a container running
> inside of a separate pid namespace is the correct process to get the
> core scheduling domain from.
>
> However, this doesn't work when the container does not run in a separate
> pid namespace or when it shares the pid namespace with another
> container. In these scenarios we can't simply call
> PR_SCHED_CORE_SHARE_FROM from the second child since we don't know the
> correct pid number to call it on in the pid namespace.
>
> (Note it is of course possible to learn the pid of the process in the
> relevant pid namespace but it is rather complex involving three separate
> processes and an AF_UNIX domain socket over which to send a message
> including struct ucred from which to learn the relevant pid. But that
> doesn't work in all cases since it requires privileges to translate
> arbitrary pids. In any case, this is not an option for performance
> reasons alone. However, I do also have a separate patchset in [1]
> allowing translation of pids between pid namespaces which will help with
> that in the future - something which I had discussed with Joel a while
> back but haven't pushed for yet since implementing it early 2020. Both
> patches are useful independent of one another.)
>
> Additionally, we ideally always want to manage the core scheduling
> domain from the first child since the first child knows the pids for the
> relevant processes in its current pid namespace. The first child knows
> the pid of the init process in the current pid namespace from which to
> pull the core scheduling domain and it knows the pid of the second child
> it created to which to apply the core scheduling domain.
>
> The core scheduling domain of the first child needs to be unaffected as
> it might run sensitive codepaths that should not be exposed in smt attacks.
>
> The new PR_CORE_SCHED_SHARE command for the PR_SCHED_CORE prctl() option
> allows to support this and other use-cases by making it possible to pull
> the core scheduling domain from a task identified via its pid and push
> it to another task identified via its pid from a third managing task:
>
> prctl(PR_SCHED_CORE, PR_SCHED_CORE_SHARE,
> <pid-to-which-to-apply-coresched-domain>,
> PR_SCHED_CORE_SCOPE_{THREAD,THREAD_GROUP,PROCESS_GROUP},
> <pid-from-which-to-take-coresched-domain>)
>
> In order to use PR_SCHED_CORE_SHARE the caller must have
> ptrace_may_access() rights to both the task from which to take the core
> scheduling domain and to the task to which to apply the core scheduling
> domain. If the caller passes zero as the 5th argument then its own core
> scheduling domain is applied to the target making the option adhere to
> regular prctl() semantics.
>
> Thanks!
> Christian
>
> Christian Brauner (3):
> pid: introduce task_by_pid()
> sched/prctl: add PR_SCHED_CORE_SHARE command
> tests: add new PR_SCHED_CORE_SHARE test
>
> arch/mips/kernel/mips-mt-fpaff.c | 14 +-----
> arch/x86/kernel/cpu/resctrl/rdtgroup.c | 19 +++-----
> block/ioprio.c | 10 +----
> include/linux/sched.h | 9 +++-
> include/uapi/linux/prctl.h | 3 +-
> kernel/cgroup/cgroup.c | 12 ++---
> kernel/events/core.c | 5 +--
> kernel/futex/syscalls.c | 20 +++------
> kernel/pid.c | 5 +++
> kernel/sched/core.c | 27 ++++--------
> kernel/sched/core_sched.c | 44 ++++++++++++++-----
> kernel/sys.c | 12 ++---
> mm/mempolicy.c | 2 +-
> tools/testing/selftests/sched/cs_prctl_test.c | 23 ++++++++++
> 14 files changed, 105 insertions(+), 100 deletions(-)
>
>
> base-commit: 136057256686de39cc3a07c2e39ef6bc43003ff6
> --
> 2.30.2
>