RFR: 8362501: Update test/hotspot/jtreg/applications/jcstress/README [v2]
David Holmes
dholmes at openjdk.org
Fri Jul 25 05:29:53 UTC 2025
On Fri, 18 Jul 2025 06:31:37 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:
>
> Update README
Grammatical updates look fine - one further one flagged. I can't review as such as I can't validate the instructions being given as I don't know how to run these tests.
Maybe @lmesnik could review?
Or @shipilev may be able to review?
test/hotspot/jtreg/applications/jcstress/README line 35:
> 33: You should specific the JAR location with jtreg option such as
> 34: -javaoption:-Djdk.test.lib.artifacts.jcstress-tests-all=
> 35: jcstress-tests-all-20241217-2035.jar when start jcstress tests in jtreg.
Suggestion:
jcstress-tests-all-20241217-2035.jar when starting jcstress tests in jtreg.
-------------
PR Review: https://git.openjdk.org/jdk/pull/26369#pullrequestreview-3054121280
PR Comment: https://git.openjdk.org/jdk/pull/26369#issuecomment-3116446288
PR Review Comment: https://git.openjdk.org/jdk/pull/26369#discussion_r2230168069
More information about the hotspot-runtime-dev
mailing list