Messages in this thread | | | Date | Tue, 01 Sep 2009 18:26:45 -0700 (PDT) | Subject | Re: [PATCH resend] tracing/events: convert NAPI's tracepoint via TRACE_EVENT | From | David Miller <> |
| |
From: Steven Rostedt <rostedt@goodmis.org> Date: Mon, 31 Aug 2009 14:09:04 -0400 (EDT)
> > On Mon, 31 Aug 2009, Ingo Molnar wrote: > >> >> * Xiao Guangrong <xiaoguangrong@cn.fujitsu.com> wrote: >> >> > - Convert NAPI's tracepoint via TRACE_EVENT macro, the output information >> > like below: >> >> I think as long as it does not touch tracing infrastructure (which >> your patches dont do in their current form) this should be >> done/merged via the networking tree. > > I agree, all changes that are in include/trace/events/ and trace point > usage can stay within the subsystem tree. > >> >> [ There might be some small collisions in define_trace.h (because >> these tracepoints move from legacy to new-style TRACE_EVENT() >> form) but that's OK. ] > > But changes to anything in include/trace or kernel/trace needs to go > throught the tracing subsystem. This includes a changes to define_trace.h.
This patch can't be split up, so I'm wondering how you suggest to handle this patch given that you have declared that define_trace.h changes aren't to go through the subsystem tree?
If we do the define_trace.h change only, we break the build (lack of macro defined for the trace).
If we do only the other parts of his patch, we get a duplicate definition.
And keep in mind that Neil and Xiao are probably going to want to do work on top of this to the networking bits. Thus if we put this patch here into the tracing tree, I'll have to develop a dependency on the tracing tree and I think that will go over like a fart in a spacesuit with the -next crowd and Stephen Rothwell in particular.
Please advise.
| |