Messages in this thread | | | Date | Tue, 17 Jul 2018 13:15:00 -0700 | From | Joel Fernandes <> | Subject | Re: [PATCH v10 2/3] tracepoint: Make rcuidle tracepoint callers use SRCU |
| |
On Tue, Jul 17, 2018 at 08:25:28PM +0200, Peter Zijlstra wrote: > On Fri, Jul 13, 2018 at 02:55:46PM -0700, Joel Fernandes wrote: > > + /* keep srcu and sched-rcu usage consistent */ \ > > + preempt_disable_notrace(); \ > > + \ > > + /* \ > > + * For rcuidle callers, use srcu since sched-rcu \ > > + * doesn't work from the idle path. \ > > + */ \ > > + if (rcuidle) \ > > + idx = srcu_read_lock_notrace(&tracepoint_srcu); \ > > + else \ > > + rcu_read_lock_sched_notrace(); \ > > That else is completely superfluous. rcu_read_lock_sched_notrace() := > prempt_disable_notrace().
Yes, sorry I missed this in your original idea. Will do it this way and repost. Thanks Peter!
- Joel
| |