On Thu, 6 Mar 2025 09:32:19 GMT, Jatin Bhateja <jbhat...@openjdk.org> wrote:

>> I think the easiest is to put a for (int i = 0; i < 1000; i++) loop around  
>> the switch statement in the run() method of  the ML_DSA_Test class 
>> (test/jdk/sun/security/provider/acvp/ML_DSA_Test.java). (This is because the 
>> intrinsics kick in after a few thousand calls of the method.)
>
> Hi @ferakocz , Yes, we should modify the test or lower the compilation 
> threshold with -Xbatch -XX:TieredCompileThreshold=0.1.
> 
> Alternatively, since the tests has a depedency on Automatic Cryptographic 
> Validation Test server I have created a simplified test which cover all the 
> security levels.
> 
> Kindly include 
> [test/hotspot/jtreg/compiler/intrinsics/signature/TestModuleLatticeDSA.java
> ](https://github.com/ferakocz/jdk/pull/1)

I have added a new command to the test 
test/jdk/sun/security/provider/acvp/Launcher.java. The line with the -Xcomp 
will invoke the intrinsics on the first call, so they will be tested.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/23860#discussion_r1991546056

Reply via email to