Closed Bug 517824 Opened 10 years ago Closed 10 years ago

Breakpad Linux Dumper: submit DWARF-on-Linux

Categories

(Toolkit :: Crash Reporting, defect)

All
Linux
defect
Not set

Tracking

()

RESOLVED FIXED

People

(Reporter: jimb, Assigned: jimb)

References

Details

I have a patch series that adds DWARF support to the Breakpad Linux dumper; the patch series is published at:

  http://hg.mozilla.org/users/jblandy_mozilla.com/breakpad-mq/

The patched dumper produces plausible output.  However, it has not yet been submitted to the Breakpad project because it hasn't been given a thorough testing.  This testing would include, at the least, compiling the same program with STABS and DWARF, dumping each, and comparing the output.  Ideally, it would include tests that provide complete source-code coverage of the dumper.
Blocks: 517829
Assignee: nobody → jim
Depends on: 524409
Depends on: 524410
No longer depends on: 524410
This now has a pretty thorough suite of unit tests, and has been submitted to Google for review.  Patches and work are in the Breakpad issue database, with URLs like:
http://breakpad.appspot.com/51004/show

51004 - Linux Breakpad Dumper: support running unit tests under valgrind or other wrapper programs.
50002 - Breakpad DWARF Reader: Also look for DWARF in sections with the proper names.
50004 - Breakpad DWARF Reader: Change LineInfoHandler::AddLine to provide the line's length.
49009 - Breakpad DWARF Parser: Improved DWARF-processing interface.
50005 - Breakpad Linux Dumper: Add DWARF support.
49006 - Google Breakpad DWARF reader: Add a handler function for DIE references.
50003 - Breakpad DWARF Reader: Add DWARF language enumeration values.
Status: NEW → ASSIGNED
All patches approved by Google, but the last one has one comment pending.

Weekend!!
All patches landed. Google Breakpad now supports the DWARF debugging format on Linux.
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.