[jdk17u] RFR: 8271567: AArch64: AES Galois CounterMode (GCM) interleaved implementation using vector instructions
Andrew Haley
aph at openjdk.java.net
Mon Oct 25 16:57:09 UTC 2021
On Mon, 25 Oct 2021 14:11:26 GMT, Andrew Dinn <adinn at openjdk.org> wrote:
> I'd be happy with "include the patch but disable the intrinsic by default". That poses a pretty low risk for users who don't need this but, as you say, should a big difference for apps that need AES GCM -- enough for them to want it despite the limited upstream baking time.
There's no point simply turning off acceleration for safety because this patch includes a partial rewrite of the code that gets executed even when the new accelerator is disabled, so it'd add risk without improving anything: a very bad bargain.
What I'll do instead is alter the patch to leave the old code unchanged when AES/GCM acceleration is disabled.
This is a bit of a "Physician, heal thyself!" moment -- it's what I'm forever asking back-porters to do for JDK 11u. Oops, my bad.
-------------
PR: https://git.openjdk.java.net/jdk17u/pull/87
More information about the jdk-updates-dev
mailing list