]> git.lizzy.rs Git - rust.git/commitdiff
auto merge of #13080 : alexcrichton/rust/possible-osx-deadlock, r=brson
authorbors <bors@rust-lang.org>
Mon, 24 Mar 2014 21:32:09 +0000 (14:32 -0700)
committerbors <bors@rust-lang.org>
Mon, 24 Mar 2014 21:32:09 +0000 (14:32 -0700)
The OSX bots have been deadlocking recently in the rustdoc tests. I have only
been able to rarely reproduce the deadlock on my local setup. When reproduced,
it looks like the child process is spinning on the malloc mutex, which I
presume is locked with no other threads to unlock it.

I'm not convinced that this is what's happening, because OSX should protect
against this with pthread_atfork by default. Regardless, running as little code
as possible in the child after fork() is normally a good idea anyway, so this
commit moves all allocation to the parent process to run before the child
executes.

After running 6k iterations of rustdoc tests, this deadlocked twice before, and
after 20k iterations afterwards, it never deadlocked. I draw the conclusion that
this is either sweeping the bug under the rug, or it did indeed fix the
underlying problem.


Trivial merge