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