RFR: JDK-8276202: LogFileOutput.invalid_file_vm asserts when being executed from a read only working directory

Thomas Stuefe stuefe at openjdk.java.net
Thu Apr 28 15:13:42 UTC 2022


On Thu, 28 Apr 2022 14:51:08 GMT, Daniel D. Daugherty <dcubed at openjdk.org> wrote:

> Typically the reason for putting temporary files in the current directory is so that they are created in a JTREG managed directory and cleaned up as needed. By putting the files in /tmp you will be putting them in a place where JTREG won't deal with them. Also, in the event of a test failure, will the file in /tmp be left behind?

Oh, Dan is right :-( With jtreg, I can use -retain to keep the temp files, otherwise jtreg cleans them up.That only works oc with cwd.

I see now how weirdly difficult this problem is. In my opinion, the file handling parts of the UL tests would have been better left as jtreg tests, not as gtests.

For the record, I would be perfectly content with just the assert->ASSERT part of this fix.

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

PR: https://git.openjdk.java.net/jdk/pull/8238


More information about the hotspot-runtime-dev mailing list