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

[Stefan Karlsson]

On Tue, 16 May 2023 13:42:07 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.
>
> Erik Österlund has updated the pull request incrementally with one additional commit since the last revision:
> 
>   White space fix

Marked as reviewed by stefank (Reviewer).

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

PR Review: https://git.openjdk.org/jdk/pull/13989#pullrequestreview-1428768327