diff options
| author |   Liu Song <liu.song11@zte.com.cn> | 2021-06-29 21:55:51 +0800 |
|---|---|---|
| committer |   Paul E. McKenney <paulmck@kernel.org> | 2021-08-06 13:41:48 -0700 |
| commit | eb880949ef41c98a203c4a033e06e05854d902ef (patch) | |
| tree | 43a94396ec4646f662ec9f082114d185edf82663 /kernel/rcu | |
| parent | rcu: Mark accesses in tree_stall.h (diff) | |
| download | linux-dev-eb880949ef41c98a203c4a033e06e05854d902ef.tar.xz linux-dev-eb880949ef41c98a203c4a033e06e05854d902ef.zip | |
rcu: Remove useless "ret" update in rcu_gp_fqs_loop()
Within rcu_gp_fqs_loop(), the "ret" local variable is set to the
return value from swait_event_idle_timeout_exclusive(), but "ret" is
unconditionally overwritten later in the code. This commit therefore
removes this useless assignment.
Signed-off-by: Liu Song <liu.song11@zte.com.cn>
Signed-off-by: Paul E. McKenney <paulmck@kernel.org>
Diffstat (limited to 'kernel/rcu')
| -rw-r--r-- | kernel/rcu/tree.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/kernel/rcu/tree.c b/kernel/rcu/tree.c index d791c4d32987..2f000e8a315d 100644 --- a/kernel/rcu/tree.c +++ b/kernel/rcu/tree.c @@ -1954,8 +1954,8 @@ static noinline_for_stack void rcu_gp_fqs_loop(void) trace_rcu_grace_period(rcu_state.name, rcu_state.gp_seq, TPS("fqswait")); WRITE_ONCE(rcu_state.gp_state, RCU_GP_WAIT_FQS); - ret = swait_event_idle_timeout_exclusive( - rcu_state.gp_wq, rcu_gp_fqs_check_wake(&gf), j); + (void)swait_event_idle_timeout_exclusive(rcu_state.gp_wq, + rcu_gp_fqs_check_wake(&gf), j); rcu_gp_torture_wait(); WRITE_ONCE(rcu_state.gp_state, RCU_GP_DOING_FQS); /* Locking provides needed memory barriers. */ |