RFR: 8318756 Create better internal buffer for AEADs

Daniel Jeliński djelinski at openjdk.org
Tue Nov 28 08:26:07 UTC 2023


On Tue, 28 Nov 2023 01:29:20 GMT, Anthony Scarpino <ascarpino at openjdk.org> wrote:

>> test/micro/org/openjdk/bench/javax/crypto/full/AESGCMBench.java line 43:
>> 
>>> 41:     AlgorithmParameterSpec getNewSpec() {
>>> 42:         iv_index = (iv_index + 1) % IV_MODULO;
>>> 43:         return new GCMParameterSpec(96, iv, iv_index, 12);
>> 
>> Can you also change tag length to 128 bits? TLS uses 128, and 128-bit tag generates a slightly different flamegraph.
>
> I'll see if I can do it cleanly. GCM spec defaults to 96bit and because CC20P1305 requires 96bit, it made the common code easier. I'm surprised you any differences which such a minor change.

FWIW, here's the relevant part of the flamegraphs:
![image](https://github.com/openjdk/jdk/assets/30433125/3a37921b-55d8-4f04-9d70-1b7b4ff9b7e2)
As you can see, the 128-bit tag is using a different intrinsic.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/16487#discussion_r1407383930



More information about the security-dev mailing list