diff options
| author |   Akira Yokosawa <akiyks@gmail.com> | 2019-03-02 17:25:19 +0900 |
|---|---|---|
| committer |   Paul E. McKenney <paulmck@linux.ibm.com> | 2019-03-26 14:37:49 -0700 |
| commit | b2eb85b49a576515fb845cb12568b173c2bedffc (patch) | |
| tree | 4b0239d53ce17b7476428f87de5084b59cf12b19 | |
| parent | MAINTAINERS: Add -rcu branch name ("dev") (diff) | |
| download | linux-dev-b2eb85b49a576515fb845cb12568b173c2bedffc.tar.xz linux-dev-b2eb85b49a576515fb845cb12568b173c2bedffc.zip | |
rcu: Move common code out of if-else block
As the result of recent addition of "rdp->core_needs_qs = false;" in
the "if" block, now both branches of the if-else have the same
assignment.
Factor it out and reduce line count.
Signed-off-by: Akira Yokosawa <akiyks@gmail.com>
Cc: Joel Fernandes <joel@joelfernandes.org>
Signed-off-by: Paul E. McKenney <paulmck@linux.ibm.com>
Acked-by: Joel Fernandes (Google) <joel@joelfernandes.org>
| -rw-r--r-- | kernel/rcu/tree.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/kernel/rcu/tree.c b/kernel/rcu/tree.c index 296131450414..5aefd36ac648 100644 --- a/kernel/rcu/tree.c +++ b/kernel/rcu/tree.c @@ -2295,12 +2295,10 @@ rcu_report_qs_rdp(int cpu, struct rcu_data *rdp) return; } mask = rdp->grpmask; + rdp->core_needs_qs = false; if ((rnp->qsmask & mask) == 0) { - rdp->core_needs_qs = false; raw_spin_unlock_irqrestore_rcu_node(rnp, flags); } else { - rdp->core_needs_qs = false; - /* * This GP can't end until cpu checks in, so all of our * callbacks can be processed during the next GP. |