RFR: 8366298: FDLeakTest sometimes takes minutes to complete on Linux [v2]
Stefan Karlsson
stefank at openjdk.org
Fri Aug 29 09:04:44 UTC 2025
On Fri, 29 Aug 2025 09:00:57 GMT, Stefan Karlsson <stefank at openjdk.org> wrote:
>> test/jdk/java/lang/ProcessBuilder/FDLeakTest/libFDLeaker.c line 54:
>>
>>> 52: if (ret != 0) {
>>> 53: return JNI_ERR;
>>> 54: }
>>
>> Hmm, if the JVM already has opened > 100 fds at this point, we get difficult to understand errors. Any future open etc call will fail.
>>
>> What I would do is to check, after the call to fopen() at line 39, check for errno==EMFILE. If that is true, write out a clear error message along the lines of "more than 100 files open?" (since that in itself would be worth to investigate; this little test should not open > 100 files, but who knows).
>
> OK. I've added error logging for both setting the limit and opening the file. I didn't check specifically for EMFILE, but I hope that's fine?
I also verified that if I set the limit to 3 I do get the "Too many open files" error message.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/26979#discussion_r2309608800
More information about the core-libs-dev
mailing list