On Tue, 7 Jul 2026 17:59:52 GMT, Coleen Phillimore <[email protected]> wrote:

>> Please review this change to allow XX configuration for specifying a 
>> different /tmp directory for the JVM to use.  In some container 
>> environments, /tmp and /proc/pid/root/tmp might not be usable and an 
>> alternate would be used.  This requires a release note and CSR.   Usage is:
>> 
>> java -XX:AltTempDir=/diags <app>
>> jps -J-XX:AltTempDir=/diags
>> jcmd -J-XX:AltTempDir=/diags <pid> <cmds>
>> 
>> Tested with a couple of tests and locally, and ran tier1-4.
>> 
>> ---------
>> - [x] I confirm that I make this contribution in accordance with the 
>> [OpenJDK Interim AI Policy](https://openjdk.org/legal/ai).
>
> Coleen Phillimore has updated the pull request incrementally with two 
> additional commits since the last revision:
> 
>  - Fix PlatformSupportImpl due to some internal code review comments.
>  - Add similar jps wording.

Changes requested by dholmes (Reviewer).

src/jdk.jcmd/share/man/jps.md line 104:

> 102: If the target JVM is started with an alternate temporary file location, 
> `jps` must
> 103: use the same temporary file location for communication; this is true by 
> default
> 104: but see the [`-XX:AltTempDir`](./java.html#-XX_AltTempDir) option.

Suggestion:

The list of JVMs is also limited to those that use the same temporary file 
location as the `jps` command. 
That is normally the case but see the 
[`-XX:AltTempDir`](./java.html#-XX_AltTempDir) option.

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

PR Review: https://git.openjdk.org/jdk/pull/31407#pullrequestreview-4649940974
PR Review Comment: https://git.openjdk.org/jdk/pull/31407#discussion_r3540474729

Reply via email to