Messages in this thread | | | Subject | Re: [RFC][PATCH 0/4] ftrace: Use -mfentry when supported (this is for x86_64 right now) | From | Steven Rostedt <> | Date | Thu, 17 Feb 2011 10:46:31 -0500 |
| |
On Fri, 2011-02-18 at 00:34 +0900, Masami Hiramatsu wrote: > (2011/02/17 22:18), Steven Rostedt wrote: > > On Thu, 2011-02-17 at 21:37 +0900, Masami Hiramatsu wrote: > > > >> Oops! with this change, current kprobes might not be able to probe > >> the entry of functions, because that is always reserved by ftrace! > >> I think we need to have some new interface for replacing each other > >> safely... > > > > Good point. I suspect that this wont be ready till .40 anyway. When I > > get a chance to work more an this, I'll also include patches where if > > -mfentry is activated kprobes will just hook to the mcount caller > > instead. Or ftrace itself :) > > Ah, that's a good idea! :) it could be done without -mfentry too. > But is that possible to modify just one mcount entry? I also worry > about the latency of enabling/disabling one entry.
I would have it go through the ftrace function tracing facility, which would handle which entry to enable/disable. It still does stopmachine. Is that an issue to enable/disable kprobes? The "fast" enable/disable could be done by the called function to just ignore the call.
Also note, if there's other callbacks that are attached to the function being traced, no stop machine is enabled. The callbacks are just a list and as long as a function has an associated callback, no code modification needs to be done to add or remove other callbacks.
> > BTW, without dynamic ftrace (no code modifying), I think we don't > need to reserve mcount code, because no one modifies it.
Correct. And even today, you can remove any kprobe code that checks for mcount without dynamic ftrace enabled. But I'm not sure if anyone enables the function tracer without dynamic ftrace, except for debugging in archs that do not support dynamic ftrace. As the overhead of this is quite high even when function tracer is disabled.
-- Steve
| |