summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorLai Jiangshan <laijs@cn.fujitsu.com>2012-11-29 16:46:04 +0800
committerPaul E. McKenney <paulmck@linux.vnet.ibm.com>2013-02-07 15:10:10 -0800
commitab4d2986e44c589aa1b647d7da5e21c2707babea (patch)
treee5400a89ca0f137125be973016c6391a701a790c
parent6e6f1b307e23201fb3e7aaf16322e80355d2a3d5 (diff)
srcu: Simple cleanup for cleanup_srcu_struct()
Pack six lines of code into two lines. Signed-off-by: Lai Jiangshan <laijs@cn.fujitsu.com> Signed-off-by: Paul E. McKenney <paulmck@linux.vnet.ibm.com>
-rw-r--r--kernel/srcu.c8
1 files changed, 2 insertions, 6 deletions
diff --git a/kernel/srcu.c b/kernel/srcu.c
index 3e43a214b4dc..7cf5baba96f9 100644
--- a/kernel/srcu.c
+++ b/kernel/srcu.c
@@ -282,12 +282,8 @@ static int srcu_readers_active(struct srcu_struct *sp)
*/
void cleanup_srcu_struct(struct srcu_struct *sp)
{
- int sum;
-
- sum = srcu_readers_active(sp);
- WARN_ON(sum); /* Leakage unless caller handles error. */
- if (sum != 0)
- return;
+ if (WARN_ON(srcu_readers_active(sp)))
+ return; /* Leakage unless caller handles error. */
free_percpu(sp->per_cpu_ref);
sp->per_cpu_ref = NULL;
}