Re: [PATCH resend] tracing/events: convert NAPI's tracepoint viaTRACE_EVENT

From: Ingo Molnar
Date: Fri Sep 04 2009 - 03:07:00 EST

* Xiao Guangrong <xiaoguangrong@xxxxxxxxxxxxxx> wrote:

> David Miller wrote:
> > 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.
> >
> This patch cooperate with my previous patch to solve the include
> file dependencies problem. So, we do better put those two patches
> together. (both in -tip tree or/add both in network tree)

I would not mind a duplicate commit here. Whoever merges later in
the merge window will resolve the (easy) conflict.

David, you definitely dont want to pull the full tracing tree into
the networking tree. It wouldnt cause technical problems in
linux-next (Git sorts out such merges just fine and both trees are
well tested) but it creates an unwanted merge window dependency: if
the tracing tree's push to Linus is delayed for whatever reason the
networking tree is delayed too. That's not really good.

It's also not good on a conceptual angle - it makes it very
difficult for Linus to reject any of the trees, due to the mingling.

A second variant would be that we could create a mini-topic (.31-rc8
based) in -tip with just the skb bits and the header file changes
and once we've tested it we could push it and you could pull that
(small, -git based) tree into the networking tree.

A third variant would be that you could do such a mini-topic branch
yourself in the networking tree and we could pull that into the
tracing tree.

All of these variants work fine with Steve's and my workload, it's
up to you and Neil - you are driving the changes so do whatever is
most convenient to you.

To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at
Please read the FAQ at