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

[Erik Österlund]

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.

This pull request has now been integrated.

Changeset: 285c833f
Author:    Erik Österlund <eosterlund at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/285c833ffacdaabe7c4955cbbafb3bc459d26784
Stats:     8 lines in 2 files changed: 4 ins; 3 del; 1 mod

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

Reviewed-by: stefank, ayang, tschatzl

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

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