Re: [PATCH] smp: generic ipi_raise tracepoint

From: Peter Zijlstra
Date: Wed May 20 2020 - 09:52:02 EST


On Wed, May 20, 2020 at 02:42:10PM +0100, Wojciech Kudla wrote:
> On 20/05/2020 14:33, Peter Zijlstra wrote:
> > We have bright shiny links like: https://lkml.kernel.org/r/$MSG-ID for
> > that. they allow me to go find the email in my local archive without
> > having to use a browser.
>
> Apologies, beginner's mistake.
>
> >> +static const char *ipi_reason_missing __tracepoint_string = "";
> >
> > That is a pretty crap reason ;-)
> >
>
> I knew this was a long shot. There is no obvious way to
> get/infer ipi reason in generic smp code at the moment.
> Any suggestions what we can put here in the meantime?
> Would "none" be more appropriate?

Depends a bit on what you actually want to achieve here.

For actual proper debugging this stuff I think I'd find it more useful
to have tracepoints one level up. So instead of tracing the IPIs, trace
the actual remote function call requests.

And in that case, trace the mask and the symbol name from csd->func.

And if you have that, who cares about the actual IPIs.