Commit 5c2aed62 authored by Jason Baron's avatar Jason Baron Committed by Rusty Russell

stop_machine: add ALL_CPUS option

-allow stop_mahcine_run() to call a function on all cpus. Calling
 stop_machine_run() with a 'ALL_CPUS' invokes this new behavior.
 stop_machine_run() proceeds as normal until the calling cpu has
 invoked 'fn'. Then, we tell all the other cpus to call 'fn'.
Signed-off-by: default avatarJason Baron <jbaron@redhat.com>
Signed-off-by: default avatarMathieu Desnoyers <mathieu.desnoyers@polymtl.ca>
Signed-off-by: default avatarRusty Russell <rusty@rustcorp.com.au>
CC: Adrian Bunk <bunk@stusta.de>
CC: Andi Kleen <andi@firstfloor.org>
CC: Alexey Dobriyan <adobriyan@gmail.com>
CC: Christoph Hellwig <hch@infradead.org>
CC: mingo@elte.hu
CC: akpm@osdl.org
parent 15bba37d
...@@ -8,11 +8,17 @@ ...@@ -8,11 +8,17 @@
#include <asm/system.h> #include <asm/system.h>
#if defined(CONFIG_STOP_MACHINE) && defined(CONFIG_SMP) #if defined(CONFIG_STOP_MACHINE) && defined(CONFIG_SMP)
#define ALL_CPUS ~0U
/** /**
* stop_machine_run: freeze the machine on all CPUs and run this function * stop_machine_run: freeze the machine on all CPUs and run this function
* @fn: the function to run * @fn: the function to run
* @data: the data ptr for the @fn() * @data: the data ptr for the @fn()
* @cpu: the cpu to run @fn() on (or any, if @cpu == NR_CPUS. * @cpu: if @cpu == n, run @fn() on cpu n
* if @cpu == NR_CPUS, run @fn() on any cpu
* if @cpu == ALL_CPUS, run @fn() first on the calling cpu, and then
* concurrently on all the other cpus
* *
* Description: This causes a thread to be scheduled on every other cpu, * Description: This causes a thread to be scheduled on every other cpu,
* each of which disables interrupts, and finally interrupts are disabled * each of which disables interrupts, and finally interrupts are disabled
......
...@@ -22,9 +22,17 @@ enum stopmachine_state { ...@@ -22,9 +22,17 @@ enum stopmachine_state {
STOPMACHINE_WAIT, STOPMACHINE_WAIT,
STOPMACHINE_PREPARE, STOPMACHINE_PREPARE,
STOPMACHINE_DISABLE_IRQ, STOPMACHINE_DISABLE_IRQ,
STOPMACHINE_RUN,
STOPMACHINE_EXIT, STOPMACHINE_EXIT,
}; };
struct stop_machine_data {
int (*fn)(void *);
void *data;
struct completion done;
int run_all;
} smdata;
static enum stopmachine_state stopmachine_state; static enum stopmachine_state stopmachine_state;
static unsigned int stopmachine_num_threads; static unsigned int stopmachine_num_threads;
static atomic_t stopmachine_thread_ack; static atomic_t stopmachine_thread_ack;
...@@ -33,6 +41,7 @@ static int stopmachine(void *cpu) ...@@ -33,6 +41,7 @@ static int stopmachine(void *cpu)
{ {
int irqs_disabled = 0; int irqs_disabled = 0;
int prepared = 0; int prepared = 0;
int ran = 0;
cpumask_of_cpu_ptr(cpumask, (int)(long)cpu); cpumask_of_cpu_ptr(cpumask, (int)(long)cpu);
set_cpus_allowed_ptr(current, cpumask); set_cpus_allowed_ptr(current, cpumask);
...@@ -58,6 +67,11 @@ static int stopmachine(void *cpu) ...@@ -58,6 +67,11 @@ static int stopmachine(void *cpu)
prepared = 1; prepared = 1;
smp_mb(); /* Must read state first. */ smp_mb(); /* Must read state first. */
atomic_inc(&stopmachine_thread_ack); atomic_inc(&stopmachine_thread_ack);
} else if (stopmachine_state == STOPMACHINE_RUN && !ran) {
smdata.fn(smdata.data);
ran = 1;
smp_mb(); /* Must read state first. */
atomic_inc(&stopmachine_thread_ack);
} }
/* Yield in first stage: migration threads need to /* Yield in first stage: migration threads need to
* help our sisters onto their CPUs. */ * help our sisters onto their CPUs. */
...@@ -136,11 +150,10 @@ static void restart_machine(void) ...@@ -136,11 +150,10 @@ static void restart_machine(void)
preempt_enable_no_resched(); preempt_enable_no_resched();
} }
struct stop_machine_data { static void run_other_cpus(void)
int (*fn)(void *); {
void *data; stopmachine_set_state(STOPMACHINE_RUN);
struct completion done; }
};
static int do_stop(void *_smdata) static int do_stop(void *_smdata)
{ {
...@@ -150,6 +163,8 @@ static int do_stop(void *_smdata) ...@@ -150,6 +163,8 @@ static int do_stop(void *_smdata)
ret = stop_machine(); ret = stop_machine();
if (ret == 0) { if (ret == 0) {
ret = smdata->fn(smdata->data); ret = smdata->fn(smdata->data);
if (smdata->run_all)
run_other_cpus();
restart_machine(); restart_machine();
} }
...@@ -173,14 +188,17 @@ struct task_struct *__stop_machine_run(int (*fn)(void *), void *data, ...@@ -173,14 +188,17 @@ struct task_struct *__stop_machine_run(int (*fn)(void *), void *data,
struct stop_machine_data smdata; struct stop_machine_data smdata;
struct task_struct *p; struct task_struct *p;
mutex_lock(&stopmachine_mutex);
smdata.fn = fn; smdata.fn = fn;
smdata.data = data; smdata.data = data;
smdata.run_all = (cpu == ALL_CPUS) ? 1 : 0;
init_completion(&smdata.done); init_completion(&smdata.done);
mutex_lock(&stopmachine_mutex); smp_wmb(); /* make sure other cpus see smdata updates */
/* If they don't care which CPU fn runs on, bind to any online one. */ /* If they don't care which CPU fn runs on, bind to any online one. */
if (cpu == NR_CPUS) if (cpu == NR_CPUS || cpu == ALL_CPUS)
cpu = raw_smp_processor_id(); cpu = raw_smp_processor_id();
p = kthread_create(do_stop, &smdata, "kstopmachine"); p = kthread_create(do_stop, &smdata, "kstopmachine");
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment