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