diff options
author | android-build-team Robot <android-build-team-robot@google.com> | 2020-04-08 02:07:06 +0000 |
---|---|---|
committer | android-build-team Robot <android-build-team-robot@google.com> | 2020-04-08 02:07:06 +0000 |
commit | 24ca8d91d1164b7a36c159020c56139e00ce13a9 (patch) | |
tree | 6b6c42e543d0d9a33efdb555a234a7dd1425e4d9 | |
parent | 2ee54cd7d0ba56ffef5d12b40dcc93b5355ab051 (diff) | |
parent | 80cd87a60222701044f423f1f8e0fef8672a851c (diff) | |
download | extras-24ca8d91d1164b7a36c159020c56139e00ce13a9.tar.gz |
Snap for 6374421 from 80cd87a60222701044f423f1f8e0fef8672a851c to rvc-d1-release
Change-Id: I47fda596e7f8d31e2587671cc7a38677bf0ca157
-rw-r--r-- | simpleperf/environment.cpp | 17 |
1 files changed, 7 insertions, 10 deletions
diff --git a/simpleperf/environment.cpp b/simpleperf/environment.cpp index e16e496f..5b2fcaa8 100644 --- a/simpleperf/environment.cpp +++ b/simpleperf/environment.cpp @@ -299,18 +299,15 @@ bool CanRecordRawData() { if (IsRoot()) { return true; } - int value; - if (!ReadPerfEventParanoid(&value) || value > -1) { - return false; - } #if defined(__ANDROID__) - // If perf_event_open() is controlled by selinux, simpleperf can't record tracepoint raw data - // unless running as root. - if (android::base::GetProperty("sys.init.perf_lsm_hooks", "") == "1") { - return false; - } + // Android R uses selinux to control perf_event_open. Whether raw data can be recorded is hard + // to check unless we really try it. And probably there is no need to record raw data in non-root + // users. + return false; +#else + int value; + return ReadPerfEventParanoid(&value) && value == -1; #endif - return true; } static const char* GetLimitLevelDescription(int limit_level) { |