Re: [PATCH v4 15/16] module: Move where we mark modules RO,X

[Steven Rostedt]

On Thu, 24 Oct 2019 22:24:55 +0200
Peter Zijlstra <peterz@xxxxxxxxxxxxx> wrote:

> On Thu, Oct 24, 2019 at 02:17:31PM -0400, Steven Rostedt wrote:
> > On Thu, 24 Oct 2019 18:43:20 +0200
> > Peter Zijlstra <peterz@xxxxxxxxxxxxx> wrote:
> >   
> > > > 
> > > >   CC [M]  drivers/gpu/drm/i915/gem/i915_gem_context.o
> > > > /work/git/linux-trace.git/kernel/trace/trace_events_hist.c: In function âregister_synth_eventâ:
> > > > /work/git/linux-trace.git/kernel/trace/trace_events_hist.c:1157:15: error: âstruct trace_event_classâ has no member named âdefine_fieldsâ; did you mean âget_fieldsâ?
> > > >   call->class->define_fields = synth_event_define_fields;
> > > >                ^~~~~~~~~~~~~
> > > >                get_fields
> > > > make[3]: *** [/work/git/linux-trace.git/scripts/Makefile.build:265: kernel/trace/trace_events_hist.o] Error 1
> > > > make[3]: *** Waiting for unfinished jobs....    
> > > 
> > > allmodconfig clean
> > > 
> > > (omg, so much __field(); fail)  
> > 
> > Well it built without warnings and passed the ftrace selftests.
> > 
> > I haven't ran it through the full suite, but that can wait for the v5.  
> 
> I'll push it out to git to the 0day robot can have a go at it. For v5
> I'm still staring at some KLP borkage. Then again, maybe I should delay
> that last bit and make that a new series.

Also note, that I'm about to travel to Lyon for Open Source Summit,
thus my looking at this will come pretty much to a stand still :-/

-- Steve