Tommy Reilly
Tommy Reilly
# Notes on how to approach this issue ## Lua has a couple existing profile solutions: 1. [LuaJIT built in signal/timer based profiler](https://luajit.org/ext_profiler.html) 2. [eBPF profiler written in C](https://github.com/apache/apisix-profiler) The...
> The nice thing about that is that debugger-level debuginfo don't have to be present in order to do this, we just need to find which Go version a binary...
> Yeah Go binaries don't even have the DWARF required for this enabled by default, you need to pass special flags for it. The only thing that cannot be stripped...
Is there a discussion somewhere of what this is fixing? For the record I think mixing non-functional cosmetic changes with functional changes is bad practice and just makes things harder...
Rebased and squashed to hopefully make further review and rebasing easier. Thanks for all the reviews so far!
This is ready for another look, I *think* all review concerns were addressed, thanks for checking it out!
@florianl and @fabled did you guys have any more concerns about this? If we think it needs more bake time can we look at merging it disabled to avoid further...
Okay some dynamic offset lookups were required to get static cgo binaries to work on amd64 and arm64. I separated out those changes in separate commits which hopefully makes it...
Thanks @fabled! Rebased and addressed all comments, I'll make a TODO to look at the pclntab stuff. @florianl can you give this another gander? Thanks!
Possibly fixed: https://github.com/open-telemetry/opentelemetry-ebpf-profiler/pull/393