-
Notifications
You must be signed in to change notification settings - Fork 253
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
strange " Could not find ELF file for ." when opening a CentOS7 perf.data file on Debian10 #551
Comments
It's not searching for |
Good, that you've found this. Should we move the following to a separate issue?
|
No, as I don't see this as an issue? you only guess without showing any proof |
You're reight about the missing proof. Rechecked with current hotspot-perfparser alone - and could not reproduce it.
With the additional extra paths the warning message vanishes, but the instructions and computing time increases.
Therefore, caching the "empty" seems to primarily drop the amount of warnings from 2 + 3 to 1 + 1. ... but I can check that with the debugger, so: This message comes from
... but again I have no idea where to cache the result. |
Describe the bug
While I don't expect the details to work (because that's a different system) the paths searched look quite wrong.
To Reproduce
Steps to reproduce the behavior:
hotspot --appPath /bin perf.data.true.v2
Expected behavior
Searching for
/lib64/libc-2.17.so
(that's the place the library is on CentOS7) instead of64/libc-2.17.so
; not searching for.
.Additional context
If I get the library from
http://mirror.centos.org/centos/7/os/x86_64/Packages/glibc-2.17-317.el7.x86_64.rpm
, extract it and add the relevant--extraLibPaths /tmp/centos7/lib64
the error messages for the strange "64" folder are removed and the recording can be seen.Furthermore, testing hints at that with each output of
we see, the perfparser binary takes longer. I guess perfparerser caches registered elf files (return of
PerfSymbolTable::registerElf()
), but only those that can be resolved.In this case it would be good to also cache "invalid" / empty returns; allowing to also decrease the wait time for perf recordings with missing elf files in general and for this repo in the specific tests - see 69ad59b#r1382617529.
The text was updated successfully, but these errors were encountered: