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

From: Ingo Molnar
Date: Mon Aug 24 2009 - 07:01:14 EST

* Paul Mundt <lethal@xxxxxxxxxxxx> wrote:

> On Mon, Aug 24, 2009 at 11:56:52AM +0200, Ingo Molnar wrote:
> > * Paul Mundt <lethal@xxxxxxxxxxxx> wrote:
> > > I think this is the source of confusion, given that I expected the
> > > interface would remain reasonable stable until all of the existing
> > > users had had a chance to catch up. [...]
> >
> > All existing upstream users of APIs were fixed up. Which was x86 and
> > s390. (and yes, we initially missed s390)
> >
> > Stable APIs are not how upstream Linux kernel development works - it
> > would be way too inflexible.
> >
> > Instead facility trees (such as the tracing tree) are either
> > expected to convert all upstream API uses (which we did), or are
> > supposed to provide different versions of APIs, if an API is so
> > widespread (used by hundreds of callsites, etc.) that it's not
> > reasonable to convert it in one go.
> No Ingo, this is not what you did, this is what you did after the
> breakage was pointed out to you. [...]

As i explained it to you back then the s390 support was 1) a few
weeks old 2) got masked due to an unrelated thing. In any case the
bug was real and it was fixed very quickly within an hour or so of
the report in the usual workflow and without any argument about it,
and in total agreement with the s390 folks.

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.

> [...] And before that, you had:
> commit 60d970c254b95ec7a0fc4c590b510253987b64a0
> Author: Ingo Molnar <mingo@xxxxxxx>
> Date: Thu Aug 13 23:37:26 2009 +0200
> tracing: Fix syscall tracing on !HAVE_FTRACE_SYSCALLS architectures
> The new syscall_regfunc()/unregfunc() functions rely on
> the existence of TIF_SYSCALL_FTRACE - but that TIF flag
> is only offered by HAVE_FTRACE_SYSCALLS.
> Cc: Frederic Weisbecker <fweisbec@xxxxxxxxx>
> Cc: Jason Baron <jbaron@xxxxxxxxxx>
> Cc: Steven Rostedt <rostedt@xxxxxxxxxxx>
> Cc: Peter Zijlstra <peterz@xxxxxxxxxxxxx>
> LKML-Reference: <new-submission>
> Signed-off-by: Ingo Molnar <mingo@xxxxxxx>
> Which you at least managed to catch before it went in to -next.

See that LKML-Reference tag? That patch too went out to lkml. Had
you chosen to participate instead of just complaining you could

Then you intentionally chose the wrong workflow and now you are
complaining 1) about an easily fixed bug that got addressed within
an hour of it reported 2) about a workflow situation you yourself
created. [and to which i suggested several alternatives how to fix

You are not being constructive at all.

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