tracing/kprobes: trace_probe->files cleanups
Get series with b4
b4 am [email protected]Versions newest first
- v1 Latest3 patches 10 repliesIn review
Patches in v1 No applies recorded yet
| # | Subject and files | Size | Replies | Applied |
|---|---|---|---|---|
| 1/3 | tracing/kprobes: Avoid perf_trace_buf_*() if ->perf_events is emptykernel/ | +8 | 3 replies | |
| 2/3 | tracing/kprobes: Kill probe_enable_lockkernel/ | +10 | 9 replies | |
| 3/3 | tracing/kprobes: Turn trace_probe->files into list_headkernel/ | +37 | 2 replies |
Reviews trailers per patch
All 3 patches have a Reviewed-by or Acked-by.
| Patch | Masami Hiramatsu |
|---|---|
| 1/3 | A-b |
| 2/3 | A-b A-b |
| 3/3 | A-b |
Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.
Cover letter [PATCH 0/3] tracing/kprobes: trace_probe->files cleanups
Hello.
I simply can't understand why trace_kprobe.c uses "struct ftrace_event_file **", the simple list_head looks much more natural and simple.
And we do not want to copy-and-paste this code to trace_uprobe.c. If there is a reason for array-of-pointers we should create the helpers in the common trace_probe.c so that uprobes can use them too.
Oleg.
kernel/trace/trace_kprobe.c | 171 ++++++++++++------------------------------ 1 files changed, 49 insertions(+), 122 deletions(-)