Re: [PATCH for 2.6.25] Markers - use synchronize_sched()

Paul E. McKenney
2008-04-01T20:46:16Z

On Tue, Apr 01, 2008 at 01:30:17PM -0700, Andrew Morton wrote:
> On Mon, 31 Mar 2008 09:16:09 -0400
> Mathieu Desnoyers <mathieu.desnoyers@polymtl.ca> wrote:
>
> > Use synchronize_sched before calling call_rcu in CONFIG_PREEMPT_RCU until we
> > have call_rcu_sched and rcu_barrier_sched in mainline. It will slow down the
> > marker operations in CONFIG_PREEMPT_RCU, but it fixes the current race against
> > the preempt_disable/enable() protected code paths.
>
> A better changelog would have described the bug which is being fixed.
>
> > Paul, is this ok ? It would be good to get this in for 2.6.25 final.
>
> Paul seems to have nodded off. I'll merge it.

Paul seems to have fat-fingered the earlier message. But he does see the
message adding this to -mm, and the patch looks good to me.

Thanx, Paul