On Thu, 11 Jun 2026 21:32:32 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:
> 
>   Use shouldMatch for logging message.

About requiring the AltTempDir to be a full pathname.  We could do so but it 
would break the tests because they'd end up with a too-long pathname (> 108).  
For "root%s" in perfMemory_posix.cpp, can we do something like this instead?

https://github.com/openjdk/jdk/pull/31407/changes#diff-35acaddf6c2af34d95ce74aa5b2841aa259dac397ae02626e4572eeb1df853fbR260

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

PR Comment: https://git.openjdk.org/jdk/pull/31407#issuecomment-4770662811

Reply via email to