RFR: 8344766: AES/CTR slow at big payloads [v2]
Jatin Bhateja
jbhateja at openjdk.org
Wed Nov 27 15:51:37 UTC 2024
On Tue, 26 Nov 2024 15:19:25 GMT, Volodymyr Paprotski <vpaprotski at openjdk.org> wrote:
>> This is a follow up to https://github.com/openjdk/jdk/pull/22086 for AES/CTR
>>
>> Before:
>>
>> Benchmark (algorithm) (dataSize) (keyLength) (provider) Mode Cnt Score Error Units
>> AESBench.decrypt AES/CTR/NoPadding 30000000 128 SunJCE thrpt 3 16.491 ± 0.356 ops/s
>> AESBench.decrypt2 AES/CTR/NoPadding 30000000 128 SunJCE thrpt 3 16.899 ± 0.013 ops/s
>> AESBench.encrypt AES/CTR/NoPadding 30000000 128 SunJCE thrpt 3 16.477 ± 1.006 ops/s
>> AESBench.encrypt2 AES/CTR/NoPadding 30000000 128 SunJCE thrpt 3 16.921 ± 0.038 ops/s
>>
>> After:
>>
>> Benchmark (algorithm) (dataSize) (keyLength) (provider) Mode Cnt Score Error Units
>> AESBench.decrypt AES/CTR/NoPadding 30000000 128 SunJCE thrpt 3 218.910 ± 1.991 ops/s
>> AESBench.decrypt2 AES/CTR/NoPadding 30000000 128 SunJCE thrpt 3 426.414 ± 2.988 ops/s
>> AESBench.encrypt AES/CTR/NoPadding 30000000 128 SunJCE thrpt 3 218.882 ± 2.446 ops/s
>> AESBench.encrypt2 AES/CTR/NoPadding 30000000 128 SunJCE thrpt 3 425.402 ± 4.205 ops/s
>
> Volodymyr Paprotski has updated the pull request incrementally with one additional commit since the last revision:
>
> fix date
Ok, we have already recorded detailed comments on the PR. This is not a compiler side change set so a simpler comment work.
-------------
Marked as reviewed by jbhateja (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/22300#pullrequestreview-2465411735
More information about the security-dev
mailing list