> Performance dropped up to 10% for 1k data after 8267125 for CPUs that do not > support the new intrinsic. Tests run were crypto.full.AESGCMBench and > crypto.full.AESGCMByteBuffer from the jmh micro benchmarks. > > The problem is each instance of GHASH allocates 96 extra longs for the > AVX512+VAES intrinsic regardless if the intrinsic is used. This extra table > space should be allocated differently so that non-supporting CPUs do not > suffer this penalty. This issue also affects non-Intel CPUs too.
Smita Kamath has updated the pull request incrementally with one additional commit since the last revision: Added a wrapper around aes-gcm intrinsic, changed data size in TestAESMain and added a new constant for htbl entries ------------- Changes: - all: https://git.openjdk.java.net/jdk/pull/5402/files - new: https://git.openjdk.java.net/jdk/pull/5402/files/4628dc3a..7ea464ae Webrevs: - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=5402&range=01 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=5402&range=00-01 Stats: 42 lines in 5 files changed: 28 ins; 1 del; 13 mod Patch: https://git.openjdk.java.net/jdk/pull/5402.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/5402/head:pull/5402 PR: https://git.openjdk.java.net/jdk/pull/5402