On Fri, 25 Jun 2021 21:04:27 GMT, Jorn Vernee <jver...@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