diff options
author | Pranith Kumar <bobby.prani@gmail.com> | 2014-07-17 20:11:01 -0400 |
---|---|---|
committer | Paul E. McKenney <paulmck@linux.vnet.ibm.com> | 2014-09-16 13:08:02 -0400 |
commit | 22c2f669611590b428647ac9a73bc63ef3989d4b (patch) | |
tree | e427b2b0edcc754196e9d422088694e3ff8e96e3 /kernel | |
parent | 35ce7f29a44a888c45c0a9f202f69e10613c5306 (diff) |
rcu: Check for have_rcu_nocb_mask instead of rcu_nocb_mask
If we configure a kernel with CONFIG_NOCB_CPU=y, CONFIG_RCU_NOCB_CPU_NONE=y and
CONFIG_CPUMASK_OFFSTACK=n and do not pass in a rcu_nocb= boot parameter, the
cpumask rcu_nocb_mask can be garbage instead of NULL.
Hence this commit replaces checks for rcu_nocb_mask == NULL with a check for
have_rcu_nocb_mask.
Signed-off-by: Pranith Kumar <bobby.prani@gmail.com>
Signed-off-by: Paul E. McKenney <paulmck@linux.vnet.ibm.com>
Tested-by: Paul Gortmaker <paul.gortmaker@windriver.com>
Diffstat (limited to 'kernel')
-rw-r--r-- | kernel/rcu/tree_plugin.h | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/kernel/rcu/tree_plugin.h b/kernel/rcu/tree_plugin.h index 31c7afb611fd..39e68bcf6d83 100644 --- a/kernel/rcu/tree_plugin.h +++ b/kernel/rcu/tree_plugin.h | |||
@@ -2578,7 +2578,7 @@ static void __init rcu_organize_nocb_kthreads(struct rcu_state *rsp) | |||
2578 | struct rcu_data *rdp_leader = NULL; /* Suppress misguided gcc warn. */ | 2578 | struct rcu_data *rdp_leader = NULL; /* Suppress misguided gcc warn. */ |
2579 | struct rcu_data *rdp_prev = NULL; | 2579 | struct rcu_data *rdp_prev = NULL; |
2580 | 2580 | ||
2581 | if (rcu_nocb_mask == NULL) | 2581 | if (!have_rcu_nocb_mask) |
2582 | return; | 2582 | return; |
2583 | if (ls == -1) { | 2583 | if (ls == -1) { |
2584 | ls = int_sqrt(nr_cpu_ids); | 2584 | ls = int_sqrt(nr_cpu_ids); |
@@ -2608,9 +2608,9 @@ static void __init rcu_organize_nocb_kthreads(struct rcu_state *rsp) | |||
2608 | /* Prevent __call_rcu() from enqueuing callbacks on no-CBs CPUs */ | 2608 | /* Prevent __call_rcu() from enqueuing callbacks on no-CBs CPUs */ |
2609 | static bool init_nocb_callback_list(struct rcu_data *rdp) | 2609 | static bool init_nocb_callback_list(struct rcu_data *rdp) |
2610 | { | 2610 | { |
2611 | if (rcu_nocb_mask == NULL || | 2611 | if (!rcu_is_nocb_cpu(rdp->cpu)) |
2612 | !cpumask_test_cpu(rdp->cpu, rcu_nocb_mask)) | ||
2613 | return false; | 2612 | return false; |
2613 | |||
2614 | rdp->nxttail[RCU_NEXT_TAIL] = NULL; | 2614 | rdp->nxttail[RCU_NEXT_TAIL] = NULL; |
2615 | return true; | 2615 | return true; |
2616 | } | 2616 | } |