[crac] RFR: Improved open file descriptors tracking [v4]

Radim Vansa duke at openjdk.org
Mon Mar 20 09:59:00 UTC 2023


On Mon, 13 Mar 2023 14:03:52 GMT, Anton Kozlov <akozlov at openjdk.org> wrote:

>> Radim Vansa has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains 16 additional commits since the last revision:
>> 
>>  - Merge remote-tracking branch 'origin/crac' into newfd
>>  - Use descriptor access rather than extending API
>>  - 8272472: StackGuardPages test doesn't build with glibc 2.34
>>    
>>    Backport-of: f77a1a156f3da9068d012d9227c7ee0fee58f571
>>  - Empty commit to trigger GHA
>>  - Drop native FDs tracking
>>  - Avoid claiming invalid FileDescriptor
>>  - Whitelist RandomAccessFile opening classpath files
>>    
>>    This is a workaround for some frameworks opening classpath files in
>>    a non-standard way.
>>  - Add tracking of FD origin
>>  - Track FileDescriptors closed by NIO
>>  - Track native FDs from EPoll
>>  - ... and 6 more: https://git.openjdk.org/crac/compare/6a0834b4...9b5e7edd
>
> src/java.base/share/classes/jdk/crac/Core.java line 138:
> 
>> 136:             fdArr[i] = claimedPairs.get(i).getKey();
>> 137:             objArr[i] = claimedPairs.get(i).getValue();
>> 138:             System.out.printf("%d %s\n", fdArr[i], objArr[i]);
> 
> Left-over from a debugging apparently

I am not so sure about that; the `objArr` is later ignored by the `VM_Crac` so I thought this it the place that should report open FDs, even though claimed by Java.

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

PR: https://git.openjdk.org/crac/pull/43


More information about the crac-dev mailing list