735829 - Report PDB name in chrome hang reports

Status: RESOLVED FIXED

(Core :: General, defect)

Description

[Vladan Djeric (:vladan)]

Currently, chrome hang reports report a variety of info about shared libraries loaded in memory including the shared library name, address information, and PDB Signature + PDB Age. This information is used for server-side symbolication of hang stacks. However, symbol files are stored on the server under their PDB name and not the shared library name, e.g. the XUL library symbols are stored in a directory named xul.pdb instead of xul.dll.

As it turns out, the shared library names do not always neatly map to their PDB filenames, e.g. on my machine c:\Windows\SysWOW64\ntdll.dll has a PDB named wntdll.pdb.

Therefore, chrome hang reports should also report the corresponding PDB name for shared libraries. The PDB name can be obtained from the library image, the same way as PDB signatures and PDB ages.

Comment 1

[Vladan Djeric (:vladan)]

Attachment: Report PDB Name, v1

Comment 2

[(no longer active)]

Comment on attachment 606671 [details] [diff] [review]
Report PDB Name, v1

Review of attachment 606671 [details] [diff] [review]:
-----------------------------------------------------------------

::: tools/profiler/shared-libraries-win32.cc
@@ +50,5 @@
>  {
>    uint32_t signature;
>    GUID pdbSignature;
>    uint32_t pdbAge;
> +  char pdbFileName[1];

Make it a char* and take out the cast below.

Comment 3

[Mounir Lamouri (:mounir)]

https://hg.mozilla.org/mozilla-central/rev/102524521984

Comment 4

[Vladan Djeric (:vladan)]

Attachment: Very minor fix: add quotes around PDB name string

Fixing small bug introduced in the first patch

Comment 5

[Vladan Djeric (:vladan)]

https://hg.mozilla.org/integration/mozilla-inbound/rev/d69e6105b5b1

Comment 6

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/d69e6105b5b1