Integrated: 8275569: Add linux-aarch64 to test-make profiles

2021-10-21 Thread Mikhailo Seledtsov
On Wed, 20 Oct 2021 18:46:14 GMT, Mikhailo Seledtsov wrote: > Please review this trivial change that adds "linux-aarch64" to test make > profile. This pull request has now been integrated. Changeset: af146501 Author:Mikhailo Seledtsov URL: https://git.openjdk.jav

Re: RFR: 8275569: Add linux-aarch64 to test-make profiles

2021-10-21 Thread Mikhailo Seledtsov
On Wed, 20 Oct 2021 18:46:14 GMT, Mikhailo Seledtsov wrote: > Please review this trivial change that adds "linux-aarch64" to test make > profile. Magnus, thank you for review. The test make build passes on linux-aarch64. - PR: https://git.openjdk.java.net/jdk/pull/6051

RFR: 8275569: Add linux-aarch64 to test-make profiles

2021-10-20 Thread Mikhailo Seledtsov
Please review this trivial change that adds "linux-aarch64" to test make profile. - Commit messages: - Add linux-aarch64 to test make profile Changes: https://git.openjdk.java.net/jdk/pull/6051/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=6051&range=00 Issue: http

Re: RFR: 8275449: Add linux-aarch64-zero build profile

2021-10-20 Thread Mikhailo Seledtsov
On Tue, 19 Oct 2021 23:22:06 GMT, Mikhailo Seledtsov wrote: > Please review this small change that adds the linux-aarch64 zero build > profile. Thanks for review. I have removed the extra brackets and re-tested. - PR: https://git.openjdk.java.net/jdk/pull/6024

Integrated: 8275449: Add linux-aarch64-zero build profile

2021-10-20 Thread Mikhailo Seledtsov
On Tue, 19 Oct 2021 23:22:06 GMT, Mikhailo Seledtsov wrote: > Please review this small change that adds the linux-aarch64 zero build > profile. This pull request has now been integrated. Changeset: 0021a2f4 Author:Mikhailo Seledtsov URL: https://git.openjdk.java.net/jdk/

Re: RFR: 8275449: Add linux-aarch64-zero build profile [v2]

2021-10-20 Thread Mikhailo Seledtsov
> Please review this small change that adds the linux-aarch64 zero build > profile. Mikhailo Seledtsov has updated the pull request incrementally with one additional commit since the last revision: Removing extra brackets per review feedback - Changes: - all:

Re: RFR: 8275449: Add linux-aarch64-zero build profile

2021-10-19 Thread Mikhailo Seledtsov
On Tue, 19 Oct 2021 23:22:06 GMT, Mikhailo Seledtsov wrote: > Please review this small change that adds the linux-aarch64 zero build > profile. Testing: - linux-aarch64 zero build - PASS - major platform builds: linux-x64, linux-aarch64, windows-x64, macosx-x64, macosx-aarch64

RFR: 8275449: Add linux-aarch64-zero build profile

2021-10-19 Thread Mikhailo Seledtsov
Please review this small change that adds the linux-aarch64 zero build profile. - Commit messages: - Add linux-aarch64-zero to jib build profiles - Add aarch64 as target CPU for libffi lookup Changes: https://git.openjdk.java.net/jdk/pull/6024/files Webrev: https://webrevs.openjdk

Re: RFR: 8274610: Add linux-aarch64 to bootcycle build profiles

2021-10-04 Thread Mikhailo Seledtsov
On Thu, 30 Sep 2021 22:01:51 GMT, Mikhailo Seledtsov wrote: > Please review this simple/trivial change that add linux-aarch64 platform to > bootcycle build profiles. Thank you Erik. - PR: https://git.openjdk.java.net/jdk/pull/5783

Integrated: 8274610: Add linux-aarch64 to bootcycle build profiles

2021-10-04 Thread Mikhailo Seledtsov
On Thu, 30 Sep 2021 22:01:51 GMT, Mikhailo Seledtsov wrote: > Please review this simple/trivial change that add linux-aarch64 platform to > bootcycle build profiles. This pull request has now been integrated. Changeset: 9914e5c4 Author:Mikhailo Seledtsov URL:

RFR: 8274610: Add linux-aarch64 to bootcycle build profiles

2021-09-30 Thread Mikhailo Seledtsov
Please review this simple/trivial change that add linux-aarch64 platform to bootcycle build profiles. - Commit messages: - Add linux-aarch64 to bootcycle build profiles Changes: https://git.openjdk.java.net/jdk/pull/5783/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=

Re: RFR: JDK-8274170: Add hooks for custom makefiles to augment jtreg test execution

2021-09-27 Thread Mikhailo Seledtsov
On Thu, 23 Sep 2021 22:02:07 GMT, Mikhailo Seledtsov wrote: > Please review this small change that adds hook for custom makefiles to > augment parameters for jtreg test execution. Thank you Erik ! - PR: https://git.openjdk.java.net/jdk/pull/5666

Integrated: JDK-8274170: Add hooks for custom makefiles to augment jtreg test execution

2021-09-27 Thread Mikhailo Seledtsov
On Thu, 23 Sep 2021 22:02:07 GMT, Mikhailo Seledtsov wrote: > Please review this small change that adds hook for custom makefiles to > augment parameters for jtreg test execution. This pull request has now been integrated. Changeset: 14100d55 Author:Mikhailo Seledtsov URL:

RFR: JDK-8274170: Add hooks for custom makefiles to augment jtreg test execution

2021-09-23 Thread Mikhailo Seledtsov
Please review this small change that adds hook for custom makefiles to augment parameters for jtreg test execution. - Commit messages: - Initial change Changes: https://git.openjdk.java.net/jdk/pull/5666/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=5666&range=00 I

Re: RFR: JDK-8274170: Add hooks for custom makefiles to augment jtreg test execution

2021-09-23 Thread Mikhailo Seledtsov
On Thu, 23 Sep 2021 22:02:07 GMT, Mikhailo Seledtsov wrote: > Please review this small change that adds hook for custom makefiles to > augment parameters for jtreg test execution. Testing: - tested with custom make file to make sure the feature works - ran a variety of test fra

Re: RFR: 8274083: Update testing docs to mention tiered testing [v5]

2021-09-23 Thread Mikhailo Seledtsov
On Thu, 23 Sep 2021 12:53:23 GMT, Aleksey Shipilev wrote: >> Now that OpenJDK has more or less complete `tier{1,2,3,4}` definitions, >> let's mention them in `testing.md`. >> >> Current patch is my braindump, I am open for suggestions :) > > Aleksey Shipilev has updated the pull request increm

Re: RFR: 8269037: jsig/Testjsig.java doesn't have to be restricted to linux only

2021-08-03 Thread Mikhailo Seledtsov
On Tue, 3 Aug 2021 20:18:11 GMT, Igor Ignatyev wrote: > Hi all, > > could you please review this small patch that enables > `runtime/jsig/Testjsig.java` test and compilation of its native library on > all platforms but windows? > from JBS: >> `runtime/jsig/Testjsig.java` test currently `@requi

Re: [jdk17] RFR: 8269037: jsig/Testjsig.java doesn't have to be restricted to linux only

2021-07-22 Thread Mikhailo Seledtsov
On Sun, 20 Jun 2021 11:08:21 GMT, Igor Ignatyev wrote: > Hi all, > > could you please review this small patch that enables > `runtime/jsig/Testjsig.java` test and compilation of its native library on > all platforms but windows? > from JBS: >> `runtime/jsig/Testjsig.java` test currently `@requ

Re: RFR 15 8242462: Residual Cleanup of rmic removal

2020-04-10 Thread mikhailo . seledtsov
Runtime test changes look good to me, Misha On 4/10/20 11:44 AM, Igor Ignatyev wrote: Hi Roger, removal of applications/ctw/modules/jdk_rmic.java and changes in doc (assuming .html was generated from .md) look good to me. adding hotspot-runtime alias to bring attention of runtime team. I'm n

Re: RFR: JDK-8242463: ProcessTools.createNativeTestProcessBuilder() in testlib needs jdk/bin on PATH on Windows

2020-04-09 Thread mikhailo . seledtsov
Hi Erik,   Looks good to me. Thanks for fixing this. Misha On 4/9/20 8:15 AM, Erik Joelsson wrote: The test open/test/hotspot/jtreg/testlibrary_tests/process/TestNativeProcessBuilder.java fails when building the JDK with VS2019. More specifically, it fails if the JDK under test is built wit

Re: RFR(S) [13] : 8226910 : make it possible to use jtreg's -match via run-test framework

2019-07-18 Thread mikhailo . seledtsov
+1 On 7/17/19 9:43 PM, David Holmes wrote: Hi Igor, This seems fine to me. Thanks, David On 17/07/2019 7:35 am, Igor Ignatyev wrote: can I get a review for this patch? http://cr.openjdk.java.net/~iignatyev//8226910/webrev.01/index.html Thanks, -- Igor On Jul 6, 2019, at 11:50 AM, Igor Ign

RFR(L): 8199256: [TESTBUG] Open source VM testbase runtime tests

2018-05-25 Thread Mikhailo Seledtsov
Please review this change that will open source VM Testbase Runtime tests. This group includes tests covering a variety of runtime functional areas: arraycopy, barrier, jni, threads, vtable and contented. These tests have been used internally for a while, and are now being open sourced. Since

RFR(L): 8199255: [TESTBUG] Open source VM testbase default methods tests

2018-05-21 Thread Mikhailo Seledtsov
Please review this change that will open source VM default method tests. These tests have been used internally for a while, and are now being open sourced. Since this is not an creation of new tests, we would like to keep the changes during this review to a minimum required for open sourcing th

Re: 8199271: [TESTBUG] open source VM testbase stress tests

2018-05-11 Thread Mikhailo Seledtsov
Looks good to me, Misha On 5/8/18, 2:23 PM, Leonid Mesnik wrote: Hi Please review this change open sourcing vm testbase stress tests. These tests have been developed a long time ago for internal test harness and don't looks very nice now. They are open sourced with minimal changes only. The

Re: RFR(M): 8200126: [TESTBUG] Open source VM runtime signal tests

2018-03-29 Thread Mikhailo Seledtsov
While testing I discovered build errors on Mac and Solaris. The following statement " BUILD_HOTSPOT_JTREG_EXECUTABLES_LIBS_exesigtest := -ljvm" was added to a Linux-only block. I have updated the make file to add this for any platform w/o conditions; the " exesigtest.c" is excluded from Windows

Re: RFR(S): 8197906: Enable CDS mode execution of jtreg tests via make

2018-02-15 Thread Mikhailo Seledtsov
at 7:26 AM, Mikhailo Seledtsov wrote: Hi Igor, Thank you for review. This is what I see from the test execution logs on Windows: // At archive creation time: -XX:SharedArchiveFile=T:/testoutput/jtreg/cds_archive.jsa -Xshare:dump // At archive use time: -vm

Re: RFR(S): 8197906: Enable CDS mode execution of jtreg tests via make

2018-02-15 Thread Mikhailo Seledtsov
Hi Magnus, Thank you for review and feedback. I will update my change based on your recommendation, the retest. If tests pass, I am planning to push. Thank you, Misha On 2/14/18, 12:04 PM, Magnus Ihse Bursie wrote: On 2018-02-14 16:26, Mikhailo Seledtsov wrote: Hi Igor, Thank you for

Re: RFR(S): 8197906: Enable CDS mode execution of jtreg tests via make

2018-02-14 Thread Mikhailo Seledtsov
RCHIVE_FILE)") otherwise, tests won't be able to find cds archive during test execution. -- Igor On Feb 13, 2018, at 8:17 PM, Mikhailo Seledtsov wrote: Please review this small change that enables execution of any jtreg test(s) in CDS mode via make. Please see bug description for details