diff options
| author | Rik van Riel <riel@redhat.com> | 2013-11-01 15:41:46 +0100 |
|---|---|---|
| committer | Ingo Molnar <mingo@kernel.org> | 2013-11-11 12:43:38 +0100 |
| commit | 7053ea1a34fa8567cb5e3c39e04ace4c5d0fbeaa (patch) | |
| tree | ed5866f5a2449e6d7b918ce92858f891c8bc1450 /kernel/trace/trace_output.c | |
| parent | sched: Remove unnecessary iteration over sched domains to update nr_busy_cpus (diff) | |
| download | linux-7053ea1a34fa8567cb5e3c39e04ace4c5d0fbeaa.tar.xz linux-7053ea1a34fa8567cb5e3c39e04ace4c5d0fbeaa.zip | |
stop_machine: Fix race between stop_two_cpus() and stop_cpus()
There is a race between stop_two_cpus, and the global stop_cpus.
It is possible for two CPUs to get their stopper functions queued
"backwards" from one another, resulting in the stopper threads
getting stuck, and the system hanging. This can happen because
queuing up stoppers is not synchronized.
This patch adds synchronization between stop_cpus (a rare operation),
and stop_two_cpus.
Reported-and-Tested-by: Prarit Bhargava <prarit@redhat.com>
Signed-off-by: Rik van Riel <riel@redhat.com>
Signed-off-by: Peter Zijlstra <peterz@infradead.org>
Acked-by: Mel Gorman <mgorman@suse.de>
Link: http://lkml.kernel.org/r/20131101104146.03d1e043@annuminas.surriel.com
Signed-off-by: Ingo Molnar <mingo@kernel.org>
Diffstat (limited to 'kernel/trace/trace_output.c')
0 files changed, 0 insertions, 0 deletions