diff options
author | Ingo Molnar <mingo@kernel.org> | 2013-09-29 16:12:54 +0200 |
---|---|---|
committer | Ingo Molnar <mingo@kernel.org> | 2013-09-29 16:12:54 +0200 |
commit | 14951f22f1cc8375ead345d2ca08455e91f2152b (patch) | |
tree | d17141520f8e8a107355d24799ff79a4e1372277 /kernel/futex_compat.c | |
parent | 9b565a8051b389e046209a5f68c93eade8de58bd (diff) | |
download | op-kernel-dev-14951f22f1cc8375ead345d2ca08455e91f2152b.zip op-kernel-dev-14951f22f1cc8375ead345d2ca08455e91f2152b.tar.gz |
Revert "perf symbols: Demangle cloned functions"
This reverts commit de95ab53645a2f0015e0f68ee723f18dce2b8b51.
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>
Diffstat (limited to 'kernel/futex_compat.c')
0 files changed, 0 insertions, 0 deletions