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  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Date:   Thu, 26 Oct 2017 09:50:13 +0100
From:   Julien Thierry <julien.thierry@....com>
To:     "linux-arm-kernel@...ts.infradead.org" 
        <linux-arm-kernel@...ts.infradead.org>,
        LKML <linux-kernel@...r.kernel.org>
Cc:     rostedt@...dmis.org, AKASHI Takahiro <takahiro.akashi@...aro.org>
Subject: arm64: ftrace: function_graph tracer not working without dynamic
 instrumentation

Hi,

On arm64, I noticed that when CONFIG_DYNAMIC_FTRACE is not enabled, 
using the function_graph tracer in ftrace alway gives empty traces.

The issues seems to be that the ftrace_trace_function is set to a 
function different than ftrace_stub when running mcount. If I allow 
mcount to continue beyond that check, I get the graph traces.

Guidelines given in Documentation/trace/ftrace-design.txt suggest that 
ftrace_trace_function != ftrace_stub should be checked before any 
function_graph related values and if the check is true then we shouldn't 
try to call ftrace_trace_function.

Investigating a bit more, ftrace_trace_function gets set 
ftrace_ops_no_ops in update_ftrace_function called when setting up the 
function_graph tracer.

The issue does not occur when dynamic ftrace is supported because we 
just replace some nops at known locations and replace them with branches.

It seemed to me that the arm64 side of things is following the 
guidelines in ftrace-design, so I am unsure whether it is intended that 
the ftrace_trace_function is not set to ftrace_stub when using the 
function_graph tracer.

Cheers,

-- 
Julien Thierry

Powered by blists - more mailing lists