std::thread::join() hangs if called after main() exits when using VS2012 RC
Tracing through Fraser's sample code in his connect bug (https://connect.microsoft.com/VisualStudio/feedback/details/747145)
with VS2012 RTM seems to show a fairly straightforward case of deadlocking. This likely isn't specific to std::thread
- likely _beginthreadex
suffers the same fate.
What I see in the debugger is the following:
On the main thread, the main()
function has completed, the process cleanup code has acquired a critical section called _EXIT_LOCK1
, called the destructor of ThreadTest
, and is waiting (indefinitely) on the second thread to exit (via the call to join()
).
The second thread's anonymous function completed and is in the thread cleanup code waiting to acquire the _EXIT_LOCK1
critical section. Unfortunately, due to the timing of things (whereby the second thread's anonymous function's lifetime exceeds that of the main()
function) the main thread already owns that critical section.
DEADLOCK.
Anything that extends the lifetime of main()
such that the second thread can acquire _EXIT_LOCK1
before the main thread avoids the deadlock situation. That's why the uncommenting the sleep in main()
results in a clean shutdown.
Alternatively if you remove the static keyword from the ThreadTest
local variable, the destructor call is moved up to the end of the main()
function (instead of in the process cleanup code) which then blocks until the second thread has exited - avoiding the deadlock situation.
Or you could add a function to ThreadTest
that calls join()
and call that function at the end of main()
- again avoiding the deadlock situation.
I realize this is an old question regarding VS2012, but the bug is still present in VS2013. For those who are stuck on VS2013, perhaps due to Microsoft's refusal to provide upgrade pricing for VS2015, I offer the following analysis and workaround.
The problem is that the mutex (at_thread_exit_mutex
) used by _Cnd_do_broadcast_at_thread_exit()
is either not yet initialized, or has already been destroyed, depending on the exact circumstances. In the former case, _Cnd_do_broadcast_at_thread_exit()
tries to initialize the mutex during shutdown, causing a deadlock. In the latter case, where the mutex has already been destroyed via the atexit stack, the program will crash on the way out.
The solution I found is to explicitly call _Cnd_do_broadcast_at_thread_exit()
(which thankfully is declared publicly) early during program startup. This has the effect of creating the mutex before anyone else tries to access it, as well as ensuring that the mutex continues to exist until the last possible moment.
So, to fix the problem, insert the following code at the bottom of a source module, for instance somewhere below main().
#pragma warning(disable:4073) // initializers put in library initialization area
#pragma init_seg(lib)
#if _MSC_VER < 1900
struct VS2013_threading_fix
{
VS2013_threading_fix()
{
_Cnd_do_broadcast_at_thread_exit();
}
} threading_fix;
#endif