[crac] RFR: Handle C/R in BasicImageReader (via reflection)

Radim Vansa rvansa at openjdk.org
Thu Nov 16 09:59:02 UTC 2023


On Wed, 15 Nov 2023 18:31:45 GMT, Anton Kozlov <akozlov at openjdk.org> wrote:

>> Fixes checkpoint failure due to open FD to modules on x86 (the Java code behaves differently on 32bit and 64bt).
>
> src/java.base/share/classes/jdk/internal/jimage/BasicImageReader.java line 109:
> 
>> 107:             channel = null;
>> 108:         } else {
>> 109:             channel = openFileChannel();
> 
> Could you avoid this refactoring to simplify future merges?

Do you mean to just copy-paste the `openFileChannel()` code? Do we rather want any change in that code go unnoticed?

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

PR Review Comment: https://git.openjdk.org/crac/pull/124#discussion_r1395441271


More information about the crac-dev mailing list