Re: [PATCH] kretprobe: produce sane stack traces

From: Steven Rostedt
Date: Thu Nov 01 2018 - 09:22:50 EST


On Thu, 1 Nov 2018 21:49:48 +1100
Aleksa Sarai <cyphar@xxxxxxxxxx> wrote:


> > > 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.

Agreed.

>
> Thanks, I just sent a v3.

Thanks. Hopefully I can take a look at it today or tomorrow. I'm still
trying to get home (sitting at yet another airport as I write this).

>
> Though, even with Steven's hooking of kprobes I think you'd still need
> to stash away the stack trace somewhere (or am I misunderstanding the
> proposal?).

I'll start working on it and then we can see what is needed when there
is actual code. I need to probably change the func graph generic
interface which will require changes in all the archs that support
function graph tracer.

-- Steve