Merge "Fix google-explicit-constructor warnings in perfprofd."

Merge "simpleperf: fix record file writer."

simpleperf: fix record file writer.

When running `run-as [package_name] ./simpleperf record...` on device,
simpleperf doesn't have permission to create temporary file in
/data/local/tmp. So create temporary files in the same directory
as perf.data instead.

Bug: http://b/30649868
Change-Id: I48178d4eb421be8d8636721ac3b29b98fa673b5d
Test: run simpleperf_unit_test.

Merge "simpleperf: allow --duration option when executing child command."

simpleperf: allow --duration option when executing child command.

Previously --duration option can't be used while executing child
command, mainly because we use `sleep XX` child command to implement
--duration option. By using libevent, the limit doesn't exist.

Bug: http://b/30405638
Change-Id: Iea1953ecf3aca48e75fb74f048ed04ad5b168630
Test: run simpleperf_unit_test.

Merge "simpleperf: remove cpu option tests."

Merge "simpleperf: reduce Record construction overhead while recording."

simpleperf: remove cpu option tests.

The cpu option tests are flaky because Cpu 0 can be offlined.

Bug: http://b/30389972
Change-Id: Ie9f57dcde957be245167c957eb029ab37496471e
Test: run simpleperf cts test.

Merge "simpleperf: fix arm unwinding."

simpleperf: fix arm unwinding.

Change-Id: Ie611504a7e5062c25466b59ebdcf8c33b51bf90c
Test: run simpleperf_unit_test.

simpleperf: reduce Record construction overhead while recording.

Avoid binary allocation and memory copy in ReadRecordsFromBuffer(),
thus reduce Record construction overhead in
EventSelectionSet::ReadMmapEventDataForFd().

Remove RecordCache used while recording. Replace it with
RecordFileWriter::SortDataSection(). For unwinding while
recording, use low watermark to make records almost sorted
when dumped from the kernel.

Bug: 30649868
Test: run simpleperf_unit_test.

Change-Id: Ie5fb942046900a5960b3c990cf4177c026eaadfb

Merge "simpleperf: fix runtest."

Merge "simpleperf: keep binary in class Record."

simpleperf: keep binary in class Record.

It removes memory copy and heap allocation/deallocation in
Record::BinaryFormat(), and is a preparation to remove memory
copy and heap allocation in Record constructor.

Bug: 30649868
Test: run simpleperf_unit_test.
Change-Id: Ic8dd80e43f7b547a9beaf896d726b56aeb5d55a2

simpleperf: fix runtest.

Use /data/local/tmp directory to pass runtest on non-rooted device.

Bug: 30413567
Change-Id: I730e0763b409f458390583dd894a644e9213e807

Merge "Fix clang-tidy warnings in iotop."

Merge "Fix google-explicit-constructor warnings in libfec."

Fix google-explicit-constructor warnings in perfprofd.

Bug: 28341362
Change-Id: Ia65b3a37e918c9bda5c197536e8b0b2b34f787a8
Test: build with WITH_TIDY=1

Fix clang-tidy warnings in iotop.

* Declare explicit conversion constructors.
* Use const reference type for parameter to avoid unnecessary copy.

Bug: 28341362
Bug: 30407689
Change-Id: Ia95a041125703ecbfac128709fd6ea0591df2103
Test: build with WITH_TIDY=1

Fix google-explicit-constructor warnings in libfec.

Bug: 28341362
Change-Id: Id7d9593e87fc8187942ef808f6ccf943a6ec023d
Test: build with WITH_TIDY=1

Merge "simpleperf: add version."

Merge "simpleperf: print warning message when failed to read symbol table from elf file."

simpleperf: add version.

Bug: http://b/30416563
Change-Id: I03e5dbae3f0944d7d6b73d4dfa252ad3396b83e6
Test: run `simpleperf --version`.

simpleperf: print warning message when failed to read symbol table from elf file.

Change return type of read elf functions to ElfStatus, and print warning message
based on situation.
Add corresponding test.

Bug: 29193936
Change-Id: Ie449f8be34cb579962cb73f4be3df9e4997270ca
Test: run simpleperf_unit_test.

Merge "simpleperf: use libevent in record command."

simpleperf: use libevent in record command.

Support read Event in IOEventLoop, test it, and use it
in record command.
Format related code and clean up some comments.

Bug: http://b/30405638
Change-Id: I3b07d50b0374be3f3e7ee9fa4d0c0ccc8ae21a2b
Test: run simpleperf_unit_test.

Merge "Fix clang-tidy performance warnings in simpleperf."

Merge "Fix clang-tidy performance warnings in perfprofd."

Fix clang-tidy performance warnings in simpleperf.

* Use const reference type for parameters to avoid unnecessary copy.
* Use faster overloaded string find function.

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

Merge "simpleperf: use libevent in stat command."

Merge "simpleperf: report lost record situation in report-sample command."

Merge "Fix clang-tidy performance warnings in libfec."

Merge "simpleperf: fix runtest."

Fix clang-tidy performance warnings in perfprofd.

* Use faster overloaded find_last_of function.

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

Fix clang-tidy performance warnings in libfec.

* Use const reference type for for-loop index variables
  to avoid unnecessary copy.

Bug: 30413223
Change-Id: I9819261087a5385fe580f805be44a2407f35cdf5
Test: build with WITH_TIDY=1

simpleperf: use libevent in stat command.

Wrap libevent in IOEventLoop, use IOEventLoop in stat command.
Add corresponding tests.

Bug: http://b/30405638
Change-Id: I78b79e0eff1365ab46dde29c2a24a2def586af79
Test: run simpleperf_unit_test.

simpleperf: report lost record situation in report-sample command.

Bug: http://b/28114205
Change-Id: I1dc1389fb62596811a27d8dab5cbd214437457ee
Test: run simpleperf_unit_test.

simpleperf: fix runtest.

1. Adjust function_fork.cpp and two_functions.cpp to make functions
not combined by compiler.
2. Download simpleperf and runtest executables on device before
testing on device.
3. Find config file in the same directory as runtest.py instead of
relying that it is in current directory.

Bug: http://b/30413567
Change-Id: I4ed9964f443c8cc75f4cb352ff1f13fb11104c89
Test: mmma system/extras/simpleperf && python system/extras/simpleperf/runtest/runtest.py

Merge "Rename lib{crypto,ssl}-host to lib{crypto,ssl}"

Merge "simplperf: improve unknown symbol print in report-sample command."

Merge "simpleperf: add --duration for stat/record command."

simpleperf: add --duration for stat/record command.

Also fix a bug in ReportCommandTest.callgraph_option.

Bug: http://b/30292111
Change-Id: I21ced0c8aa7ec10c60c4fc1ddc6e5dfd256feeb1
Test: run simpleperf_unit_test.

Rename lib{crypto,ssl}-host to lib{crypto,ssl}

There's no need to use a different name for target and host moudles. In
Soong, it's better to use the same for both, as target and host modules
can be defined at the same time.

Change-Id: I4b3ffd0b4abcd10490578d623ae140df5c19dc76

simplperf: improve unknown symbol print in report-sample command.

Add * for all unknown symbols, so it is easier to write program to
identify unknown symbols.
Use dso[+ip_offset] for unknown symbols.

Bug: 28114205
Change-Id: I14cfffe6673a9aedc62cc288c1b2d724ab411f5d

Merge "Add runconuid command to xbin"

Add runconuid command to xbin

Change-Id: Iebaabb316682acd4415673f1ef3727a25922225d

Merge "simpleperf: add option for generating java code for report_sample.proto."

Merge "simpleperf: add thread_id in report_sample's output."

simpleperf: add option for generating java code for report_sample.proto.

Bug: 28114205
Change-Id: Iaa13b47b32c148c4770b4215e1dc2369538592f7

simpleperf: add thread_id in report_sample's output.

Also fix enum Type in report_sample.proto.

Bug: 28114205
Change-Id: Ia4e6a31a69095c8183544e2d53c8ceddc10ccb51

Merge "simpleperf: let simpleperf running record/stat command die if parent exits."

simpleperf: let simpleperf running record/stat command die if parent exits.

Bug: 28114205
Change-Id: I310b9d7a4f52ad5bacf892eba34dbf531a3a7245

Merge "simpleperf: fix unwind stack size."

Merge "simpleperf: show dso[+vaddr_in_file] for unknown symbols."

simpleperf: fix unwind stack size.

Currently, to work around the bug in N9, simpleperf uses
all dumped user stack for unwinding. However, not all of
the data is valid, which can make unwinding result annoying.
So fix this by using only valid stack data if dyn_size != 0.

Bug: 29574526
Test: run simpleperf_unit_test and use simpleperf manually.

Change-Id: Idaa82f3156215fc4fb902d96de290e43726b0e17

Merge "simpleperf: change default callgraph to use caller mode."

Merge "simpleperf: fix dumping symbols for unwinded ips."

simpleperf: show dso[+vaddr_in_file] for unknown symbols.

It gives more information than just unknown symbols.
Add --no-show-ip option to disable this additional detail.

Bug: 29772268
Change-Id: Ie8067f95b5fdc65806044e229ee12095367d115a
Test: run simpleperf_unit_test.

simpleperf: change default callgraph to use caller mode.

Callgraph in caller mode shows how functions call others, i.e.,
main() -> func1() -> func2(). Instead, callgraph in callee mode shows
how functions are called by others, i.e., func2() <- func1() <- main().
It seems callgraph in caller mode is easier to understand than callee
mode. So change to default option to use caller mode.

Bug: 29574526
Change-Id: I85f104ef7c974bbf7cafd290563e762ba1e0a084
Test: manually check if the default call graph changes.

simpleperf: fix dumping symbols for unwinded ips.

Previous code dumps symbols for unwinded ips regardless of
--dump-symbols option. Fix it to be controled by --dump-symbols
option.

Bug: 29581559
Change-Id: Ie7a79bf5275c13b0df4352e4aedc702f34b7b899
Test: run simpleperf_unit_test.

Merge "simpleperf: adjust mapped buffer size in record command."

simpleperf: adjust mapped buffer size in record command.

Instead of fixing the mapped buffer size, we can adjust
it at runtime. So users don't need to adjust -m manually.

Bug: 29574526
Change-Id: Icb580df3d60f8d2cf554c0d4139e6f7f64b19f8f
Test: run simpleperf_unit_test.

Merge "simpleperf: check dump stack size and adjust its default value."

simpleperf: check dump stack size and adjust its default value.

Improve error message by checking if dump stack size > 65528.
And adjust the default dump stack size to 65528, because I
find that it is the value I always want to use.

Bug: 29574526
Change-Id: I8f16dcf3a86a477f17d81fd387bf4dfa0dc0b341
Test: run simpleperf_unit_test.

Merge "Convert libpagemap Android.mk to Android.bp"

Merge "simpleperf: combine mapped buffer used for the same event and cpu."

Merge "Basic A/B bootloader tests via bootctl"

simpleperf: combine mapped buffer used for the same event and cpu.

Non root users have a size limit of buffers used for profiling.
By combining buffers used for the same event and cpu, we can
reduce buffer cost when monitoring multiple threads, thus
avoid mmap() failures.

Increase default value for -m option for non system wide profiling.

Bug: 28911532
Change-Id: I91148061eb54840c144cf72e7bb901e7b74897ec
Test: run simpleperf_unit_test.

Basic A/B bootloader tests via bootctl

Change-Id: Ie7868cc205859c657e905e60ab6928517f02edb8

Merge "simpleperf: fix test record_cmd.dump_symbols."

Convert libpagemap Android.mk to Android.bp

Change-Id: I551a7f7ce46267bb609e5ed8bd8dafc00b9fbab7

Merge "simpleperf: check kernel symbol addresses before dumping them."

simpleperf: check kernel symbol addresses before dumping them.

If kernel symbols have zero addresses, there is no need to dump them,
and we can give useful suggestion.

Bug: 29574526
Test: manually set /proc/sys/kernel/kptr_restrict and run simpleperf record.
Change-Id: I850531c7e4c5315a44e08cf3b73852e77fef8eb8

Merge "Fix google-explicit-constructor warnings in simpleperf."

Merge "Fix google-explicit-constructor warnings in memory_replay."

Fix google-explicit-constructor warnings in memory_replay.

* Declare explicit conversion constructors.

Bug: 28341362
Change-Id: Ia6613392bd68f170ec2d7b0b033b7476aff27e0f
Test: build with clang-tidy

Fix google-explicit-constructor warnings in simpleperf.

* Declare explicit conversion constructors.

Bug: 28341362
Change-Id: I9d5b764fe004e291cbdd212617df6f385c31df25
Test: build with clang-tidy

Merge "simpleperf: add min_vaddr in DsoRecord."

simpleperf: add min_vaddr in DsoRecord.

Min virtual address of a shared library is needed when mapping ip
addresses to function symbols. So we should dump it in DsoRecord.

Bug: 28114205
Test: run simpleperf_unit_test.

Change-Id: Ib986ee598281cf60caa3a2c5408100b9e7678143

Merge "simpleperf: change the separator of --symbols option from comma to semicolon."

simpleperf: fix test record_cmd.dump_symbols.

The test can fail when dumping an unknown symbol in a new shared
library. The shared library is recorded in DsoRecord, but the
unknown symbol may have been dumped. To fix this, don't dump
unknown symbols, and make the test not enforcing DsoRecords
and SymbolRecords to occur.

Bug: 28114205
Change-Id: Ib01d0753710c68dd9538d8b635c6d9207fe7aa1c
Test: run simpleperf_unit_test.

simpleperf: change the separator of --symbols option from comma to semicolon.

Function name of c++ can have comma, like func2(int, int). So it is not
proper to use comma to separate function names.

Bug: 29826956
Change-Id: I9da4bc178232dad53abdc5735358f3feb608fee4
Test: run simpleperf_unit_test.

Merge "Generate output of simpleperf in csv format"

Generate output of simpleperf in csv format

CSV format is easier to parse and could be useful for testing.

Change-Id: I444bddfa181b48c79ec48ef7595b5d8a25e62e12

Merge "Remove call to no-op klog_init."

Merge "simpleperf: don't warn if child process was killed by simpleperf."

simpleperf: don't warn if child process was killed by simpleperf.

Bug: 29574526
Change-Id: I3ae95ed95703bcd380ed086d3fbf7ae2830feed5

Merge "simpleperf: give suggestion when mmap fails."

Merge "simpleperf: fix build id check of files in symfs."

simpleperf: give suggestion when mmap fails.

Bug: 29574526
Change-Id: I72cf2d0e1d15a561314d3c043650479b2c6773ff

simpleperf: fix build id check of files in symfs.

In dso.cpp, build_id_map_ should use path_ instead of GetAccessiblePath() as the key.
However, patch https://android-review.googlesource.com/#/c/175654/ wrongly used
GetAccessiblePath() as the key in build_id_map_. This patch fixes the error and add
corresponding test.

Check if file in symfs exists before using it as debug file path.
If the build id of debug file path doesn't match the one in build_id_map_, output
warning to user.

Bug: 28911532
Test: run simpleperf_unit_test.
Change-Id: I21bca508359a492245db4cba5d287005363cd465

Merge "simpleperf: add --symfs option for record command."

simpleperf: add --symfs option for record command.

When running record command with -g or --dump-symbols command,
files with symbol table and debug information are needed. Similar
to report command, we can add --symfs option to record command.

Bug: 28911532
Test: run simpleperf_unit_test.

Change-Id: I8e2b6320ca29c8de78b4f217cd25e1ea4383150e

Merge "simpleperf: add symbol for .plt section."

simpleperf: add symbol for .plt section.

This avoid reporting unknown symbols when there are samples in .plt section.

Bug: 28911532
Test: run simpleperf_unit_test.
Change-Id: I62cb08776c99951ff845e98f0f601859d25ece5c

Merge "Simpleperf: add auto generated summaries in stat command."

Simpleperf: add auto generated summaries in stat command.

When there are summaries monitoring one event type in user
space and kernel space at the same time period, we can automatically
generate a summary combining the results generated in user space and
kernel space.
This can help to decrease the number of needed hardware counters.

Also adjust scale check when deciding whether an event is monitored
all the time.

Bug: 29213742
Change-Id: I272e21420700c10fccb1336a1e60138db8d13b3d
Test: run simpleperf_unit_test.

Merge "simpleperf: add symbol filter for report command."

Merge "simpleperf: fix RecordCache."

simpleperf: add symbol filter for report command.

By adding --symbols option, we can report for selected functions.
It helps to filter out functions we don't care, especially when
we are using `simpleperf report --sort vaddr_in_file` to profile
inside a function.

Bug: 29826956

Change-Id: I4de9dfe1a376db097f6163aa780a079cbc33689a