sched: fix exit_mm vs membarrier (v4)
authorMathieu Desnoyers <mathieu.desnoyers@efficios.com>
Tue, 20 Oct 2020 13:47:13 +0000 (09:47 -0400)
committerPeter Zijlstra <peterz@infradead.org>
Thu, 29 Oct 2020 10:00:30 +0000 (11:00 +0100)
commit5bc78502322a5e4eef3f1b2a2813751dc6434143
tree2c07a22fd6161ab1504634bd4785b0d0e52531dd
parent45da7a2b0af8fa29dff2e6ba8926322068350fce
sched: fix exit_mm vs membarrier (v4)

exit_mm should issue memory barriers after user-space memory accesses,
before clearing current->mm, to order user-space memory accesses
performed prior to exit_mm before clearing tsk->mm, which has the
effect of skipping the membarrier private expedited IPIs.

exit_mm should also update the runqueue's membarrier_state so
membarrier global expedited IPIs are not sent when they are not
needed.

The membarrier system call can be issued concurrently with do_exit
if we have thread groups created with CLONE_VM but not CLONE_THREAD.

Here is the scenario I have in mind:

Two thread groups are created, A and B. Thread group B is created by
issuing clone from group A with flag CLONE_VM set, but not CLONE_THREAD.
Let's assume we have a single thread within each thread group (Thread A
and Thread B).

The AFAIU we can have:

Userspace variables:

int x = 0, y = 0;

CPU 0                   CPU 1
Thread A                Thread B
(in thread group A)     (in thread group B)

x = 1
barrier()
y = 1
exit()
exit_mm()
current->mm = NULL;
                        r1 = load y
                        membarrier()
                          skips CPU 0 (no IPI) because its current mm is NULL
                        r2 = load x
                        BUG_ON(r1 == 1 && r2 == 0)

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
Signed-off-by: Peter Zijlstra (Intel) <peterz@infradead.org>
Link: https://lkml.kernel.org/r/20201020134715.13909-2-mathieu.desnoyers@efficios.com
include/linux/sched/mm.h
kernel/exit.c
kernel/sched/membarrier.c