diff options
| author | Lai Jiangshan <laijs@cn.fujitsu.com> | 2012-11-29 16:46:04 +0800 |
|---|---|---|
| committer | Paul E. McKenney <paulmck@linux.vnet.ibm.com> | 2013-02-07 15:10:10 -0800 |
| commit | ab4d2986e44c589aa1b647d7da5e21c2707babea (patch) | |
| tree | e5400a89ca0f137125be973016c6391a701a790c /kernel/srcu.c | |
| parent | 6e6f1b307e23201fb3e7aaf16322e80355d2a3d5 (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>
Diffstat (limited to 'kernel/srcu.c')
| -rw-r--r-- | kernel/srcu.c | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/kernel/srcu.c b/kernel/srcu.c index 3e43a214b4d..7cf5baba96f 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; } |