Make sure the entire for loop has stop_cpus_in_progress set.

Cc: Valentin Schneider <[email protected]>
Cc: Aaron Lu <[email protected]>
Cc: [email protected]
Cc: [email protected]
Cc: Pawan Gupta <[email protected]>
Cc: Phil Auld <[email protected]>
Cc: [email protected]
Cc: Tim Chen <[email protected]>
Cc: [email protected]
Cc: [email protected]
Cc: [email protected]
Cc: Julien Desfossez <[email protected]>
Cc: [email protected]
Cc: Nishanth Aravamudan <[email protected]>
Cc: Aubrey Li <[email protected]>
Cc: Mel Gorman <[email protected]>
Cc: [email protected]
Cc: Paolo Bonzini <[email protected]>
Signed-off-by: Peter Zijlstra (Intel) <[email protected]>
Link: 
https://lkml.kernel.org/r/0fd8fd4b99b9b9aa88d8b2dff897f7fd0d88f72c.1559129225.git.vpil...@digitalocean.com
---
 kernel/stop_machine.c |    2 ++
 1 file changed, 2 insertions(+)

--- a/kernel/stop_machine.c
+++ b/kernel/stop_machine.c
@@ -383,6 +383,7 @@ static bool queue_stop_cpus_work(const s
         */
        preempt_disable();
        stop_cpus_in_progress = true;
+       barrier();
        for_each_cpu(cpu, cpumask) {
                work = &per_cpu(cpu_stopper.stop_work, cpu);
                work->fn = fn;
@@ -391,6 +392,7 @@ static bool queue_stop_cpus_work(const s
                if (cpu_stop_queue_work(cpu, work))
                        queued = true;
        }
+       barrier();
        stop_cpus_in_progress = false;
        preempt_enable();
 


Reply via email to