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

Reply via email to