On Tue, 6 Jul 2021 15:41:27 GMT, Weijun Wang <wei...@openjdk.org> wrote:
>> Anthony Scarpino has updated the pull request incrementally with one >> additional commit since the last revision: >> >> merge AESGCMByteArray with existing AESGCMBench > > test/micro/org/openjdk/bench/javax/crypto/full/AESGCMBench.java line 58: > >> 56: SecretKeySpec ks; >> 57: GCMParameterSpec gcm_spec; >> 58: byte[] aad; > > Not an expert on this, but why no more AAD? For a performance test there isn't any value in adding an AAD. It's data sitting in a buffer that an be easily replicated with a multipart operation ------------- PR: https://git.openjdk.java.net/jdk/pull/4672