RFR: 8321369: Unproblemlist gc/cslocker/TestCSLocker.java
Thomas Schatzl
tschatzl at openjdk.org
Wed Dec 6 10:37:43 UTC 2023
On Wed, 6 Dec 2023 02:52:29 GMT, David Holmes <dholmes 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.
thanks @dholmes-ora for the review.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/16970#issuecomment-1842606045
More information about the hotspot-dev
mailing list