From jvanek at openjdk.org Tue Apr 1 10:52:30 2025 From: jvanek at openjdk.org (=?UTF-8?B?SmnFmcOt?= =?UTF-8?B?IA==?= =?UTF-8?B?VmFuxJtr?=) Date: Tue, 1 Apr 2025 10:52:30 GMT Subject: RFR: 7903774: make all tests combinations printing nicer and/or configurable [v9] In-Reply-To: References: <3wZeI91CX_iFefOZJdSxdmZFSyzjd8-w1PJXYufNxrM=.77abf5c8-c5ca-45b9-9b8b-e5c0a7e9b478@github.com> Message-ID: On Mon, 30 Dec 2024 16:54:02 GMT, Ji?? Van?k wrote: >> This is still WIP, but already on row what [CODETOOLS-7903774](https://bugs.openjdk.org/browse/CODETOOLS-7903774) suggested. >> >> It will be finished once all desired output combinations are included, but already now is doing the usage of listing much more comfortable and code readable. >> >> From: >> >> `TESTS` - as introduced by https://github.com/openjdk/jcstress/pull/149 >> `COMBINATIONS` - as introduced in https://github.com/openjdk/jcstress/pull/149 verbose mode (thus surpassing the `-v` impact >> `ALL_TESTS` - all tests as it was before https://github.com/openjdk/jcstress/pull/149 >> `COMBINATIONS_GROUPED ` - as suggested in https://github.com/openjdk/jcstress/pull/149#discussion_r1668678910 >> `COMBINATIONS_JSON` same as combinations, but a valid json >> `COMBINATIONS_GROUPS` same as COMBINATIONS_GROUPED only instead of enumeration in each group, only count of members will be printed >> >> >> * TESTS are already implemented as ALL_MATCHING >> * COMBINATIONS as ALL_MATCHING_COMBINATIONS >> * ALL_TESTS as ALL >> >> COMBINATIONS_GROUPED, and COMBINATIONS_GROUPS are under development. >> The _JSON variants are tobe decided, but ability to JQL it sounds nice. >> >> Suggestion to better names welcomed. > > Ji?? Van?k has updated the pull request incrementally with two additional commits since the last revision: > > - Implemented proper json listings > > - id can be used both flatten (so equal to plaintext) or properly > strcutured for JQ (default) > - listing can be redirecd to stderr to use with `jq` directly > - ALL now corectly return all tests I still have faith! ------------- PR Comment: https://git.openjdk.org/jcstress/pull/153#issuecomment-2768960912 From jvanek at openjdk.org Tue Apr 1 10:53:47 2025 From: jvanek at openjdk.org (=?UTF-8?B?SmnFmcOt?= =?UTF-8?B?IA==?= =?UTF-8?B?VmFuxJtr?=) Date: Tue, 1 Apr 2025 10:53:47 GMT Subject: RFR: 7903754: jcstress should die asap - with report - on broken jvm/hw, instead of waiting on kill [v4] In-Reply-To: References:

Message-ID: On Fri, 21 Mar 2025 12:33:40 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 > > Ji?? Van?k has updated the pull request incrementally with one additional commit since the last revision: > > replaced incorrect System.out by output I still have faith! ------------- PR Comment: https://git.openjdk.org/jcstress/pull/157#issuecomment-2768961542 From jvanek at openjdk.org Wed Apr 23 07:49:53 2025 From: jvanek at openjdk.org (=?UTF-8?B?SmnFmcOt?= =?UTF-8?B?IA==?= =?UTF-8?B?VmFuxJtr?=) Date: Wed, 23 Apr 2025 07:49:53 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. Not sure with the faith any more... ------------- PR Comment: https://git.openjdk.org/jcstress/pull/158#issuecomment-2823375799 From jvanek at openjdk.org Fri Apr 25 15:04:05 2025 From: jvanek at openjdk.org (=?UTF-8?B?SmnFmcOt?= =?UTF-8?B?IA==?= =?UTF-8?B?VmFuxJtr?=) Date: Fri, 25 Apr 2025 15:04:05 GMT Subject: RFR: 7903750: TimeBudget (-tb) does not fulfill its promisses [v7] In-Reply-To: References:

Message-ID: On Wed, 26 Mar 2025 14:01:37 GMT, Ji?? Van?k wrote: >> Added warning, when time budget is to small, and final run will run at least two times longer. > > Ji?? Van?k has updated the pull request incrementally with one additional commit since the last revision: > > Removed duplicated construcion of timebudget Loosing the faith... ------------- PR Comment: https://git.openjdk.org/jcstress/pull/161#issuecomment-2830776464 From jvanek at openjdk.org Mon Apr 28 18:10:01 2025 From: jvanek at openjdk.org (=?UTF-8?B?SmnFmcOt?= =?UTF-8?B?IA==?= =?UTF-8?B?VmFuxJtr?=) Date: Mon, 28 Apr 2025 18:10:01 GMT Subject: RFR: 7903671: jcstress: Update buffer tests for JDK-8318966 In-Reply-To: References: Message-ID: On Wed, 26 Feb 2025 17:50:28 GMT, Ji?? Van?k wrote: > part 1: Detecting jdk of target VM and exuding illegible tests > > This is prequel to full fixing of CODETOOLS-7903671. This PR detects target JDK version and then disables illegible tests. > part 2 will be done in templates of affected tests and logic around generating the classes from them. In meantime this should server. > > The jdk detection will be most likely used anyway. WDYT? If we agree on similar approach, I will create a new bug for this PR only. loosing faith:( ------------- PR Comment: https://git.openjdk.org/jcstress/pull/159#issuecomment-2836070132 From jvanek at openjdk.org Mon Apr 28 18:10:01 2025 From: jvanek at openjdk.org (=?UTF-8?B?SmnFmcOt?= =?UTF-8?B?IA==?= =?UTF-8?B?VmFuxJtr?=) Date: Mon, 28 Apr 2025 18:10:01 GMT Subject: RFR: 7903889: jcstress: document used properites (and env variables) [v5] In-Reply-To: References: <_id_iOGQYGai8cFr-oaXer0R8UivNlj7-ZavmtRzetU=.1c07c8af-058d-4a9e-b83b-3a23bfb68776@github.com> Message-ID: On Sat, 25 Jan 2025 12:27:14 GMT, Ji?? Van?k wrote: >> 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 loosing faith:( ------------- PR Comment: https://git.openjdk.org/jcstress/pull/156#issuecomment-2836070262 From jvanek at openjdk.org Tue Apr 29 14:31:02 2025 From: jvanek at openjdk.org (=?UTF-8?B?SmnFmcOt?= =?UTF-8?B?IA==?= =?UTF-8?B?VmFuxJtr?=) Date: Tue, 29 Apr 2025 14:31:02 GMT Subject: RFR: 7903754: jcstress should die asap - with report - on broken jvm/hw, instead of waiting on kill [v4] In-Reply-To: References:

Message-ID: <4iDw4vM_hcEBqx1EGjz04qD2CNwyHfh5RyHtaUrtdPg=.e760a0d7-f7cb-4856-b6ee-4760cec2630c@github.com> On Fri, 21 Mar 2025 12:33:40 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 > > Ji?? Van?k has updated the pull request incrementally with one additional commit since the last revision: > > replaced incorrect System.out by output loosing faith ------------- PR Comment: https://git.openjdk.org/jcstress/pull/157#issuecomment-2839150593 From jvanek at openjdk.org Tue Apr 29 14:31:02 2025 From: jvanek at openjdk.org (=?UTF-8?B?SmnFmcOt?= =?UTF-8?B?IA==?= =?UTF-8?B?VmFuxJtr?=) Date: Tue, 29 Apr 2025 14:31:02 GMT Subject: RFR: 7903774: make all tests combinations printing nicer and/or configurable [v9] In-Reply-To: References: <3wZeI91CX_iFefOZJdSxdmZFSyzjd8-w1PJXYufNxrM=.77abf5c8-c5ca-45b9-9b8b-e5c0a7e9b478@github.com> Message-ID: On Mon, 30 Dec 2024 16:54:02 GMT, Ji?? Van?k wrote: >> This is still WIP, but already on row what [CODETOOLS-7903774](https://bugs.openjdk.org/browse/CODETOOLS-7903774) suggested. >> >> It will be finished once all desired output combinations are included, but already now is doing the usage of listing much more comfortable and code readable. >> >> From: >> >> `TESTS` - as introduced by https://github.com/openjdk/jcstress/pull/149 >> `COMBINATIONS` - as introduced in https://github.com/openjdk/jcstress/pull/149 verbose mode (thus surpassing the `-v` impact >> `ALL_TESTS` - all tests as it was before https://github.com/openjdk/jcstress/pull/149 >> `COMBINATIONS_GROUPED ` - as suggested in https://github.com/openjdk/jcstress/pull/149#discussion_r1668678910 >> `COMBINATIONS_JSON` same as combinations, but a valid json >> `COMBINATIONS_GROUPS` same as COMBINATIONS_GROUPED only instead of enumeration in each group, only count of members will be printed >> >> >> * TESTS are already implemented as ALL_MATCHING >> * COMBINATIONS as ALL_MATCHING_COMBINATIONS >> * ALL_TESTS as ALL >> >> COMBINATIONS_GROUPED, and COMBINATIONS_GROUPS are under development. >> The _JSON variants are tobe decided, but ability to JQL it sounds nice. >> >> Suggestion to better names welcomed. > > Ji?? Van?k has updated the pull request incrementally with two additional commits since the last revision: > > - Implemented proper json listings > > - id can be used both flatten (so equal to plaintext) or properly > strcutured for JQ (default) > - listing can be redirecd to stderr to use with `jq` directly > - ALL now corectly return all tests loosing faith ------------- PR Comment: https://git.openjdk.org/jcstress/pull/153#issuecomment-2839150262