[jdk17] RFR: 8269240: java/foreign/stackwalk/TestAsyncStackWalk.java test failed with concurrent GC [v2]

[David Holmes]

On Fri, 25 Jun 2021 21:04:27 GMT, Jorn Vernee <jvernee at openjdk.org> wrote:

>> Jorn Vernee has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Assert frame is correct type in frame_data_for_frame
>
> src/hotspot/share/runtime/safepoint.cpp line 931:
> 
>> 929:     // See if return type is an oop.
>> 930:     bool return_oop = nm->method()->is_returning_oop();
>> 931:     HandleMark hm(self);
> 
> I was seeing an `assert(_handle_mark_nesting > 1) failed: memory leak: allocating handle outside HandleMark` when the existing code allocates the Handle for `return_value` in the code down below. It seems to be a simple case of a missing handle mark, so I've added it here. (looking at the stack trace in the error log, the caller frame is native code, so I don't think we can expect the caller to have a HandleMark).

How does native code reach a safepoint polling point?

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

PR: https://git.openjdk.java.net/jdk17/pull/149