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