Merge "simpleperf: make inferno flamegraph resizable."
am: 442f469d08

Change-Id: Id479db60cd9d47e4df37b0a8be38623de0eaafe3

Merge "simpleperf: make inferno flamegraph resizable."

Merge "simpleperf: fix scripts based on test on darwin/windows."
am: b137bb2c74

Change-Id: I330a053333edb0d2b686d6dd1a63147951d614cf

Merge "simpleperf: fix scripts based on test on darwin/windows."

simpleperf: make inferno flamegraph resizable.

By using jqueryui resizable and set the width of each
node by percentage, we can make the flamegraph resizable.

Bug: http://b/64035530
Test: run inferno.sh.
Change-Id: I1d56e4eecbceaa6189eff25995d20f758a8b4111

simpleperf: fix scripts based on test on darwin/windows.

1. fix inferno.bat to correctly pass arguments.
2. type `inferno.bat` instead of `./inferno.bat`.
3. call `inferno.bat` in python should use shell=True.
4. fix the way to open browser on darwin/windows.

Bug: http://b/64035530
Test: run test.py on all platforms.

Change-Id: I5e22907d0c25186b129bf82e0f83d3b71e28ffb2

Merge "simpleperf: support --profile_from_launch option in app_profiler.py."
am: 0e6b67e308

Change-Id: I66d0e1d240d972ec9c39290a5b7ab43b85c80c75

Merge "simpleperf: support --profile_from_launch option in app_profiler.py."

simpleperf: support --profile_from_launch option in app_profiler.py.

Bug: http://b/34108866
Test: run test.py.
Change-Id: I6476b02fe31bf3f949e61640a8637582b09f270e

Merge "simpleperf: add missing scripts in simpleperf_script.zip."
am: e1a9f9267f

Change-Id: I73e6afb0159518089b5841ccc34e92630f0353b6

Merge "simpleperf: add missing scripts in simpleperf_script.zip."

simpleperf: add missing scripts in simpleperf_script.zip.

Bug: http://b/64035530
Test: run `make sdk`.
Change-Id: I2e60ca530076ea46664a0afa0602a456786b03e6

Merge "simpleperf: add doc for --trace-offcpu option."
am: bc53c51acc

Change-Id: Ic465e25c7683627469e113d1d2273bcdda5e1232

Merge "simpleperf: add doc for --trace-offcpu option."

simpleperf: add doc for --trace-offcpu option.

Bug: http://b/37572306
Test: run `remarkable README.md`.
Change-Id: I353dbc480a5af2755cde35132addaff890e715c5

Merge "simpleperf: fix inferno and test content of report.html."
am: 001e76be63

Change-Id: Id915fae598b6538f2a9e6ccb6454bdb7e075900e

Merge "simpleperf: add run_simpleperf_on_device.py."
am: 72dd31f567

Change-Id: I7ba9c0c46a3fcbb70cd30aaa10d2f5ca0ba789b0

Merge "simpleperf: fix building simpleperf_script.zip."
am: 88fed2d53e

Change-Id: I79dedd05ddf937726e11d164d78c306409490b04

Merge "simpleperf: fix inferno and test content of report.html."

Merge "simpleperf: add run_simpleperf_on_device.py."

Merge "simpleperf: fix building simpleperf_script.zip."

Merge "simpleperf: update simpleperf prebuilts to build 4282339."
am: 24bd27006a

Change-Id: I5e36c3fbaac216141ac5b80543ca1814583c79b7

Merge "simpleperf: update simpleperf prebuilts to build 4282339."

simpleperf: add run_simpleperf_on_device.py.

It is added mainly to show in README.md how to detect whether
trace-offcpu is supported on device, by
 `python run_simpleperf_on_device.py list --show-features`.

Bug: http://b/32834638
Test: run test.py.
Change-Id: I33a6fb81b650293faec233eb6a735bac7cb5e215

simpleperf: update simpleperf prebuilts to build 4282339.

Taken from branch aosp-master.
Bug: None
Test: None

Change-Id: I9349e8e646b2b7c999c7f540fd1f0d346ff15889

simpleperf: fix building simpleperf_script.zip.

The previous way of building simpleperf_script.zip uses "../" in zip file
makes unzip complain. So remove "../" by moving the makefile to the parent
directory.

Also include inferno in simpleperf_script.zip.

Bug: http://b/64035530
Test: run `make sdk -j30`.
Change-Id: Ic4359c5cf68e1475a605d58883a2451ca43bbe24

simpleperf: fix inferno and test content of report.html.

Inferno uses sample count to decide the width of each method,
but this is not accurate. Fix it by using event count instead.

Besides that, refactor inferno.py a little, get record cmd
and device info from perf.data, and test content of report.html
generated by inferno.

Also fix a flakey test TestExamplePureJava.test_app_profiler_with_ctrl_c.

Bug: http://b/64035530
Test: run test.py.
Change-Id: Ia57fcbd28b4242b4251b063bd38a58da7b93eba0

Merge "simpleperf: export more info through report_lib_interface."
am: 78b3cc8387

Change-Id: I4a0081b2b56b8ebc20529fd71d6e4ebeb8329712

Merge "simpleperf: export more info through report_lib_interface."

Merge "simpleperf: format inferno code."
am: 504d653008

Change-Id: I3fb3ca1ca306e58bfb720eadd12e0f6659c88b9d

Merge "simpleperf: format inferno code."

Merge "simpleperf: fix link of README.md."
am: e96387ab5e

Change-Id: I10da85da18156e0b50ebeff275df6eb3d4803e21

Merge "simpleperf: fix link of README.md."

simpleperf: export more info through report_lib_interface.

Make below changes to better suppport inferno:
1. Save product properties of device in perf.data.

2. Add a python/c api GetFeatureSection. It is used to support
reading record cmd and meta info from perf.data.

3. Remove old api GetNextMetaInfo, because meta info can be
exported by GetFeatureSection more easily.

4. When reading perf.data in record_file_reader.cpp, remove
callchain at and below ip == 0 to avoid caller's effort.

Also move to use android-base/properties.h.

Bug: http://b/64035530
Test: run simpleperf_unit_test and test.py.
Change-Id: Ib6743a09167e2b7cd65a12f17d991bc1ac628588

simpleperf: fix link of README.md.

Bug: None
Test: None
Change-Id: If06288b0b5b2b2ec20a7fc4f930c5ca3d0198978

Merge "simpleperf: improve warning msg."
am: 668f19e1a9

Change-Id: I6c122f13e6bd8dcd2f3d7c93c9e8a1f65381bfd0

Merge "simpleperf: improve warning msg."

simpleperf: improve warning msg.

The warning of missing symbol for each lib is shown twice,
once in recording, and once in reporting. This is redundant and
maybe misleading. Because when reporting we actually don't know
if we have collected enough symbols in perf.data. So move warning
of missing symbols to debug info if we have symbols in perf.data.

Use customized StderrLogger, because simpleperf doesn't need
time,pid,tid information in log.

Bug: http://b/29574526
Test: run simpleperf manually and run simpleperf_unit_test.
Change-Id: I9baf6d8fdcd63907681f2daa45b8fad6bf7e2516

simpleperf: format inferno code.

Format python code using autopep8.
Change Javascript code to ES6, and format it using eslint.

Bug: http://b/64035530
Test: run inferno.
Change-Id: I8b3553bf49423275da8eb3ad6fe3d8bb8dcf0a5b

Merge "simpleperf: use app_profiler.py in inferno."
am: c2adeafb3c

Change-Id: I8c4566f13540c115cd61fe35627ce57802bde9ca

Merge "simpleperf: use app_profiler.py in inferno."

Merge "simpleperf: fix stat cmd."
am: 45da8349f7

Change-Id: I4b659d142223167041058e4b91399d1010b7854e

Merge "simpleperf: fix stat cmd."

Merge "simpleperf: add --exit-with-parent option in record cmd."
am: d51b849069

Change-Id: I90867cfa7195d8bc60b0c898df3d841c75610f7a

Merge "simpleperf: add --exit-with-parent option in record cmd."

simpleperf: fix stat cmd.

Previous CL wrongly makes kernel dump samples for stat cmd.

Bug: http://b/64489160
Test: run simpleperf_unit_test.
Change-Id: I4f5c08839e283b2361e47d61310e5161433824bb

simpleperf: add --exit-with-parent option in record cmd.

This makes it optional whether simpleperf stops recording
when the parent dies.

Bug: http://b/64391339
Test: run simpleperf_unit_test.
Change-Id: I0827a17d6fedbb9763347df76da9dc981fea6312

simpleperf: use app_profiler.py in inferno.

Also add tests to check if we can run inferno successfully.
More tests will be added to check report results of inferno.

Bug: http://b/64035530
Test: run test.py.
Change-Id: Ie5dd1f5cb47c8c7a2ae68f8614cf81b68e3c7dbf

Merge "simpleperf: allow recording events in different speed."
am: f9ed7b5507

Change-Id: Ia1eaafc0c7b825289014ebe362f4419957f934ec

Merge "Fix fd ownership issue in PrintNativeInfo/GetNativeInfo."
am: 6278bc9fd8

Change-Id: I4d20cf4891f041c323af90f2a92c63337606eeb9

Merge "simpleperf: allow recording events in different speed."

Merge "Fix fd ownership issue in PrintNativeInfo/GetNativeInfo."

simpleperf: allow recording events in different speed.

Currently record command interface only allows one sample freq or sample period
for all events. This is not convenient when recording both non tracepoint events
and tracepoint events. This CL allows setting different sample speed for
different events. For example, for "-f 1000 -e cpu-cycles -c 1 sched:sched_switch",
"-f 1000" applies to cpu-cycles, and "-c 1" applies to sched:sched_switch.

It also fixes a bug about trace-offcpu: if "-f 1000 --trace-offcpu" is used,
the sched:sched_switch is be samples with "-f 1000". But we want to sample it
with "-c 1".

Also change the order of options in the help msg of record cmd to make it more
readable. Remove -F option. Because adding it seems not useful.

Bug: http://b/37572306
Test: run simpleperf_unit_test.

Change-Id: Ifdbd27c8f9fec49aade0e0e6ce624d8114042020

Fix fd ownership issue in PrintNativeInfo/GetNativeInfo.

Bug: N/A
Test: ran tests
Change-Id: I10bdeeb38404583560685bf47c58f6770fc9050b
Signed-off-by: Ivan Maidanski <i.maidanski@samsung.com>

Merge "simpleperf: support profiling native programs in app_profiler.py."
am: 51bdff6ccb

Change-Id: I12717142bd37241bbb7ca3c27b4122b74c316dce

Merge "simpleperf: export meta info in report_sample.proto."
am: 312cbae4fa

Change-Id: I4000bd49bf50a2a8314087a1dbaa2740ab9837be

Merge "simpleperf: support profiling native programs in app_profiler.py."

Merge "simpleperf: export meta info in report_sample.proto."

Merge "Fix clang-tidy performance-* warnings in system/extras."
am: a386d2168e

Change-Id: I0c63d2b7c1316b00c60d02faf2c08f77283ae4aa

simpleperf: export meta info in report_sample.proto.

To support perf.data generated with --trace-offcpu option,
update interface in report_sample.proto:
1. Add meta info to show all event types.
2. Add event_type_id in each sample to show which even type it belongs to.

Bug: http://b/37572306
Test: run simpleperf_unit_test.
Change-Id: I2878979ec2023904df1006ce353dcf233b6a2642

Merge "Fix clang-tidy performance-* warnings in system/extras."

simpleperf: support profiling native programs in app_profiler.py.

This is a preparation of using app_profiler.py in inferno.

In app_profiler.py:
Add -np option to profile native programs like surfaceflinger.
Add -cmd option to profile command lines.
Add --arch option to set the app arch, because we don't guess the app arch
when profiling command lines.
Add corresponding tests in test.py.

Also improve the way of detecting whether the device supports trace-offcpu.

Bug: http://b/63006886
Test: run test.py.

Change-Id: Id899063d9a94beec67b22fd7710cda7a438557fb

Fix clang-tidy performance-* warnings in system/extras.

* Use const reference parameter type to avoid unnecessary copy.
* Use more efficient overloaded string methods.

Bug: 30407689
Bug: 30411878
Test: build with WITH_TIDY=1
Change-Id: I558d482910c8a53c042d876848e35cdce8b8c15b

Merge "simpleperf: fix supporting ctrl-c when using `adb shell simpleperf xxx`."
am: b3734fda74

Change-Id: I89b8b5f864306487cbb6ca35d1adb657a0385199

Merge "simpleperf: fix supporting ctrl-c when using `adb shell simpleperf xxx`."

simpleperf: fix supporting ctrl-c when using `adb shell simpleperf xxx`.

Also fix a problem that binary_cache_builder.py tries to pull /dev/zero.

Bug: None.
Test: run test.py TestExamplePureJava.test_app_profiler_with_ctrl_c.
Change-Id: Ie99af6795bb1e81ae6e93f8b7a8907d49c048694

Merge "simpleperf: update simpleperf prebuilts to build 4237700."
am: 76163e6103

Change-Id: Ia19468d3c21852b73d2f58816e2e0ca59500cb5c

Merge "simpleperf: update simpleperf prebuilts to build 4237700."

simpleperf: update simpleperf prebuilts to build 4237700.

Taken from branch aosp-master.
Test: None.

Change-Id: Idcf75340fb9c0d7b8c81f64eeb2cb94a4c432224

Merge "simpleperf: split README.md in doc directory."
am: 3b7e114229

Change-Id: Ic836b59b5a8b7c5f653c87d902abb15df1258da7

Merge "simpleperf: split README.md in doc directory."

simpleperf: split README.md in doc directory.

Move README.md to doc directory, split inferno doc into inferno.md,
move pictures from scripts/inferno/ to doc/.
Improve the content about showing flamegraphs in README.md.

Bug: http://b/64123842
Test: No.
Change-Id: Ia3ebd0bbbda21fa816b94cd349c901b5c7713564

Merge "simpleperf: add --show-features option in list cmd."
am: 2ad09b2ffd

Change-Id: I2bce8ed7bf6ad13cadc01b92d07da661cb454a2e

Merge "simpleperf: add --show-features option in list cmd."

Merge "Update for v4.12.3 kernel headers."
am: 6dfe8cde2c

Change-Id: I9bf957357f35b995b000770689319a508e723510

Merge "Update for v4.12.3 kernel headers."

simpleperf: add --show-features option in list cmd.

It is used to show features supported on the device.

Bug: http://b/64147273
Test: run simpleperf_unit_test.
Change-Id: Idb7821e74d1a23f8988ef2207696114498713f47

Merge "simpleperf add more script tests."
am: 4ba39d11d3

Change-Id: I379bf269c3a1d441ebc935e63d21ca988a5aad77

Merge "simpleperf add more script tests."

simpleperf add more script tests.

Add tests for Trace offcpu.
Add test for jni call.
Build testdata when running test.py.

Bug: http://b/63006886
Test: run test.py.

Change-Id: I5a87b8d2a8f59ead858c1b78a78c7e78847854f4

Update for v4.12.3 kernel headers.

Modify the host side linux byteorder.h to set the defines for little
endian. The new kernel headers require the defines, or there will be
compile errors.

Test: Ran all host side unit tests.
Change-Id: I05f0ea0695a56525af3cd808f2ae9a8d6b5c49db

Merge "ext4_utils: enable uninit_bg feature for ext4 mke2fs"
am: d4c959ba79

Change-Id: Id07f9f71e63c14706e6b0e9a161c7444163efa4d

Merge "ext4_utils: enable uninit_bg feature for ext4 mke2fs"

ext4_utils: enable uninit_bg feature for ext4 mke2fs

Some devices have very slow storage media. This feature can speed
up filesystem creation time noticeably.

Bug: 63809106
Bug: 64101366
Change-Id: Id4886822035808bcb279dbf6f81989fee9cc0ace
(cherry picked from commit 794223c633652b714db804729b5f57d3689a1f91)

Merge "simpleperf: fix README.md to show inferno pictures correctly."
am: 6979f2d842

Change-Id: Ie03268117b782b8b45cc86f462763f4691702d6f

Merge "simpleperf: fix README.md to show inferno pictures correctly."

Merge "simpleperf: update simpleperf prebuilts to build 4219235."
am: 6fd3f9b50d

Change-Id: I4c2ce4b296b2c17dbaef8b7b5676b003e9d3ee6c

Merge "simpleperf: update simpleperf prebuilts to build 4219235."

simpleperf: update simpleperf prebuilts to build 4219235.

Taken from branch aosp-master.

Test: none.

Change-Id: I64c9a60b4251549a22d44e9444824a439d5f4cc1

simpleperf: fix README.md to show inferno pictures correctly.

Bug: http://b/63006886
Test: none.
Change-Id: If27d2f0d91e2121c3d58c0b5451dee98934f2d40

Merge "Fix a memory leak"
am: 880d4e8574

Change-Id: I411fd53aa80fc790773b8878bdeff06a4be40b51

Merge "Fix a memory leak"

Fix a memory leak

This fixes the following memory leak:
system/extras/tests/tcp_nuke_addr/tcp_nuke_addr_test.cpp:146:20:
warning: Potential memory leak [clang-analyzer-cplusplus.NewDeleteLeaks]

Namely, clang was upset that we didn't free the std::threads we new'ed
up. Immediately detach()ing them gets us the same effect.

(Context: We're trying to clean up scary-sounding warnings like this in
Android. I realize that, in this case, a leak is harmless, but...)

Bug: 27101951
Test: mma. Memory leak warning is gone. Ran the test on bullhead for
60s; seemed to work (last line of output is "60s: 246 cps, total 25432")

Change-Id: I4b6b5679430968e01ffea5bc10309a4ced04ba4e

Merge "simpleperf: move inferno to scripts directory."
am: 0cffcbee0d

Change-Id: Ieb2f1bbe13f60df81553dd320d46aded1c00cec0

Merge "simpleperf: move inferno to scripts directory."

Merge "simpleperf: support reporting trace offcpu data in simpleperf_report_lib."
am: be51cd0e9a

Change-Id: I28b4d69e575ca6258bb5da2b40b7309d3c0d844b

Merge "simpleperf: support reporting trace offcpu data in simpleperf_report_lib."

simpleperf: move inferno to scripts directory.

This is a preparation for integrating inferno with python scripts under scripts/.

Bug: http://b/64035530
Test: run inferno.py.
Change-Id: I60ee1c30f650307aa58a038448cddb4eef43acb5

Merge "Remove LOCAL_CLANG and clang: true"
am: 481127a308

Change-Id: Ibbf20780454133924b3292156968b57008c3f73d

Merge "Remove LOCAL_CLANG and clang: true"