diff options
author | Steven Rostedt <srostedt@redhat.com> | 2012-04-30 09:17:03 -0400 |
---|---|---|
committer | Steven Rostedt <rostedt@goodmis.org> | 2012-05-16 20:00:29 -0400 |
commit | b732d439cb43336cd6d7e804ecb2c81193ef63b0 (patch) | |
tree | 8c89aec44ef3fbc6ff3e3afbf74ca5ded1354bc1 /tools/perf/builtin-report.c | |
parent | e4f5d5440bb860a3e8942ca8f7277a7f31798965 (diff) | |
download | blackbird-op-linux-b732d439cb43336cd6d7e804ecb2c81193ef63b0.tar.gz blackbird-op-linux-b732d439cb43336cd6d7e804ecb2c81193ef63b0.zip |
ftrace: Remove selecting FRAME_POINTER with FUNCTION_TRACER
The function tracer will enable the -pg option with gcc, which requires
that frame pointers. When FRAME_POINTER is defined in the kernel config
it adds the gcc option -fno-omit-frame-pointer which causes some problems
on some architectures. For those architectures, the FRAME_POINTER select
was not set.
When FUNCTION_TRACER was selected on these architectures that can not have
-fno-omit-frame-pointer, the -pg option is still set. But when
FRAME_POINTER is not selected, the kernel config would add the gcc option
-fomit-frame-pointer. Adding this option is incompatible with -pg
even on archs that do not need frame pointers with -pg.
The answer to this was to just not add either -fno-omit-frame-pointer
or -fomit-frame-pointer on these archs that want function tracing
but do not set FRAME_POINTER.
As it turns out, for archs that require frame pointers for function
tracing, the same can be used. If gcc requires frame pointers with
-pg, it will simply add it. The best thing to do is not select FRAME_POINTER
when function tracing is selected, and let gcc add it if needed.
Only add the -fno-omit-frame-pointer when something else selects
FRAME_POINTER, but do not add -fomit-frame-pointer if function tracing
is selected.
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
Diffstat (limited to 'tools/perf/builtin-report.c')
0 files changed, 0 insertions, 0 deletions