On Thu, 9 Jul 2026 12:30:43 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:
> 
>   Moved down AltTempDir in jinfo.md

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

> 1569:     } else {
> 1570:       if (!is_writable_directory(AltTempDir)) {
> 1571:         log_warning(os)("Warning: AltTempDir is not an existing or 
> writable directory");

Nit: Should we add this line here as well as at line the 1568: `AltTempDir = 
nullptr;` ?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/31407#discussion_r3557092591

Reply via email to