[jdk17u-dev] Integrated: 8251216: Implement MD5 intrinsics on AArch64

[Dmitry Chuyko]

On Thu, 20 Jan 2022 14:22:06 GMT, Dmitry Chuyko <dchuyko at openjdk.org> wrote:

> As mentioned in the original review MD5 can't be considered a secure hash. But it is still widely used as a lightweight generic hash function, and that's true not only for a legacy software. It is in fact not as lightweight as expected, e.g. hardware accelerated SHA-1 and SHA-256 work much faster on N1. We can sweeten the pill at least for the latest LTS release by 25-30% acceleration with this change.
> 
> Original patch applies almost cleanly. There was a minor fuzz, and "code_size2 = 45000" increase had to be re-applied manually.
> 
> Micro-benchmark results in openjdk.bench.javax.crypto.full.MessageDigestBench on Graviton 2 show improvements identical to original numbers on Altra.
> 
> Testing: tier1, tier2, test/hotspot/jtreg/compiler/intrinsics/sha (aarch64)

This pull request has now been integrated.

Changeset: 49cf332a
Author:    Dmitry Chuyko <dchuyko at openjdk.org>
Committer: Paul Hohensee <phh at openjdk.org>
URL:       https://git.openjdk.java.net/jdk17u-dev/commit/49cf332a25f63d7e0fb81db7fdbc4ef36873569c
Stats:     199 lines in 4 files changed: 193 ins; 1 del; 5 mod

8251216: Implement MD5 intrinsics on AArch64

Reviewed-by: phh, luhenry
Backport-of: 088b244ec6d9393a1fcd2233fa5b4cf46f9ae0dd

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

PR: https://git.openjdk.java.net/jdk17u-dev/pull/110