1643047 - Crash in [@ IPCError-browser | ShutDownKill | mozilla::BackgroundHangMonitor::NotifyActivity ]

Status: RESOLVED WORKSFORME

(Core :: XPCOM, defect)

Description

[Simon Giesecke [:sg] [he/him]]

This bug is for crash report bp-5d6b45b1-43b4-47bf-a395-0b7550200603.

Top 10 frames of crashing thread:

0 ntdll.dll NtWaitForAlertByThreadId 
1 ntdll.dll RtlAcquireSRWLockExclusive 
2 xul.dll mozilla::BackgroundHangMonitor::NotifyActivity toolkit/components/backgroundhangmonitor/BackgroundHangMonitor.cpp:758
3 xul.dll nsThread::ProcessNextEvent xpcom/threads/nsThread.cpp:1170
4 xul.dll NS_ProcessNextEvent xpcom/threads/nsThreadUtils.cpp:501
5 xul.dll mozilla::ipc::MessagePump::Run ipc/glue/MessagePump.cpp:87
6 xul.dll MessageLoop::RunHandler ipc/chromium/src/base/message_loop.cc:308
7 xul.dll MessageLoop::Run ipc/chromium/src/base/message_loop.cc:290
8 xul.dll nsBaseAppShell::Run widget/nsBaseAppShell.cpp:137
9 xul.dll nsAppShell::Run widget/windows/nsAppShell.cpp:430

At least with this signature, reports have started with Nightly build 20200509091423

Comment 1

[Sean Feng [:sefeng211]]

I noticed a couple of shutdown hangs with this signature on Sept 7th Nightlies. The hangs were caused by RtlSleepConditionVariableSRW.

Comment 2

[Sean Feng [:sefeng211]]

Nika, do you know who might be the best person to look into this?

Comment 3

[Andrew McCreight [:mccr8]]

Maybe Doug could take a look? I think the hang monitor might be Nightly-only, in which case it isn't a huge issue. It doesn't look to be too common.

Comment 4

[Alex Thayer [:alexical] (she/her)]

Yeah, BHR is Nightly-only. I am not too concerned about this, though it will be relevant if we ever want to enable BHR past Nightly.

Comment 5

[BugBot [:suhaib / :marco/ :calixte]]

Closing because no crashes reported for 12 weeks.