Re: [PATCH v3 2/4] tracing: Make syscall_(un)regfunc arch-specific

[Ingo Molnar]

* Paul Mundt <lethal@xxxxxxxxxxxx> wrote:

> On Mon, Aug 24, 2009 at 01:00:40PM +0200, Ingo Molnar wrote:
>
> > And the thing is, s390 was always a pleasure to deal with. I 
> > cannot say the same about anything SH. SH always was a huge PITA 
> > to deal with (to me at least), mainly because you seem to have a 
> > false sense of entitlement: you think everyone else must be 
> > perfect and must not affect you while you can sit on your own 
> > little island not worrying about the rest of the world - who 
> > develops and tests core kernel facilities for you. It doesnt 
> > work like that.
> 
> If you had bothered to read any of my earlier mail you would 
> realize that this is complete and utter nonsense.

first you say it's nonsense, then you basically back up my points:

> [...] I expect there to be breakage, and I expect that I will be 
> the one fixing it up (even when most of the time that breakage is 
> not even in my own tree). Unsurprisingly, this is largely a result 
> of SH being one of the first architectures testing these new core 
> kernel facilities, which is something you seem intent on just 
> writing off completely. [...]

... the thing is, you didnt Cc: any of the tracing commits in the SH 
tree either to lkml or to the tracing folks who are developing those 
facilities:

  c652d78: sh: Add ftrace syscall tracing support

  From: Matt Fleming <matt@xxxxxxxxxxxxxxxxx>
  Date: Mon, 6 Jul 2009 20:16:33 +0900
  Subject: [PATCH] sh: Add ftrace syscall tracing support

  [...]

  Signed-off-by: Matt Fleming <matt@xxxxxxxxxxxxxxxxx>
  Signed-off-by: Paul Mundt <lethal@xxxxxxxxxxxx>

That 'could' have been Cc:-ed to Frederic and Steve, perhaps Jason 
and me as well.

You basically treat SH as your own private tree, with little need to 
communicate with anyone outside the SH hackers. Which is fine, as 
long as you are willing to deal with the other side of the coin: 
that others wont be aware of your changes.

Or, at minimum, to not scream loudly when the flip side beats you.

	Ingo
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/