OSDN Git Service
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."
Yabin Cui [Thu, 7 Jul 2016 17:56:47 +0000 (10:56 -0700)]
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
Yabin Cui [Thu, 7 Jul 2016 18:43:54 +0000 (18:43 +0000)]
Merge \"simpleperf: add symbol for .plt section.\"
am:
13a21e1df3
Change-Id: I0e5dc7646c4225d2226b4ba0ba67a19c883a23e1
Yabin Cui [Thu, 7 Jul 2016 18:23:40 +0000 (18:23 +0000)]
Merge "simpleperf: add symbol for .plt section."
Yabin Cui [Thu, 7 Jul 2016 01:29:00 +0000 (18:29 -0700)]
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
Yabin Cui [Thu, 7 Jul 2016 17:18:04 +0000 (17:18 +0000)]
Merge \"Simpleperf: add auto generated summaries in stat command.\"
am:
11615de832
Change-Id: I41e95398507e62f442d948273c12cb65e5d1730d
Yabin Cui [Thu, 7 Jul 2016 17:09:48 +0000 (17:09 +0000)]
Merge "Simpleperf: add auto generated summaries in stat command."
Yabin Cui [Thu, 23 Jun 2016 03:58:52 +0000 (20:58 -0700)]
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.
Yabin Cui [Thu, 7 Jul 2016 00:15:12 +0000 (00:15 +0000)]
Merge \"simpleperf: add symbol filter for report command.\"
am:
90c2cd36c6
Change-Id: Iacd1999b02878ed129409a4d538f0d8d0b141fe9
Yabin Cui [Thu, 7 Jul 2016 00:15:10 +0000 (00:15 +0000)]
Merge \"simpleperf: fix RecordCache.\"
am:
fb1f20eafe
Change-Id: If177bbdb589e92aab7d81c1831071561c0c5cdf1
Yabin Cui [Thu, 7 Jul 2016 00:04:31 +0000 (00:04 +0000)]
Merge "simpleperf: add symbol filter for report command."
Yabin Cui [Thu, 7 Jul 2016 00:04:18 +0000 (00:04 +0000)]
Merge "simpleperf: fix RecordCache."
Yabin Cui [Wed, 6 Jul 2016 19:26:13 +0000 (12:26 -0700)]
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
Yabin Cui [Wed, 6 Jul 2016 19:01:35 +0000 (12:01 -0700)]
simpleperf: fix RecordCache.
RecordCache::Push(vector<..>) doesn't update last_time_, this makes
RecordCache don't pop any record before PopAll().
Bug:
29581559
Change-Id: Icea806346b7ad812e606eaf05747797b766ebd71
Test: run simpleperf_unit_test.
Yabin Cui [Sat, 2 Jul 2016 00:18:07 +0000 (00:18 +0000)]
Merge \"Simpleperf: add vaddr_in_file sort key in report command.\"
am:
5f99cdc04b
Change-Id: I3fcd9d63a592f1e30b3623448124f33f22091840
Yabin Cui [Sat, 2 Jul 2016 00:10:46 +0000 (00:10 +0000)]
Merge "Simpleperf: add vaddr_in_file sort key in report command."
Elliott Hughes [Wed, 29 Jun 2016 23:18:12 +0000 (16:18 -0700)]
Remove call to no-op klog_init.
Change-Id: I3a55449f594c629edc75e4aa5b91b27c72ecd8ca
Yabin Cui [Wed, 29 Jun 2016 19:18:11 +0000 (12:18 -0700)]
Simpleperf: add vaddr_in_file sort key in report command.
Currently report command can't report more details than function name.
After adding vaddr_in_file sort key, it can report the place of the
instruction being sampled. vaddr_in_file is the instruction's virtual
address in elf file, which matches output generated by objdump.
Bug:
29826956
Test: run simpleperf_unit_test.
Change-Id: Ifad4dfb7c60014a03c01ffbfd0b972858f1a4884
Rom Lemarchand [Tue, 28 Jun 2016 21:55:23 +0000 (21:55 +0000)]
Merge \"pagingtest: fix large read errors\"
am:
d937202a3b
Change-Id: If90cfaa7ed7769b760c57aace300fa2b10bed8df
Treehugger Robot [Tue, 28 Jun 2016 21:30:42 +0000 (21:30 +0000)]
Merge "pagingtest: fix large read errors"
Than McIntosh [Tue, 28 Jun 2016 18:12:53 +0000 (18:12 +0000)]
Merge \"Switch to new encoding scheme for large records.\"
am:
0380bb5f26
Change-Id: Ib6daa6183bcfb989ef52b71d4c9d012fd6884529
Than McIntosh [Tue, 28 Jun 2016 18:07:12 +0000 (18:07 +0000)]
Merge "Switch to new encoding scheme for large records."
Yabin Cui [Tue, 28 Jun 2016 17:18:23 +0000 (17:18 +0000)]
Merge \"Simpleperf: Add SPLIT and SPLIT_END records to handle big records.\"
am:
6d20b67da5
Change-Id: I105cbdd688b50ce64450e8819b8d7ef28c15b838
Treehugger Robot [Tue, 28 Jun 2016 17:09:26 +0000 (17:09 +0000)]
Merge "Simpleperf: Add SPLIT and SPLIT_END records to handle big records."
Than McIntosh [Tue, 28 Jun 2016 15:50:11 +0000 (11:50 -0400)]
Switch to new encoding scheme for large records.
Move to the new SPLIT/SPLIT_END encoding scheme used
by simpleperf for records with oversized (more than 2^16 bytes)
payloads.
Bug:
29818021
Change-Id: I40126748dbe9b07b107c85b891f7aafbc556ae1f
Rom Lemarchand [Tue, 28 Jun 2016 14:51:33 +0000 (07:51 -0700)]
pagingtest: fix large read errors
Some kernels have a limit on the length of reads from urandom - fix
Change-Id: I67dda25c2e43121da1470a56e40d03c59fd5fabe
Yabin Cui [Tue, 28 Jun 2016 00:21:37 +0000 (17:21 -0700)]
resolve merge conflicts of
a486a4b to nyc-dev-plus-aosp
Change-Id: Iad2f0b498d7abd2eb52116a32517bfee20129f05
Yabin Cui [Mon, 27 Jun 2016 23:00:20 +0000 (16:00 -0700)]
Simpleperf: fix flaky tests.
Simpleperf stat/record command may need 1 second to set security.perf_harden
property. So the monitored child threads should run longer than 1 second.
Bug:
29763505
Change-Id: I9bf6dcba03f3fc5c91130f38a08ce0062e104103
Yabin Cui [Fri, 24 Jun 2016 00:11:14 +0000 (17:11 -0700)]
Simpleperf: Add SPLIT and SPLIT_END records to handle big records.
Previously we split KernelSymbolRecord because it is > 65535. Then
I found TracingDataRecord can also be > 65535. So it is better to
handle big records when reading and writing perf.data.
record_file_writer.cpp splits a big record into multiple SPLIT
records followed by a SPLIT_END record, and record_file_reader.cpp
restores the big record when reading SPLIT and SPLIT_END records.
Also Add RecordHeader to represent record having size > 65535.
Bug:
29581559
Change-Id: I0b4556988f77b3431c7f1a28fce65cf225d6a067
Test: run simpleperf_unit_test.
Dan Willemsen [Thu, 23 Jun 2016 01:12:30 +0000 (01:12 +0000)]
Merge \\"Dist simpleperf executables in SDK targets\\" am:
965a99e493
am:
aab6d80c06
Change-Id: I1d71111e4238ca9a2b87045d46485369e0271984
Dan Willemsen [Thu, 23 Jun 2016 01:09:32 +0000 (01:09 +0000)]
Merge \"Dist simpleperf executables in SDK targets\"
am:
965a99e493
Change-Id: I35d4bb08697e32c889a5d0a4baa82de7ec79c9b8
Treehugger Robot [Thu, 23 Jun 2016 01:00:39 +0000 (01:00 +0000)]
Merge "Dist simpleperf executables in SDK targets"
Yabin Cui [Thu, 23 Jun 2016 00:02:11 +0000 (00:02 +0000)]
Merge \\"Simpleperf: use libc++_static on host.\\" am:
185f314239
am:
b76a0affbd
Change-Id: I9d6da22c618df56e5da08413ceb7eca6ce965a32
Yabin Cui [Wed, 22 Jun 2016 23:59:16 +0000 (23:59 +0000)]
Merge \"Simpleperf: use libc++_static on host.\"
am:
185f314239
Change-Id: Ia1b208c8e6120caef5fc7bb0cd2b0ab488fe1d32
Yabin Cui [Wed, 22 Jun 2016 23:49:12 +0000 (23:49 +0000)]
Merge "Simpleperf: use libc++_static on host."
Dan Willemsen [Wed, 22 Jun 2016 23:41:36 +0000 (16:41 -0700)]
Dist simpleperf executables in SDK targets
Bug:
28911532
Change-Id: I5ec387ee1bbdff1e62788aa1fe3995cb44b37754
Yabin Cui [Wed, 22 Jun 2016 22:58:43 +0000 (22:58 +0000)]
Merge \\"Simpleperf: report lost record count and warn if 10% records are lost.\\" am:
142b63f4e7
am:
d3e37d55f1
Change-Id: I0cad51ed45ababb34d91f8b2a1359920da3b6ae4
Yabin Cui [Wed, 22 Jun 2016 22:56:08 +0000 (22:56 +0000)]
Merge \"Simpleperf: report lost record count and warn if 10% records are lost.\"
am:
142b63f4e7
Change-Id: Id09c49521f8ae2d0e7b11a9607e6c5d2bb1cbad6
Yabin Cui [Wed, 22 Jun 2016 22:48:25 +0000 (22:48 +0000)]
Merge "Simpleperf: report lost record count and warn if 10% records are lost."
Yabin Cui [Wed, 22 Jun 2016 22:30:37 +0000 (15:30 -0700)]
Simpleperf: use libc++_static on host.
Bug:
28911532
Change-Id: I5cc97fc35f5c5225ea81bdaec04cc22d9a99790f
Test: Use readelf to make sure simpleperf on host doesn't dynamically link libc++.so.