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

[Doerr, Martin]

Hi Tobias,

thanks for fixing. Looks good. Removing the extra bool flag is a nice cleanup.

I'd rate this issue P4 because it's only to get a clean error message instead of a crash when the initial threads don't come up. Right?

Best regards,
Martin


-----Original Message-----
From: Tobias Hartmann <tobias.hartmann at oracle.com> 
Sent: Dienstag, 30. Oktober 2018 10:52
To: hotspot compiler <hotspot-compiler-dev at openjdk.java.net>; Doerr, Martin <martin.doerr at sap.com>
Subject: [12] RFR(S): 8213014: Crash in CompileBroker::make_thread due to OOM

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