RFR: 8258836: JNI local refs exceed capacity getDiagnosticCommandInfo [v6]

Chris Plummer cjplummer at openjdk.java.net
Fri Jan 22 20:21:48 UTC 2021


On Fri, 22 Jan 2021 09:37:45 GMT, Severin Gehwolf <sgehwolf at openjdk.org> wrote:

>> This patch adds some explicit capacity for local refs. New regression test
>> fails prior and passes after the patch.
>> 
>> Thoughts?
>
> Severin Gehwolf 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 10 additional commits since the last revision:
> 
>  - Merge branch 'master' into JDK-8258836-check-jni-mbeanserver
>  - Fix copyright and one more PopLocalFrame
>  - Improve popping of local frames in the error case
>  - Make HOTSPOT_DIAGNOSTIC_MXBEAN_NAME final
>  - Merge branch 'master' into JDK-8258836-check-jni-mbeanserver
>  - Actually assign the variable returned from PopLocalFrame
>  - Merge test files into one
>  - Adress review feedback from dholmes
>  - Merge branch 'master' into JDK-8258836-check-jni-mbeanserver
>  - 8258836: JNI local refs exceed capacity getDiagnosticCommandInfo

Looks good.

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

Marked as reviewed by cjplummer (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/2130


More information about the serviceability-dev mailing list