On Wed, Jul 01, 2015 at 12:07:30PM +0200, Peter Zijlstra wrote:
> On Tue, Jun 30, 2015 at 02:48:30PM -0700, Paul E. McKenney wrote:
> > From: "Paul E. McKenney" <paul...@linux.vnet.ibm.com>
>
> This seems like a good place to explain why this is a desirable thing,
> no?
>
> Why would you want to limit this?
Good point! I added the following to the commit log:
Currently, the expedited grace period unconditionally awakens the
normal grace period anytime there is a grace period in progress.
However, this can be too much of a good thing for the following
reasons: (1) It can result in excessive CPU consumption on
the part of the RCU grace-period kthread, (2) The resulting
variations in normal grace-period latency may cause confusion
(as Josh Triplett points out), and (3) In many cases, reducing
normal grace-period latency will be of little or no value.
This commit therefore does the wakeups only once per ten jiffies
or every fourth grace period, whichever is more frequent.
Does that help?
Thanx, Paul
> > Signed-off-by: Paul E. McKenney <paul...@linux.vnet.ibm.com>
> > ---
> > kernel/rcu/tree.c | 15 ++++++++++++---
> > 1 file changed, 12 insertions(+), 3 deletions(-)
> >
> > diff --git a/kernel/rcu/tree.c b/kernel/rcu/tree.c
> > index 308b6acb4260..247aa1120c4c 100644
> > --- a/kernel/rcu/tree.c
> > +++ b/kernel/rcu/tree.c
> > @@ -3505,10 +3505,19 @@ void synchronize_sched_expedited(void)
> > !atomic_read(&rsp->expedited_need_qs));
> >
> > rcu_exp_gp_seq_end(rsp);
> > - mutex_unlock(&rnp->exp_funnel_mutex);
> > smp_mb(); /* ensure subsequent action seen after grace period. */
> > - if (rsp->gp_kthread && rcu_gp_in_progress(rsp))
> > - wake_up(&rsp->gp_wq);
> > + if (rsp->gp_kthread && rcu_gp_in_progress(rsp)) {
> > + static unsigned long nextgp;
> > + static unsigned long nextjiffy;
> > +
> > + if (time_after_eq(jiffies, nextgp) ||
> > + ULONG_CMP_GE(rsp->gpnum, nextgp)) {
> > + nextgp = rsp->gpnum + 4;
> > + nextjiffy = jiffies + 10;
> > + wake_up(&rsp->gp_wq);
> > + }
> > + }
> > + mutex_unlock(&rnp->exp_funnel_mutex);
> >
> > put_online_cpus();
> > }
> > --
> > 1.8.1.5
> >
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
