OSDN Git Service
Chih-Hung Hsieh [Thu, 28 Jul 2016 15:59:56 +0000 (15:59 +0000)]
Merge \"Fix clang-tidy performance warnings in simpleperf.\"
am:
43b194dc15
Change-Id: Ie7206846246bd50fbc758e972b83596bcacb3faf
Chih-hung Hsieh [Thu, 28 Jul 2016 15:50:06 +0000 (15:50 +0000)]
Merge "Fix clang-tidy performance warnings in simpleperf."
Chih-Hung Hsieh [Thu, 28 Jul 2016 06:25:58 +0000 (06:25 +0000)]
Merge \"Fix clang-tidy performance warnings in perfprofd.\"
am:
c2643f1c69
Change-Id: I7cbd828531927d517b319ca3795eae2c3c049cc7
Chih-hung Hsieh [Thu, 28 Jul 2016 01:19:16 +0000 (01:19 +0000)]
Merge "Fix clang-tidy performance warnings in perfprofd."
Chih-Hung Hsieh [Wed, 27 Jul 2016 22:22:01 +0000 (15:22 -0700)]
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
Yabin Cui [Wed, 27 Jul 2016 23:54:12 +0000 (23:54 +0000)]
Merge \"simpleperf: use libevent in stat command.\"
am:
cab924fae5
Change-Id: Id3f2e39a1aebddcdc6e548595b394d4eedcf3e0f
Yabin Cui [Wed, 27 Jul 2016 23:49:35 +0000 (23:49 +0000)]
Merge "simpleperf: use libevent in stat command."
Yabin Cui [Wed, 27 Jul 2016 23:48:28 +0000 (23:48 +0000)]
Merge \"simpleperf: report lost record situation in report-sample command.\"
am:
fdd7b1f5f3
Change-Id: Ic5af58a491b03d7800b0886a995062249a2f0b7b
Treehugger Robot [Wed, 27 Jul 2016 23:40:39 +0000 (23:40 +0000)]
Merge "simpleperf: report lost record situation in report-sample command."
Chih-Hung Hsieh [Wed, 27 Jul 2016 23:23:29 +0000 (23:23 +0000)]
Merge \"Fix clang-tidy performance warnings in libfec.\"
am:
1f6947bcd9
Change-Id: I7dd8acceacec22365ffb0157704cf839771a1e18
Treehugger Robot [Wed, 27 Jul 2016 23:21:41 +0000 (23:21 +0000)]
Merge "Fix clang-tidy performance warnings in libfec."
Yabin Cui [Wed, 27 Jul 2016 23:12:00 +0000 (23:12 +0000)]
Merge \"simpleperf: fix runtest.\"
am:
b40840ff74
Change-Id: I734ce3ec710c9e8fdf65e976893d956d4e391cb7
Treehugger Robot [Wed, 27 Jul 2016 23:03:32 +0000 (23:03 +0000)]
Merge "simpleperf: fix runtest."
Chih-Hung Hsieh [Wed, 27 Jul 2016 22:16:00 +0000 (15:16 -0700)]
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
Chih-Hung Hsieh [Wed, 27 Jul 2016 22:06:58 +0000 (15:06 -0700)]
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
Yabin Cui [Tue, 26 Jul 2016 22:03:27 +0000 (15:03 -0700)]
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.
Yabin Cui [Wed, 27 Jul 2016 02:49:15 +0000 (19:49 -0700)]
simpleperf: report lost record situation in report-sample command.
Bug: http://b/
28114205
Change-Id: I1dc1389fb62596811a27d8dab5cbd214437457ee
Test: run simpleperf_unit_test.
Yabin Cui [Tue, 26 Jul 2016 22:57:10 +0000 (15:57 -0700)]
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
Dan Willemsen [Sat, 23 Jul 2016 00:29:27 +0000 (00:29 +0000)]
Merge \"Rename lib{crypto,ssl}-host to lib{crypto,ssl}\"
am:
5b46f5747c
Change-Id: I513a2be90fee7b96d1e11bb45d93fc9f315c6e63
Dan Willemsen [Sat, 23 Jul 2016 00:22:28 +0000 (00:22 +0000)]
Merge "Rename lib{crypto,ssl}-host to lib{crypto,ssl}"
Yabin Cui [Fri, 22 Jul 2016 22:59:21 +0000 (22:59 +0000)]
Merge \"simplperf: improve unknown symbol print in report-sample command.\"
am:
7682f60edc
Change-Id: I1738643c840698d65adecd67a586a02506dd26e1
Treehugger Robot [Fri, 22 Jul 2016 22:50:18 +0000 (22:50 +0000)]
Merge "simplperf: improve unknown symbol print in report-sample command."
Yabin Cui [Fri, 22 Jul 2016 21:36:46 +0000 (21:36 +0000)]
Merge \"simpleperf: add --duration for stat/record command.\"
am:
53fdddd794
Change-Id: Ibc911eb9be4666dae97ae179248863a0622541a8
Treehugger Robot [Fri, 22 Jul 2016 21:24:33 +0000 (21:24 +0000)]
Merge "simpleperf: add --duration for stat/record command."
Yabin Cui [Fri, 22 Jul 2016 01:32:53 +0000 (18:32 -0700)]
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.
Dan Willemsen [Thu, 21 Jul 2016 21:16:53 +0000 (14:16 -0700)]
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
Yabin Cui [Thu, 21 Jul 2016 19:29:47 +0000 (12:29 -0700)]
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
Zach Riggle [Thu, 21 Jul 2016 15:30:27 +0000 (15:30 +0000)]
Merge \"Add runconuid command to xbin\"
am:
a33edd664d
Change-Id: I7f8403cb1f355c25c46659f9e5b0261ebe302791
Zach Riggle [Thu, 21 Jul 2016 15:23:14 +0000 (15:23 +0000)]
Merge "Add runconuid command to xbin"
Zach Riggle [Thu, 21 Jul 2016 15:21:54 +0000 (11:21 -0400)]
Add runconuid command to xbin
Change-Id: Iebaabb316682acd4415673f1ef3727a25922225d
Yabin Cui [Thu, 21 Jul 2016 01:33:14 +0000 (01:33 +0000)]
Merge \"simpleperf: add option for generating java code for report_sample.proto.\"
am:
68320c1559
Change-Id: I26415fadd7e8be6866e8ace0a68354d81f4fa0f4
Yabin Cui [Thu, 21 Jul 2016 01:33:13 +0000 (01:33 +0000)]
Merge \"simpleperf: add thread_id in report_sample\'s output.\"
am:
0611ad5a36
Change-Id: Ib3b22e80e1f59827aa3ed65413ef2344c7ede390
Yabin Cui [Thu, 21 Jul 2016 01:12:19 +0000 (01:12 +0000)]
Merge "simpleperf: add option for generating java code for report_sample.proto."
Yabin Cui [Thu, 21 Jul 2016 01:12:05 +0000 (01:12 +0000)]
Merge "simpleperf: add thread_id in report_sample's output."
Yabin Cui [Wed, 20 Jul 2016 00:15:12 +0000 (17:15 -0700)]
simpleperf: add option for generating java code for report_sample.proto.
Bug:
28114205
Change-Id: Iaa13b47b32c148c4770b4215e1dc2369538592f7
Yabin Cui [Tue, 19 Jul 2016 23:07:54 +0000 (16:07 -0700)]
simpleperf: add thread_id in report_sample's output.
Also fix enum Type in report_sample.proto.
Bug:
28114205
Change-Id: Ia4e6a31a69095c8183544e2d53c8ceddc10ccb51
Yabin Cui [Fri, 15 Jul 2016 22:29:04 +0000 (22:29 +0000)]
Merge \"simpleperf: let simpleperf running record/stat command die if parent exits.\"
am:
20908fe1d7
Change-Id: Ibb27d164b498d88878517d9f100fcd71f8b71aed
Yabin Cui [Fri, 15 Jul 2016 22:17:45 +0000 (22:17 +0000)]
Merge "simpleperf: let simpleperf running record/stat command die if parent exits."
Yabin Cui [Fri, 15 Jul 2016 21:08:48 +0000 (14:08 -0700)]
simpleperf: let simpleperf running record/stat command die if parent exits.
Bug:
28114205
Change-Id: I310b9d7a4f52ad5bacf892eba34dbf531a3a7245
Yabin Cui [Thu, 14 Jul 2016 23:06:31 +0000 (23:06 +0000)]
Merge \"simpleperf: fix unwind stack size.\"
am:
7be7fee1e7
Change-Id: I549984ef043ecb05e2131d02901359d61efa879d
Treehugger Robot [Thu, 14 Jul 2016 22:58:35 +0000 (22:58 +0000)]
Merge "simpleperf: fix unwind stack size."
Yabin Cui [Thu, 14 Jul 2016 22:37:59 +0000 (22:37 +0000)]
Merge \"simpleperf: show dso[+vaddr_in_file] for unknown symbols.\"
am:
71f3fbc91b
Change-Id: Id47db344814bb5c9ab71509c7a6ccc45080ab645
Treehugger Robot [Thu, 14 Jul 2016 22:22:10 +0000 (22:22 +0000)]
Merge "simpleperf: show dso[+vaddr_in_file] for unknown symbols."
Yabin Cui [Thu, 14 Jul 2016 21:44:57 +0000 (21:44 +0000)]
Merge \"simpleperf: change default callgraph to use caller mode.\"
am:
fd16291c83
Change-Id: Idde73a7d1220aaac208debd835a1145639a5b5a4
Yabin Cui [Thu, 14 Jul 2016 21:44:56 +0000 (21:44 +0000)]
Merge \"simpleperf: fix dumping symbols for unwinded ips.\"
am:
1ebe5df8ae
Change-Id: Ib54f74775238a1827ceccb27b224e53ac53604c7
Yabin Cui [Thu, 14 Jul 2016 21:29:33 +0000 (14:29 -0700)]
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
Yabin Cui [Thu, 14 Jul 2016 21:34:59 +0000 (21:34 +0000)]
Merge "simpleperf: change default callgraph to use caller mode."
Yabin Cui [Thu, 14 Jul 2016 21:34:41 +0000 (21:34 +0000)]
Merge "simpleperf: fix dumping symbols for unwinded ips."
Yabin Cui [Thu, 14 Jul 2016 20:26:19 +0000 (13:26 -0700)]
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.
Yabin Cui [Thu, 14 Jul 2016 18:50:31 +0000 (11:50 -0700)]
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.
Yabin Cui [Thu, 14 Jul 2016 18:13:41 +0000 (11:13 -0700)]
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.
Yabin Cui [Thu, 14 Jul 2016 02:35:28 +0000 (02:35 +0000)]
Merge \"simpleperf: adjust mapped buffer size in record command.\"
am:
e5b5cf07c7
Change-Id: Ifff95d2239dc83267bb0dad870480bd4d29b9341
Treehugger Robot [Thu, 14 Jul 2016 02:17:21 +0000 (02:17 +0000)]
Merge "simpleperf: adjust mapped buffer size in record command."
Yabin Cui [Thu, 14 Jul 2016 00:06:50 +0000 (17:06 -0700)]
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.
Yabin Cui [Wed, 13 Jul 2016 20:56:09 +0000 (20:56 +0000)]
Merge \"simpleperf: check dump stack size and adjust its default value.\"
am:
7a6b3fdff5
Change-Id: Ifbdce7abc65f089e35a0ecbcfedc81fc808282b6
Yabin Cui [Wed, 13 Jul 2016 20:47:40 +0000 (20:47 +0000)]
Merge "simpleperf: check dump stack size and adjust its default value."
Yabin Cui [Wed, 13 Jul 2016 19:18:18 +0000 (12:18 -0700)]
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.
Colin Cross [Wed, 13 Jul 2016 00:34:40 +0000 (00:34 +0000)]
Merge \"Convert libpagemap Android.mk to Android.bp\"
am:
6941c09c77
Change-Id: I88ca7d0201da079bfb66c3c3a0ed8db0f83c6621
Colin Cross [Wed, 13 Jul 2016 00:22:29 +0000 (00:22 +0000)]
Merge "Convert libpagemap Android.mk to Android.bp"
Yabin Cui [Tue, 12 Jul 2016 23:24:43 +0000 (23:24 +0000)]
Merge \"simpleperf: combine mapped buffer used for the same event and cpu.\"
am:
70e5ffea32
Change-Id: I4df54144afd5ada55a05c77f61c9ff3087bfbe1e
Treehugger Robot [Tue, 12 Jul 2016 23:19:36 +0000 (23:19 +0000)]
Merge "simpleperf: combine mapped buffer used for the same event and cpu."
Jed Estep [Tue, 12 Jul 2016 22:54:56 +0000 (22:54 +0000)]
Merge \"Basic A/B bootloader tests via bootctl\"
am:
0ef925afc0
Change-Id: I3f438cef798c59c856326b389a1b162cdd31e33a
Daniel Rosenberg [Tue, 12 Jul 2016 22:47:52 +0000 (22:47 +0000)]
Merge "Basic A/B bootloader tests via bootctl"
Yabin Cui [Fri, 8 Jul 2016 20:56:48 +0000 (13:56 -0700)]
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.
Jed Estep [Fri, 20 May 2016 23:58:59 +0000 (16:58 -0700)]
Basic A/B bootloader tests via bootctl
Change-Id: Ie7868cc205859c657e905e60ab6928517f02edb8
Yabin Cui [Tue, 12 Jul 2016 22:06:39 +0000 (22:06 +0000)]
Merge \"simpleperf: fix test record_cmd.dump_symbols.\"
am:
b17b6c3edf
Change-Id: I271713aad752d90d40dc3ae19ee5aa6ba7adab43
Treehugger Robot [Tue, 12 Jul 2016 21:57:48 +0000 (21:57 +0000)]
Merge "simpleperf: fix test record_cmd.dump_symbols."
Colin Cross [Tue, 12 Jul 2016 20:03:02 +0000 (13:03 -0700)]
Convert libpagemap Android.mk to Android.bp
Change-Id: I551a7f7ce46267bb609e5ed8bd8dafc00b9fbab7
Yabin Cui [Tue, 12 Jul 2016 21:16:22 +0000 (21:16 +0000)]
Merge \"simpleperf: check kernel symbol addresses before dumping them.\"
am:
7c3c50097e
Change-Id: I1a77f9e0f8ff19eb4c524cb13a9fd6ec98c38c14
Yabin Cui [Tue, 12 Jul 2016 21:14:08 +0000 (21:14 +0000)]
Merge "simpleperf: check kernel symbol addresses before dumping them."
Chih-Hung Hsieh [Tue, 12 Jul 2016 20:59:59 +0000 (20:59 +0000)]
Merge \"Fix google-explicit-constructor warnings in simpleperf.\"
am:
ee87476e38
Change-Id: I2d76b015f42b24e5e467b406b9f560a682b20f69
Chih-Hung Hsieh [Tue, 12 Jul 2016 20:59:57 +0000 (20:59 +0000)]
Merge \"Fix google-explicit-constructor warnings in memory_replay.\"
am:
aca5561017
Change-Id: I2f562b4bd4ae9a8186598a9deecf78d9046a06a1
Yabin Cui [Fri, 8 Jul 2016 18:27:48 +0000 (11:27 -0700)]
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
Chih-hung Hsieh [Tue, 12 Jul 2016 20:48:58 +0000 (20:48 +0000)]
Merge "Fix google-explicit-constructor warnings in simpleperf."
Chih-hung Hsieh [Tue, 12 Jul 2016 20:48:49 +0000 (20:48 +0000)]
Merge "Fix google-explicit-constructor warnings in memory_replay."
Chih-Hung Hsieh [Tue, 12 Jul 2016 18:47:28 +0000 (11:47 -0700)]
Fix google-explicit-constructor warnings in memory_replay.
* Declare explicit conversion constructors.
Bug:
28341362
Change-Id: Ia6613392bd68f170ec2d7b0b033b7476aff27e0f
Test: build with clang-tidy
Chih-Hung Hsieh [Tue, 12 Jul 2016 18:35:16 +0000 (11:35 -0700)]
Fix google-explicit-constructor warnings in simpleperf.
* Declare explicit conversion constructors.
Bug:
28341362
Change-Id: I9d5b764fe004e291cbdd212617df6f385c31df25
Test: build with clang-tidy
Yabin Cui [Tue, 12 Jul 2016 01:44:16 +0000 (01:44 +0000)]
Merge \"simpleperf: add min_vaddr in DsoRecord.\"
am:
78f176baaf
Change-Id: Ibb2df64090c24129299587515c7661d6137a2f16
Yabin Cui [Tue, 12 Jul 2016 01:36:00 +0000 (01:36 +0000)]
Merge "simpleperf: add min_vaddr in DsoRecord."
Yabin Cui [Tue, 12 Jul 2016 00:04:54 +0000 (17:04 -0700)]
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
Yabin Cui [Tue, 12 Jul 2016 00:17:49 +0000 (00:17 +0000)]
Merge \"simpleperf: change the separator of --symbols option from comma to semicolon.\"
am:
2d6e0d6511
Change-Id: I97562f44f1624b7e8cc7e3325942368f64e492ca
Yabin Cui [Tue, 12 Jul 2016 00:04:40 +0000 (00:04 +0000)]
Merge "simpleperf: change the separator of --symbols option from comma to semicolon."
Yabin Cui [Mon, 11 Jul 2016 23:27:52 +0000 (16:27 -0700)]
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.
Yabin Cui [Mon, 11 Jul 2016 20:50:01 +0000 (13:50 -0700)]
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.
Zhiting Zhu [Mon, 11 Jul 2016 20:18:29 +0000 (20:18 +0000)]
Merge \"Generate output of simpleperf in csv format\"
am:
3c614cb52d
Change-Id: Ib9dc3e74de66bb9bff2081ac13e4185a1a0c2149
swan [Mon, 11 Jul 2016 20:09:46 +0000 (20:09 +0000)]
Merge "Generate output of simpleperf in csv format"
Zhiting Zhu [Sun, 10 Jul 2016 02:38:08 +0000 (19:38 -0700)]
Generate output of simpleperf in csv format
CSV format is easier to parse and could be useful for testing.
Change-Id: I444bddfa181b48c79ec48ef7595b5d8a25e62e12
Elliott Hughes [Fri, 8 Jul 2016 17:05:27 +0000 (17:05 +0000)]
Merge \"Remove call to no-op klog_init.\"
am:
2b66a6f2bd
Change-Id: I5fd576a1e9c05c035d8618091a72f9c5a5f90333
Elliott Hughes [Fri, 8 Jul 2016 17:00:24 +0000 (17:00 +0000)]
Merge "Remove call to no-op klog_init."
Yabin Cui [Thu, 7 Jul 2016 23:14:51 +0000 (23:14 +0000)]
Merge \"simpleperf: don\'t warn if child process was killed by simpleperf.\"
am:
e26676ef00
Change-Id: I2471c50bbab5a8246f243e54fc280e9b050fa45c
Yabin Cui [Thu, 7 Jul 2016 23:14:50 +0000 (23:14 +0000)]
Merge \"simpleperf: give suggestion when mmap fails.\"
am:
57dac93cf3
Change-Id: I352cd43a30770fc235b65e53aebb08c52f462c31
Yabin Cui [Thu, 7 Jul 2016 23:06:32 +0000 (23:06 +0000)]
Merge "simpleperf: don't warn if child process was killed by simpleperf."
Yabin Cui [Thu, 7 Jul 2016 22:00:10 +0000 (15:00 -0700)]
simpleperf: don't warn if child process was killed by simpleperf.
Bug:
29574526
Change-Id: I3ae95ed95703bcd380ed086d3fbf7ae2830feed5
Yabin Cui [Thu, 7 Jul 2016 23:04:42 +0000 (23:04 +0000)]
Merge "simpleperf: give suggestion when mmap fails."
Yabin Cui [Thu, 7 Jul 2016 22:21:02 +0000 (22:21 +0000)]
Merge \"simpleperf: fix build id check of files in symfs.\"
am:
3077c433d5
Change-Id: If76de1f0a61d1184491b03fb6fdc685287a95450
Treehugger Robot [Thu, 7 Jul 2016 22:15:02 +0000 (22:15 +0000)]
Merge "simpleperf: fix build id check of files in symfs."
Yabin Cui [Thu, 7 Jul 2016 21:42:54 +0000 (14:42 -0700)]
simpleperf: give suggestion when mmap fails.
Bug:
29574526
Change-Id: I72cf2d0e1d15a561314d3c043650479b2c6773ff
Yabin Cui [Thu, 7 Jul 2016 20:53:33 +0000 (13:53 -0700)]
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
Yabin Cui [Thu, 7 Jul 2016 20:55:43 +0000 (20:55 +0000)]
Merge \"simpleperf: add --symfs option for record command.\"
am:
a4f2c631cb
Change-Id: I3a560d98de1b1cd7f2a1302fd9d21f3a9d35cc0e
Yabin Cui [Thu, 7 Jul 2016 20:46:13 +0000 (20:46 +0000)]
Merge "simpleperf: add --symfs option for record command."