Age | Commit message (Collapse) | Author |
|
Use the 2-space configuration by default and disable any formatting for
the demo/ subdirectory. The following command was used to generate this
change:
$ find . \( -name \*.cpp -o -name \*.h \) -exec clang-format \
--style=file -i {} \;
Test: mm
Change-Id: I89ec1f18f6e352f40cfa1a770087f3b586b1d7cb
|
|
The change is to reduce sample lost rate when recording
dwarf-based callgraph.
It includes below changes:
1. Add RecordBuffer class to store record data.
2. Add RecordReadThread to create a separate high priority
thread reading records from kernel buffer to a RecordBuffer.
3. Cut stack data in sample records when free space in
record buffer is below low level.
4. Drop sample records when free space in record buffer is
below critical level.
5. Use different record buffer sizes for system wide profiling
and non system wide profiling.
6. Refactor code replacing regs and stack data to callchains
in SampleRecord.
On walleye, set cpu percentage for profiling to 50:
$ ./old_simpleperf record -a -g --duration 30 --log debug
simpleperf I cmd_record.cpp:545] Samples recorded: 80524. Samples lost: 22993.
$ ./new_simpleperf record -a -g --duration 30 --log debug
simpleperf I cmd_record.cpp:555] Samples recorded: 99776. Samples lost: 0.
Bug: 110174247
Test: run simpleperf_unit_test.
Test: run simpleperf manually.
Change-Id: I10c8a090abc36e9feb712357cbb20a20b205af14
|
|
And other small changes:
add time when building comm record.
move some Move*BinaryFormat to utils.h.
Handle wrong symbol whoes address can be ULLONG_MAX.
Bug: http://b/30974760
Test: simpleperf_unit_test.
Change-Id: I2956d3c4b781c580fe93a6e5b77e0469f7f4f43f
|
|
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
|
|
To better support kernel profiling, record kernel symbols in perf.data
when necessary. An option --no-dump-kernel-symbols is added in
record command to always avoid recording kernel symbols.
The way to handle all zero /proc/modules and /proc/kallsyms is
improved. Add Better support in finding symbols for kernel modules.
Bug: 27403614
Change-Id: I470151c54f8a45ad1c101c1b94490e33d7fd7485
|
|
Bug: 19483574
Change-Id: Ie2acd8a157bca9ad3c01a2e4b37e139aba89670f
|
|
Bug: 19483574
Change-Id: Ia65cb12804a6dffa440501736a6229b2f7248958
|