Re: [PATCH RT v2 3/3] rcu: Disable use_softirq on PREEMPT_RT

2019-08-23 Thread Sebastian Andrzej Siewior
On 2019-08-21 16:40:18 [-0700], Paul E. McKenney wrote: > Save a couple of lines? > > static bool use_softirq = !IS_ENABLED(CONFIG_PREEMPT_RT_FULL); > > And if I understand your point above, the module_param() might be > able to be the same either way given the new RCU. But if not, > at least we

Re: [PATCH RT v2 3/3] rcu: Disable use_softirq on PREEMPT_RT

2019-08-22 Thread Joel Fernandes
On Thu, Aug 22, 2019 at 02:31:17PM -0500, Scott Wood wrote: > On Thu, 2019-08-22 at 09:59 -0400, Joel Fernandes wrote: > > On Wed, Aug 21, 2019 at 06:19:06PM -0500, Scott Wood wrote: > > > I think the prohibition on use_softirq can be dropped once RT gets the > > > latest RCU code, but the question

Re: [PATCH RT v2 3/3] rcu: Disable use_softirq on PREEMPT_RT

2019-08-22 Thread Scott Wood
On Thu, 2019-08-22 at 09:59 -0400, Joel Fernandes wrote: > On Wed, Aug 21, 2019 at 06:19:06PM -0500, Scott Wood wrote: > > I think the prohibition on use_softirq can be dropped once RT gets the > > latest RCU code, but the question of what use_softirq should default > > to on PREEMPT_RT remains. >

Re: [PATCH RT v2 3/3] rcu: Disable use_softirq on PREEMPT_RT

2019-08-22 Thread Paul E. McKenney
On Thu, Aug 22, 2019 at 09:59:53AM -0400, Joel Fernandes wrote: > On Wed, Aug 21, 2019 at 06:19:06PM -0500, Scott Wood wrote: > > Besides restoring behavior that used to be default on RT, this avoids > > a deadlock on scheduler locks: > > > > [ 136.894657] 039: ===

Re: [PATCH RT v2 3/3] rcu: Disable use_softirq on PREEMPT_RT

2019-08-22 Thread Joel Fernandes
On Wed, Aug 21, 2019 at 06:19:06PM -0500, Scott Wood wrote: > Besides restoring behavior that used to be default on RT, this avoids > a deadlock on scheduler locks: > > [ 136.894657] 039: > [ 136.900401] 039: WARNING: possible recursive locking detect

Re: [PATCH RT v2 3/3] rcu: Disable use_softirq on PREEMPT_RT

2019-08-21 Thread Paul E. McKenney
On Wed, Aug 21, 2019 at 06:19:06PM -0500, Scott Wood wrote: > Besides restoring behavior that used to be default on RT, this avoids > a deadlock on scheduler locks: > > [ 136.894657] 039: > [ 136.900401] 039: WARNING: possible recursive locking detect