Skip to content

tracing: Minor fixes and clean ups

Steven Rostedt lkml stable 1 version New

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest5 patches 0 repliesNew

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/5samples: ftrace: Save required argument registers in sample trampolinessamples/ftrace/ftrace-direct-too.c+8 -60 replies
2/5arm64: ftrace: Add direct call trampoline samples supportarch/arm64/Kconfig samples/ftrace/ftrace-direct-modify.c samples/ftrace/ftrace-direct-multi-modify.c +3 more+151 -00 replies
3/5x86/ftrace: Remove unsued extern declaration ftrace_regs_caller_ret()arch/x86/kernel/ftrace.c+0 -10 replies
4/5tracing: arm64: Avoid missing-prototype warningsarch/arm64/include/asm/ftrace.h arch/arm64/include/asm/syscall.h arch/arm64/kernel/syscall.c +4 more+22 -50 replies
5/5tracing/user_events: Fix struct arg size match checkkernel/trace/trace_events_user.c+3 -00 replies

Reviews trailers per patch

4 of 5 patches have a Reviewed-by or Acked-by.

PatchSteven Rostedt (Google)Mark RutlandCatalin MarinasMasami Hiramatsu (Google)
1/5R-b R-b A-b
2/5R-b T-b A-b
3/5A-b
4/5A-b
5/5

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes. Scroll the table sideways for all 4 reviewers.

Read the v1 thread

Cover letter [for-linus][PATCH 0/5] tracing: Minor fixes and clean ups

Tracing fixes and clean ups:

- Fix some missing-prototype warnings

- Fix user events struct args (did not include size of struct) When creating a user event, the "struct" keyword is to denote that the size of the field will be passed in. But the parsing failed to handle this case.

- Fix sample code for direct trampolines.
The sample code for direct trampolines attached to handle_mm_fault(). But the prototype changed and the direct trampoline sample code was not updated. Direct trampolines needs to have the arguments correct otherwise it can fail or crash the system.

- Remove unused ftrace_regs_caller_ret() prototype.

git://git.kernel.org/pub/scm/linux/kernel/git/trace/linux-trace.git
trace/core

Head SHA1: 46b77ea28faeed56f1d1b63f9049c40b9eb3cbf5

Arnd Bergmann (1):
tracing: arm64: Avoid missing-prototype warnings

Beau Belgrave (1):
tracing/user_events: Fix struct arg size match check

Florent Revest (2):
samples: ftrace: Save required argument registers in sample trampolines
arm64: ftrace: Add direct call trampoline samples support

YueHaibing (1):
x86/ftrace: Remove unsued extern declaration ftrace_regs_caller_ret()

----

 arch/arm64/Kconfig                          |  2 ++
 arch/arm64/include/asm/ftrace.h             |  4 +++
 arch/arm64/include/asm/syscall.h            |  3 +++
 arch/arm64/kernel/syscall.c                 |  3 ---
 arch/x86/kernel/ftrace.c                    |  1 -
 include/linux/ftrace.h                      |  8 ++++++
 kernel/trace/fgraph.c                       |  1 +
 kernel/trace/ftrace_internal.h              |  5 ++--
 kernel/trace/trace_events_user.c            |  3 +++
 kernel/trace/trace_kprobe_selftest.c        |  3 +++
 samples/ftrace/ftrace-direct-modify.c       | 34 ++++++++++++++++++++++++
 samples/ftrace/ftrace-direct-multi-modify.c | 40 +++++++++++++++++++++++++++++
 samples/ftrace/ftrace-direct-multi.c        | 25 ++++++++++++++++++
 samples/ftrace/ftrace-direct-too.c          | 40 ++++++++++++++++++++++++-----
 samples/ftrace/ftrace-direct.c              | 24 +++++++++++++++++
 15 files changed, 184 insertions(+), 12 deletions(-)