RFR: 8308043: Deadlock in TestCSLocker.java due to blocking GC while allocating

[Thomas Schatzl]

On Mon, 15 May 2023 14:26:42 GMT, Erik Österlund <eosterlund at openjdk.org> wrote:

> The TestCSLocker.java test spawns a thread that grabs the GC locker, and then wait for the first thread to run some java code and then get signal back to release the GC locker. All of this while another thread is allocating garbage and triggering GCs. Naturally, if the thread that is to signal the release of the GC locker requires GC in order to make progress, we will end up with a deadlock that leads to a timeout. As it turns out, that does indeed happen. A println statement is performed, which in its internal implementation performs an allocation, which requires GC. I think any GC can spuriously fail here, but it seems more likely with generational ZGC for whatever reason. While it seems really shady to wait with the GC locker held while a Java thread executing Java code is supposed to make progress, in general, I think the test can be fixed by removing the println statement causing the allocation. I have run the test 200 times, and it's no longer failing with generational ZGC.

Marked as reviewed by tschatzl (Reviewer).

test/hotspot/jtreg/gc/cslocker/TestCSLocker.java line 53:

> 51: 
> 52:         // check timeout to success deadlocking
> 53:         while(System.currentTimeMillis() < startTime + timeout) {

In addition to @albertnetymk's comment, please also fix the whitespace after the `while` and the bracket.

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

PR Review: https://git.openjdk.org/jdk/pull/13989#pullrequestreview-1426849604
PR Review Comment: https://git.openjdk.org/jdk/pull/13989#discussion_r1194021093