RFR: 8273297: AES/GCM non-AVX512+VAES CPUs suffer after 8267125 [v4]

Smita Kamath svkamath at openjdk.java.net
Tue Sep 21 22:16:37 UTC 2021


> 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:

  Fixed code-style standard to have a space between plus, minus and combo operators

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

Changes:
  - all: https://git.openjdk.java.net/jdk/pull/5402/files
  - new: https://git.openjdk.java.net/jdk/pull/5402/files/19b0d547..8756d301

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=5402&range=03
 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=5402&range=02-03

  Stats: 6 lines in 1 file changed: 0 ins; 0 del; 6 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



More information about the security-dev mailing list