On Wed, 24 Jun 2026 02:52:13 GMT, Volodymyr Paprotski <[email protected]> 
wrote:

> With `quadKeccak`, it is possible to have 3 extra `keccak`s calls that are 
> really noops.. teach `SHA3Parallel.squeeze()` how many operations it really 
> should be doing.
> 
> While it is possible to match precisely the number of keccak calls required.. 
> the previous implementation always called doubleKeccak, so collapsing the odd 
> numbers to the closest even.
> 
> PS:  `make install-hsdis test 
> TEST="micro:org.openjdk.bench.javax.crypto.full.SignatureBench.MLDSA" 
> MICRO="JAVA_OPTIONS=-XX:+UnlockDiagnosticVMOptions 
> -XX:-UseSHA3Intrinsics;FORK=1;ITER=3;TIME=10;WARMUP_ITER=7;WARMUP_TIME=10;OPTIONS=-prof
>  perfasm -p algorithm=ML-DSA-65"`
> 
> ---------
> - [x] I confirm that I make this contribution in accordance with the [OpenJDK 
> Interim AI Policy](https://openjdk.org/legal/ai).

src/java.base/share/classes/sun/security/provider/SHA3Parallel.java line 106:

> 104:         }
> 105: 
> 106:         for (int i = 0; i < NRPAR; i++) {

nr iterations is enough here.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/31648#discussion_r3469825713

Reply via email to