Skip to content

ftrace/kprobes/x86:

Steven Rostedt lkml stable 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest3 patches 7 repliesIn review

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/3ftrace: Fix updating of filters for shared global_ops filterskernel/trace/ftrace.c+25 -11 reply
2/3ftrace: Check both notrace and filter for old hashkernel/trace/ftrace.c+20 -72 replies
3/3ftrace/jprobes/x86: Fix conflict between jprobes and function graph tracingarch/x86/kernel/kprobes/core.c+15 -54 replies

Reviews trailers per patch

All 3 patches have a Reviewed-by or Acked-by.

PatchMasami Hiramatsu
1/3R-b
2/3R-b
3/3A-b

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v1 thread

Cover letter [PATCH 0/3] ftrace/kprobes/x86:

Vasily Averin reported that combining jprobes with function tracing resulted in a kernel crash.

I first thought this was related to the fentry code and wrote up a few nice (albeit complex) patches that could handle it. But then during my testing I discovered that it also affected non fentry code and i386. I had to revert all my changes and just disable function graph tracing before the jprobe is called, and re-enable it afterward. This seems to be the easiest fix, although if fentry is enabled, we lose the tracing of the function that was being probed. But since it would have crashed otherwise, this solution is a major improvement.

While debugging this I also uncovered some other bugs in the function graph trampoline logic. Nothing too major, but worthy enough to be fixed now and sent to stable.

I can still use the patches I have done for fentry to let the probed function be traced (as well as the jprobe handlers) if fentry is being used. But that is something for the next merge window and not needed now.

Steven Rostedt (Red Hat) (3):
ftrace: Fix updating of filters for shared global_ops filters
ftrace: Check both notrace and filter for old hash ftrace/jprobes/x86: Fix conflict between jprobes and function graph tracing

----

 arch/x86/kernel/kprobes/core.c | 20 ++++++++++++----
 kernel/trace/ftrace.c          | 53 +++++++++++++++++++++++++++++++++++-------
 2 files changed, 60 insertions(+), 13 deletions(-)