Closed
Bug 1270190
Opened 9 years ago
Closed 8 years ago
Missing symbols for Windows ntdll.dll@0x4bb7a and kernelbase.dll@0x10ab
Categories
(Socorro :: General, task)
Socorro
General
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: mccr8, Assigned: ted)
References
Details
The signature [@ shutdownhang | ntdll.dll@0x4bb7a ] is currently a top crash on Nightly. It would be nice if ntdll.dll@0x4bb7a was symbolized. Also, the next frame kernelbase.dll@0x10ab should be symbolized. They look related to the Windows implementation of CondVar. This would let us add them to the skip list or the irrelevant list.
https://crash-stats.mozilla.com/report/index/2ac95b88-47d4-4c72-a7a0-015712160428
Comment 1•9 years ago
|
||
Peter, who handles Windows symbol import nowadays?
mccr8 FWIW since this is a server-side change it needs to be in the Socorro product instead of the Toolkit/client world.
Component: Breakpad Integration → General
Flags: needinfo?(peterbe)
Product: Toolkit → Socorro
Reporter | ||
Comment 2•9 years ago
|
||
(In reply to Benjamin Smedberg [:bsmedberg] from comment #1)
> mccr8 FWIW since this is a server-side change it needs to be in the Socorro
> product instead of the Toolkit/client world.
Thanks for the information. I guess symbolizing in the client would not be practical...
Comment 3•9 years ago
|
||
(In reply to Benjamin Smedberg [:bsmedberg] from comment #1)
> Peter, who handles Windows symbol import nowadays?
>
Ted does. He still has a manual process whereby he uses CSV files spat out by our processor. I don't remember if he uses TaskCluster or just a script on his laptop.
We initiated a project to entirely fix that with a real-time symbol fetcher thing. But we couldn't get it started because deep down it relies on a Windows only executable and our only way to run that would be by using Wine and that's not an option.
I believe the latest, potential idea is if that executable has been or can be ported to *nix so we can proceed to build that real-time symbol-fetcher thing. Ted's keeping an eye on it.
Flags: needinfo?(peterbe)
Assignee | ||
Comment 4•9 years ago
|
||
This crash is on Win64. Unfortunately we are not fetching symbols for Win64 system libraries(bug 1140358), because we currently can't get stack unwind info for them, because dump_syms needs to have the EXE or DLL file present to dump stack unwind info, and the script I use for scraping missing symbols from Microsoft's symbol server uses the missing-symbols CSV as input, which does not have enough info to fetch the associated DLLs.
We were previously dumping symbols for system libraries on Win64, but without stack unwind info we'd get symbols but really awful stacks, so I disabled that so as to not produce misleading results.
We either need to fix bug 1140937, fix bug 1117741, or check if this unlanded Breakpad dump_syms patch allows us to get the unwind info out of PDB files from Microsoft's symbol server:
https://codereview.chromium.org/1773183003/
(In reply to Peter Bengtsson [:peterbe] from comment #3)
> I believe the latest, potential idea is if that executable has been or can
> be ported to *nix so we can proceed to build that real-time symbol-fetcher
> thing. Ted's keeping an eye on it.
That's something I've poked at but not something I'm actively working on.
Updated•9 years ago
|
Comment 5•9 years ago
|
||
[@ shutdownhang | ntdll.dll@0x4bb7a] is the top #6 crash in Nightly 20160509040557 but it's actually an aggregation of different shutdown hangs. Fixing this would break them apart.
Reporter | ||
Comment 6•9 years ago
|
||
[@ shutdownhang | ntdll.dll@0xa5164] is similarly the top #2 Windows crasher on 5/11.
Comment 8•8 years ago
|
||
(In reply to Ted Mielczarek [:ted.mielczarek] from comment #4)
> to dump stack unwind info, and the script I use for scraping missing symbols
> from Microsoft's symbol server uses the missing-symbols CSV as input, which
> does not have enough info to fetch the associated DLLs.
What else information are needed for fetching the DLLs? Since VS can show the symbols properly by openning the minidump.
Flags: needinfo?(ted)
Comment 9•8 years ago
|
||
(In reply to Ting-Yu Chou [:ting] from comment #8)
> What else information are needed for fetching the DLLs? Since VS can show
> the symbols properly by openning the minidump.
I found it in bug 1140937, filename and the code_id.
Assignee | ||
Comment 11•8 years ago
|
||
(In reply to Andrew McCreight [:mccr8] from comment #0)
> The signature [@ shutdownhang | ntdll.dll@0x4bb7a ] is currently a top crash
> on Nightly. It would be nice if ntdll.dll@0x4bb7a was symbolized. Also, the
> next frame kernelbase.dll@0x10ab should be symbolized. They look related to
> the Windows implementation of CondVar. This would let us add them to the
> skip list or the irrelevant list.
>
> https://crash-stats.mozilla.com/report/index/2ac95b88-47d4-4c72-a7a0-
> 015712160428
Since fixing bug 1140358 we have acquired symbols for the modules in this report, such as:
http://symbols.mozilla.org/ntdll.pdb/0A2571A3E36A4F909B719773CC176B032/ntdll.sym
This should be fixed in the general case, as we are now backfilling Win64 symbols just like we've historically done for Win32 symbols (thanks to peterbe fixing bug 1140937). If you look at the graph for that crash signature you can see that it has dropped to virtually zero since we started getting symbols:
https://crash-stats.mozilla.com/signature/?product=Firefox&signature=shutdownhang%20%7C%20ntdll.dll%400x4bb7a#graphs
We should be able to reprocess these crashes to get good signatures out of the existing reports, but that's not a self-service action right now. (We recently gained the ability to self-service reprocess individual crash IDs, but not yet by signature etc.) I filed bug 1280213 for that.
Assignee: nobody → ted
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Reporter | ||
Comment 12•8 years ago
|
||
Thanks, Ted! I posted about this on the Uptime list, for people who are looking at Nightly crashes to be aware of it.
Assignee | ||
Comment 13•8 years ago
|
||
The crash from comment 0 winds up with the signature [@ shutdownhang | WaitForSingleObjectEx | PR_WaitCondVar | mozilla::CondVar::Wait | nsEventQueue::GetEvent ] after reprocessing:
https://crash-stats.mozilla.com/report/index/2ac95b88-47d4-4c72-a7a0-015712160428
You need to log in
before you can comment on or make changes to this bug.
Description
•