Can I please get a review of this change which addresses the issue noted in 
https://bugs.openjdk.org/browse/JDK-8367597? 

As noted in that issue, on certain occasions, during shutdown of the JVM, the 
logging in `Runtime.exit()` generates a `NullPointerException`. The issue is 
due to the JVM not being fully initialized when (concurrently) the signal 
handler thread calls the `Runtime.exit()` code which then attempts to log a 
message.

The commit in this PR skips the logging from `Runtime.exit()` if the JVM isn't 
fully initialized at that point in time. I've manually verified that this 
change indeed addresses the issue by running the same tests on the hosts where 
this was previously failing. No new regressions tests have been introduced 
given the nature of the issue.

tier1 testing with this change completed without any related issues and 
additional tier testing is currently in progress.

-------------

Commit messages:
 - 8367597: Runtime.exit logging failed: Cannot invoke 
"java.lang.Module.getClassLoader()" because "m" is null

Changes: https://git.openjdk.org/jdk/pull/27270/files
  Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=27270&range=00
  Issue: https://bugs.openjdk.org/browse/JDK-8367597
  Stats: 5 lines in 1 file changed: 4 ins; 0 del; 1 mod
  Patch: https://git.openjdk.org/jdk/pull/27270.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/27270/head:pull/27270

PR: https://git.openjdk.org/jdk/pull/27270

Reply via email to