RFR: 8288047: Accelerate Poly1305 on x86_64 using AVX512 instructions [v7]

Volodymyr Paprotski duke at openjdk.org
Fri Nov 4 14:40:46 UTC 2022


On Tue, 18 Oct 2022 22:51:51 GMT, Sandhya Viswanathan <sviswanathan at openjdk.org> wrote:

>> Volodymyr Paprotski has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 12 commits:
>> 
>>  - Merge remote-tracking branch 'origin/master' into avx512-poly
>>  - address Jamil's review
>>  - invalidkeyexception and some review comments
>>  - extra whitespace character
>>  - assembler checks and test case fixes
>>  - Merge remote-tracking branch 'origin/master' into avx512-poly
>>  - Merge remote-tracking branch 'origin' into avx512-poly
>>  - further restrict UsePolyIntrinsics with supports_avx512vlbw
>>  - missed white-space fix
>>  - - Fix whitespace and copyright statements
>>    - Add benchmark
>>  - ... and 2 more: https://git.openjdk.org/jdk/compare/9d3b4ef2...38d9e83c
>
> src/java.base/share/classes/com/sun/crypto/provider/Poly1305.java line 286:
> 
>> 284:      * numeric values.
>> 285:      */
>> 286:     private void setRSVals() { //throws InvalidKeyException {
> 
> The R and S check for invalid key (all bytes zero) could be submitted as a separate PR. 
> It is not related to the Poly1305 acceleration.

done, added a flag

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

PR: https://git.openjdk.org/jdk/pull/10582



More information about the security-dev mailing list