]> git.baikalelectronics.ru Git - kernel.git/commit
Revert "perf symbols: Demangle cloned functions"
authorIngo Molnar <mingo@kernel.org>
Sun, 29 Sep 2013 14:12:54 +0000 (16:12 +0200)
committerIngo Molnar <mingo@kernel.org>
Sun, 29 Sep 2013 14:12:54 +0000 (16:12 +0200)
commitf408ef6ce911ee1d7e2dd1c58c1a672282026d69
treed17141520f8e8a107355d24799ff79a4e1372277
parentdee2ffaadb7eb252147b3dafd0850523736ddb0f
Revert "perf symbols: Demangle cloned functions"

This reverts commit a7d900f10dbeed430b23453601ca2b048e4583c4.

Markus Trippelsdorf reported that this commit broke 'perf top':

 > I just see a gray screen with no text at all. Sometimes the
 > following error messages are printed:
 >
 >  *** Error in `perf': invalid fastbin entry (free): 0x00000000029b18c0
 >  ***
 >  *** Error in `perf': malloc(): memory corruption (fast): 0x0000000000ee0b10 ***

While this code is fixable, the commit itself fails on several levels:

 - it should have been a separate helper function
 - why the heck does it do strchr() twice
 - it casts a const char * over into char *
 - sloppy style
 - it's not even a regression fix!

So lets revert it and re-try the patch in v3.13.

Reported-by: Markus Trippelsdorf <markus@trippelsdorf.de>
Cc: Andi Kleen <ak@linux.intel.com>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Cc: Arnaldo Carvalho de Melo <acme@redhat.com>
Cc: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Ingo Molnar <mingo@kernel.org>
tools/perf/util/symbol-elf.c