From jvanek at openjdk.org  Fri Sep 12 10:07:29 2025
From: jvanek at openjdk.org (=?UTF-8?B?SmnFmcOt?= =?UTF-8?B?IA==?= =?UTF-8?B?VmFuxJtr?=)
Date: Fri, 12 Sep 2025 10:07:29 GMT
Subject: RFR: 7903750: TimeBudget (-tb) does not fulfill its promisses [v8]
In-Reply-To: <yvrVpJthMXPojWdswm9nkwQp_CB3FX3NAavvnBBEsqQ=.43a41d60-bf3d-4d53-a266-a1a3e07cb01f@github.com>
References: <CjY4lFNJTGY5E65LBRuh_-6soydH4EuO1l-EW09Xq_Y=.e89a2b57-c330-4e64-a60b-d6de9bc44d19@github.com>
 <yvrVpJthMXPojWdswm9nkwQp_CB3FX3NAavvnBBEsqQ=.43a41d60-bf3d-4d53-a266-a1a3e07cb01f@github.com>
Message-ID: <e-K46UnafARSaNBFHT_EN7rFJ3Q4GxQ_26dxQG513p0=.7e35768a-ce35-48aa-85af-c2f7211b88a6@github.com>

On Mon, 23 Jun 2025 08:22:32 GMT, Ji?? Van?k <jvanek at openjdk.org> 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 with a new target base due to a merge or a rebase. The pull request now contains 11 commits:
> 
>  - Removed duplicated construcion of timebudget
>  - directly return new instance without intermediate variable
>    
>    Co-authored-by: Galder Zamarre?o <galder at redhat.com>
>  - Removed table like formatting for time budget warnning
>  - Removed unnecesasary NPE check in getTimeBudget
>    
>    The NPE workaround was causing NPE to be only delayed for later
>  - Removed unnecessary shortening of MS before next conversion
>    
>    -        ms -= TimeUnit.HOURS.toMillis(hours);
>    -        ms -= TimeUnit.MINUTES.toMillis(minutes);
>  - Used  WARNING: and FATAL: instead of +++ WARNING - and +++ FATAL -
>  - renamed getNiceMsTimeDate to formatMsToDaysAndTime
>  - Replaced System.out by preset out
>  - added warning if timebudget is to small
>  - print time budfget also for listing
>    
>    Required two small refactorings to get nice code reused
>  - ... and 1 more: https://git.openjdk.org/jcstress/compare/1d143cbd...cbedff70

Loosing the faith...

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

PR Comment: https://git.openjdk.org/jcstress/pull/161#issuecomment-3284647785

From jvanek at openjdk.org  Fri Sep 12 10:08:18 2025
From: jvanek at openjdk.org (=?UTF-8?B?SmnFmcOt?= =?UTF-8?B?IA==?= =?UTF-8?B?VmFuxJtr?=)
Date: Fri, 12 Sep 2025 10:08:18 GMT
Subject: RFR: 7903671: jcstress: Update buffer tests for JDK-8318966 [v2]
In-Reply-To: <e_2LcCtpzHvbAN8J10sVZubkZEor3wqZK_KXKpCshX4=.f8d10c60-f967-4f54-997e-74b3c82dda1c@github.com>
References: <t9Uz4xkYl2aeoYzWd61sqxNG7b9ADpv4tnEotznqDWY=.392bc5a6-4349-40c3-96bb-5ee5c56f7cfc@github.com>
 <e_2LcCtpzHvbAN8J10sVZubkZEor3wqZK_KXKpCshX4=.f8d10c60-f967-4f54-997e-74b3c82dda1c@github.com>
Message-ID: <AjM9zH3ECL07NVlu7FdX8eCen9kOnqQdxIaV7KT5RD8=.f5985ac2-38a5-4ab0-bc79-f3be66dc98ff@github.com>

On Mon, 23 Jun 2025 09:10:54 GMT, Ji?? Van?k <jvanek at openjdk.org> 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.
>
> Ji?? Van?k has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains two commits:
> 
>  - Excluding jdk23+ illegible tests on jdk23+
>  - Added  detection on target JDK version

Loosing the faith...

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

PR Comment: https://git.openjdk.org/jcstress/pull/159#issuecomment-3284648492

From jvanek at openjdk.org  Fri Sep 12 10:08:19 2025
From: jvanek at openjdk.org (=?UTF-8?B?SmnFmcOt?= =?UTF-8?B?IA==?= =?UTF-8?B?VmFuxJtr?=)
Date: Fri, 12 Sep 2025 10:08:19 GMT
Subject: RFR: 7903889: jcstress: document used properites (and env
 variables) [v6]
In-Reply-To: <eWh5p7h78HWx9T1xvKAiCeGsZ99osMoKkBvVaYDHdeU=.17fb1ca8-d62f-4687-b24e-3887aabdcb16@github.com>
References: <_id_iOGQYGai8cFr-oaXer0R8UivNlj7-ZavmtRzetU=.1c07c8af-058d-4a9e-b83b-3a23bfb68776@github.com>
 <eWh5p7h78HWx9T1xvKAiCeGsZ99osMoKkBvVaYDHdeU=.17fb1ca8-d62f-4687-b24e-3887aabdcb16@github.com>
Message-ID: <K-2KMZAFRJC2SUkWUDVz7SITaFqNgM7fb-DlviI99CI=.09dab267-adb6-420f-a78e-b762f0c9b90b@github.com>

On Mon, 23 Jun 2025 08:49:27 GMT, Ji?? Van?k <jvanek at openjdk.org> 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 with a new target base due to a merge or a rebase. The pull request now contains four commits:
> 
>  - Moved declaration of individual proeprties to ceparate classes by family
>  - Extracted properties encapsualted in objects
>  - Removed autoformatted whitespaces
>  - 7903889: jcstress: document used properites (and env variables)
>    
>    Known properties unifed in one class and printed in verbose help:
>    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

Loosing the faith...

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

PR Comment: https://git.openjdk.org/jcstress/pull/156#issuecomment-3284649823

From jvanek at openjdk.org  Fri Sep 12 10:08:22 2025
From: jvanek at openjdk.org (=?UTF-8?B?SmnFmcOt?= =?UTF-8?B?IA==?= =?UTF-8?B?VmFuxJtr?=)
Date: Fri, 12 Sep 2025 10:08:22 GMT
Subject: RFR: 7903754: jcstress: Implement fail-on-error run option [v7]
In-Reply-To: <G1eeWyZglpxUI1XjLEOH7-SMuN_kRZQylxUa-po3voY=.17cf713d-d107-45bc-82d4-bed72b4542aa@github.com>
References: <YXTGh5WwXYP31kLN1WM5AYuPb5xoKOpoCjI0TFZ4diQ=.25589405-3afa-47a8-947e-4c70e3f14fde@github.com>
 <G1eeWyZglpxUI1XjLEOH7-SMuN_kRZQylxUa-po3voY=.17cf713d-d107-45bc-82d4-bed72b4542aa@github.com>
Message-ID: <qSiNBOO-NJEUN6ZxJ91H4-JzX37WJ-Qe6VGOnEGIUrQ=.af2a3df7-7444-466c-b87b-4b327376f188@github.com>

On Mon, 23 Jun 2025 08:21:56 GMT, Ji?? Van?k <jvanek at openjdk.org> 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 with a new target base due to a merge or a rebase. The pull request now contains 11 commits:
> 
>  - Removed counting of groups.
>    
>    It do not fill its purpose in real life
>  - Simplified to simpe -foe
>    
>    Both existing swithces with parameter changed to simngle -foe swithc,
>    whcih do not have any parameter, and if used , jcstress ends as soon as
>    possible after first error.
>    
>    Dropped the %% relative metric. Was good, but was never deployed afaik.
>    Simplifed test counting, now only tests are counted (no longer group)
>    Kept possibility to forbid exiting in the middle of group via property
>    
>    Kept posibility to set limit both  for absolute and relative treshold,
>    so user can still wait for more errors to appear before it is canceled.
>  - replaced incorrect System.out by output
>  - Experimental counting of softErrors through property
>  - Fix in logic
>  - 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
>  - ... and 1 more: https://git.openjdk.org/jcstress/compare/1d143cbd...389dd675

Loosing the faith...

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

PR Comment: https://git.openjdk.org/jcstress/pull/157#issuecomment-3284649310

From jvanek at openjdk.org  Fri Sep 12 10:08:23 2025
From: jvanek at openjdk.org (=?UTF-8?B?SmnFmcOt?= =?UTF-8?B?IA==?= =?UTF-8?B?VmFuxJtr?=)
Date: Fri, 12 Sep 2025 10:08:23 GMT
Subject: RFR: 7903774: make all tests combinations printing nicer and/or
 configurable [v10]
In-Reply-To: <zVJdkSKOd-xSB7pe5fbrb3DzXwQQZzF9xieAwtsVY0Q=.e64f16e5-c615-4e29-984a-8fa723833fd0@github.com>
References: <3wZeI91CX_iFefOZJdSxdmZFSyzjd8-w1PJXYufNxrM=.77abf5c8-c5ca-45b9-9b8b-e5c0a7e9b478@github.com>
 <zVJdkSKOd-xSB7pe5fbrb3DzXwQQZzF9xieAwtsVY0Q=.e64f16e5-c615-4e29-984a-8fa723833fd0@github.com>
Message-ID: <SWMevx4jZiL48lqvjU1MNZkjYXZSpqrer4IamCqVVzk=.3714e459-43e6-46a2-9f81-f21d076730b3@github.com>

On Mon, 23 Jun 2025 08:45:52 GMT, Ji?? Van?k <jvanek at openjdk.org> 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 with a new target base due to a merge or a rebase. The pull request now contains 10 commits:
> 
>  - 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
>  - Added property to control json variants fragemnts
>    
>    if true, the key will be written as:
>    "[actor1, actor2], spinLoopStyle: Thread.onSpinWait(), threads: 2, forkId: 0, maxFootprintMB: 64, compileMode: 7, shClass: (PG 0, CG 0), (PG 0, CG 0), strideSize: 256, strideCount: 40, cpuMap: null, [-XX:+UseBiasedLocking, -XX:+StressLCM, -XX:+StressGCM, -XX:+StressIGVN, -XX:+StressCCP, -XX:StressSeed=yyyyyyyy]"
>    
>    otherwise (default), it will be written as
>    
>    ...
>    "actorNames": ["actor1", "actor2"],
>    "spinLoopStyle": "Thread.onSpinWait()",
>    "threads": 2,
>    "forkId": 0,
>    "maxFootprintMB": 64,
>    "compileMode": 7,
>    "shClass": "(PG 0, CG 0), (PG 0, CG 0)",
>    "strideSize": 256,
>    "strideCount": 40,
>    "cpuMap": "null",
>    "jvmArgs": [-XX:+UseBiasedLocking, -XX:+StressLCM, -XX:+StressGCM, -XX:+StressIGVN, -XX:+StressCCP, -XX:StressSeed=yyyyyyyy]"
>    ...
>  - Implemented naive json formatting
>    
>    To cut away the header one can use: | sed '0,/^All.*[0-9]\+$/d'
>  - Implemented totaling of listing only
>  - Fixed spacing in help
>  - Removed redundnant hyphen
>  - implemented MATCHING_(INVERTED)GROUPS(_COUNT)
>  - Refactored lsiting so it takes proper argument
>    
>    instead of misusing verbose switch
>    and thus returned the ability to print allavailable tetest
>  - refactored listTests() to its own class

Loosing the faith...

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

PR Comment: https://git.openjdk.org/jcstress/pull/153#issuecomment-3284650293

From jvanek at openjdk.org  Fri Sep 12 10:08:26 2025
From: jvanek at openjdk.org (=?UTF-8?B?SmnFmcOt?= =?UTF-8?B?IA==?= =?UTF-8?B?VmFuxJtr?=)
Date: Fri, 12 Sep 2025 10:08:26 GMT
Subject: RFR: 7903819: jcstress do not print structuralised results [v2]
In-Reply-To: <6FMz9T4k7cDvZyBPXZrnTG1YCh758jJXoxGAJpGYIro=.657b12f6-3cad-4920-a7c7-2723c3e2203e@github.com>
References: <IUgbbJHyAWPYwqSmseeNKscyUtXdVSUpXQESdL-TRIw=.f1a12b3c-9ff7-4d28-86ce-75de4bb01c30@github.com>
 <6FMz9T4k7cDvZyBPXZrnTG1YCh758jJXoxGAJpGYIro=.657b12f6-3cad-4920-a7c7-2723c3e2203e@github.com>
Message-ID: <bXQEuNWxT9TLItiHO2W0F7tGIe3R18eA9GN7qPWlps8=.b4630687-e6ad-4257-94b3-0c6e42f76e4b@github.com>

On Mon, 23 Jun 2025 08:50:02 GMT, Ji?? Van?k <jvanek at openjdk.org> 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.
>
> Ji?? Van?k has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains 25 additional commits since the last revision:
> 
>  - Dropped the dom-based generation
>    
>    Without stream the serilializer was unnecessarily big
>  - Started migration to proepr DOM
>  - Revert "FIXME Added fast fake results generator REMOVE IT"
>    
>    This reverts commit 0d758a79012dcb11f2a98b74a326b58809a97599.
>    The junit result looks good. Now testing on real data
>  - Fixed few noted comments/variables
>  - made default output xsd valid again
>    
>    and keeping option to enable the widely used, but not standart parts for
>    higher readability
>  - changed comments proeprty to comment out non-standart elements
>    
>    soem of the wide spread features like nested testsuites and properties
>    are nmot part of xsd... those will be commented out b default, and moved
>    to fail/err if possible
>  - Unifide testcase print
>  - Support for time attribute  and removed some dnagling code
>  - <testsuite> now prints correct summary
>    
>    stil not perfect...
>    mia: interesting? time?
>    and move to dom....
>  - Not printing the seed argument
>  - ... and 15 more: https://git.openjdk.org/jcstress/compare/d62cc929...870f1605

Loosing the faith...

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

PR Comment: https://git.openjdk.org/jcstress/pull/158#issuecomment-3284648903