Dave Huseby [:huseby] Assignee
	Description • 11 years ago

It is assumed that the dwarf info can help fix stack walks when the profile sig handler fires and the PC is in the prologue/epilogue of a function.  When that occurs, the arm exception handling index/microcode doesn't work because the stack frame isn't fully set up/taken down.

This bug is to track the investigation of how dwarf information helps fix this problem, and can we steal/use it to make the stack walking more robust when using the arm exception handling index data.

This is not a high priority since the having the profile sig handler fire when the PC is in an prologue/epilogue is relatively rare.  And this is just a profiler, if we lose a small percentage of samples due to bogus stack walks, we'll still get plenty of useful profiling data.

	Dave Huseby [:huseby] Assignee
	Comment 1 • 11 years ago

I'll take a look at what DWARF data can give us that we may be able to use to fix the PC when the profile signal handler gets called during a function prologue/epilogue.

	Dave Huseby [:huseby] Assignee
	Comment 6 • 11 years ago

I attached some useful documents on this subject.  The current state of my research is this:

1. We generate, but strip DWARF information from our binaries.
2. DWARF not only includes prologue/epilogue annotations for locations in the binary, but also explicit call frame information specifically for stack unwinding.
3. We may be able to extract the bits we need during the stripping process for use by the profiler.  Or alternatively, we could try to rework the build scripts so they don't strip out the DWARF data.

	Dave Huseby [:huseby] Assignee
	Comment 7 • 11 years ago

GCC supports -gsplit-dwarf which causes the compiler to put as much of the DWARF debugging information into a separate file with a .dwo extension.  This could be a shortcut towards using DWARF data in the profiler without having to deal with bigger binaries as a result.

	Dave Huseby [:huseby] Assignee
	Comment 8 • 11 years ago

Scratch that, -gsplit-dwarf seems to have been removed from GCC.  At least it isn't present in the toolchain we use to compile B2G.

	Dave Huseby [:huseby] Assignee
	Comment 9 • 11 years ago

My notes are here: https://wiki.mozilla.org/User:Dhuseby#Bug_895162_Notes