On Tue, Feb 28, 2017 at 01:49:06PM +0100, Peter Zijlstra wrote: > On Tue, Feb 28, 2017 at 01:45:07PM +0100, Peter Zijlstra wrote: > > On Wed, Jan 18, 2017 at 10:17:32PM +0900, Byungchul Park wrote: > > > + /* > > > + * struct held_lock does not have an indicator whether in nmi. > > > + */ > > > + int nmi; > > > > Do we really need this? Lockdep doesn't really know about NMI context, > > so its weird to now partially introduce it. > > That is, see how nmi_enter() includes lockdep_off().
Indeed. OK. I will fix it.

