RFR: 8321369: Unproblemlist gc/cslocker/TestCSLocker.java

David Holmes dholmes at openjdk.org
Wed Dec 6 02:55:33 UTC 2023


On Tue, 5 Dec 2023 10:19:02 GMT, Thomas Schatzl <tschatzl at openjdk.org> wrote:

> Hi all,
> 
>   please review this fix to unproblemlist gc/cs/TestCSLocker.java; the CR to fix this [JDK-8310480](https://bugs.openjdk.org/browse/JDK-8310480) has already been closed as duplicate of [JDK-8318706](https://bugs.openjdk.org/browse/JDK-8318706) that removed the GCLocker for G1 which is the cause for the issue.
> 
> Note that the test has only been problemlisted for linux-x64 previously, so it has already been run for a long time for other platforms and other collectors, so my testing did not extensively try all the other platforms/gc combinations (I did try with Serial and Parallel a few times with no issues; ZGC is excluded anyway in the test, and Shenandoah also does not use the GCLocker).
> 
> Testing: test case with g1, gha
> 
> Thanks,
>   Thomas

Looks good and trivial. Thanks for cleaning up the PL.

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

Marked as reviewed by dholmes (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/16970#pullrequestreview-1766542397


More information about the hotspot-dev mailing list