RFR: JDK-8271567: AArch64: AES Galois CounterMode (GCM) interleaved implementation using vector instructions [v8]

Andrew Haley aph at openjdk.java.net
Tue Sep 14 16:15:11 UTC 2021


On Tue, 14 Sep 2021 07:48:03 GMT, Xin Liu <xliu at openjdk.org> wrote:

>> Andrew Haley has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Whitespace
>
> src/hotspot/cpu/aarch64/macroAssembler_aarch64.hpp line 1300:
> 
>> 1298:         Register tmp3);
>> 1299: 
>> 1300:   void ghash_modmul_wide (int index, FloatRegister result,
> 
> Is there definition and reference of this?

Removed.

> src/hotspot/cpu/aarch64/stubGenerator_aarch64.cpp line 2863:
> 
>> 2861:     //
>> 2862:     // int result = len;
>> 2863:     // while (len-- > 0) {
> 
> I see that algorithm code comes from CounterMode.implCrypt, but  while (len-- > 0) seems not to be exactly same as algorithm here. I think it should be `while (len > 0)`
> 
> `blockSize()` at line 2865 should be `blockSize`

Done.

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

PR: https://git.openjdk.java.net/jdk/pull/5390


More information about the hotspot-dev mailing list