diff options
author | Yabin Cui <yabinc@google.com> | 2016-08-25 10:46:40 -0700 |
---|---|---|
committer | Yabin Cui <yabinc@google.com> | 2016-08-25 10:46:40 -0700 |
commit | cb6c901e01739ec10942556b6171029b1117c772 (patch) | |
tree | a13106140d8b3f6537da10cf0261b53849f05d22 /simpleperf/dso.cpp | |
parent | db7a1c571ac23c23b621dce5ce23612591df69df (diff) | |
download | extras-cb6c901e01739ec10942556b6171029b1117c772.tar.gz |
simpleperf: print file path used for reading symbols.
It helps users to find out which files are used for
reporting.
Bug: http://b/29574526
Change-Id: I3d608e61c50471632c50bf6e8f6f9e45c63fc4b4
Test: run `simpleperf report --log verbose` manually.
Diffstat (limited to 'simpleperf/dso.cpp')
-rw-r--r-- | simpleperf/dso.cpp | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/simpleperf/dso.cpp b/simpleperf/dso.cpp index 69049e28..bd6c1baa 100644 --- a/simpleperf/dso.cpp +++ b/simpleperf/dso.cpp @@ -265,6 +265,7 @@ static void VmlinuxSymbolCallback(const ElfFileSymbol& elf_symbol, Dso* dso) { bool CheckReadSymbolResult(ElfStatus result, const std::string& filename) { if (result == ElfStatus::NO_ERROR) { + LOG(VERBOSE) << "Read symbols from " << filename << " successfully"; return true; } else if (result == ElfStatus::NO_SYMBOL_TABLE) { // Lacking symbol table isn't considered as an error but worth reporting. @@ -374,7 +375,7 @@ bool Dso::LoadElfFile() { std::bind(ElfFileSymbolCallback, std::placeholders::_1, this, SymbolFilterForDso)); if (result == ElfStatus::NO_ERROR) { - return true; + return CheckReadSymbolResult(result, "/usr/lib/debug" + path_); } } ElfStatus result = ParseSymbolsFromElfFile( |