RFR: 8356102: TestJcmdOutput, JcmdWithNMTDisabled and DumpSharedDictionary hs/tier1 tests fail on static-jdk [v3]

David Holmes dholmes at openjdk.org
Wed May 14 04:02:54 UTC 2025


On Wed, 14 May 2025 01:49:43 GMT, Jiangli Zhou <jiangli at openjdk.org> wrote:

>> Please review the change to set `-Dcompile.jdk=<jdk>` when launching child processes in TestJcmdOutput, JcmdWithNMTDisabled and DumpSharedDictionary hs/tier1 tests.
>> 
>> These three tests fail on static-jdk as `jdk.test.lib.JDKToolFinder.getJDKTool()` fails to locate the `jcmd` tool. `JDKToolFinder.getJDKTool()` locates the requested tool from the path specified in 'test.jdk' system property first, then using the path specified in 'compile.jdk' system property. Currently when running jtreg tests on static-jdk, jtreg '-compilejdk' is set to a regular JDK binary. Populating the 'compile.jdk' system property from jtreg when creating the child process would allow `JDKToolFinder.getJDKTool()` (executing in the child process) to locate the tool from 'compile.jdk' path.
>> 
>> Tested all three tests on static-jdk. All three tests pass with the fix.
>
> Jiangli Zhou has updated the pull request incrementally with two additional commits since the last revision:
> 
>  - Revert to https://github.com/openjdk/jdk/pull/25018/commits/6fd8d7ef2de6f0b4f6bed50e4255e1091834b68b.
>  - Revert test/lib/jdk/test/lib/process/ProcessTools.java.

Looks good. Thanks

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

Marked as reviewed by dholmes (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/25018#pullrequestreview-2838651553


More information about the hotspot-runtime-dev mailing list