Re: [Patch v2 0/4] tracing: trivial cleanup

From: Steven Rostedt
Date: Mon Sep 14 2020 - 18:55:33 EST


On Fri, 28 Aug 2020 11:42:57 +0800
Wei Yang <richard.weiyang@xxxxxxxxxxxxxxxxx> wrote:

> Steven,
>
> Would you like to pick this up?
>

Hmm, patch 1 and 2 have been accepted (different subjects though):

746cf3459f11859 ("tracing: Simplify defining of the next event id")
36b8aacf2a483ba ("tracing: Save one trace_event->type by using __TRACE_LAST_TYPE")

I'm not sure why I didn't pick up patches 3 and 4. I'm looking into that
now.

-- Steve


> On Sun, Jul 12, 2020 at 09:10:32AM +0800, Wei Yang wrote:
> >Some trivial cleanup for tracing.
> >
> >v2:
> > * drop patch 1
> > * merge patch 4 & 5
> > * introduce a new patch change the return value of tracing_init_dentry()
> >
> >Wei Yang (4):
> > tracing: simplify the logic by defining next to be "lasst + 1"
> > tracing: save one trace_event->type by using __TRACE_LAST_TYPE
> > tracing: toplevel d_entry already initialized
> > tracing: make tracing_init_dentry() returns an integer instead of a
> > d_entry pointer
> >
> > kernel/trace/trace.c | 36 ++++++++++++++--------------
> > kernel/trace/trace.h | 2 +-
> > kernel/trace/trace_dynevent.c | 8 +++----
> > kernel/trace/trace_events.c | 9 ++-----
> > kernel/trace/trace_events_synth.c | 9 +++----
> > kernel/trace/trace_functions_graph.c | 8 +++----
> > kernel/trace/trace_hwlat.c | 8 +++----
> > kernel/trace/trace_kprobe.c | 10 ++++----
> > kernel/trace/trace_output.c | 14 +++++------
> > kernel/trace/trace_printk.c | 8 +++----
> > kernel/trace/trace_stack.c | 12 +++++-----
> > kernel/trace/trace_stat.c | 8 +++----
> > kernel/trace/trace_uprobe.c | 9 ++++---
> > 13 files changed, 66 insertions(+), 75 deletions(-)
> >
> >--
> >2.20.1 (Apple Git-117)
>