On Wed, 8 Jul 2026 21:05:12 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: > > With @kevinjwalls rewrite for getLocalVmId that ignores temp directories. > Rewrite tests to actually delete temp files. Fix VirtualMachineImpl to get > "unable to access target filesystem" failure and not bypass the same > filesystem check. src/jdk.jcmd/share/man/jinfo.md line 61: > 59: use the same temporary file location for communication; this is true by > default > 60: but see the [`-XX:AltTempDir`](./java.html#-XX_AltTempDir) option. > 61: Nit: I would move this after the next paragraph (so it's the same as jmap). Note for svc folk: the preceding part about using -d64 is obsolete and should be removed. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/31407#discussion_r3547942575
