Integrated: 8351145: RISC-V: only enable some crypto intrinsic when AvoidUnalignedAccess == false

[Hamlin Li]

On Tue, 4 Mar 2025 16:11:41 GMT, Hamlin Li <mli at openjdk.org> wrote:

> Hi,
> Can you help to review the patch?
> 
> Depending whether a cpu supports fast misaligned access or not, the misaligned access can impact the performance a lot.
> Some crypto intrinsic implementation on riscv do not consider data alignment and just use `ld` to load input byte array, and seems there is no way to do it, the main reason is that at java API level, the input byte array to these JVM intrinsic could be part of a real java array, so the input byte array could be 1/2...7 byte aligned.
> And with the introduction of COH, it would be even complicated to do the input data alignment.
> 
> So, for the consistency of performance, seems it's better to disable these intrinsics when AvoidUnalignedAccess == true.
> And the user can still enable the intrinsics explicitly on a CPU with AvoidUnalignedAccess == true if they want so.
> 
> Thanks!

This pull request has now been integrated.

Changeset: af9af7e9
Author:    Hamlin Li <mli at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/af9af7e90f7dab5adc7b89b76eb978d269e863de
Stats:     10 lines in 1 file changed: 4 ins; 4 del; 2 mod

8351145: RISC-V: only enable some crypto intrinsic when AvoidUnalignedAccess == false

Reviewed-by: fyang, rehn

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

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