From jvanek at openjdk.org Sat Jan 4 14:49:24 2025 From: jvanek at openjdk.org (=?UTF-8?B?SmnFmcOt?= =?UTF-8?B?IA==?= =?UTF-8?B?VmFuxJtr?=) Date: Sat, 4 Jan 2025 14:49:24 GMT Subject: RFR: 7903889: jcstress: document used properites (and env variables) Message-ID: <_id_iOGQYGai8cFr-oaXer0R8UivNlj7-ZavmtRzetU=.1c07c8af-058d-4a9e-b83b-3a23bfb68776@github.com> The verbose help can be removed, the individual properties with defaults can be wrapped to its class. Even the OptionSpec and OptionFormatter can be used if needed No Env. vars were found ------------- Commit messages: - 7903889: jcstress: document used properites (and env variables) Changes: https://git.openjdk.org/jcstress/pull/156/files Webrev: https://webrevs.openjdk.org/?repo=jcstress&pr=156&range=00 Issue: https://bugs.openjdk.org/browse/CODETOOLS-7903889 Stats: 240 lines in 7 files changed: 180 ins; 26 del; 34 mod Patch: https://git.openjdk.org/jcstress/pull/156.diff Fetch: git fetch https://git.openjdk.org/jcstress.git pull/156/head:pull/156 PR: https://git.openjdk.org/jcstress/pull/156 From jvanek at openjdk.org Sat Jan 4 14:53:59 2025 From: jvanek at openjdk.org (=?UTF-8?B?SmnFmcOt?= =?UTF-8?B?IA==?= =?UTF-8?B?VmFuxJtr?=) Date: Sat, 4 Jan 2025 14:53:59 GMT Subject: RFR: 7903889: jcstress: document used properites (and env variables) [v2] In-Reply-To: <_id_iOGQYGai8cFr-oaXer0R8UivNlj7-ZavmtRzetU=.1c07c8af-058d-4a9e-b83b-3a23bfb68776@github.com> References: <_id_iOGQYGai8cFr-oaXer0R8UivNlj7-ZavmtRzetU=.1c07c8af-058d-4a9e-b83b-3a23bfb68776@github.com> Message-ID: > The verbose help can be removed, the individual properties with defaults can be wrapped to its class. Even the OptionSpec and OptionFormatter can be used if needed > > No Env. vars were found Ji?? Van?k has updated the pull request incrementally with one additional commit since the last revision: Removed autoformatted whitespaces ------------- Changes: - all: https://git.openjdk.org/jcstress/pull/156/files - new: https://git.openjdk.org/jcstress/pull/156/files/1761d670..ced87e28 Webrevs: - full: https://webrevs.openjdk.org/?repo=jcstress&pr=156&range=01 - incr: https://webrevs.openjdk.org/?repo=jcstress&pr=156&range=00-01 Stats: 14 lines in 1 file changed: 0 ins; 0 del; 14 mod Patch: https://git.openjdk.org/jcstress/pull/156.diff Fetch: git fetch https://git.openjdk.org/jcstress.git pull/156/head:pull/156 PR: https://git.openjdk.org/jcstress/pull/156 From jvanek at openjdk.org Sat Jan 4 17:37:26 2025 From: jvanek at openjdk.org (=?UTF-8?B?SmnFmcOt?= =?UTF-8?B?IA==?= =?UTF-8?B?VmFuxJtr?=) Date: Sat, 4 Jan 2025 17:37:26 GMT Subject: RFR: 7903889: jcstress: document used properites (and env variables) [v3] In-Reply-To: <_id_iOGQYGai8cFr-oaXer0R8UivNlj7-ZavmtRzetU=.1c07c8af-058d-4a9e-b83b-3a23bfb68776@github.com> References: <_id_iOGQYGai8cFr-oaXer0R8UivNlj7-ZavmtRzetU=.1c07c8af-058d-4a9e-b83b-3a23bfb68776@github.com> Message-ID: > The purpose of this PR is to move properties and properties based initializations to single place, and to document them, and to enable printing the listing with the documentation via help. > > No Env. vars were found. > > The verbose help can be removed, the individual properties with defaults can be wrapped to its class. Even the OptionSpec and OptionFormatter can be used if needed Ji?? Van?k has updated the pull request incrementally with one additional commit since the last revision: Extracted properties encapsualted in objects ------------- Changes: - all: https://git.openjdk.org/jcstress/pull/156/files - new: https://git.openjdk.org/jcstress/pull/156/files/ced87e28..4303a351 Webrevs: - full: https://webrevs.openjdk.org/?repo=jcstress&pr=156&range=02 - incr: https://webrevs.openjdk.org/?repo=jcstress&pr=156&range=01-02 Stats: 161 lines in 1 file changed: 97 ins; 4 del; 60 mod Patch: https://git.openjdk.org/jcstress/pull/156.diff Fetch: git fetch https://git.openjdk.org/jcstress.git pull/156/head:pull/156 PR: https://git.openjdk.org/jcstress/pull/156 From jvanek at openjdk.org Sat Jan 4 17:43:27 2025 From: jvanek at openjdk.org (=?UTF-8?B?SmnFmcOt?= =?UTF-8?B?IA==?= =?UTF-8?B?VmFuxJtr?=) Date: Sat, 4 Jan 2025 17:43:27 GMT Subject: RFR: 7903889: jcstress: document used properites (and env variables) [v4] In-Reply-To: <_id_iOGQYGai8cFr-oaXer0R8UivNlj7-ZavmtRzetU=.1c07c8af-058d-4a9e-b83b-3a23bfb68776@github.com> References: <_id_iOGQYGai8cFr-oaXer0R8UivNlj7-ZavmtRzetU=.1c07c8af-058d-4a9e-b83b-3a23bfb68776@github.com> Message-ID: > The purpose of this PR is to move properties and properties based initializations to single place, and to document them, and to enable printing the listing with the documentation via help. > > No Env. vars were found. > > The verbose help can be removed, the individual properties with defaults can be wrapped to its class. Even the OptionSpec and OptionFormatter can be used if needed Ji?? Van?k has refreshed the contents of this pull request, and previous commits have been removed. The incremental views will show differences compared to the previous content of the PR. The pull request contains one new commit since the last revision: Extracted properties encapsualted in objects ------------- Changes: - all: https://git.openjdk.org/jcstress/pull/156/files - new: https://git.openjdk.org/jcstress/pull/156/files/4303a351..c67cbf44 Webrevs: - full: https://webrevs.openjdk.org/?repo=jcstress&pr=156&range=03 - incr: https://webrevs.openjdk.org/?repo=jcstress&pr=156&range=02-03 Stats: 2 lines in 1 file changed: 0 ins; 2 del; 0 mod Patch: https://git.openjdk.org/jcstress/pull/156.diff Fetch: git fetch https://git.openjdk.org/jcstress.git pull/156/head:pull/156 PR: https://git.openjdk.org/jcstress/pull/156 From jvanek at openjdk.org Tue Jan 7 14:47:29 2025 From: jvanek at openjdk.org (=?UTF-8?B?SmnFmcOt?= =?UTF-8?B?IA==?= =?UTF-8?B?VmFuxJtr?=) Date: Tue, 7 Jan 2025 14:47:29 GMT Subject: RFR: 7903754: jcstress should die asap - with report - on broken jvm/hw, instead of waiting on kill Message-ID: Initial PoC It currently show how to set up argument, and how the framework will be terminated. Feedback welcomed. Should be finished soon ------------- Commit messages: - Repalced list.size by counting collector - Implemented main terminating logic - Moved the dying message to correct place - Refactored ConsoleReport and Failfast to share test counting logic - typo - 7903754: jcstress should die asap - with report - on broken jvm/hw, instead of waiting on kill Changes: https://git.openjdk.org/jcstress/pull/157/files Webrev: https://webrevs.openjdk.org/?repo=jcstress&pr=157&range=00 Issue: https://bugs.openjdk.org/browse/CODETOOLS-7903754 Stats: 353 lines in 6 files changed: 318 ins; 30 del; 5 mod Patch: https://git.openjdk.org/jcstress/pull/157.diff Fetch: git fetch https://git.openjdk.org/jcstress.git pull/157/head:pull/157 PR: https://git.openjdk.org/jcstress/pull/157 From jvanek at openjdk.org Tue Jan 7 14:47:29 2025 From: jvanek at openjdk.org (=?UTF-8?B?SmnFmcOt?= =?UTF-8?B?IA==?= =?UTF-8?B?VmFuxJtr?=) Date: Tue, 7 Jan 2025 14:47:29 GMT Subject: RFR: 7903754: jcstress should die asap - with report - on broken jvm/hw, instead of waiting on kill In-Reply-To: References: Message-ID: On Sun, 5 Jan 2025 15:18:53 GMT, Ji?? Van?k wrote: > Initial PoC > > It currently show how to set up argument, and how the framework will be terminated. Feedback welcomed. Should be finished soon The PR is now moreover done. I will run it through even ore testing through. ------------- PR Comment: https://git.openjdk.org/jcstress/pull/157#issuecomment-2575473754 From jvanek at openjdk.org Tue Jan 7 15:32:10 2025 From: jvanek at openjdk.org (=?UTF-8?B?SmnFmcOt?= =?UTF-8?B?IA==?= =?UTF-8?B?VmFuxJtr?=) Date: Tue, 7 Jan 2025 15:32:10 GMT Subject: RFR: 7903754: jcstress should die asap - with report - on broken jvm/hw, instead of waiting on kill [v2] In-Reply-To: References: Message-ID: > Initial PoC > > It currently show how to set up argument, and how the framework will be terminated. Feedback welcomed. Should be finished soon Ji?? Van?k has updated the pull request incrementally with one additional commit since the last revision: Fix in logic ------------- Changes: - all: https://git.openjdk.org/jcstress/pull/157/files - new: https://git.openjdk.org/jcstress/pull/157/files/1dcbd95a..fae6ff11 Webrevs: - full: https://webrevs.openjdk.org/?repo=jcstress&pr=157&range=01 - incr: https://webrevs.openjdk.org/?repo=jcstress&pr=157&range=00-01 Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod Patch: https://git.openjdk.org/jcstress/pull/157.diff Fetch: git fetch https://git.openjdk.org/jcstress.git pull/157/head:pull/157 PR: https://git.openjdk.org/jcstress/pull/157 From jvanek at openjdk.org Sat Jan 25 12:27:14 2025 From: jvanek at openjdk.org (=?UTF-8?B?SmnFmcOt?= =?UTF-8?B?IA==?= =?UTF-8?B?VmFuxJtr?=) Date: Sat, 25 Jan 2025 12:27:14 GMT Subject: RFR: 7903889: jcstress: document used properites (and env variables) [v5] In-Reply-To: <_id_iOGQYGai8cFr-oaXer0R8UivNlj7-ZavmtRzetU=.1c07c8af-058d-4a9e-b83b-3a23bfb68776@github.com> References: <_id_iOGQYGai8cFr-oaXer0R8UivNlj7-ZavmtRzetU=.1c07c8af-058d-4a9e-b83b-3a23bfb68776@github.com> Message-ID: > The purpose of this PR is to move properties and properties based initializations to single place, and to document them, and to enable printing the listing with the documentation via help. > > No Env. vars were found. > > The verbose help can be removed, the individual properties with defaults are wrapped to its class. The OptionSpec and OptionFormatter can be used if needed Ji?? Van?k has updated the pull request incrementally with one additional commit since the last revision: Moved declaration of individual proeprties to ceparate classes by family ------------- Changes: - all: https://git.openjdk.org/jcstress/pull/156/files - new: https://git.openjdk.org/jcstress/pull/156/files/c67cbf44..cf4d4125 Webrevs: - full: https://webrevs.openjdk.org/?repo=jcstress&pr=156&range=04 - incr: https://webrevs.openjdk.org/?repo=jcstress&pr=156&range=03-04 Stats: 644 lines in 10 files changed: 398 ins; 243 del; 3 mod Patch: https://git.openjdk.org/jcstress/pull/156.diff Fetch: git fetch https://git.openjdk.org/jcstress.git pull/156/head:pull/156 PR: https://git.openjdk.org/jcstress/pull/156 From jvanek at openjdk.org Tue Jan 28 11:39:12 2025 From: jvanek at openjdk.org (=?UTF-8?B?SmnFmcOt?= =?UTF-8?B?IA==?= =?UTF-8?B?VmFuxJtr?=) Date: Tue, 28 Jan 2025 11:39:12 GMT Subject: RFR: 7903819: jcstress do not print structuralised results In-Reply-To: References: Message-ID: On Fri, 10 Jan 2025 15:46:48 GMT, Ji?? Van?k wrote: > Jcstress are generating very good html report, which is however not exactly great for automated comapriosns over various systyems and architectures. This pr is adding junit-like, configurable report. Hello! I'm now happy with the generated stuff. The xmls have 100-500mb based on type/number of failures (note, that htmls have 200-700mb based on failures). I have tried them with https://github.com/jenkinsci/report-jtreg-plugin/ and junit plugin, and they behave as expected. I will post exemplar files (with real failures) ------------- PR Comment: https://git.openjdk.org/jcstress/pull/158#issuecomment-2618751696 From jvanek at openjdk.org Tue Jan 28 11:39:12 2025 From: jvanek at openjdk.org (=?UTF-8?B?SmnFmcOt?= =?UTF-8?B?IA==?= =?UTF-8?B?VmFuxJtr?=) Date: Tue, 28 Jan 2025 11:39:12 GMT Subject: RFR: 7903819: jcstress do not print structuralised results Message-ID: Jcstress are generating very good html report, which is however not exactly great for automated comapriosns over various systyems and architectures. This pr is adding junit-like, configurable report. ------------- Commit messages: - Dropped the dom-based generation - Started migration to proepr DOM - Revert "FIXME Added fast fake results generator REMOVE IT" - Fixed few noted comments/variables - made default output xsd valid again - changed comments proeprty to comment out non-standart elements - Unifide testcase print - Support for time attribute and removed some dnagling code - now prints correct summary - Not printing the seed argument - ... and 15 more: https://git.openjdk.org/jcstress/compare/95384bab...910b6ba5 Changes: https://git.openjdk.org/jcstress/pull/158/files Webrev: https://webrevs.openjdk.org/?repo=jcstress&pr=158&range=00 Issue: https://bugs.openjdk.org/browse/CODETOOLS-7903819 Stats: 757 lines in 6 files changed: 735 ins; 4 del; 18 mod Patch: https://git.openjdk.org/jcstress/pull/158.diff Fetch: git fetch https://git.openjdk.org/jcstress.git pull/158/head:pull/158 PR: https://git.openjdk.org/jcstress/pull/158 From jvanek at openjdk.org Tue Jan 28 11:45:06 2025 From: jvanek at openjdk.org (=?UTF-8?B?SmnFmcOt?= =?UTF-8?B?IA==?= =?UTF-8?B?VmFuxJtr?=) Date: Tue, 28 Jan 2025 11:45:06 GMT Subject: RFR: 7903819: jcstress do not print structuralised results In-Reply-To: References: Message-ID: On Fri, 10 Jan 2025 15:46:48 GMT, Ji?? Van?k wrote: > Jcstress are generating very good html report, which is however not exactly great for automated comapriosns over various systyems and architectures. This pr is adding junit-like, configurable report. java -jar ../tests-all/target/jcstress.jar -tb 1m -c 2 -t org.openjdk.jcstress.tests.copy.manual.objects.plain.S.* all passing:
junit-sparse.xml
junit-full.xml
------------- PR Comment: https://git.openjdk.org/jcstress/pull/158#issuecomment-2618765732 From jvanek at openjdk.org Tue Jan 28 11:53:06 2025 From: jvanek at openjdk.org (=?UTF-8?B?SmnFmcOt?= =?UTF-8?B?IA==?= =?UTF-8?B?VmFuxJtr?=) Date: Tue, 28 Jan 2025 11:53:06 GMT Subject: RFR: 7903819: jcstress do not print structuralised results In-Reply-To: References: Message-ID: On Fri, 10 Jan 2025 15:46:48 GMT, Ji?? Van?k wrote: > Jcstress are generating very good html report, which is however not exactly great for automated comapriosns over various systyems and architectures. This pr is adding junit-like, configurable report. crafted failures (but real). The selected jdk had known bug in shenandoah, which was more likely to occure with below JAVA_TOOL_OPTIONS: `JAVA_TOOL_OPTIONS="-XX:+UnlockDiagnosticVMOptions -XX:+UnlockExperimentalVMOptions -XX:+UseShenandoahGC -Djcstress.console.printIntervalMs=3600000 -XX:StressSeed=2039903058" ../java-17-openjdk-17.0.13.0.11-1.portable.jdk.x86_64/bin/java -jar ../tests-all/target/jcstress.jar -tb 1m -c 2 -t org.openjdk.jcstress.tests.copy.manual.objects.plain.S.* `
junit-sparse.xml
junit-full.xml
------------- PR Comment: https://git.openjdk.org/jcstress/pull/158#issuecomment-2618780904 From jvanek at openjdk.org Tue Jan 28 11:59:59 2025 From: jvanek at openjdk.org (=?UTF-8?B?SmnFmcOt?= =?UTF-8?B?IA==?= =?UTF-8?B?VmFuxJtr?=) Date: Tue, 28 Jan 2025 11:59:59 GMT Subject: RFR: 7903819: jcstress do not print structuralised results In-Reply-To: References: Message-ID: On Fri, 10 Jan 2025 15:46:48 GMT, Ji?? Van?k wrote: > Jcstress are generating very good html report, which is however not exactly great for automated comapriosns over various systyems and architectures. This pr is adding junit-like, configurable report. `JAVA_TOOL_OPTIONS="-XX:+UnlockDiagnosticVMOptions -XX:+UnlockExperimentalVMOptions -XX:+UseShenandoahGC" ../java-17-openjdk-17.0.13.0.11-1.portable.jdk.x86_64/bin/java -jar ../tests-all/target/jcstress.jar -tb 1m -c 2 -t org.openjdk.jcstress.tests.copy.manual.objects.plain.S.* ` natural failures:
junit-sparse.xml
`JAVA_TOOL_OPTIONS="-XX:+UnlockDiagnosticVMOptions -XX:+UnlockExperimentalVMOptions -XX:+UseShenandoahGC" ../java-17-openjdk-17.0.13.0.11-1.portable.jdk.x86_64/bin/java -jar ../tests-all/target/jcstress.jar -tb 1m -c 2 -t org.openjdk.jcstress.tests.copy.manual.objects.plain.S.* ` natural failures:
junit-full.xml
------------- PR Comment: https://git.openjdk.org/jcstress/pull/158#issuecomment-2618796263 PR Comment: https://git.openjdk.org/jcstress/pull/158#issuecomment-2618800237