Re: [PATCH] perf report: fix off-by-one for non-activation frames

From: Namhyung Kim
Date: Mon May 15 2017 - 21:58:07 EST


On Mon, May 15, 2017 at 05:13:06PM +0200, Milian Wolff wrote:
> On Monday, May 15, 2017 5:04:44 PM CEST Milian Wolff wrote:
> > As the documentation for dwfl_frame_pc says, frames that
> > are no activation frames need to have their program counter
> > decremented by one to properly find the function of the caller.
>
> Note that this leaves the perf build against libunwind in the current, broken
> state. I do not know how to detect the activation property there. Does anyone
> else? See elfutils source code for what it is doing:
>
> https://sourceware.org/git/?p=elfutils.git;a=blob;f=libdwfl/
> dwfl_frame_pc.c;h=296c815b9c73f42d79ac1778d2a0c420b89ee4eb;hb=HEAD

It seems that you can use unw_is_signal_frame().

Thanks,
Namhyung