Re: RFR: 8271707: migrate tests to use jdk.test.whitebox.WhiteBox

2022-07-08 Thread Leonid Mesnik
On Thu, 7 Jul 2022 20:43:09 GMT, Coleen Phillimore wrote: > This change uses sed to change sun.hotspot.WhiteBox to > jdk.test.whitebox.Whitebox, and sun/hotspot/Whitebox similarly. Due to > indirect inclusions of some of the test libraries, changing a few wasn't a > reliable option, and I nee

Re: RFR: 8275662: remove test/lib/sun/hotspot

2022-07-08 Thread Leonid Mesnik
On Fri, 8 Jul 2022 19:46:17 GMT, Coleen Phillimore wrote: > This change removes the last remnants of sun/hotspot/WhiteBox.java and other > classes, and uses the versions in jdk/test/whitebox. > I used sed to change sun.hotspot.{gc,code,cpuinfo} to jdk.test.whitebox and > deleted the old files a

Re: RFR: JDK-8304163: Move jdk.internal.module.ModuleInfoWriter to the test library [v2]

2023-03-19 Thread Leonid Mesnik
On Sat, 18 Mar 2023 19:14:09 GMT, Mandy Chung wrote: >> `ModuleInfoWriter` is not used by the runtime. Move it to the test library >> as `jdk.test.lib.util.ModuleInfoWriter`. The tests are updated to use the >> test library instead. `ModuleInfoWriter` depends on `jdk.internal.module` >>

Re: RFR: 8315097: Rename createJavaProcessBuilder

2023-08-28 Thread Leonid Mesnik
On Mon, 28 Aug 2023 15:54:08 GMT, Leo Korinth wrote: > Rename createJavaProcessBuilder so that it is not used by mistake instead of > createTestJvm. > > I have used the following sed script: `find -name "*.java" | xargs -n 1 sed > -i -e > "s/createJavaProcessBuilder(/createJavaProcessBuilderI

Re: RFR: 8315097: Rename createJavaProcessBuilder [v6]

2023-10-24 Thread Leonid Mesnik
On Tue, 24 Oct 2023 07:49:30 GMT, Leo Korinth wrote: >> This pull request renames `createJavaProcessBuilder` to >> `createLimitedTestJavaProcessBuilder` and renames `createTestJvm` to >> `createTestJavaProcessBuilder`. Both are implemented through a private >> `createJavaProcessBuilder`. It al

Re: RFR: 8321713: Harmonize executeTestJvm with create[Limited]TestJavaProcessBuilder [v3]

2023-12-12 Thread Leonid Mesnik
On Mon, 11 Dec 2023 14:06:43 GMT, Stefan Karlsson wrote: >> [JDK-8315097](https://bugs.openjdk.org/browse/JDK-8315097): 'Rename >> createJavaProcessBuilder' changed the name of the ProcessTools helper >> functions used to create `ProcessBuilder`s used to spawn new java test >> processes. >> >

Re: RFR: 8346050: Update BuildTestLib.gmk to build whole testlibrary [v2]

2025-02-07 Thread Leonid Mesnik
> The fix add remaining classes to the testlibrary jar and fix some warnings in > security-related classes. Leonid Mesnik has updated the pull request incrementally with one additional commit since the last revision: Update make/test/BuildTestLib.gmk Co-authored-by: Erik Jo

RFR: 8346050: Update BuildTestLib.gmk to build whole testlibrary

2025-02-07 Thread Leonid Mesnik
The fix add remaining classes to the testlibrary jar and fix some warnings in security-related classes. - Commit messages: - 8346050: Update BuildTestLib.gmk to build whole testlibrary Changes: https://git.openjdk.org/jdk/pull/23525/files Webrev: https://webrevs.openjdk.org/?repo

Re: RFR: 8346049: jdk/test/lib/security/timestamp/TsaServer.java warnings

2025-02-05 Thread Leonid Mesnik
On Wed, 5 Feb 2025 12:53:09 GMT, Fernando Guallini wrote: > When compiling the test class: jdk/test/lib/security/timestamp/TsaServer.java > > > Two warning are shown: > > > test/lib/security/timestamp/TsaServer.java:56: warning: [this-escape] > possible 'this' escape before subclass is ful

Re: RFR: 8346050: Update BuildTestLib.gmk to build whole testlibrary [v3]

2025-02-11 Thread Leonid Mesnik
> The fix add remaining classes to the testlibrary jar and fix some warnings in > security-related classes. Leonid Mesnik has updated the pull request incrementally with two additional commits since the last revision: - Merge branch '8346050' of https://github.com/lmesnik/

Re: RFR: 8339891: Several sun/security/ssl/SSLSessionImpl/* tests override test.java.opts [v2]

2025-01-22 Thread Leonid Mesnik
On Wed, 22 Jan 2025 14:00:53 GMT, Sonia Zaldana Calles wrote: >> Hi all, >> >> This PR addresses [8339891](https://bugs.openjdk.org/browse/JDK-8339891). >> With this patch, "test.java.opts" aggregates on options set via jtreg as >> opposed to overriding them. >> >> Thanks, >> Sonia > > So

RFR: 8318098: Update jfr tests to replace keyword jfr with vm.flagless

2025-01-23 Thread Leonid Mesnik
A lot of jfr tests are very specific. Currently all of them are marked with jfr keyword and excluded when VM flags are set. It makes sense to mark them with @requires to be complaint with all other openjdk tests. The next step is to review tests and remove vm.flagless from tests that should b

Re: RFR: 8339891: Several sun/security/ssl/SSLSessionImpl/* tests override test.java.opts

2025-01-21 Thread Leonid Mesnik
On Thu, 7 Nov 2024 15:44:56 GMT, Sonia Zaldana Calles wrote: > Hi all, > > This PR addresses [8339891](https://bugs.openjdk.org/browse/JDK-8339891). > With this patch, "test.java.opts" aggregates on options set via jtreg as > opposed to overriding them. > > Thanks, > Sonia @SoniaZaldana

Re: RFR: 8351034: Add AVX-512 intrinsics for ML-DSA [v7]

2025-03-17 Thread Leonid Mesnik
On Wed, 12 Mar 2025 19:19:08 GMT, Ferenc Rakoczi wrote: >> By using the AVX-512 vector registers the speed of the computation of the >> ML-DSA algorithms (key generation, document signing, signature verification) >> can be approximately doubled. > > Ferenc Rakoczi has updated the pull request i

Re: RFR: 8351412: Add AVX-512 intrinsics for ML-KEM [v2]

2025-04-30 Thread Leonid Mesnik
On Wed, 30 Apr 2025 07:48:32 GMT, Ferenc Rakoczi wrote: >> By using the AVX-512 vector registers the speed of the computation of the >> ML-KEM algorithms (key generation, encapsulation, decapsulation) can be >> approximately doubled. > > Ferenc Rakoczi has updated the pull request incrementally

Integrated: 8346050: Update BuildTestLib.gmk to build whole testlibrary

2025-02-18 Thread Leonid Mesnik
On Fri, 7 Feb 2025 21:15:36 GMT, Leonid Mesnik wrote: > The fix add remaining classes to the testlibrary jar and fix some warnings in > security-related classes. This pull request has now been integrated. Changeset: 62d93f2a Author:Leonid Mesnik URL: https://git.openjdk.o

Re: RFR: 8351034: Add AVX-512 intrinsics for ML-DSA [v5]

2025-03-09 Thread Leonid Mesnik
On Thu, 6 Mar 2025 17:37:33 GMT, Ferenc Rakoczi wrote: >> By using the AVX-512 vector registers the speed of the computation of the >> ML-DSA algorithms (key generation, document signing, signature verification) >> can be approximately doubled. > > Ferenc Rakoczi has updated the pull request in

Re: RFR: 8351034: Add AVX-512 intrinsics for ML-DSA [v6]

2025-03-12 Thread Leonid Mesnik
On Wed, 12 Mar 2025 13:42:33 GMT, Ferenc Rakoczi wrote: >> By using the AVX-512 vector registers the speed of the computation of the >> ML-DSA algorithms (key generation, document signing, signature verification) >> can be approximately doubled. > > Ferenc Rakoczi has updated the pull request i

Re: RFR: 8346050: Update BuildTestLib.gmk to build whole testlibrary [v4]

2025-02-14 Thread Leonid Mesnik
> The fix add remaining classes to the testlibrary jar and fix some warnings in > security-related classes. Leonid Mesnik has updated the pull request incrementally with two additional commits since the last revision: - added copyrights - replaced commens fix with s

Re: RFR: 8346050: Update BuildTestLib.gmk to build whole testlibrary [v4]

2025-02-14 Thread Leonid Mesnik
On Fri, 14 Feb 2025 15:24:53 GMT, Weijun Wang wrote: >> Leonid Mesnik has updated the pull request incrementally with two additional >> commits since the last revision: >> >> - added copyrights >> - replaced commens fix with supress > > test/lib/jdk/test

Re: RFR: 8356171: Increase timeout for testcases as preparation for change of default timeout factor

2025-05-08 Thread Leonid Mesnik
On Thu, 8 May 2025 16:43:10 GMT, Leo Korinth wrote: >> This change tries to add timeout to individual testcases so that I am able >> to run them with a timeout factor of 1 in the future (JDK-8260555). >> >> The first commit changes the timeout factor to 0.7, so that I can run tests >> and test

Re: RFR: 8356171: Increase timeout for testcases as preparation for change of default timeout factor

2025-05-08 Thread Leonid Mesnik
On Thu, 8 May 2025 17:03:03 GMT, Leo Korinth wrote: >> doc/testing.md line 385: >> >>> 383: (`-timeoutFactor`). Also, some test cases that programmatically wait a >>> 384: certain amount of time will apply this factor. If we run in >>> 385: interpreted mode (`-Xcomp`), [RunTest.gmk](../make/RunT

Re: RFR: 8351412: Add AVX-512 intrinsics for ML-KEM [v7]

2025-05-20 Thread Leonid Mesnik
On Tue, 20 May 2025 17:49:14 GMT, Ferenc Rakoczi wrote: >> By using the AVX-512 vector registers the speed of the computation of the >> ML-KEM algorithms (key generation, encapsulation, decapsulation) can be >> approximately doubled. > > Ferenc Rakoczi has updated the pull request incrementally

Re: RFR: 8351412: Add AVX-512 intrinsics for ML-KEM [v7]

2025-05-20 Thread Leonid Mesnik
On Tue, 20 May 2025 17:49:14 GMT, Ferenc Rakoczi wrote: >> By using the AVX-512 vector registers the speed of the computation of the >> ML-KEM algorithms (key generation, encapsulation, decapsulation) can be >> approximately doubled. > > Ferenc Rakoczi has updated the pull request incrementally