[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <alpine.DEB.2.00.0904161053590.6495@gandalf.stny.rr.com>
Date: Thu, 16 Apr 2009 10:55:00 -0400 (EDT)
From: Steven Rostedt <rostedt@...dmis.org>
To: Zhaolei <zhaolei@...fujitsu.com>
cc: linux-kernel@...r.kernel.org
Subject: Re: [PATCH 2/2] ftrace: Code cleanup for
kernel/trace/trace_events_stage_*.h
On Mon, 6 Apr 2009, Steven Rostedt wrote:
>
> On Fri, 3 Apr 2009, Zhaolei wrote:
>
> > Add TRACE_FORMAT's define for trace_events_stage_2.h.
> > Although it is already defined in trace_events_stage_1.h, we should make each
> > function independence.
> >
> > Move TP_fast_assign's define from trace_events_stage_2.h to
> > trace_events_stage_3.h because it is used there.
> >
> > Unify TRACE_EVENT's 5th argument's name to "assign"
> >
> > Impact: cleanup, no functionality changed
> >
> > Signed-off-by: Zhao Lei <zhaolei@...fujitsu.com>
>
> I see what you are doing here, but I'm a little hesitant to apply it.
> I'm getting ready to travel, so I do not have the time to look deeper at
> this today. I'll try to do it while I'm traveling.
Hi Zhao,
As you probably noticed, the staging code has been combined into
include/trace/ftrace.h
Is this patch (or a variant) still needed?
-- Steve
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists