linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* rcu: enable rcu_normal_after_boot by default for RT
@ 2018-11-01 23:02 Paul E. McKenney
  2018-11-08 15:52 ` Sebastian Andrzej Siewior
  0 siblings, 1 reply; 3+ messages in thread
From: Paul E. McKenney @ 2018-11-01 23:02 UTC (permalink / raw)
  To: lcapitulino, julia, bigeasy; +Cc: linux-rt-users, linux-kernel

> The forcing of an expedited grace period is an expensive and very
> RT-application unfriendly operation, as it forcibly preempts all running
> tasks on CPUs which are preventing the gp from expiring.
> 
> By default, as a policy decision, disable the expediting of grace
> periods (after boot) on configurations which enable PREEMPT_RT_FULL.
> 
> Suggested-by: Luiz Capitulino <lcapitulino@redhat.com>
> Signed-off-by: Julia Cartwright <julia@ni.com>
> Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de>

In case it matters:

Acked-by: Paul E. McKenney <paulmck@linux.ibm.com>

Alternatively, any reason that I should not pull this into -rcu?

							Thanx, Paul

> diff --git a/kernel/rcu/update.c b/kernel/rcu/update.c
> index f56c0fbdf22e..12027723abaf 100644
> --- a/kernel/rcu/update.c
> +++ b/kernel/rcu/update.c
> @@ -67,7 +67,7 @@ extern int rcu_expedited; /* from sysctl */
>  module_param(rcu_expedited, int, 0);
>  extern int rcu_normal; /* from sysctl */
>  module_param(rcu_normal, int, 0);
> -static int rcu_normal_after_boot;
> +static int rcu_normal_after_boot = IS_ENABLED(CONFIG_PREEMPT_RT_FULL);
>  module_param(rcu_normal_after_boot, int, 0);
>  #endif /* #ifndef CONFIG_TINY_RCU */
>  


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: rcu: enable rcu_normal_after_boot by default for RT
  2018-11-01 23:02 rcu: enable rcu_normal_after_boot by default for RT Paul E. McKenney
@ 2018-11-08 15:52 ` Sebastian Andrzej Siewior
  2018-11-08 16:12   ` Paul E. McKenney
  0 siblings, 1 reply; 3+ messages in thread
From: Sebastian Andrzej Siewior @ 2018-11-08 15:52 UTC (permalink / raw)
  To: Paul E. McKenney; +Cc: lcapitulino, julia, linux-rt-users, linux-kernel

On 2018-11-01 16:02:35 [-0700], Paul E. McKenney wrote:
> > The forcing of an expedited grace period is an expensive and very
> > RT-application unfriendly operation, as it forcibly preempts all running
> > tasks on CPUs which are preventing the gp from expiring.
> > 
> > By default, as a policy decision, disable the expediting of grace
> > periods (after boot) on configurations which enable PREEMPT_RT_FULL.
> > 
> > Suggested-by: Luiz Capitulino <lcapitulino@redhat.com>
> > Signed-off-by: Julia Cartwright <julia@ni.com>
> > Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
> 
> In case it matters:
> 
> Acked-by: Paul E. McKenney <paulmck@linux.ibm.com>
> 
> Alternatively, any reason that I should not pull this into -rcu?
because we don't have CONFIG_PREEMPT_RT_FULL yet.
> 							Thanx, Paul

Sebastian

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: rcu: enable rcu_normal_after_boot by default for RT
  2018-11-08 15:52 ` Sebastian Andrzej Siewior
@ 2018-11-08 16:12   ` Paul E. McKenney
  0 siblings, 0 replies; 3+ messages in thread
From: Paul E. McKenney @ 2018-11-08 16:12 UTC (permalink / raw)
  To: Sebastian Andrzej Siewior
  Cc: lcapitulino, julia, linux-rt-users, linux-kernel

On Thu, Nov 08, 2018 at 04:52:26PM +0100, Sebastian Andrzej Siewior wrote:
> On 2018-11-01 16:02:35 [-0700], Paul E. McKenney wrote:
> > > The forcing of an expedited grace period is an expensive and very
> > > RT-application unfriendly operation, as it forcibly preempts all running
> > > tasks on CPUs which are preventing the gp from expiring.
> > > 
> > > By default, as a policy decision, disable the expediting of grace
> > > periods (after boot) on configurations which enable PREEMPT_RT_FULL.
> > > 
> > > Suggested-by: Luiz Capitulino <lcapitulino@redhat.com>
> > > Signed-off-by: Julia Cartwright <julia@ni.com>
> > > Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
> > 
> > In case it matters:
> > 
> > Acked-by: Paul E. McKenney <paulmck@linux.ibm.com>
> > 
> > Alternatively, any reason that I should not pull this into -rcu?

> because we don't have CONFIG_PREEMPT_RT_FULL yet.

Fair enough.  This particular case works without CONFIG_PREEMPT_RT_FULL
defined, but up to you guys.

							Thanx, Paul


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2018-11-08 16:12 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-11-01 23:02 rcu: enable rcu_normal_after_boot by default for RT Paul E. McKenney
2018-11-08 15:52 ` Sebastian Andrzej Siewior
2018-11-08 16:12   ` Paul E. McKenney

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).