Re: [PATCH 2/2] ftrace: Adjust priority of ftrace module notifier

2016-01-29 Thread Josh Poimboeuf
On Fri, Jan 29, 2016 at 10:49:50AM -0500, Steven Rostedt wrote: > On Fri, 29 Jan 2016 09:45:05 -0600 > Josh Poimboeuf wrote: > > > > If we do that then I still think it would be a good idea to split up the > > livepatch notifiers, with: > > > > - INT_MAX-1 for coming so that relocations are all

Re: [PATCH 2/2] ftrace: Adjust priority of ftrace module notifier

2016-01-29 Thread Steven Rostedt
On Fri, 29 Jan 2016 09:45:05 -0600 Josh Poimboeuf wrote: > If we do that then I still think it would be a good idea to split up the > livepatch notifiers, with: > > - INT_MAX-1 for coming so that relocations are all written before any > other notifiers (besides ftrace) get a chance to run. >

Re: [PATCH 2/2] ftrace: Adjust priority of ftrace module notifier

2016-01-29 Thread Josh Poimboeuf
On Fri, Jan 29, 2016 at 09:38:00AM -0500, Steven Rostedt wrote: > On Fri, 29 Jan 2016 01:43:47 -0500 > Jessica Yu wrote: > > > > --- > > kernel/trace/ftrace.c | 7 ++- > > 1 file changed, 6 insertions(+), 1 deletion(-) > > > > diff --git a/kernel/trace/ftrace.c b/kernel/trace/ftrace.c > >

Re: [PATCH 2/2] ftrace: Adjust priority of ftrace module notifier

2016-01-29 Thread Steven Rostedt
On Fri, 29 Jan 2016 01:43:47 -0500 Jessica Yu wrote: > --- > kernel/trace/ftrace.c | 7 ++- > 1 file changed, 6 insertions(+), 1 deletion(-) > > diff --git a/kernel/trace/ftrace.c b/kernel/trace/ftrace.c > index eca592f..bdd7bfc 100644 > --- a/kernel/trace/ftrace.c > +++ b/kernel/trace/ftr

[PATCH 2/2] ftrace: Adjust priority of ftrace module notifier

2016-01-28 Thread Jessica Yu
Adjust the priority of the ftrace module notifier such that it will run before the livepatch coming module notifier but after the livepatch going module modifier. This fixes a notifier ordering issue in which the ftrace module notifier (and hence ftrace_module_enable()) for COMING modules was bein