On Mon, Aug 11, 2014 at 03:49:03PM -0700, Paul E. McKenney wrote: > From: "Paul E. McKenney" <[email protected]> > > In theory, synchronize_sched() requires a read-side critical section to > order against. In practice, preemption can be thought of as being > disabled across every machine instruction. So this commit removes > the redundant preempt_disable() from rcu_note_voluntary_context_switch().
> #define rcu_note_voluntary_context_switch(t) \
> do { \
> - preempt_disable(); /* Exclude synchronize_sched(); */ \
> if (ACCESS_ONCE((t)->rcu_tasks_holdout)) \
> ACCESS_ONCE((t)->rcu_tasks_holdout) = 0; \
> - preempt_enable(); \
> } while (0)
But that's more than 1 instruction.
pgp6umPct2dsK.pgp
Description: PGP signature
