diff options
author | Alexander Gordeev <agordeev@redhat.com> | 2015-02-11 09:42:37 -0500 |
---|---|---|
committer | Paul E. McKenney <paulmck@linux.vnet.ibm.com> | 2015-02-26 15:11:52 -0500 |
commit | 27153acbe1141ceecf098ca5d24c2ae2714c1a5f (patch) | |
tree | e9582d6d28a27b75aab17fbc0a94dd7e3fb38ea5 | |
parent | c517d838eb7d07bbe9507871fab3931deccff539 (diff) |
rcu: Remove unnecessary condition check in rcu_qsctr_help()
When the ->curtail and ->donetail pointers differ, ->rcucblist
always points to the beginning of the current list and thus
cannot be NULL. Therefore, the check ->rcucblist != NULL is
redundant and this commit removes it.
Cc: "Paul E. McKenney" <paulmck@linux.vnet.ibm.com>
Signed-off-by: Alexander Gordeev <agordeev@redhat.com>
Signed-off-by: Paul E. McKenney <paulmck@linux.vnet.ibm.com>
-rw-r--r-- | kernel/rcu/tiny.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/kernel/rcu/tiny.c b/kernel/rcu/tiny.c index cc9ceca7bde1..d4e7fe5f3baf 100644 --- a/kernel/rcu/tiny.c +++ b/kernel/rcu/tiny.c | |||
@@ -103,8 +103,7 @@ EXPORT_SYMBOL(__rcu_is_watching); | |||
103 | static int rcu_qsctr_help(struct rcu_ctrlblk *rcp) | 103 | static int rcu_qsctr_help(struct rcu_ctrlblk *rcp) |
104 | { | 104 | { |
105 | RCU_TRACE(reset_cpu_stall_ticks(rcp)); | 105 | RCU_TRACE(reset_cpu_stall_ticks(rcp)); |
106 | if (rcp->rcucblist != NULL && | 106 | if (rcp->donetail != rcp->curtail) { |
107 | rcp->donetail != rcp->curtail) { | ||
108 | rcp->donetail = rcp->curtail; | 107 | rcp->donetail = rcp->curtail; |
109 | return 1; | 108 | return 1; |
110 | } | 109 | } |