On Mon, 13 Jul 2026 12:17:09 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 one 
> additional commit since the last revision:
> 
>   Serguei review.

Still good.

src/hotspot/os/linux/os_linux.cpp line 1561:

> 1559: 
> 1560: // If it is not an absolute path, revert back to hardcoded /tmp. If the 
> directory is non existant or not
> 1561: // writeable give a warning but use AltTempDir. In the latter case, we 
> may be connecting to a process that is

writable?

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

Marked as reviewed by dholmes (Reviewer).

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

Reply via email to