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