Re: [PATCH] rcu: Remove superfluous versions of rcu_read_lock_sched_held()

2016-03-23 Thread Boqun Feng
On Wed, Mar 23, 2016 at 09:38:46AM -0700, Paul E. McKenney wrote: > On Wed, Mar 23, 2016 at 11:11:48PM +0800, Boqun Feng wrote: > > Currently, we have four versions of rcu_read_lock_sched_held(), > > depending on the combined choices on PREEMPT_COUNT and DEBUG_LOCK_ALLOC. > > But we actually don't

Re: [PATCH] rcu: Remove superfluous versions of rcu_read_lock_sched_held()

2016-03-23 Thread Paul E. McKenney
On Wed, Mar 23, 2016 at 11:11:48PM +0800, Boqun Feng wrote: > Currently, we have four versions of rcu_read_lock_sched_held(), > depending on the combined choices on PREEMPT_COUNT and DEBUG_LOCK_ALLOC. > But we actually don't need to specialize those for PREEMPT_COUNT=n > kernel. Because: > > 1.

[PATCH] rcu: Remove superfluous versions of rcu_read_lock_sched_held()

2016-03-23 Thread Boqun Feng
Currently, we have four versions of rcu_read_lock_sched_held(), depending on the combined choices on PREEMPT_COUNT and DEBUG_LOCK_ALLOC. But we actually don't need to specialize those for PREEMPT_COUNT=n kernel. Because: 1. For the implementations in DEBUG_LOCK_ALLOC=n kernel, we can use