Re: [PATCH] kretprobe: produce sane stack traces

From: Masami Hiramatsu
Date: Thu Nov 01 2018 - 06:13:55 EST


On Thu, 1 Nov 2018 00:39:12 +1100
Aleksa Sarai <cyphar@xxxxxxxxxx> wrote:

> On 2018-10-31, Steven Rostedt <rostedt@xxxxxxxxxxx> wrote:
> > > Anyway, until that merge happens, this patch looks good to avoid
> > > this issue for generic solution (e.g. for the arch which doesn't
> > > supports retstack).
> >
> > I think its time to come up with an algorithm that makes function graph
> > work with multiple users, and have kretprobes be able to hook into it
> > just like kprobes hooks into function tracer.
> >
> > I have some ideas on how to get this done, and will try to have an RFC
> > patch set ready by plumbers.
>
> Should I continue working on this patchset?

Yes, until we finally introduce Steven's algorithm on all arch (yeah, we still
have some archs which don't support graph-tracer but supports kprobes),
I think your patch is the best fix for this issue.

Thank you,

--
Masami Hiramatsu <mhiramat@xxxxxxxxxx>