Re: [PATCH 0/3] ftrace, orc, x86, tracing: Fix stack traces again

From: Ingo Molnar
Date: Wed Jan 24 2018 - 02:47:03 EST



* Steven Rostedt <rostedt@xxxxxxxxxxx> wrote:

>
> With the new ORC unwinder, ftrace stack tracing became disfunctional.
>
> One was that ORC didn't know how to handle the ftrace callbacks in
> general (which Josh fixed). The other was that ORC would just bail
> if it hit a dynamically allocated trampoline. I added a check to
> the ORC unwinder to see if the trampoline belonged to ftrace, and
> if it did, use the orc entry of the static trampoline that was used
> to create the dynamic one (it would be identical).
>
> Finally, I noticed that the skip values of the stack tracing is out
> of whack. I went through and fixed them.
>
> Anyone have any issues with these patches? I'm starting my tests on
> them now and if all goes well, I plan on pushing them to Linus
> hopefully tonight.

No fundamental objections from me, assuming they are well tested.

Thanks,

Ingo