Skip to content

uprobes: kill uprobe_trace_consumer and other cleanups

Oleg Nesterov lkml 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/5uprobes: Ensure inode != NULL in create_trace_uprobe()kernel/trace/trace_uprobe.c+4 -20 replies
2/5uprobes: Introduce trace_uprobe->enabledkernel/trace/trace_uprobe.c+6 -20 replies
3/5uprobes: Kill uprobe_trace_consumer, embed uprobe_consumer into trace_uprobekernel/trace/trace_uprobe.c+11 -330 replies
4/5uprobes: Kill set_trace_uprobe_filter_func()kernel/trace/trace_uprobe.c+45 -520 replies
5/5uprobes: Kill trace_uprobe->enabled, introduce is_trace_uprobe_enabled()kernel/trace/trace_uprobe.c+7 -60 replies

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.

Read the v1 thread

Cover letter [PATCH 0/5] uprobes: kill uprobe_trace_consumer and other cleanups

Hello.

On top of
"[PATCH 0/4] uprobes: Teach debug/tracing/uprobe_events to do the filtering" I sent yesterday.

I tried to test that series, seems to work...

This one tries to cleanup the code. No functional changes. Almost untested so far, but please review.

Since I had to look at this code, I simply can't resist. uprobe_trace_consumer complicates the code for no reason and imho should die.

Oleg.

 kernel/trace/trace_uprobe.c |  146 ++++++++++++++++++-------------------------
 1 files changed, 62 insertions(+), 84 deletions(-)