Re: [PATCH] perf script: Show correct offsets for DWARF-based unwinding

[Jiri Olsa]

On Tue, Jul 03, 2018 at 05:35:55PM +0530, Sandipan Das wrote:

SNIP

> 
> After:
> 
>   # perf report --stdio --no-children -s sym,srcline -g address
> 
>   # Samples: 1  of event 'probe_libc:inet_pton'
>   # Event count (approx.): 1
>   #
>   # Overhead  Symbol                Source:Line
>   # ........  ....................  ...........
>   #
>      100.00%  [.] __GI___inet_pton  inet_pton.c
>               |
>               ---gaih_inet.constprop.7 getaddrinfo.c:537
>                  getaddrinfo getaddrinfo.c:2304
>                  main ping.c:519
>                  generic_start_main.isra.0 libc-start.c:308
>                  __libc_start_main libc-start.c:102
>   ...
> 
>   # perf script -F comm,ip,sym,symoff,srcline,dso
> 
>   ping
>               7fffb38aaf28 __GI___inet_pton+0x8 (/usr/lib64/libc-2.26.so)
>     inet_pton.c:68
>               7fffb385fa53 gaih_inet.constprop.7+0xf43 (/usr/lib64/libc-2.26.so)
>     getaddrinfo.c:537
>               7fffb38605b3 getaddrinfo+0x163 (/usr/lib64/libc-2.26.so)
>     getaddrinfo.c:2304
>                  130782d6f main+0x3df (/usr/bin/ping)
>     ping.c:519
>               7fffb377369f generic_start_main.isra.0+0x13f (/usr/lib64/libc-2.26.so)
>     libc-start.c:308
>               7fffb3773897 __libc_start_main+0xb7 (/usr/lib64/libc-2.26.so)
>     libc-start.c:102
> 
> Fixes: 67540759151a ("perf unwind: Use addr_location::addr instead of ip for entries")
> Signed-off-by: Sandipan Das <sandipan@xxxxxxxxxxxxx>

looks good to me, Milian?

Acked-by: Jiri Olsa <jolsa@xxxxxxxxxx>

thanks,
jirka