lkml.org 
[lkml]   [2015]   [Nov]   [4]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: perf related lockdep bug
On Wed, Nov 04, 2015 at 04:36:51PM +0100, Peter Zijlstra wrote:
> On Wed, Nov 04, 2015 at 03:20:58PM +0100, Peter Zijlstra wrote:
> > On Wed, Nov 04, 2015 at 05:48:38AM -0800, Paul E. McKenney wrote:
> > > This problem is caused by the IPI handler interrupting the RCU read-side
> > > critical section. One way to prevent the IPI from doing this is to
> > > disable interrupts across the RCU read-side critical section instead
> > > of merely disabling preemption. This is a reasonable approach given
> > > that acquiring the scheduler locks is going to disable interrupts
> > > in any case.
> > >
> > > The (untested) patch below takes this approach.
> > >
> > > Thoughts?
> >
> > Yes, this should work, but now I worry I need to go audit all of perf
> > and sched for this :/
>
> I can't find any other sites just now, so let me queue this.

Works for me, thank you! It passes light rcutorture testing, but then
again so did the version without this commit. :-/

I will queue the needed documentation updates separately. I don't see
these as urgent, so probably next merge window.

> I also had a brief look if you used any other locks under rnp->lock, but
> aside from the printk and sched things it seems clean.

Whew! ;-)

Thanx, Paul



\
 
 \ /
  Last update: 2015-11-04 17:01    [W:0.084 / U:0.040 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site