diff options
| author | Steven Rostedt (Red Hat) <rostedt@goodmis.org> | 2015-07-16 19:24:54 +0200 |
|---|---|---|
| committer | Steven Rostedt <rostedt@goodmis.org> | 2015-07-21 04:30:50 +0200 |
| commit | 72ac426a5bb0cec572d26b4456f8c1e14601694e (patch) | |
| tree | 637bfb15ecc273161b4a5d49e6abc7b136fca6ac /include | |
| parent | ring-buffer: Reorganize function locations (diff) | |
| download | linux-72ac426a5bb0cec572d26b4456f8c1e14601694e.tar.xz linux-72ac426a5bb0cec572d26b4456f8c1e14601694e.zip | |
tracing: Clean up stack tracing and fix fentry updates
Akashi Takahiro was porting the stack tracer to arm64 and found some
issues with it. One was that it repeats the top function, due to the
stack frame added by the mcount caller and added by itself. This
was added when fentry came in, and before fentry created its own stack
frame. But x86's fentry now creates its own stack frame, and there's
no need to insert the function again.
This also cleans up the code a bit, where it doesn't need to do something
special for fentry, and doesn't include insertion of a duplicate
entry for the called function being traced.
Link: http://lkml.kernel.org/r/55A646EE.6030402@linaro.org
Some-suggestions-by: Jungseok Lee <jungseoklee85@gmail.com>
Some-suggestions-by: Mark Rutland <mark.rutland@arm.com>
Reported-by: AKASHI Takahiro <takahiro.akashi@linaro.org>
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
Diffstat (limited to 'include')
0 files changed, 0 insertions, 0 deletions