RFR: 8339686: java/foreign/TestMappedHandshake.java fails with assert(depth < max_critical_stack_depth) failed: can't have more than 10 critical frames [v2]

[Martin Doerr]

On Mon, 9 Sep 2024 15:30:47 GMT, Maurizio Cimadamore <mcimadamore at openjdk.org> wrote:

>> Maurizio Cimadamore has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Drop spurious change
>
> src/java.base/share/classes/java/nio/MappedMemoryUtils.java line 128:
> 
>> 126:     static {
>> 127:         registerNatives();
>> 128:         isLoaded0(0, 0, 0);
> 
> This spurious line was added for debugging reasons in a previous PR - it should have been removed. Doing it now. This is probably the cause of https://bugs.openjdk.org/browse/JDK-8339780

Yes. Thanks for fixing it! Maybe you can add that issue to this PR.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/20914#discussion_r1750492934