[12] RFR(S): 8213014: Crash in CompileBroker::make_thread due to OOM

[David Holmes]

Tobias writes:
> Hi,
> 
> please review the following patch:
> https://bugs.openjdk.java.net/browse/JDK-8213014
> http://cr.openjdk.java.net/~thartmann/8213014/webrev.00/
> 
> The code in compileBroker.cpp:806 dereferences 'comp' which is NULL if creation of the sweeper
> thread fails. I've also removed the 'compiler_thread' argument and replaced it by a "comp != NULL"
> check. In addition, the calls to create_thread_oop() in init_compiler_sweeper_threads() need to
> CHECK for exceptions.
> 
> Thanks,
> Tobias

This looks fine to me Tobias.

The only query I have (existing issue) is with the dynamic threads case: 
if we fail to create the osThread should we not at some point call 
delete on the CompilerThread/SweeperThread instance?

Thanks,
David