traced_perf: add to android build by default (initially disabled)
Will separately check whether packageinfo
group is necessary (or effectively covered by DAC_READ_SEARCH).
Bug: 144281346
Change-Id: If84d91a4c31ec04df6617da9374bf5087e991bd7
diff --git a/Android.bp b/Android.bp
index 781d421..09646d2 100644
--- a/Android.bp
+++ b/Android.bp
@@ -7525,6 +7525,9 @@
"libprocinfo",
"libunwindstack",
],
+ init_rc: [
+ "traced_perf.rc",
+ ],
generated_headers: [
"perfetto_protos_perfetto_common_cpp_gen_headers",
"perfetto_protos_perfetto_common_zero_gen_headers",
@@ -7592,6 +7595,7 @@
],
required: [
"libperfetto_android_internal",
+ "traced_perf",
"trigger_perfetto",
],
}
diff --git a/tools/gen_android_bp b/tools/gen_android_bp
index 7c1c444..59d5a11 100755
--- a/tools/gen_android_bp
+++ b/tools/gen_android_bp
@@ -84,6 +84,7 @@
target_initrc = {
'//src/traced/service:traced': {'perfetto.rc'},
'//src/profiling/memory:heapprofd': {'heapprofd.rc'},
+ '//src/profiling/perf:traced_perf': {'traced_perf.rc'},
}
target_host_supported = [
@@ -168,7 +169,9 @@
('include_dirs', {'bionic/libc/kernel'}),
],
'traced_probes': [
- ('required', {'libperfetto_android_internal', 'trigger_perfetto'}),
+ ('required', {'libperfetto_android_internal',
+ 'trigger_perfetto',
+ 'traced_perf'}),
],
'libperfetto_android_internal': [('static_libs', {'libhealthhalutils'}),],
'trace_processor_shell': [
diff --git a/traced_perf.rc b/traced_perf.rc
new file mode 100644
index 0000000..b38b7c4
--- /dev/null
+++ b/traced_perf.rc
@@ -0,0 +1,34 @@
+# Copyright (C) 2020 The Android Open Source Project
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+# http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+
+service traced_perf /system/bin/traced_perf
+ class late_start
+ disabled
+ # socket for receiving /proc/pid/{maps,mem} file descriptors
+ socket traced_perf stream 0666 root root
+ user nobody
+ # readproc to stat(/proc/pid) to find out UID of processes, and read
+ # /proc/pid/cmdline.
+ group nobody readproc
+ # KILL for sending BIONIC_SIGNAL_PROFILER.
+ # DAC_READ_SEARCH for stack unwinding and on-device symbolization (requires
+ # opening libraries/executables for sections not already mapped in).
+ capabilities KILL DAC_READ_SEARCH
+ writepid /dev/cpuset/foreground/tasks
+
+on property:persist.traced_perf.enable=1
+ start traced_perf
+on property:persist.traced_perf.enable=0
+ stop traced_perf
+