RFR: 8247281: migrate ObjectMonitor::_object to OopStorage [v5]
Daniel D.Daugherty
dcubed at openjdk.java.net
Wed Sep 16 15:45:32 UTC 2020
On Wed, 16 Sep 2020 00:20:16 GMT, Serguei Spitsyn <sspitsyn at openjdk.org> wrote:
>> I've taken a first pass at creating a CSR:
>> JDK-8253121 migrate ObjectMonitor::_object to OopStorage
>> https://bugs.openjdk.java.net/browse/JDK-8253121
>
> Just a minor comment.
> The line 1754 can be deleted as the JVMTI_HEAP_REFERENCE_MONITOR reference type will be never encountered:
>
> 1750 static jvmtiHeapRootKind toJvmtiHeapRootKind(jvmtiHeapReferenceKind kind) {
> 1751 switch (kind) {
> 1752 case JVMTI_HEAP_REFERENCE_JNI_GLOBAL: return JVMTI_HEAP_ROOT_JNI_GLOBAL;
> 1753 case JVMTI_HEAP_REFERENCE_SYSTEM_CLASS: return JVMTI_HEAP_ROOT_SYSTEM_CLASS;
> 1754 case JVMTI_HEAP_REFERENCE_MONITOR: return JVMTI_HEAP_ROOT_MONITOR;
> 1755 case JVMTI_HEAP_REFERENCE_STACK_LOCAL: return JVMTI_HEAP_ROOT_STACK_LOCAL;
> 1756 case JVMTI_HEAP_REFERENCE_JNI_LOCAL: return JVMTI_HEAP_ROOT_JNI_LOCAL;
> 1757 case JVMTI_HEAP_REFERENCE_THREAD: return JVMTI_HEAP_ROOT_THREAD;
> 1758 case JVMTI_HEAP_REFERENCE_OTHER: return JVMTI_HEAP_ROOT_OTHER;
> 1759 default: ShouldNotReachHere(); return JVMTI_HEAP_ROOT_OTHER;
> 1760 }
> 1761 }
>
> Other than that the update in this file looks okay to me.
I cleaned that up. The only references to JVMTI_HEAP_REFERENCE_MONITOR and
JVMTI_HEAP_ROOT_MONITOR that remain are in the spec:
$ egrep -r 'JVMTI_HEAP_REFERENCE_MONITOR|JVMTI_HEAP_ROOT_MONITOR' src/hotspot
src/hotspot/share/prims/jvmti.xml: <constant id="JVMTI_HEAP_REFERENCE_MONITOR" num="23">
src/hotspot/share/prims/jvmti.xml: <constant id="JVMTI_HEAP_ROOT_MONITOR" num="3">
src/hotspot/share/prims/jvmti.xml: <code>JVMTI_HEAP_ROOT_MONITOR</code>,
-------------
PR: https://git.openjdk.java.net/jdk/pull/135
More information about the serviceability-dev
mailing list