rcu: Make expedited IPI handler return after handling critical section
authorPaul E. McKenney <paulmck@linux.ibm.com>
Wed, 28 Nov 2018 18:37:42 +0000 (10:37 -0800)
committerPaul E. McKenney <paulmck@linux.ibm.com>
Fri, 25 Jan 2019 23:28:24 +0000 (15:28 -0800)
During expedited RCU grace-period initialization, IPIs are sent to
all non-idle online CPUs.  The IPI handler checks to see if the CPU is
in quiescent state, reporting one if so.  This handler looks at three
different cases: (1) The CPU is not in an rcu_read_lock()-based critical
section, (2) The CPU is in the process of exiting an rcu_read_lock()-based
critical section, and (3) The CPU is in an rcu_read_lock()-based critical
section.  In case (2), execution falls through into case (3).

This is harmless from a functionality viewpoint, but can result in
needless overhead during an improbable corner case.  This commit therefore
adds the "return" statement needed to prevent fall-through.

Signed-off-by: Paul E. McKenney <paulmck@linux.ibm.com>
kernel/rcu/tree_exp.h

index 928fe58..6d4eb46 100644 (file)
@@ -697,6 +697,7 @@ static void sync_rcu_exp_handler(void *unused)
                        WRITE_ONCE(t->rcu_read_unlock_special.b.exp_hint, true);
                }
                raw_spin_unlock_irqrestore_rcu_node(rnp, flags);
+               return;
        }
 
        /*