diff options
| author | Ingo Molnar <mingo@elte.hu> | 2009-03-12 22:20:49 -0400 |
|---|---|---|
| committer | Ingo Molnar <mingo@elte.hu> | 2009-04-03 06:23:02 -0400 |
| commit | b1f77b0581b8fd837acb4a973f7d5496cae6efee (patch) | |
| tree | 4faa2e4285c31dd70d1d20b4172d1eed55ba7330 /kernel | |
| parent | 255d11bc910fd38153156a0c5ebb256657290882 (diff) | |
kmemtrace, rcu: fix linux/rcutree.h and linux/rcuclassic.h dependencies
Impact: build fix for all non-x86 architectures
We want to remove percpu.h from rcuclassic.h/rcutree.h (for upcoming
kmemtrace changes) but that would break the DECLARE_PER_CPU based
declarations in these files.
Move the quiescent counter management functions to their respective
RCU implementation .c files - they were slightly above the inlining
limit anyway.
Cc: Pekka Enberg <penberg@cs.helsinki.fi>
Cc: Eduard - Gabriel Munteanu <eduard.munteanu@linux360.ro>
Cc: paulmck@linux.vnet.ibm.com
LKML-Reference: <1237898630.25315.83.camel@penberg-laptop>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Diffstat (limited to 'kernel')
| -rw-r--r-- | kernel/rcuclassic.c | 23 | ||||
| -rw-r--r-- | kernel/rcutree.c | 28 |
2 files changed, 45 insertions, 6 deletions
diff --git a/kernel/rcuclassic.c b/kernel/rcuclassic.c index 654c640a6b9c..0f2b0b311304 100644 --- a/kernel/rcuclassic.c +++ b/kernel/rcuclassic.c | |||
| @@ -65,6 +65,7 @@ static struct rcu_ctrlblk rcu_ctrlblk = { | |||
| 65 | .lock = __SPIN_LOCK_UNLOCKED(&rcu_ctrlblk.lock), | 65 | .lock = __SPIN_LOCK_UNLOCKED(&rcu_ctrlblk.lock), |
| 66 | .cpumask = CPU_BITS_NONE, | 66 | .cpumask = CPU_BITS_NONE, |
| 67 | }; | 67 | }; |
| 68 | |||
| 68 | static struct rcu_ctrlblk rcu_bh_ctrlblk = { | 69 | static struct rcu_ctrlblk rcu_bh_ctrlblk = { |
| 69 | .cur = -300, | 70 | .cur = -300, |
| 70 | .completed = -300, | 71 | .completed = -300, |
| @@ -73,8 +74,26 @@ static struct rcu_ctrlblk rcu_bh_ctrlblk = { | |||
| 73 | .cpumask = CPU_BITS_NONE, | 74 | .cpumask = CPU_BITS_NONE, |
| 74 | }; | 75 | }; |
| 75 | 76 | ||
| 76 | DEFINE_PER_CPU(struct rcu_data, rcu_data) = { 0L }; | 77 | static DEFINE_PER_CPU(struct rcu_data, rcu_data); |
| 77 | DEFINE_PER_CPU(struct rcu_data, rcu_bh_data) = { 0L }; | 78 | static DEFINE_PER_CPU(struct rcu_data, rcu_bh_data); |
| 79 | |||
| 80 | /* | ||
| 81 | * Increment the quiescent state counter. | ||
| 82 | * The counter is a bit degenerated: We do not need to know | ||
| 83 | * how many quiescent states passed, just if there was at least | ||
| 84 | * one since the start of the grace period. Thus just a flag. | ||
| 85 | */ | ||
| 86 | void rcu_qsctr_inc(int cpu) | ||
| 87 | { | ||
| 88 | struct rcu_data *rdp = &per_cpu(rcu_data, cpu); | ||
| 89 | rdp->passed_quiesc = 1; | ||
| 90 | } | ||
| 91 | |||
| 92 | void rcu_bh_qsctr_inc(int cpu) | ||
| 93 | { | ||
| 94 | struct rcu_data *rdp = &per_cpu(rcu_bh_data, cpu); | ||
| 95 | rdp->passed_quiesc = 1; | ||
| 96 | } | ||
| 78 | 97 | ||
| 79 | static int blimit = 10; | 98 | static int blimit = 10; |
| 80 | static int qhimark = 10000; | 99 | static int qhimark = 10000; |
diff --git a/kernel/rcutree.c b/kernel/rcutree.c index 97ce31579ec0..a2015edfe167 100644 --- a/kernel/rcutree.c +++ b/kernel/rcutree.c | |||
| @@ -72,11 +72,31 @@ EXPORT_SYMBOL_GPL(rcu_lock_map); | |||
| 72 | .n_force_qs_ngp = 0, \ | 72 | .n_force_qs_ngp = 0, \ |
| 73 | } | 73 | } |
| 74 | 74 | ||
| 75 | struct rcu_state rcu_state = RCU_STATE_INITIALIZER(rcu_state); | 75 | static struct rcu_state rcu_state = RCU_STATE_INITIALIZER(rcu_state); |
| 76 | DEFINE_PER_CPU(struct rcu_data, rcu_data); | 76 | static DEFINE_PER_CPU(struct rcu_data, rcu_data); |
| 77 | 77 | ||
| 78 | struct rcu_state rcu_bh_state = RCU_STATE_INITIALIZER(rcu_bh_state); | 78 | static struct rcu_state rcu_bh_state = RCU_STATE_INITIALIZER(rcu_bh_state); |
| 79 | DEFINE_PER_CPU(struct rcu_data, rcu_bh_data); | 79 | static DEFINE_PER_CPU(struct rcu_data, rcu_bh_data); |
| 80 | |||
| 81 | /* | ||
| 82 | * Increment the quiescent state counter. | ||
| 83 | * The counter is a bit degenerated: We do not need to know | ||
| 84 | * how many quiescent states passed, just if there was at least | ||
| 85 | * one since the start of the grace period. Thus just a flag. | ||
| 86 | */ | ||
| 87 | void rcu_qsctr_inc(int cpu) | ||
| 88 | { | ||
| 89 | struct rcu_data *rdp = &per_cpu(rcu_data, cpu); | ||
| 90 | rdp->passed_quiesc = 1; | ||
| 91 | rdp->passed_quiesc_completed = rdp->completed; | ||
| 92 | } | ||
| 93 | |||
| 94 | void rcu_bh_qsctr_inc(int cpu) | ||
| 95 | { | ||
| 96 | struct rcu_data *rdp = &per_cpu(rcu_bh_data, cpu); | ||
| 97 | rdp->passed_quiesc = 1; | ||
| 98 | rdp->passed_quiesc_completed = rdp->completed; | ||
| 99 | } | ||
| 80 | 100 | ||
| 81 | #ifdef CONFIG_NO_HZ | 101 | #ifdef CONFIG_NO_HZ |
| 82 | DEFINE_PER_CPU(struct rcu_dynticks, rcu_dynticks) = { | 102 | DEFINE_PER_CPU(struct rcu_dynticks, rcu_dynticks) = { |
