You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
hi, have been using (and loving) samply for a few weeks. it's really an incredible tool, thank you.
my goal is to profile nodejs and electron processes. the former already works quite well on linux, thanks to all the work done towards jitdump support.
i also see several mentions of jitdump macos support, in the changelog and in commits like 2732e49
my understanding is that jitdumps are ELF-based, but it appears that's still the standard format ingested on macos? thus, programs are expected to output jitdumps the same way on both linux and macos, along with the map files in /tmp.
I was able to start emitting perf-pid.map and jit-pid.dump files.
The map files are working in samply 🎉
The jitdumps are being picked by the dyld preload, and I confirmed that samply is able to parse the file to see the load/unwind sections. However, symbolization doesn't work for some reason. Any pointers?
hi, have been using (and loving)
samply
for a few weeks. it's really an incredible tool, thank you.my goal is to profile nodejs and electron processes. the former already works quite well on linux, thanks to all the work done towards jitdump support.
i also see several mentions of jitdump macos support, in the changelog and in commits like 2732e49
my understanding is that jitdumps are ELF-based, but it appears that's still the standard format ingested on macos? thus, programs are expected to output jitdumps the same way on both linux and macos, along with the map files in /tmp.
in the case of nodejs, it appears the jitdump code is all gated away to be linux-specific atm: https://github.com/nodejs/node/blob/985262a4a8221d993df82e134623d8437d92b4c5/deps/v8/src/diagnostics/perf-jit.cc#L32-L33
am i correct that the first step here would be to enable this codepath inside nodejs on darwin? what else would be required.
EDIT: found https://bugzilla-dev.allizom.org/show_bug.cgi?id=1827214 so it looks like I am on the right track
The text was updated successfully, but these errors were encountered: