linux-kernel - Re: [PATCHSET 00/21] perf tools: Add support to accumulate hist periods (v9)

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Tue, 1 Apr 2014 07:28:06 +0000
From:	Namhyung Kim <namhyung@...nel.org>
To:	Arun Sharma <asharma@...com>
Cc:	Arnaldo Carvalho de Melo <acme@...nel.org>,
	Peter Zijlstra <a.p.zijlstra@...llo.nl>,
	Ingo Molnar <mingo@...nel.org>,
	Paul Mackerras <paulus@...ba.org>,
	Namhyung Kim <namhyung.kim@....com>,
	LKML <linux-kernel@...r.kernel.org>,
	Jiri Olsa <jolsa@...hat.com>, David Ahern <dsahern@...il.com>,
	Frederic Weisbecker <fweisbec@...il.com>,
	Andi Kleen <andi@...stfloor.org>,
	Rodrigo Campos <rodrigo@...g.com.ar>
Subject: Re: [PATCHSET 00/21] perf tools: Add support to accumulate hist
 periods (v9)

Hi Arun,

On Mon, Mar 31, 2014 at 2:26 PM, Arun Sharma <asharma@...com> wrote:
> On 3/20/14, 11:06 AM, Namhyung Kim wrote:
>>
>> Hello,
>>
>> This is a new attempt to implement cumulative hist period report.
>> This work begins from Arun's SORT_INCLUSIVE patch [1] but I completely
>> rewrote it from scratch.
>
>
> While testing this patch series, we found error messages which look like
> this:
>
> Out of bounds address found:
>
> Addr:   10370
> DSO:    /usr/local/lib/libgcc_s.so.1 d
> Map:    7f1b0c953000-7f1b0c968000
> Symbol: 102d0-102e9 g _Unwind_DeleteException
> Arch:   x86_64
> Kernel: 3.10.23+
> Tools:  3.13.rc1.g374a4d
>
> Not all samples will be on the annotation output.
>
> Please report to linux-kernel@...r.kernel.org

Hmm.. interesting.  is it perf top right?

>
> I first suspected it to be caused by this patch series, but I'm able to
> reproduce without these patches as of this commit:
>
> a51e87c perf tools: Remove unused simple_strtoul() function

Ah, it's good to know :)

>
> gdb attributes 0x10370 to a different/known symbol.
>
> (gdb) x /i 0x10370
>    0x10370 <get_cie_encoding+160>:      cmp    $0x4c,%dl
>
> Is this known? Could this possibly be caused by stale histogram entries from
> unmapped/remapped shared libs?

Possibly.

Anyway the addr which perf reported is a mapped address so that it's
pointless to use the addr directly - it's 7f1b0c963370 in fact.

What was the exact command line though - did you use any filter
(--comms, --dsos, --symbols) or event modifiers?  Those are another
possible culprits since map searching code touched by recent changes.

I'm not able to reproduce the problem on my machine.  It'd be great if
you could bisect or let me know how to reproduce it easily.

Thanks,
Namhyung
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/