diff options
author | Yabin Cui <yabinc@google.com> | 2016-05-25 14:08:05 -0700 |
---|---|---|
committer | Yabin Cui <yabinc@google.com> | 2016-05-31 14:16:14 -0700 |
commit | b42129797f17f990ea5a88c6bdc7446000cc361c (patch) | |
tree | cbc62824b5cd6bbaa0e012b21529d240fa0a868c /simpleperf/cmd_report_test.cpp | |
parent | c8642b2dcfd60f12a0cdbd1f9361967d264d2a3a (diff) | |
download | extras-b42129797f17f990ea5a88c6bdc7446000cc361c.tar.gz |
simpleperf: record kernel symbols in perf.data.
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
Diffstat (limited to 'simpleperf/cmd_report_test.cpp')
-rw-r--r-- | simpleperf/cmd_report_test.cpp | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/simpleperf/cmd_report_test.cpp b/simpleperf/cmd_report_test.cpp index 4c2b4978..befa1111 100644 --- a/simpleperf/cmd_report_test.cpp +++ b/simpleperf/cmd_report_test.cpp @@ -274,6 +274,12 @@ TEST_F(ReportCommandTest, report_more_than_one_event_types) { ASSERT_NE(content.find("cpu-clock"), std::string::npos); } +TEST_F(ReportCommandTest, report_kernel_symbol) { + Report(PERF_DATA_WITH_KERNEL_SYMBOL); + ASSERT_TRUE(success); + ASSERT_NE(content.find("perf_event_comm_output"), std::string::npos); +} + #if defined(__linux__) static std::unique_ptr<Command> RecordCmd() { |