RFR: CODETOOLS-7903748 - jcstress: Test list should honor concurrency settings [v11]

[Aleksey Shipilev]

On Mon, 8 Jul 2024 16:02:08 GMT, Jiří Vaněk <jvanek at openjdk.org> wrote:

>> This is extracting  List<TestConfig> configs =prepareRunProgram(classes, tests);
>> with all he HW/switches setup to separated method and reusing it in `-l` mode
>> 
>> I'm aware of triplicated removal of -agentlib, and will clean it up as
>> CODETOOLS-7903756 will progress.
>> 
>> -l now honours also verbose mode, in which it prints not just matching
>> tests but all really run tests, and thus enabling much more easy
>> determining of all tests
>> 
>> help adjusted.
>> 
>> Maybe I'm missing plain quick initial all tests metod now, but with 
>> artificial -c MAX it seems doing exactly that
>
> Jiří Vaněk has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Update jcstress-core/src/main/java/org/openjdk/jcstress/JCStress.java
>   
>   Co-authored-by: Aleksey Shipilëv <shipilev at amazon.de>

One more leftover.

jcstress-core/src/main/java/org/openjdk/jcstress/Main.java line 27:

> 25: package org.openjdk.jcstress;
> 26: 
> 27: import org.openjdk.jcstress.infra.runners.TestConfig;

Don't need this import now?

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

PR Review: https://git.openjdk.org/jcstress/pull/149#pullrequestreview-2163798658
PR Review Comment: https://git.openjdk.org/jcstress/pull/149#discussion_r1668925742