RFR: 8259374: Make ThreadInVMfromNative have ResetNoHandleMark [v2]

David Holmes dholmes at openjdk.java.net
Fri Jan 8 06:59:02 UTC 2021


On Thu, 7 Jan 2021 22:21:07 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:

>> Please review this straightforward change to add a ResetNoHandleMark in the ThreadInVMfromNative transition class so that callers don't have to do this.  Tested with tier1-3.
>> This was broken out of a larger change for JDK-8258032 Reconsider LEAF entry restrictions.
>
> Coleen Phillimore has updated the pull request incrementally with one additional commit since the last revision:
> 
>   fix grammar

src/hotspot/share/runtime/interfaceSupport.inline.hpp line 195:

> 193: 
> 194: class ThreadInVMfromNative : public ThreadStateTransition {
> 195:   ResetNoHandleMark __rnhm;

We should add a constructor for RNHM that takes the current thread so that we don't have to call Thread::current() unnecessarily in the existing RNHM constructor.

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

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


More information about the hotspot-dev mailing list