RFR: 8362501: Update test/hotspot/jtreg/applications/jcstress/README [v4]

Aleksey Shipilev shade at openjdk.org
Mon Jul 28 09:08:56 UTC 2025


On Mon, 28 Jul 2025 08:31:33 GMT, SendaoYan <syan at openjdk.org> wrote:

>> Hi all,
>> 
>> Currently, this is no documentation on how to run the application/jcstress tests. I think it will be useful to complement the document test/hotspot/jtreg/applications/jcstress/README on how to run the jcstress tests in jtreg.
>> 
>> No behaviour has been change, only update the ducumentation, no risk.
>
> SendaoYan has updated the pull request incrementally with one additional commit since the last revision:
> 
>   The way to
>   get the specified build you can reference the guide

Looks better. I think we can be less specific in the guidance, see below.

test/hotspot/jtreg/applications/jcstress/README line 34:

> 32: You should specific the JAR location with jtreg option such as
> 33: -javaoption:-Djdk.test.lib.artifacts.jcstress-tests-all=
> 34: jcstress-tests-all-20241217-2035.jar when starting jcstress tests in jtreg.

Suggestion:

successfully. These tests require a build of org.openjdk.jcstress:jcstress-tests-all.
If this artifact could not be found automatically, you can build it using jcstress
guide[1], and then pass the JAR location with JDK option such as 
-Djdk.test.lib.artifacts.jcstress-tests-all=jcstress-tests-all.jar, 
either with explicit -javaoption: or TEST_VM_OPTS test variable.

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

PR Review: https://git.openjdk.org/jdk/pull/26369#pullrequestreview-3061389801
PR Review Comment: https://git.openjdk.org/jdk/pull/26369#discussion_r2235459431


More information about the hotspot-runtime-dev mailing list