lkml.org 
[lkml]   [2018]   [Aug]   [29]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH] locking/lockdep: Delete unnecesary #include
From
Date
On Wed, 2018-08-29 at 09:49 -0400, Steven Rostedt wrote:
> On Wed, 29 Aug 2018 11:31:21 +0200
> Peter Zijlstra <peterz@infradead.org> wrote:
>
> > On Tue, Aug 28, 2018 at 09:33:15PM +0100, Ben Hutchings wrote:
> > > Commit c3bc8fd637a9 ("tracing: Centralize preemptirq tracepoints and u
> > > nify their usage") added the inclusion of <trace/events/preemptirq.h>.
> > > liblockdep doesn't have a stub version of that header so now fails to
> > > build.
> > >
> > > However, commit bff1b208a5d1 ("tracing: Partial revert of "tracing:
> > > Centralize preemptirq tracepoints and unify their usage"") removed the
> > > use of functions declared in that header. So delete the #include.
> > >
> > > Fixes: c3bc8fd637a9 ("tracing: Centralize preemptirq tracepoints ...")
> > > Fixes: bff1b208a5d1 ("tracing: Partial revert of "tracing: Centralize ...")
> >
> > There's no actual breakage because of this extra include, is there?
>
> I believe the user tool in tools/lib/lockdep fails because of it.
> Although, I just tried building it myself, and it fails because of
> nmi.h header file.

Sasha already sent fixes to fix that and another missing definition,
but they haven't been applied.

Ben.

--
Ben Hutchings
For every complex problem
there is a solution that is simple, neat, and wrong.


[unhandled content-type:application/pgp-signature]
\
 
 \ /
  Last update: 2018-08-29 20:31    [W:0.055 / U:1.636 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site