[8u] RFO (Request For Opinion) 7092821: Backport to jdk8u: JDK-7092821: java.security.Provider.getService() is synchronized and became scalability bottleneck

[Andrew Hughes]

On 16/07/2020 19:50, Hohensee, Paul wrote:
> One of our engineers would like to backport this patch because it improves performance, but it includes far more than just the changes to Provider.java necessary to get the performance improvement. In the context of moving to TLS 1.3, Martin referred to 7092821 in
> 
> https://mail.openjdk.java.net/pipermail/jdk8u-dev/2020-May/011763.html
> 
> 
> “The classes/sun/security/ssl/SunJSSE.java file was not deleted -even
> though it has been modified by 8196584- and is an exception too. The
> reason is that this file includes changes from 7092821, which escape
> sun/security/ssl and would be better handled separately. We could have
> done file-replacement here and created a new Step to remove them, but it
> is probably not worth the effort. Note that 7092821 is required for
> parity with Oracle's JDK and will be introduced later. At some point in
> the future, there should be almost no differences with JDK 11.0.7's
> SunJSSE.java file.”
> 
> Amazon has been running with just the changes to Provider.java for some time with no issues. Is there interest in a partial backport of just those changes? If not, is now the time to do a complete backport?
> 
> Thanks,
> Paul
> 
> 

For the OpenJDK 11.0.8 release, we had to include a last minute
regression fix (JDK-8248505) for a bug (JDK-8246613) which itself was a
late regression fix for JDK-7092821.

With that in mind, I'd like this to be out in the wild longer before it
is considered for a stable JDK like 8u. The experience with 11u suggests
that, even though 7092821 went into 12u, its full impact wasn't felt
until it was in an LTS JDK (11u).

In general, I think there should be a very good case (certainly greater
than parity) for including a performance enhancement in a stable release.

Thanks,
-- 
Andrew :)

Senior Free Java Software Engineer
OpenJDK Package Owner
Red Hat, Inc. (http://www.redhat.com)

PGP Key: ed25519/0xCFDA0F9B35964222 (hkp://keys.gnupg.net)
Fingerprint = 5132 579D D154 0ED2 3E04  C5A0 CFDA 0F9B 3596 4222