RFR: 8338383: Implement JEP 491: Synchronize Virtual Threads without Pinning [v22]

Alan Bateman alanb at openjdk.org
Thu Oct 31 20:30:52 UTC 2024


On Thu, 31 Oct 2024 19:59:00 GMT, Dean Long <dlong at openjdk.org> wrote:

>> Patricio Chilano Mateo has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Fix typos in comments
>
> src/java.base/share/classes/sun/security/ssl/X509TrustManagerImpl.java line 57:
> 
>> 55:     static {
>> 56:         try {
>> 57:             MethodHandles.lookup().ensureInitialized(AnchorCertificates.class);
> 
> Why is this needed?  A comment would help.

That's probably a good idea.  It’s caused by pinning due to the sun.security.util.AnchorCertificates’s class initializer, some of the http client tests are running into this. Once monitors are out of the way then class initializers, both executing, and waiting for, will be a priority.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/21565#discussion_r1825127591


More information about the serviceability-dev mailing list